Fix black and mypy warnings

Author: Michal Ploski

Diff for: aws_lambda_powertools/event_handler/api_gateway.py

@@ -836,7 +836,6 @@ def route(
     # Override _compile_regex to exclude trailing slashes for route resolution
     @staticmethod
     def _compile_regex(rule: str, base_regex: str = _ROUTE_REGEX):
-
         return super(APIGatewayRestResolver, APIGatewayRestResolver)._compile_regex(rule, "^{}/*$")
 
 

Diff for: aws_lambda_powertools/logging/utils.py

@@ -12,7 +12,6 @@ def copy_config_to_registered_loggers(
     exclude: Optional[Set[str]] = None,
     include: Optional[Set[str]] = None,
 ) -> None:
-
     """Copies source Logger level and handler to all registered loggers for consistent formatting.
 
     Parameters

Diff for: aws_lambda_powertools/utilities/feature_flags/schema.py

@@ -272,7 +272,6 @@ def __init__(self, rule: Dict[str, Any], rule_name: str, logger: Optional[Union[
         self.logger = logger or logging.getLogger(__name__)
 
     def validate(self):
-
         if not self.conditions or not isinstance(self.conditions, list):
             self.logger.debug(f"Condition is empty or invalid for rule={self.rule_name}")
             raise SchemaValidationError(f"Invalid condition, rule={self.rule_name}")

Diff for: aws_lambda_powertools/utilities/idempotency/persistence/dynamodb.py

@@ -154,7 +154,7 @@ def _put_record(self, data_record: DataRecord) -> None:
         if data_record.in_progress_expiry_timestamp is not None:
             item[self.in_progress_expiry_attr] = {"N": str(data_record.in_progress_expiry_timestamp)}
 
-        if self.payload_validation_enabled:
+        if self.payload_validation_enabled and data_record.payload_hash:
             item[self.validation_key_attr] = {"S": data_record.payload_hash}
 
         now = datetime.datetime.now()

Diff for: tests/e2e/idempotency/handlers/parallel_execution_handler.py

@@ -12,7 +12,6 @@
 
 @idempotent(persistence_store=persistence_layer)
 def lambda_handler(event, context):
-
     time.sleep(5)
 
     return event

Diff for: tests/e2e/idempotency/handlers/ttl_cache_expiration_handler.py

@@ -14,7 +14,6 @@
 
 @idempotent(config=config, persistence_store=persistence_layer)
 def lambda_handler(event, context):
-
     time_now = time.time()
 
     return {"time": str(time_now)}

Diff for: tests/e2e/idempotency/handlers/ttl_cache_timeout_handler.py

@@ -14,7 +14,6 @@
 
 @idempotent(config=config, persistence_store=persistence_layer)
 def lambda_handler(event, context):
-
     sleep_time: int = event.get("sleep") or 0
     time.sleep(sleep_time)
 

Diff for: tests/e2e/parameters/infrastructure.py

@@ -34,7 +34,6 @@ def create_resources(self):
         )
 
     def _create_app_config(self, function: Function):
-
         service_name = build_service_name()
 
         cfn_application = appconfig.CfnApplication(
@@ -82,7 +81,6 @@ def _create_app_config_freeform(
         function: Function,
         service_name: str,
     ):
-
         cfn_configuration_profile = appconfig.CfnConfigurationProfile(
             self.stack,
             "appconfig-profile",

Diff for: tests/functional/feature_flags/test_feature_flags.py

@@ -315,6 +315,7 @@ def test_flags_conditions_rule_match_multiple_actions_multiple_rules_multiple_co
 
 # check a case where the feature exists but the rule doesn't match so we revert to the default value of the feature
 
+
 # Check IN/NOT_IN/KEY_IN_VALUE/KEY_NOT_IN_VALUE/VALUE_IN_KEY/VALUE_NOT_IN_KEY conditions
 def test_flags_match_rule_with_in_action(mocker, config):
     expected_value = True
@@ -775,6 +776,7 @@ def test_get_configuration_with_envelope_and_raw(mocker, config):
 ## Inequality test cases
 ##
 
+
 # Test not equals
 def test_flags_not_equal_no_match(mocker, config):
     expected_value = False