diff options
author | Zuul <zuul@review.opendev.org> | 2023-05-09 23:45:15 +0000 |
---|---|---|
committer | Gerrit Code Review <review@openstack.org> | 2023-05-09 23:45:15 +0000 |
commit | 832275015af5f187105e1de2558f7811b64a6d03 (patch) | |
tree | 1603d59c6d3f9017de1614423e7f250d4d04ca8b /ironic/tests/unit/drivers | |
parent | 1d0818cba2d948902ca056e6f82f873838415c0e (diff) | |
parent | 03cd9788e6ad658641b0c3328d10636c5aab96a4 (diff) | |
download | ironic-832275015af5f187105e1de2558f7811b64a6d03.tar.gz |
Merge "Support longer checksums for redfish firmware upgrade"
Diffstat (limited to 'ironic/tests/unit/drivers')
-rw-r--r-- | ironic/tests/unit/drivers/modules/redfish/test_firmware_utils.py | 24 |
1 files changed, 24 insertions, 0 deletions
diff --git a/ironic/tests/unit/drivers/modules/redfish/test_firmware_utils.py b/ironic/tests/unit/drivers/modules/redfish/test_firmware_utils.py index 61bc23e48..0bbc38eba 100644 --- a/ironic/tests/unit/drivers/modules/redfish/test_firmware_utils.py +++ b/ironic/tests/unit/drivers/modules/redfish/test_firmware_utils.py @@ -256,6 +256,30 @@ class FirmwareUtilsTestCase(base.TestCase): mock_compute_file_checksum.assert_called_with( file_path, algorithm='sha1') + @mock.patch.object(fileutils, 'compute_file_checksum', autospec=True) + def test_verify_checksum_sha256(self, mock_compute_file_checksum): + checksum = 'a' * 64 + file_path = '/tmp/bios.exe' + mock_compute_file_checksum.return_value = checksum + node = mock.Mock(uuid='9f0f6795-f74e-4b5a-850e-72f586a92435') + + firmware_utils.verify_checksum(node, checksum, file_path) + + mock_compute_file_checksum.assert_called_with( + file_path, algorithm='sha256') + + @mock.patch.object(fileutils, 'compute_file_checksum', autospec=True) + def test_verify_checksum_sha512(self, mock_compute_file_checksum): + checksum = 'a' * 128 + file_path = '/tmp/bios.exe' + mock_compute_file_checksum.return_value = checksum + node = mock.Mock(uuid='9f0f6795-f74e-4b5a-850e-72f586a92435') + + firmware_utils.verify_checksum(node, checksum, file_path) + + mock_compute_file_checksum.assert_called_with( + file_path, algorithm='sha512') + @mock.patch.object(os, 'makedirs', autospec=True) @mock.patch.object(shutil, 'copyfile', autospec=True) @mock.patch.object(os, 'link', autospec=True) |