Merge "Cast volume-related ids to str"

4 files changed, 25 insertions, 25 deletions

diff --git a/nova/tests/api/openstack/volume/test_snapshots.py b/nova/tests/api/openstack/volume/test_snapshots.py
index 0ef112ee9..95efc539b 100644
--- a/nova/tests/api/openstack/volume/test_snapshots.py
+++ b/nova/tests/api/openstack/volume/test_snapshots.py
@@ -81,7 +81,7 @@ class SnapshotApiTest(test.TestCase):
     def test_snapshot_create(self):
         self.stubs.Set(volume.api.API, "create_snapshot", stub_snapshot_create)
         self.stubs.Set(volume.api.API, 'get', fakes.stub_volume_get)
-        snapshot = {"volume_id": 12,
+        snapshot = {"volume_id": '12',
                 "force": False,
                 "display_name": "Snapshot Test Name",
                 "display_description": "Snapshot Test Desc"}
@@ -99,7 +99,7 @@ class SnapshotApiTest(test.TestCase):
         self.stubs.Set(volume.api.API, "create_snapshot_force",
             stub_snapshot_create)
         self.stubs.Set(volume.api.API, 'get', fakes.stub_volume_get)
-        snapshot = {"volume_id": 12,
+        snapshot = {"volume_id": '12',
                 "force": True,
                 "display_name": "Snapshot Test Name",
                 "display_description": "Snapshot Test Desc"}
@@ -131,12 +131,11 @@ class SnapshotApiTest(test.TestCase):
                           snapshot_id)
 
     def test_snapshot_show(self):
-        snapshot_id = 123
-        req = fakes.HTTPRequest.blank('/v1/snapshots/%d' % snapshot_id)
-        resp_dict = self.controller.show(req, snapshot_id)
+        req = fakes.HTTPRequest.blank('/v1/snapshots/123')
+        resp_dict = self.controller.show(req, 123)
 
         self.assertTrue('snapshot' in resp_dict)
-        self.assertEqual(resp_dict['snapshot']['id'], snapshot_id)
+        self.assertEqual(resp_dict['snapshot']['id'], '123')
 
     def test_snapshot_show_invalid_id(self):
         snapshot_id = 234
@@ -155,7 +154,7 @@ class SnapshotApiTest(test.TestCase):
         self.assertEqual(len(resp_snapshots), 1)
 
         resp_snapshot = resp_snapshots.pop()
-        self.assertEqual(resp_snapshot['id'], 123)
+        self.assertEqual(resp_snapshot['id'], '123')
 
 
 class SnapshotSerializerTest(test.TestCase):
diff --git a/nova/tests/api/openstack/volume/test_types.py b/nova/tests/api/openstack/volume/test_types.py
index e16d853e5..462880443 100644
--- a/nova/tests/api/openstack/volume/test_types.py
+++ b/nova/tests/api/openstack/volume/test_types.py
@@ -112,6 +112,7 @@ class VolumeTypesApiTest(test.TestCase):
         res_dict = self.controller.show(req, 1)
 
         self.assertEqual(1, len(res_dict))
+        self.assertEqual('1', res_dict['volume_type']['id'])
         self.assertEqual('vol_type_1', res_dict['volume_type']['name'])
 
     def test_volume_types_show_not_found(self):
diff --git a/nova/tests/api/openstack/volume/test_volumes.py b/nova/tests/api/openstack/volume/test_volumes.py
index e22e8ac13..9e9b5ff71 100644
--- a/nova/tests/api/openstack/volume/test_volumes.py
+++ b/nova/tests/api/openstack/volume/test_volumes.py
@@ -56,12 +56,12 @@ class VolumeApiTest(test.TestCase):
                                'displayName': 'Volume Test Name',
                                'attachments': [{'device': '/',
                                                 'serverId': 'fakeuuid',
-                                                'id': 1,
-                                                'volumeId': 1}],
+                                                'id': '1',
+                                                'volumeId': '1'}],
                                'volumeType': 'vol_type_name',
                                'snapshotId': None,
                                'metadata': {},
