Configure linter

Author: Janos Tolgyesi

dynamodb_mapping/dynamodb_mapping.py

@@ -18,7 +18,7 @@
     import mypy_boto3_dynamodb
     DynamoDBTable = mypy_boto3_dynamodb.service_resource.Table
 except ImportError:
-    DynamoDBTable = Any # type: ignore
+    DynamoDBTable = Any  # type: ignore
 
 DynamoDBKeyPrimitiveTypes = (str, bytes, bytearray, int, Decimal)
 """DynamoDB primary key primitive choices."""
@@ -49,7 +49,7 @@
 """DynamoDB value type choices."""
 
 DynamoDBValue = Union[
-    bytes, bytearray, str, int, Decimal,bool,
+    bytes, bytearray, str, int, Decimal, bool,
     Set[int], Set[Decimal], Set[str], Set[bytes], Set[bytearray],
     Sequence[Any], Mapping[str, Any], None,
 ]
@@ -60,6 +60,7 @@
 
 logger = logging.getLogger(__name__)
 
+
 def _boto3_session_from_config(config: Dict[str, Any]) -> Optional[boto3.Session]:
     if "aws_access_key_id" in config and "aws_secret_access_key" in config:
         return boto3.Session(
@@ -71,6 +72,7 @@ def _boto3_session_from_config(config: Dict[str, Any]) -> Optional[boto3.Session
     else:
         return None
 
+
 def get_key_names(table: DynamoDBTable) -> DynamoDBKeyName:
     """Gets the key names of the DynamoDB table.
 
@@ -121,6 +123,7 @@ def create_tuple_keys(key: DynamoDBKeySimplified) -> DynamoDBKeyAny:
     else:
         return cast(DynamoDBKeySimple, (key,))
 
+
 def _log_keys_from_params(key_params: Dict[str, DynamoDBKeyPrimitive]) -> str:
     log_keys = list(key_params.values())
     res = log_keys[0] if len(log_keys) == 1 else log_keys
@@ -194,7 +197,8 @@ class DynamoDBItemAccessor(dict):
         initial_data (Dict): The initial item data.
     """
 
-    def __init__(self,
+    def __init__(
+        self,
         parent: "DynamoDBMapping",
         item_keys: DynamoDBKeySimplified,
         initial_data: DynamoDBItemType,
@@ -268,7 +272,8 @@ class DynamoDBMapping(MutableMapping):
     the values are one of the `permitted DynamoDB value types`_. The ``DynamoDBItemType`` type
     reflects the possible item types.
 
-    .. _permitted DynamoDB value types: https://boto3.amazonaws.com/v1/documentation/api/latest/reference/customizations/dynamodb.html
+    .. _permitted DynamoDB value types: \
+        https://boto3.amazonaws.com/v1/documentation/api/latest/reference/customizations/dynamodb.html
 
     Args:
         table_name: The name of the DynamoDB table.
@@ -278,7 +283,7 @@ class DynamoDBMapping(MutableMapping):
     """
 
     def __init__(
-        self, table_name: str, boto3_session: Optional[boto3.session.Session]=None, **kwargs
+        self, table_name: str, boto3_session: Optional[boto3.session.Session] = None, **kwargs
     ) -> None:
         session = (
             boto3_session or
@@ -294,7 +299,7 @@ def _create_key_param(self, keys: DynamoDBKeySimplified) -> Dict[str, DynamoDBKe
         tuple_keys = create_tuple_keys(keys)
         if len(tuple_keys) != len(self.key_names):
             raise ValueError(f"You must provide a value for each of {self.key_names} keys.")
-        param = { name: value for name, value in zip(self.key_names, tuple_keys) }
+        param = {name: value for name, value in zip(self.key_names, tuple_keys)}
         return param
 
     def scan(self, **kwargs) -> Iterator[DynamoDBItemType]:
@@ -356,7 +361,7 @@ def get_item(self, keys: DynamoDBKeySimplified, **kwargs) -> DynamoDBItemAccesso
         key_params = self._create_key_param(keys)
         logger.debug("Performing a get_item operation on %s table", self.table.name)
         response = self.table.get_item(Key=key_params, **kwargs)
-        if not "Item" in response:
+        if "Item" not in response:
             raise KeyError(_log_keys_from_params(key_params))
         data = response["Item"]
         return DynamoDBItemAccessor(parent=self, item_keys=keys, initial_data=data)
@@ -404,12 +409,13 @@ def del_item(self, keys: DynamoDBKeySimplified, check_existing=True, **kwargs) -
                 :meth:`~DynamoDBTable.delete_item` operation.
         """
         key_params = self._create_key_param(keys)
-        if check_existing and not keys in self.keys():
+        if check_existing and keys not in self.keys():
             raise KeyError(_log_keys_from_params(key_params))
         logger.debug("Performing a delete_item operation on %s table", self.table.name)
         self.table.delete_item(Key=key_params, **kwargs)
 
-    def modify_item(self,
+    def modify_item(
+        self,
         keys: DynamoDBKeySimplified,
         modifications: DynamoDBItemType,
         **kwargs

setup.cfg

@@ -16,5 +16,7 @@ universal = 1
 
 [flake8]
 exclude = docs
+max-line-length = 100
+
 [tool:pytest]
 collect_ignore = ['setup.py']

tests/test_dynamodb_mapping.py

@@ -5,8 +5,9 @@
 import pytest
 
 
-from dynamodb_mapping import dynamodb_mapping
+# from dynamodb_mapping import DynamoDBMapping
 
+# mapping = DynamoDBMapping("foobar")
 
 @pytest.fixture
 def response():

tox.ini

@@ -23,4 +23,3 @@ deps =
 commands =
     pip install -U pip
     pytest --basetemp={envtmpdir}
-