From abfe0f4bf3295a4bc807b7e4e6b46a16061b29de Mon Sep 17 00:00:00 2001
From: Cyril Roelandt <cyril@redhat.com>
Date: Thu, 3 May 2018 15:54:00 +0200
Subject: Image show: print human readable string when the virtual size is
 unknown

Currently, when the virtual size of an image is not known, "None" is displayed.
To a regular user, it feels like a programming error. We try and make things
clearer by using a "human readable" string instead.

Change-Id: Id7b8799356857d9bc58cc8a3677024fe1a7f4f56
Partial-Bug: #1665037
---
 glanceclient/tests/unit/test_utils.py | 38 +++++++++++++++++++++++++++++++++++
 1 file changed, 38 insertions(+)

(limited to 'glanceclient/tests')

diff --git a/glanceclient/tests/unit/test_utils.py b/glanceclient/tests/unit/test_utils.py
index a63ee80..3ef585a 100644
--- a/glanceclient/tests/unit/test_utils.py
+++ b/glanceclient/tests/unit/test_utils.py
@@ -138,6 +138,44 @@ class TestUtils(testtools.TestCase):
 +--------------------------------------+--------------------------------------+
 | b8e1c57e-907a-4239-aed8-0df8e54b8d2d | ['Name1', 'Tag_123', 'veeeery long'] |
 +--------------------------------------+--------------------------------------+
+''',
+                         output_list.getvalue())
+
+    def test_print_image_virtual_size_available(self):
+        image = {'id': '42', 'virtual_size': 1337}
+        saved_stdout = sys.stdout
+        try:
+            sys.stdout = output_list = six.StringIO()
+            utils.print_image(image)
+        finally:
+            sys.stdout = saved_stdout
+
+        self.assertEqual('''\
++--------------+-------+
+| Property     | Value |
++--------------+-------+
+| id           | 42    |
+| virtual_size | 1337  |
++--------------+-------+
+''',
+                         output_list.getvalue())
+
+    def test_print_image_virtual_size_not_available(self):
+        image = {'id': '42', 'virtual_size': None}
+        saved_stdout = sys.stdout
+        try:
+            sys.stdout = output_list = six.StringIO()
+            utils.print_image(image)
+        finally:
+            sys.stdout = saved_stdout
+
+        self.assertEqual('''\
++--------------+---------------+
+| Property     | Value         |
++--------------+---------------+
+| id           | 42            |
+| virtual_size | Not available |
++--------------+---------------+
 ''',
                          output_list.getvalue())
 
-- 
cgit v1.2.1