-                               'id': 1,
+                               'id': '1',
                                'createdAt': datetime.datetime(1, 1, 1,
                                                               1, 1, 1),
                                'size': 100}}
@@ -84,12 +84,12 @@ class VolumeApiTest(test.TestCase):
                                  'displayName': 'displayname',
                                  'attachments': [{'device': '/',
                                                   'serverId': 'fakeuuid',
-                                                  'id': 1,
-                                                  'volumeId': 1}],
+                                                  'id': '1',
+                                                  'volumeId': '1'}],
                                  'volumeType': 'vol_type_name',
                                  'snapshotId': None,
                                  'metadata': {},
-                                 'id': 1,
+                                 'id': '1',
                                  'createdAt': datetime.datetime(1, 1, 1,
                                                                 1, 1, 1),
                                  'size': 1}]}
@@ -104,12 +104,12 @@ class VolumeApiTest(test.TestCase):
                                  'displayName': 'displayname',
                                  'attachments': [{'device': '/',
                                                   'serverId': 'fakeuuid',
-                                                  'id': 1,
-                                                  'volumeId': 1}],
+                                                  'id': '1',
+                                                  'volumeId': '1'}],
                                  'volumeType': 'vol_type_name',
                                  'snapshotId': None,
                                  'metadata': {},
-                                 'id': 1,
+                                 'id': '1',
                                  'createdAt': datetime.datetime(1, 1, 1,
                                                                 1, 1, 1),
                                  'size': 1}]}
@@ -124,12 +124,12 @@ class VolumeApiTest(test.TestCase):
                                'displayName': 'displayname',
                                'attachments': [{'device': '/',
                                                 'serverId': 'fakeuuid',
-                                                'id': 1,
-                                                'volumeId': 1}],
+                                                'id': '1',
+                                                'volumeId': '1'}],
                                'volumeType': 'vol_type_name',
                                'snapshotId': None,
                                'metadata': {},
-                               'id': 1,
+                               'id': '1',
                                'createdAt': datetime.datetime(1, 1, 1,
                                                               1, 1, 1),
                                'size': 1}}
diff --git a/nova/tests/integrated/test_volumes.py b/nova/tests/integrated/test_volumes.py
index 8a0721ca1..8a6089595 100644
--- a/nova/tests/integrated/test_volumes.py
+++ b/nova/tests/integrated/test_volumes.py
@@ -118,29 +118,29 @@ class VolumesTest(integrated_helpers._IntegratedTestBase):
 
         create_actions = driver.LoggingVolumeDriver.logs_like(
                             'create_volume',
-                            id=created_volume_id)
+                            id=int(created_volume_id))
         LOG.debug("Create_Actions: %s" % create_actions)
 
         self.assertEquals(1, len(create_actions))
         create_action = create_actions[0]
-        self.assertEquals(create_action['id'], created_volume_id)
+        self.assertEquals(create_action['id'], int(created_volume_id))
         self.assertEquals(create_action['availability_zone'], 'nova')
         self.assertEquals(create_action['size'], 1)
 
         export_actions = driver.LoggingVolumeDriver.logs_like(
                             'create_export',
-                            id=created_volume_id)
+                            id=int(created_volume_id))
         self.assertEquals(1, len(export_actions))
         export_action = export_actions[0]
-        self.assertEquals(export_action['id'], created_volume_id)
+        self.assertEquals(export_action['id'], int(created_volume_id))
         self.assertEquals(export_action['availability_zone'], 'nova')
 
         delete_actions = driver.LoggingVolumeDriver.logs_like(
                             'delete_volume',
-                            id=created_volume_id)
+                            id=int(created_volume_id))
         self.assertEquals(1, len(delete_actions))
         delete_action = export_actions[0]
-        self.assertEquals(delete_action['id'], created_volume_id)
+        self.assertEquals(delete_action['id'], int(created_volume_id))
 
     def test_create_volume_with_metadata(self):
         """Creates a volume with metadata."""