Lint clean-up, testing fixes

backend/bin/run_tests.sh

@@ -3,12 +3,13 @@ REPORT="$1"
 
 # Run CDK tests, tracking code coverage in a new data file
 pytest --cov=. --cov-config=.coveragerc tests || exit "$?"
-for dir in lambdas delete-objects-lambda
-(
-  cd "$dir"
-  # Run lambda tests, appending data to the same data file
-  pytest --cov=. --cov-config=.coveragerc --cov-append tests
-) || exit "$?"
+for dir in lambdas delete-objects-lambda; do
+  (
+    cd "$dir"
+    # Run lambda tests, appending data to the same data file
+    pytest --cov=. --cov-config=.coveragerc --cov-append tests
+  ) || exit "$?"
+done
 
 # Run a coverage report with the combined data
 coverage html --fail-under=90

backend/common_constructs/python_function.py

@@ -23,7 +23,7 @@ def __init__(
         }
         defaults.update(kwargs)
 
-        super().__init__(
+        super().__init__(  # pylint: disable=missing-kwoa
             scope, construct_id,
             bundling=BundlingOptions(command_hooks=TestingHooks()),
             runtime=Runtime.PYTHON_3_12,

backend/delete-objects-lambda/.coveragerc

@@ -0,0 +1,12 @@
+[run]
+include =
+  **/*.py
+data_file = ../.coverage
+
+omit =
+  */cdk.out/*
+  */smoke-test/*
+  */tests/*
+
+[report]
+skip_empty = true

backend/lambdas/tests/function/__init__.py

@@ -21,7 +21,7 @@ class TstFunction(TstLambdas):
     Base class to set up Moto mocking and create mock AWS resources for functional testing
     """
 
-    def setUp(self):
+    def setUp(self):  # pylint: disable=invalid-name
         super().setUp()
 
         self._os_patch = patch.dict(os.environ, {

backend/stacks/persistent_stack/__init__.py

@@ -41,7 +41,7 @@ def __init__(
             access_logs_bucket=self.access_logs_bucket,
             encryption_key=self.shared_encryption_key,
             removal_policy=removal_policy,
-            auto_delete_objects=False if environment_name == 'prod' else True
+            auto_delete_objects=environment_name != 'prod'
         )
 
         self.board_users = BoardUsers(