chore: fix test failure (#797)

* chore: fix test failure

This error was introduced in https://github.com/amundsen-io/amundsenfrontendlibrary/commit/0b47694ea74cbbef34e03eb45f29643b16a1332a

I do not understand why that changeset passed CI (it shouldn't have), but
once it landed into master it didn't. But regardless, fix
Signed-off-by: Dorian Johnson <2020@dorianj.net>

* chore: fix lint errors
Signed-off-by: Dorian Johnson <2020@dorianj.net>

amundsen_application/api/metadata/v0.py

@@ -14,8 +14,8 @@ from amundsen_application.log.action_log import action_logging
 
 from amundsen_application.models.user import load_user, dump_user
 
-from amundsen_application.api.utils.metadata_utils import is_table_editable, marshall_table_partial,\
-    marshall_table_full,marshall_dashboard_partial, marshall_dashboard_full, TableUri
+from amundsen_application.api.utils.metadata_utils import is_table_editable, marshall_table_partial, \
+    marshall_table_full, marshall_dashboard_partial, marshall_dashboard_full, TableUri
 from amundsen_application.api.utils.request_utils import get_query_param, request_metadata
 
 

amundsen_application/api/utils/metadata_utils.py

@@ -83,7 +83,7 @@ def _parse_editable_rule(rule: MatchRuleObject,
     return True
 
 
-def is_table_editable(schema_name: str, table_name: str, cfg: any=None) -> bool:
+def is_table_editable(schema_name: str, table_name: str, cfg: Any = None) -> bool:
     if cfg is None:
         cfg = app.config
 
@@ -109,7 +109,8 @@ def marshall_table_full(table_dict: Dict) -> Dict:
     table: Table = schema.load(table_dict).data
     results: Dict[str, Any] = schema.dump(table).data
 
-    results['is_editable'] = is_table_editable(results['schema'], results['name'])
+    is_editable = is_table_editable(results['schema'], results['name'])
+    results['is_editable'] = is_editable
 
     # TODO - Cleanup https://github.com/lyft/amundsen/issues/296
     #  This code will try to supplement some missing data since the data here is incomplete.

tests/unit/api/metadata/test_v0.py

@@ -747,7 +747,6 @@ class MetadataTest(unittest.TestCase):
             )
             self.assertEqual(response.status_code, HTTPStatus.OK)
 
-
     @responses.activate
     def test_put_column_description_denied(self) -> None:
         """

tests/unit/utils/test_metadata_utils.py

@@ -178,7 +178,6 @@ class TableEditabilityWrapper(unittest.TestCase):
         self.assertTrue(is_table_editable('not_uneditable_schema', 'anytable', mockConfig))
         self.assertFalse(is_table_editable('uneditable_schema', 'anytable', mockConfig))
 
-
     def test_schema_table_match_rule(self) -> None:
         mockConfig = {
             'UNEDITABLE_SCHEMAS': [''],