v1.0.0
Reviews test quality across unit, integration, and E2E layers — test isolation, assertion clarity, factory patterns, and coverage gaps. Tests that pass but do not actually verify behavior create false confidence that leads to production incidents.
A review skill for test code quality, structure, and coverage strategy.
Run this on any PR that adds or modifies tests. Especially valuable during test suite refactors or when test flakiness becomes a recurring problem.
Test what the code does (outputs, side effects), not how it does it (internal method calls, private state). Implementation-coupled tests break every time you refactor, even when behavior is unchanged — making tests a liability instead of a safety net.
Every test must be independent — no shared mutable state, no execution order dependencies. When tests share state, they pass in isolation but fail together (or worse, fail randomly), creating flaky CI that wastes hours of debugging time.
Use factory functions (e.g., `createUser({role: 'admin'})`) instead of static JSON fixtures. Factories let you create exactly the data each test needs with sensible defaults, while fixtures force you to maintain large JSON files where a change to one test's data breaks another test.
BeforeMerge scans your pull requests against all 3 Testing Review rules automatically. Get actionable feedback before code ships.