Address flake8 lint errors

Author: achilleasa

examples/expose-application.py

@@ -12,7 +12,6 @@
 from juju.model import Model
 from juju.application import ExposedEndpoint
 
-import logging
 
 async def main():
     model = Model()

juju/application.py

@@ -341,19 +341,16 @@ async def unexpose(self, exposed_endpoints=None):
             if facade_version < 13:
                 raise JujuError("controller does not support granular expose parameters; applying this change would unexpose the application")
 
-            log.debug("Unexposing endpoints %s of %s",
-                      ",".join(exposed_endpoints), self.name)
+            log.debug("Unexposing endpoints %s of %s", ",".join(exposed_endpoints), self.name)
             return await app_facade.Unexpose(application=self.name,
                                              exposed_endpoints=exposed_endpoints)
 
         # Just expose the entire application
         log.debug("Unexposing %s", self.name)
         return await app_facade.Unexpose(application=self.name)
 
-
     async def get_config(self):
         """Return the configuration settings dict for this application.
-
         """
         app_facade = self._facade()
 

tests/unit/test_application.py

@@ -94,28 +94,28 @@ async def test_expose_endpoints_on_older_controller(self, mock_conn):
         # Case 1: exposed_endpoints includes an entry with a space list.
         with self.assertRaises(JujuError):
             await app.expose(exposed_endpoints={
-             "": ExposedEndpoint(to_spaces=["alpha"]),
+                "": ExposedEndpoint(to_spaces=["alpha"]),
             })
 
         # Case 2: exposed_endpoints only includes the wildcard endpoints key
         # with a non-wildcard CIDR.
         with self.assertRaises(JujuError):
             await app.expose(exposed_endpoints={
-             "": ExposedEndpoint(to_cidrs=["0.0.0.0/0", "10.0.0.0/24"]),
+                "": ExposedEndpoint(to_cidrs=["0.0.0.0/0", "10.0.0.0/24"]),
             })
 
         # Case 3: exposed_endpoints has a single entry for the
         # non-wildcard endpoint.
         with self.assertRaises(JujuError):
             await app.expose(exposed_endpoints={
-             "": ExposedEndpoint(to_cidrs=["0.0.0.0/0", "10.0.0.0/24"]),
+                "": ExposedEndpoint(to_cidrs=["0.0.0.0/0", "10.0.0.0/24"]),
             })
 
         # Case 4: exposed_endpoints has multiple keys.
         with self.assertRaises(JujuError):
             await app.expose(exposed_endpoints={
-             "foo": ExposedEndpoint(to_cidrs=["0.0.0.0/0"]),
-             "bar": ExposedEndpoint(to_spaces=["alpha"]),
+                "foo": ExposedEndpoint(to_cidrs=["0.0.0.0/0"]),
+                "bar": ExposedEndpoint(to_spaces=["alpha"]),
             })
 
         # Check that we call the facade with the right arity.
@@ -139,7 +139,7 @@ async def test_unexpose_endpoints_on_older_controller(self, mock_conn):
         # If we try to unexpose individual endpoints on an older controller
         # (app facade < 13) we should get an error back.
         with self.assertRaises(JujuError):
-            await app.unexpose(exposed_endpoints=["outer","inner"])
+            await app.unexpose(exposed_endpoints=["outer", "inner"])
 
         # Check that we call the facade with the right arity.
         await app.unexpose()

tests/unit/test_bundle.py

@@ -634,14 +634,14 @@ def test_dict_params_with_exposed_endpoints_data(self):
                           "requires": [],
                           "application": "application",
                           "exposed_endpoints": {
-                            "": {
-                                "to-spaces": ["alpha"],
-                                "to-cidrs": ["10.0.0.0/24"]
-                            },
-                            "foo": {
-                                "to-spaces": ["alien"],
-                                "to-cidrs": ["0.0.0.0/0", "::/0"]
-                            }
+                              "": {
+                                  "to-spaces": ["alpha"],
+                                  "to-cidrs": ["10.0.0.0/24"]
+                              },
+                              "foo": {
+                                  "to-spaces": ["alien"],
+                                  "to-cidrs": ["0.0.0.0/0", "::/0"]
+                              }
                           }}, change.__dict__)
 
 
@@ -678,14 +678,14 @@ async def test_run(self, event_loop):
 
         model.applications["application1"].expose.assert_called_once()
         model.applications["application1"].expose.assert_called_with({
-                "": {
-                    "to-spaces": ["alpha"],
-                    "to-cidrs": ["10.0.0.0/24"]
-                },
-                "foo": {
-                    "to-spaces": ["alien"],
-                    "to-cidrs": ["0.0.0.0/0", "::/0"]
-                }
+            "": {
+                "to-spaces": ["alpha"],
+                "to-cidrs": ["10.0.0.0/24"]
+            },
+            "foo": {
+                "to-spaces": ["alien"],
+                "to-cidrs": ["0.0.0.0/0", "::/0"]
+            }
         })