KAB-46 flake8

Author: tomasfejfar

kbcstorage/tables.py

@@ -14,7 +14,6 @@
 from kbcstorage.tables_metadata import TablesMetadata
 
 
-
 class Tables(Endpoint):
     """
     Tables Endpoint

kbcstorage/tables_metadata.py

@@ -7,11 +7,7 @@
     http://docs.keboola.apiary.io/#reference/metadata/table-metadata
 """
 import json
-import tempfile
-import os
 from kbcstorage.base import Endpoint
-from kbcstorage.files import Files
-from kbcstorage.jobs import Jobs
 
 
 class TablesMetadata(Endpoint):
@@ -67,32 +63,32 @@ def delete(self, table_id, metadata_id):
         self._delete(url)
 
     def create(self, table_id, provider, metadata, columns_metadata):
-          """
-          Post metadata to a table.
-
-          Args:
-              table_id (str): Table id
-              provider (str): Provider of the metadata
-              metadata (list): List of metadata dictionaries with 'key' and 'value'
-              columns_metadata (dict): Dictionary with column metadata
-
-          Returns:
-              response_body: The parsed json from the HTTP response.
-
-          Raises:
-              requests.HTTPError: If the API request fails.
-          """
-          if not isinstance(table_id, str) or table_id == '':
-              raise ValueError("Invalid table_id '{}'.".format(table_id))
-
-          url = '{}/{}/metadata'.format(self.base_url, table_id)
-          headers = {
-              'Content-Type': 'application/json',
-              'X-StorageApi-Token': self.token
-          }
-          data = {
-              "provider": provider,
-              "metadata": metadata,
-              "columnsMetadata": columns_metadata
-          }
-          return self._post(url, data=json.dumps(data), headers=headers)
+        """
+        Post metadata to a table.
+
+        Args:
+            table_id (str): Table id
+            provider (str): Provider of the metadata
+            metadata (list): List of metadata dictionaries with 'key' and 'value'
+            columns_metadata (dict): Dictionary with column metadata
+
+        Returns:
+            response_body: The parsed json from the HTTP response.
+
+        Raises:
+            requests.HTTPError: If the API request fails.
+        """
+        if not isinstance(table_id, str) or table_id == '':
+            raise ValueError("Invalid table_id '{}'.".format(table_id))
+
+        url = '{}/{}/metadata'.format(self.base_url, table_id)
+        headers = {
+            'Content-Type': 'application/json',
+            'X-StorageApi-Token': self.token
+        }
+        data = {
+            "provider": provider,
+            "metadata": metadata,
+            "columnsMetadata": columns_metadata
+        }
+        return self._post(url, data=json.dumps(data), headers=headers)

tests/functional/test_tables.py

@@ -5,7 +5,6 @@
 from requests import exceptions
 from kbcstorage.tables import Tables
 from kbcstorage.buckets import Buckets
-from kbcstorage.tables_metadata import TablesMetadata
 from tests.base_test_case import BaseTestCase
 
 
@@ -345,13 +344,13 @@ def test_table_with_metadata(self):
             table_id=table_id,
             provider='test',
             metadata=[{
-                'key' : 'test_table_with_metadata',
-                'value' : 'success'
+                'key': 'test_table_with_metadata',
+                'value': 'success'
                 }],
             columns_metadata={
-                'col1' : [{
-                    'key' : 'test_column_with_metadata',
-                    'value' : 'success'
+                'col1': [{
+                    'key': 'test_column_with_metadata',
+                    'value': 'success'
                     }]
                 })
 
@@ -376,7 +375,7 @@ def test_table_with_metadata(self):
             self.assertIn('timestamp', table_info['columnMetadata']['col1'][0])
             self.assertEqual('success', table_info['columnMetadata']['col1'][0]['value'])
 
-        listedMetadata = self.tables.metadata.list(table_id=table_id);
+        listedMetadata = self.tables.metadata.list(table_id=table_id)
 
         with self.subTest("Test metadata key in list response"):
             self.assertEqual(1, len(listedMetadata))
@@ -386,6 +385,6 @@ def test_table_with_metadata(self):
 
         self.tables.metadata.delete(table_id=table_id, metadata_id=listedMetadata[0]['id'])
 
-        listedMetadata = self.tables.metadata.list(table_id=table_id);
+        listedMetadata = self.tables.metadata.list(table_id=table_id)
         with self.subTest('Test metadata can was deleted'):
-            self.assertEqual(0, len(listedMetadata))
+            self.assertEqual(0, len(listedMetadata))