Summary
MeterReadingControllerTest.shouldReturn200WhenExists exhibits Spring-context-bootstrap flakiness: it failed once in CI on PR #116 and passed on retry with no code change.
Distinct from #78 (cross-platform timestamp-resolution flakiness) — this one is context-bootstrap timing, not timestamp precision.
Action
- Reproduce (run the test repeatedly, e.g.
mvn test -pl openespi-datacustodian -Dtest=MeterReadingControllerTest#shouldReturn200WhenExists -Dsurefire.rerunFailingTestsCount=0 in a loop).
- Identify the bootstrap race — likely shared mutable context state,
@MockBean re-init, or an ordering dependency between tests in the class.
- Fix the root cause (proper test isolation /
@DirtiesContext only if unavoidable), not by adding blanket retries.
Acceptance criteria
Source
PR_116_FOLLOWUP_NOTES.md item 4 (post-PR #116, 2026-05-19).
Summary
MeterReadingControllerTest.shouldReturn200WhenExistsexhibits Spring-context-bootstrap flakiness: it failed once in CI on PR #116 and passed on retry with no code change.Distinct from #78 (cross-platform timestamp-resolution flakiness) — this one is context-bootstrap timing, not timestamp precision.
Action
mvn test -pl openespi-datacustodian -Dtest=MeterReadingControllerTest#shouldReturn200WhenExists -Dsurefire.rerunFailingTestsCount=0in a loop).@MockBeanre-init, or an ordering dependency between tests in the class.@DirtiesContextonly if unavoidable), not by adding blanket retries.Acceptance criteria
Source
PR_116_FOLLOWUP_NOTES.mditem 4 (post-PR #116, 2026-05-19).