| name | java-test-architecture |
| description | Use this skill when statically reviewing a JVM test suite's architecture for soundness and non-flakiness: JUnit 5 lifecycle and isolation (shared mutable static state, test-instance lifecycle, order dependence, time/locale/timezone dependence, and unguarded junit.jupiter.execution.parallel usage), Testcontainers discipline (singleton-container-with-Ryuk-reuse vs per-test @Container, explicit Wait strategies vs Thread.sleep), ArchUnit layering/cycle rules including FreezingArchRule brownfield adoption, and test-quality smells (assertion-free tests, over-mocking, coverage theater, missing negative tests) expressed via AssertJ/Mockito. Also trigger when a user reports a flaky JVM test and wants root-cause triage. Reads test source, ArchUnit rule definitions, and sanitized build/test configuration only; it never invokes a JDK, runs mvn/gradle test, starts a JUnit runner, or opens a Testcontainers/Docker daemon connection. |
java-test-architecture
Purpose
This skill statically reviews JVM test suite architecture for soundness and non-flakiness. A test suite is only sound if tests are isolated from each other (no shared mutable static state leaking across methods, no implicit order dependence), deterministic regardless of wall-clock time, locale, or timezone, safe to run in parallel only where resource contention is explicitly guarded, disciplined in how Testcontainers instances are shared or scoped and how they signal readiness, architecturally enforced via ArchUnit with a sane brownfield-adoption path, and verifying real behavior rather than accumulating assertion-free or over-mocked tests that inflate coverage without catching regressions. The review produces a severity-ranked, evidence-labelled finding list plus a root-cause classification for any reported flaky test.
Trigger conditions
- A user provides JUnit 5 test classes, base test classes, or junit-platform.properties/parallel-execution configuration and asks whether the suite is sound or safe to parallelize.
- A user provides Testcontainers usage (container fields, @Container annotations, Wait strategy calls or their absence) and asks about container-sharing strategy or CI startup time/flakiness.
- A user provides ArchUnit rule classes and asks how to introduce layering/cycle enforcement on an existing codebase without breaking the build.
- A user reports a specific flaky JVM test (passes locally, fails in CI; passes alone, fails in a suite; fails intermittently) and wants root-cause triage rather than a blanket retry.
- A user provides a test class and asks whether it actually verifies behavior (assertion strength, mock usage, missing negative cases) rather than just producing green coverage.
When not to use
- The task is running the test suite, invoking a JDK/build tool, or starting a real Testcontainers/Docker daemon — this skill is static-review only and never executes anything.
- The task is generic cross-framework flaky-test quarantine policy or CI retry-configuration audit for a non-JVM stack — route to the generic test-flakiness-triage-agent.
- The task is cross-language coverage-percentage gate policy or a framework-agnostic mock-quality rubric — route to the generic test-coverage-quality-review-agent; this skill owns only the JUnit5/AssertJ/Mockito-specific instantiation.
- The task is CI pipeline mechanics (job sharding, parallel job-matrix wiring, artifact retention, secret exposure in pipeline YAML) — route to the ci-test-pipeline-review-agent.
- The task is JPA/Hibernate fetch-strategy or connection-pool correctness, deserialization/parser RCE surface, or JDK upgrade posture — route to the respective java-* sibling agent.
Lean operating rules
- CRITICAL — flag shared mutable static state (static fields, singletons, un-cleared ThreadLocal, static caches, System properties) read or written across test methods without @TestInstance(PER_CLASS) discipline or an explicit @BeforeEach/@AfterEach reset; it is the leading cause of order-dependent and cross-test-pollution failures.
- CRITICAL — flag junit.jupiter.execution.parallel.enabled=true adopted without per-class/per-method @Execution(CONCURRENT) opt-in scoping and @ResourceLock/@Isolated guards on every shared resource (static state, env vars, System.setProperty, shared ports/files, a shared Testcontainers instance); block until contention is guarded, since parallelism turns a latent static-state bug nondeterministic.
- HIGH — flag System.currentTimeMillis()/Instant.now()/LocalDate.now()/default Locale or TimeZone used without an injected fixed Clock or a pinned Locale/TimeZone; these fail at day/month boundaries, under DST, on non-UTC CI runners, or under a non-en-US default locale.
- HIGH — flag execution-order dependence: a test that only passes because a prior test mutated shared state, or an implicit assumption of declaration order without a stated @TestMethodOrder — JUnit 5 does not guarantee order by default.
- HIGH — for Testcontainers, flag a heavyweight container (database, broker) restarted per test method when a singleton-container-with-Ryuk-reuse pattern would avoid the cost, and separately flag a shared/singleton container whose state is not reset between tests; name which isolation boundary the chosen sharing model requires and confirm it is enforced.
- HIGH — flag Thread.sleep() used as a Testcontainers or async readiness wait; require an explicit Wait strategy (Wait.forHttp, Wait.forListeningPort, Wait.forLogMessage, or a HealthcheckStrategy) sized to the real startup signal instead.
- HIGH — flag assertion-free tests (exercise code, call no assertion, assert no thrown exception) and tautological assertions (assertTrue(true), assertEquals(x, x)) as coverage that verifies nothing.
- HIGH — flag over-mocking: mocking a value object, pure function, or trivial collaborator so the test only verifies a mock was called rather than observing resulting behavior. Prefer AssertJ state assertions over Mockito verify() unless the collaborator is genuinely side-effecting.
- MEDIUM — flag a component with visible validation/error-handling/exception paths whose suite has no negative or boundary test for those paths; name the specific missing case rather than asserting the suite generically needs more tests.
- MEDIUM — for ArchUnit layering/cycle rules introduced on a brownfield codebase with existing violations, recommend FreezingArchRule.freeze(rule) with a visible, owned plan to shrink the freeze store; never recommend suppressing the rule or widening the freeze store to relicense existing violations.
- MEDIUM — when triaging a reported flaky JVM test, classify root cause as shared static/mutable state, order dependence, time/locale/timezone dependence, unguarded parallelism, async/Testcontainers timing, or external-resource nondeterminism, and give the fix by category; a blanket @RepeatedTest or CI auto-retry must not be the primary recommendation.
References
Load these only when needed:
Response minimum
Return, at minimum:
- A verdict (pass / pass-with-conditions / block) and an evidence level stating which test source, configuration, and ArchUnit rules were provided.
- Lifecycle/isolation findings covering shared static state, test-instance lifecycle, order dependence, time/locale/timezone dependence, and parallel-execution guards.
- Testcontainers discipline findings (container-sharing strategy and Wait-strategy-vs-sleep usage) and ArchUnit findings (layering/cycle rules and FreezingArchRule adoption status).
- Test-quality findings covering assertion-free tests, over-mocking, coverage theater, and missing negative/boundary tests.
- A severity-labelled finding list (critical / high / medium / low), each carrying an evidence-basis label.
- For a reported flaky test: a root-cause category (from the fixed taxonomy) and a fix by category, not a blanket retry/quarantine.
- Safe next actions and open questions naming exactly what source/config the user must still supply.
