Updates based on discussion in PR #206

* Remove lambda-related Makefile commands
* Replace caplog.at_level with caplog.set_level in CLI test
* Remove coverage from Pipfile

Makefile

@@ -69,11 +69,6 @@ publish-dev: dist-dev ## Build, tag and push (intended for developer-based manua
 	docker push $(ECR_URL_DEV):latest
 	docker push $(ECR_URL_DEV):`git describe --always`
 
-### If this is a Lambda repo, uncomment the two lines below     ###
-# update-lambda-dev: ## Updates the lambda with whatever is the most recent image in the ecr (intended for developer-based manual update)
-#	aws lambda update-function-code --function-name $(FUNCTION_DEV) --image-uri $(ECR_URL_DEV):latest
-
-
 ### Terraform-generated manual shortcuts for deploying to Stage. This requires  ###
 ###   that ECR_NAME_STAGE, ECR_URL_STAGE, and FUNCTION_STAGE environment        ###
 ###   variables are set locally by the developer and that the developer has     ###
@@ -90,9 +85,5 @@ publish-stage: ## Only use in an emergency
 	docker push $(ECR_URL_STAGE):latest
 	docker push $(ECR_URL_STAGE):`git describe --always`
 
-### If this is a Lambda repo, uncomment the two lines below     ###
-# update-lambda-stage: ## Updates the lambda with whatever is the most recent image in the ecr (intended for developer-based manual update)
-#	aws lambda update-function-code --function-name $(FUNCTION_STAGE) --image-uri $(ECR_URL_STAGE):latest
-
 run-dev: # Run in dev against Alma sandbox
 	aws ecs run-task --cluster alma-integrations-creditcardslips-ecs-dev --task-definition alma-integrations-creditcardslips-ecs-dev --launch-type="FARGATE" --network-configuration '{ "awsvpcConfiguration": {"subnets": ["subnet-0488e4996ddc8365b", "subnet-022e9ea19f5f93e65"],"securityGroups": ["sg-095372030a26c7753"],"assignPublicIp": "DISABLED"}}'

Pipfile

@@ -12,7 +12,6 @@ sentry-sdk = "*"
 [dev-packages]
 black = "*"
 boto3-stubs = "*"
-coverage = "*"
 coveralls = "*"
 freezegun = "*"
 moto = "*"

tests/test_cli.py

@@ -18,31 +18,30 @@ def test_cli_options_from_env(caplog, runner):
 
 
 def test_cli_all_options_passed(caplog, runner):
-    with caplog.at_level(logging.DEBUG):
-        result = runner.invoke(
-            main,
-            [
-                "--source-email",
-                "[email protected]",
-                "--recipient-email",
-                "[email protected]",
-                "--recipient-email",
-                "[email protected]",
-                "--date",
-                "2023-01-02",
-                "--log-level",
-                "debug",
-            ],
-        )
-        assert result.exit_code == 0
-        assert "Logger 'root' configured with level=DEBUG" in caplog.text
-        assert (
-            "Command called with options: {'source_email': '[email protected]', "
-            "'recipient_email': ('[email protected]', '[email protected]'), "
-            "'date': '2023-01-02', 'log_level': 'debug'}" in caplog.text
-        )
-        assert (
-            "Credit card slips processing complete for date 2023-01-02. Email sent to "
-            "recipient(s) ('[email protected]', '[email protected]')"
-            in caplog.text
-        )
+    caplog.set_level(logging.DEBUG)
+    result = runner.invoke(
+        main,
+        [
+            "--source-email",
+            "[email protected]",
+            "--recipient-email",
+            "[email protected]",
+            "--recipient-email",
+            "[email protected]",
+            "--date",
+            "2023-01-02",
+            "--log-level",
+            "debug",
+        ],
+    )
+    assert result.exit_code == 0
+    assert "Logger 'root' configured with level=DEBUG" in caplog.text
+    assert (
+        "Command called with options: {'source_email': '[email protected]', "
+        "'recipient_email': ('[email protected]', '[email protected]'), "
+        "'date': '2023-01-02', 'log_level': 'debug'}" in caplog.text
+    )
+    assert (
+        "Credit card slips processing complete for date 2023-01-02. Email sent to "
+        "recipient(s) ('[email protected]', '[email protected]')" in caplog.text
+    )