Skip to content

Commit 67c9fe8

Browse files
committed
chore: Add comments to justify missing testing coverage
1 parent 1ea19aa commit 67c9fe8

File tree

1 file changed

+5
-0
lines changed

1 file changed

+5
-0
lines changed

src/main.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -400,6 +400,9 @@ def handler(event: dict, context) -> str: # pylint: disable=unused-argument, to
400400

401401
# Toggle local logging
402402
if show_log_locally:
403+
# This is a nightmare to test as it's really hard to get to.
404+
# At some point we should look to make a wrapper for logging
405+
# so it can be tested more easily.
403406
logging.basicConfig(
404407
filename="debug.log",
405408
filemode="w",
@@ -454,6 +457,8 @@ def handler(event: dict, context) -> str: # pylint: disable=unused-argument, to
454457
updated_team_history = create_dictionary(gh, copilot_teams, existing_team_history)
455458

456459
# Write updated team history to S3
460+
# This line isn't covered by tests as it's painful to get to.
461+
# The function itself is tested though.
457462
update_s3_object(s3, BUCKET_NAME, "teams_history.json", updated_team_history)
458463
else:
459464
local_path = "output/teams_history.json"

0 commit comments

Comments
 (0)