commit b34c45f8847ce9a0edca6eff293028d711ab9fba
author Zuul <zuul@review.opendev.org> Tue Sep 03 20:23:26 2024 +0000
committer Gerrit Code Review <review@openstack.org> Tue Sep 03 20:23:26 2024 +0000
tree 0838353d71cb81daf8126a89f04c4f350eac3b5e
parent 3ba975fce29779f779b0f48ea1e0c422a35f6821
parent df14157e3ac6a008b6e0bc4e018bf9d8c9091484

Merge "Fix misuse of assertIsNone"

tempest/api/image/base.py

diff --git a/tempest/api/image/base.py b/tempest/api/image/base.py
index 0cc088a..c2f067c 100644
--- a/tempest/api/image/base.py
+++ b/tempest/api/image/base.py
@@ -177,8 +177,8 @@
         # If we added the location directly, the image goes straight
         # to active and no hashing is done
         self.assertEqual('active', image['status'])
-        self.assertIsNone(None, image['os_hash_algo'])
-        self.assertIsNone(None, image['os_hash_value'])
+        self.assertIsNone(image['os_hash_algo'])
+        self.assertIsNone(image['os_hash_value'])
 
         return image
 
@@ -201,8 +201,8 @@
 
         # The image should still be active and still have no hashes
         self.assertEqual('active', image['status'])
-        self.assertIsNone(None, image['os_hash_algo'])
-        self.assertIsNone(None, image['os_hash_value'])
+        self.assertIsNone(image['os_hash_algo'])
+        self.assertIsNone(image['os_hash_value'])
 
         # The direct_url should still match the first location
         if 'direct_url' in image: