Merge "Remove invalid block_device_mapping volume_size of ''"

1 files changed, 17 insertions, 0 deletions

diff --git a/nova/tests/test_db_api.py b/nova/tests/test_db_api.py
index cfed3ab3b..bf7cc003a 100644
--- a/nova/tests/test_db_api.py
+++ b/nova/tests/test_db_api.py
@@ -19,6 +19,7 @@
 
 """Unit tests for the DB API."""
 
+import copy
 import datetime
 import types
 import uuid as stdlib_uuid
@@ -3343,6 +3344,22 @@ class BlockDeviceMappingTestCase(test.TestCase):
             if bdm['device_name'] == values['device_name']:
                 return bdm
 
+    def test_scrub_empty_str_values_no_effect(self):
+        values = {'volume_size': 5}
+        expected = copy.copy(values)
+        sqlalchemy_api._scrub_empty_str_values(values, ['volume_size'])
+        self.assertEqual(values, expected)
+
+    def test_scrub_empty_str_values_empty_string(self):
+        values = {'volume_size': ''}
+        sqlalchemy_api._scrub_empty_str_values(values, ['volume_size'])
+        self.assertEqual(values, {})
+
+    def test_scrub_empty_str_values_empty_unicode(self):
+        values = {'volume_size': u''}
+        sqlalchemy_api._scrub_empty_str_values(values, ['volume_size'])
+        self.assertEqual(values, {})
+
     def test_block_device_mapping_create(self):
         bdm = self._create_bdm({})
         self.assertFalse(bdm is None)