Repository navigation
test(providers): record live Mailchimp report responses for M6 - #183
Open
TheAggressive wants to merge 1 commit into
Open
TheAggressive wants to merge 1 commit into
TheAggressive wants to merge 1 commit into
Conversation
Captured on 2026-10-17 from the live account before the subscription ends, so normalized reporting (#67) can be built and tested without it: aggregate report endpoints for a campaign that was sent, the report of an unsent draft, a missing report, and audience stats. Member-level endpoints were not called; IDs, names, subjects, sender details, and the recipient domain are replaced with placeholders. The recording shows delivery_status "delivered" for a send whose only recipient soft-bounced, so "delivered" means the send finished, not inbox delivery. A guard test now proves every recorded Mailchimp fixture states its source and contains no email address or credential. Part of #67. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Part of #67 (M6). These responses were recorded from the live Mailchimp account before its subscription ends, so normalized reporting can be built and tested without the account.
What this adds
tests/Fixtures/Mailchimp/reports.jsonrecords read-only, aggregate endpoints:200, with nothing sent);404error shape);example.com, list names, subjects, and sender details are{{field}}placeholders, and links are removed.Mailchimp_Fixture_Privacy_Testchecks every recorded Mailchimp fixture: it must state its source and contain no email address or credential. It also pins the aggregate shapes that reporting will rely on.Findings for #67
delivery_status.statussaysdeliveredeven when the only recipient soft-bounced. It means the send finished, not that mail reached an inbox, so reporting must not show it as inbox delivery.proxy_excluded_*(Apple Mail Privacy Protection). Reporting should say which one it shows.200report with zero counts. Reporting should not take that as a sent campaign.Evidence
Mailchimp_Fixture_Privacy_Testpasses (4 tests), and Prettier is clean.🤖 Generated with Claude Code