parsedmarc

amorfo77/parsedmarc

Fork 0

mirror of https://github.com/domainaware/parsedmarc.git synced 2026-07-22 08:24:55 +00:00

Files

T

History

4fa8cfb1e7 Skip the results email when no reports were parsed (#837 )

* Skip the results email when no reports were parsed (#200)

The [smtp] (and Microsoft Graph) results email was sent unconditionally
whenever the transport was configured, so an empty run — an empty inbox,
or one where every message was invalid — still emailed a zip of
headers-only CSVs. The email step is now skipped with an INFO log when
the run produced no aggregate, failure, or SMTP TLS reports.

The regression test was verified to fail against the unfixed code
(send_email was called once with a headers-only zip). Six existing tests
that asserted on the email path with all-empty mocked results now feed a
real parsed sample aggregate report through the actual zip/CSV-building
code instead.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

* Add Graph-path regression test for the empty-run email guard

Copilot review on #837 pointed out that only the SMTP transport had a
skip regression test, so a refactor narrowing the guard to the SMTP
branch could silently reintroduce headers-only zips via Microsoft
Graph. The new test was verified to fail against exactly that
narrowing (send_message called once with a headers-only zip).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>

---------

Co-authored-by: Claude Fable 5 <noreply@anthropic.com>

2026-07-21 20:49:41 -04:00

__init__.py

Split tests.py into per-module tests/test_<module>.py (#774 )

2026-05-20 19:29:09 -04:00

test_cli.py

Skip the results email when no reports were parsed (#837 )

2026-07-21 20:49:41 -04:00

test_elastic.py

Migrate Elasticsearch output to the elasticsearch-py 8.x client (#822 )

2026-07-13 13:50:25 -04:00

test_gelf.py

Make the whole codebase pass pyright cleanly and enforce it in CI (#798 )