test: cover ComplianceRegistry.all() and stop excluding index.ts from coverage (fixes #45)#58
Conversation
alphacrack
left a comment
There was a problem hiding this comment.
This is the strongest PR of the set — the toContainEqual(S3BucketCompliance.report()) sync assertion is exactly the drift-catcher #45 asked for, the per-entry shape checks will hold future resources to the same bar, and removing the !src/**/index.ts coverage exclusion closes the blind spot (note it also un-hides the resources/*/index.ts barrels, so thanks for confirming the suite still passes with coverage collected there).
|
Re-checked against the new main: MERGEABLE/CLEAN — including One sequencing note: this branch is stacked above #57, so its diff currently carries #57's |
Adds tests asserting ComplianceRegistry.all() returns in-sync S3 reports with valid controls, and removes the index.ts exclusion from jest coverage collection (now 100% line coverage).