Blackify openstack.placement

Black used with the '-l 79 -S' flags.

A future change will ignore this commit in git-blame history by adding a
'git-blame-ignore-revs' file.

Change-Id: If63b878036e2af53cfb4bc2ff29872253ba21597
Signed-off-by: Stephen Finucane <stephenfin@redhat.com>

openstack/placement/placement_service.py

@@ -16,6 +16,7 @@ from openstack import service_description
 
 class PlacementService(service_description.ServiceDescription):
     """The placement service."""
+
     supported_versions = {
         '1': _proxy.Proxy,
     }

openstack/placement/v1/_proxy.py

@@ -70,7 +70,9 @@ class Proxy(proxy.Proxy):
         :rtype: :class:`~openstack.placement.v1.resource_class.ResourceClass`
         """
         return self._update(
-            _resource_class.ResourceClass, resource_class, **attrs,
+            _resource_class.ResourceClass,
+            resource_class,
+            **attrs,
         )
 
     def get_resource_class(self, resource_class):
@@ -87,7 +89,8 @@ class Proxy(proxy.Proxy):
             resource class matching the criteria could be found.
         """
         return self._get(
-            _resource_class.ResourceClass, resource_class,
+            _resource_class.ResourceClass,
+            resource_class,
         )
 
     def resource_classes(self, **query):
@@ -149,7 +152,9 @@ class Proxy(proxy.Proxy):
         :rtype: :class:`~openstack.placement.v1.resource_provider.ResourceProvider`
         """  # noqa: E501
         return self._update(
-            _resource_provider.ResourceProvider, resource_provider, **attrs,
+            _resource_provider.ResourceProvider,
+            resource_provider,
+            **attrs,
         )
 
     def get_resource_provider(self, resource_provider):
@@ -166,7 +171,8 @@ class Proxy(proxy.Proxy):
             resource provider matching the criteria could be found.
         """
         return self._get(
-            _resource_provider.ResourceProvider, resource_provider,
+            _resource_provider.ResourceProvider,
+            resource_provider,
         )
 
     def find_resource_provider(self, name_or_id, ignore_missing=True):

openstack/placement/v1/resource_provider.py

@@ -29,7 +29,12 @@ class ResourceProvider(resource.Resource):
     # Filters
 
     _query_mapping = resource.QueryParameters(
-        'name', 'member_of', 'resources', 'in_tree', 'required', id='uuid',
+        'name',
+        'member_of',
+        'resources',
+        'in_tree',
+        'required',
+        id='uuid',
     )
 
     # The parent_provider_uuid and root_provider_uuid fields were introduced in

openstack/tests/functional/placement/v1/test_resource_provider.py

@@ -15,7 +15,6 @@ from openstack.tests.functional import base
 
 
 class TestResourceProvider(base.BaseFunctionalTest):
-
     def setUp(self):
         super().setUp()
         self._set_operator_cloud(interface='admin')
@@ -29,7 +28,8 @@ class TestResourceProvider(base.BaseFunctionalTest):
 
     def tearDown(self):
         sot = self.conn.placement.delete_resource_provider(
-            self._resource_provider)
+            self._resource_provider
+        )
         self.assertIsNone(sot)
         super().tearDown()
 
@@ -39,7 +39,8 @@ class TestResourceProvider(base.BaseFunctionalTest):
 
     def test_get(self):
         sot = self.conn.placement.get_resource_provider(
-            self._resource_provider.id)
+            self._resource_provider.id
+        )
         self.assertEqual(self.NAME, sot.name)
 
     def test_list(self):

openstack/tests/unit/placement/v1/test_proxy.py

@@ -17,7 +17,6 @@ from openstack.tests.unit import test_proxy_base as test_proxy_base
 
 
 class TestPlacementProxy(test_proxy_base.TestProxyBase):
-
     def setUp(self):
         super().setUp()
         self.proxy = _proxy.Proxy(self.session)

openstack/tests/unit/placement/v1/test_resource_class.py

@@ -19,7 +19,6 @@ FAKE = {
 
 
 class TestResourceClass(base.TestCase):
-
     def test_basic(self):
         sot = rc.ResourceClass()
         self.assertEqual(None, sot.resource_key)
@@ -33,8 +32,8 @@ class TestResourceClass(base.TestCase):
         self.assertFalse(sot.allow_patch)
 
         self.assertDictEqual(
-            {'limit': 'limit', 'marker': 'marker'},
-            sot._query_mapping._mapping)
+            {'limit': 'limit', 'marker': 'marker'}, sot._query_mapping._mapping
+        )
 
     def test_make_it(self):
         sot = rc.ResourceClass(**FAKE)

openstack/tests/unit/placement/v1/test_resource_provider.py

@@ -21,7 +21,6 @@ FAKE = {
 
 
 class TestResourceProvider(base.TestCase):
-
     def test_basic(self):
         sot = rp.ResourceProvider()
         self.assertEqual(None, sot.resource_key)
@@ -45,12 +44,14 @@ class TestResourceProvider(base.TestCase):
                 'required': 'required',
                 'id': 'uuid',
             },
-            sot._query_mapping._mapping)
+            sot._query_mapping._mapping,
+        )
 
     def test_make_it(self):
         sot = rp.ResourceProvider(**FAKE)
         self.assertEqual(FAKE['uuid'], sot.id)
         self.assertEqual(FAKE['name'], sot.name)
         self.assertEqual(
-            FAKE['parent_provider_uuid'], sot.parent_provider_id,
+            FAKE['parent_provider_uuid'],
+            sot.parent_provider_id,
         )