General cleanup of unused objects

* Remove unused function fake_get_remote_image_service
* Remove unused local function fake_execute2
* stub_out_key_pair_funcs() does nothing, remove it.
* Removed usage of enumerate()

Closes-Bug: #1286714
Closes-Bug: #1286712
Closes-Bug: #1286742
Closes-Bug: #1286809
Change-Id: I76413c886f6903df2adcf4a4ceb8990486a2ff2f

cinder/tests/api/contrib/test_extended_snapshot_attributes.py

@@ -91,7 +91,7 @@ class ExtendedSnapshotAttributesTest(test.TestCase):
         res = self._make_request(url)
 
         self.assertEqual(res.status_int, 200)
-        for i, snapshot in enumerate(self._get_snapshots(res.body)):
+        for snapshot in self._get_snapshots(res.body):
             self.assertSnapshotAttributes(snapshot,
                                           project_id='fake',
                                           progress='0%')

cinder/tests/api/fakes.py

@@ -28,7 +28,6 @@ from cinder.api.v2 import limits
 from cinder.api.v2 import router
 from cinder.api import versions
 from cinder import context
-from cinder import exception as exc
 from cinder.openstack.common import timeutils
 from cinder import wsgi
 
@@ -83,20 +82,6 @@ def wsgi_app(inner_app_v2=None, fake_auth=True, fake_auth_context=None,
     return mapper
 
 
-def stub_out_key_pair_funcs(stubs, have_key_pair=True):
-    def key_pair(context, user_id):
-        return [dict(name='key', public_key='public_key')]
-
-    def one_key_pair(context, user_id, name):
-        if name == 'key':
-            return dict(name='key', public_key='public_key')
-        else:
-            raise exc.KeypairNotFound(user_id=user_id, name=name)
-
-    def no_key_pair(context, user_id):
-        return []
-
-
 class FakeToken(object):
     id_count = 0
 

cinder/tests/api/v1/test_snapshot_metadata.py

@@ -137,7 +137,6 @@ class SnapshotMetaDataTest(test.TestCase):
     def setUp(self):
         super(SnapshotMetaDataTest, self).setUp()
         self.volume_api = cinder.volume.api.API()
-        fakes.stub_out_key_pair_funcs(self.stubs)
         self.stubs.Set(cinder.db, 'volume_get', return_volume)
         self.stubs.Set(cinder.db, 'snapshot_get', return_snapshot)
         self.stubs.Set(cinder.db, 'snapshot_metadata_get',

cinder/tests/api/v1/test_volume_metadata.py

@@ -122,7 +122,6 @@ class volumeMetaDataTest(test.TestCase):
     def setUp(self):
         super(volumeMetaDataTest, self).setUp()
         self.volume_api = cinder.volume.api.API()
-        fakes.stub_out_key_pair_funcs(self.stubs)
         self.stubs.Set(cinder.db, 'volume_get', return_volume)
         self.stubs.Set(cinder.db, 'volume_metadata_get',
                        return_volume_metadata)

cinder/tests/api/v2/test_snapshot_metadata.py

@@ -137,7 +137,6 @@ class SnapshotMetaDataTest(test.TestCase):
     def setUp(self):
         super(SnapshotMetaDataTest, self).setUp()
         self.volume_api = cinder.volume.api.API()
-        fakes.stub_out_key_pair_funcs(self.stubs)
         self.stubs.Set(cinder.db, 'volume_get', return_volume)
         self.stubs.Set(cinder.db, 'snapshot_get', return_snapshot)
         self.stubs.Set(cinder.db, 'snapshot_metadata_get',

cinder/tests/api/v2/test_volume_metadata.py

@@ -123,7 +123,6 @@ class volumeMetaDataTest(test.TestCase):
     def setUp(self):
         super(volumeMetaDataTest, self).setUp()
         self.volume_api = volume_api.API()
-        fakes.stub_out_key_pair_funcs(self.stubs)
         self.stubs.Set(db, 'volume_get', return_volume)
         self.stubs.Set(db, 'volume_metadata_get',
                        return_volume_metadata)

cinder/tests/brick/test_brick_linuxscsi.py

@@ -106,15 +106,6 @@ class LinuxSCSITestCase(test.TestCase):
                    )
             return out, None
 
-        def fake_execute2(*cmd, **kwargs):
-            out = ("350002ac20398383d dm-3 3PARdata,VV\n"
-                   "size=2.0G features='0' hwhandler='0' wp=rw\n"
-                   "`-+- policy='round-robin 0' prio=-1 status=active\n"
-                   "  |- 0:0:0:1  sde 8:64 active undef running\n"
-                   "  `- 2:0:0:1  sdf 8:80 active undef running\n"
-                   )
-            return out, None
-
         self.stubs.Set(self.linuxscsi, '_execute', fake_execute)
 
         info = self.linuxscsi.find_multipath_device('/dev/sde')

cinder/tests/image/fake.py

@@ -229,8 +229,6 @@ def FakeImageService_reset():
 
 
 def stub_out_image_service(stubs):
-    def fake_get_remote_image_service(context, image_href):
-        return (FakeImageService(), image_href)
     stubs.Set(cinder.image.glance, 'get_remote_image_service',
               lambda x, y: (FakeImageService(), y))
     stubs.Set(cinder.image.glance, 'get_default_image_service',