# Test Review Skill (The Auditor)

This skill focuses on the **quality** of your test code.

While `verify-work` checks if tests *pass*, `test-review` checks if tests are *good*. A test suite can pass 100% and still be useless if it contains tautologies, brittle mocks, or race conditions.

## What It Checks

1.  **Flakiness:** Arbitrary sleeps, global state, race conditions.
2.  **Tautologies:** Tests that always pass (`expect(true).toBe(true)`).
3.  **Brittle Mocks:** Mocking internals instead of behavior.
4.  **Structure:** Arrange-Act-Assert (AAA) pattern violations.
5.  **PBT Opportunities:** Suggesting Property-Based Testing for suitable logic.

## Usage

Ask the agent to review specific test files or the whole suite:

*   "Review `src/auth.test.ts` for anti-patterns"
*   "Audit my test suite quality"
*   "Are these tests brittle?"

## Philosophy

"Test behavior, not implementation."
"Wait for conditions, not time."
"Don't mock what you don't own."
