[imageutils] Fix __str__ for QemuImgInfo

Code is calling 'appened' on a list; correct this to 'append'. Closes-bug: #1996426 Change-Id: I8729cf180f92f43519c942e22f3b285377a5612f
openstack · Nov 11, 2022 · d49d594 · d49d594
1 parent cb8076b
commit d49d594
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/oslo_utils/imageutils.py b/oslo_utils/imageutils.py
@@ -100,7 +100,7 @@ def __str__(self):
         if self.encrypted:
             lines.append("encrypted: %s" % self.encrypted)
         if self.format_specific:
-            lines.appened("format_specific: %s" % self.format_specific)
+            lines.append("format_specific: %s" % self.format_specific)
         return "\n".join(lines)
 
     def _canonicalize(self, field):

diff --git a/oslo_utils/tests/test_imageutils.py b/oslo_utils/tests/test_imageutils.py
@@ -264,6 +264,9 @@ def test_qemu_img_info(self, mock_deprecate):
         self.assertEqual(13168640, image_info.disk_size)
         self.assertEqual("bar", image_info.format_specific["data"]["foo"])
         self.assertEqual('yes', image_info.encrypted)
+        # test for Bug #1996426
+        expected_str = "format_specific: {'data': {'foo': 'bar'}}"
+        self.assertIn(expected_str, str(image_info))
 
     @mock.patch("debtcollector.deprecate")
     def test_qemu_img_info_blank(self, mock_deprecate):