Testing Patterns Skill
Test Structure (AAA Pattern)
def test_function_does_expected_thing():
"""Test description explaining what and why."""
# Arrange - Set up test data and preconditions
input_data = {"key": "value"}
expected = "result"
# Act - Execute the code under test
result = function_under_test(input_data)
# Assert - Verify the outcome
assert result == expected
Test Organization
tests/
├── conftest.py # Shared fixtures
├── unit/ # Unit tests (isolated)
│ ├── test_search_core.py
│ └── test_utils.py
├── integration/ # Integration tests
│ └── test_api.py
└── e2e/ # End-to-end tests
└── test_workflow.py
Quality Targets
| Metric |
Target |
| Coverage |
>70% overall |
| New code |
100% |
| Core modules |
>80% |
| No flaky tests |
0 |
Validation contract regressions
Mirrored validators can agree on a generated snapshot while disagreeing on real
inputs. Execute the same accepted/rejected fixtures against the real backend
validator and the client evaluator. Include N/N+1 boundaries, collection/item
limits, omitted/null/empty/default values, nested paths, conditional branches,
hydrated updates versus fresh DTOs, Unicode units and previously valid inputs.
Test the actual HTTP path for malformed input and unchanged persisted state
after a pre-write refusal. Test zero transport calls for a local refusal,
normal transport for a valid payload, and field correction/resubmission.
Run deterministic-generation and reviewed-exclusion checks separately from
behavioral parity. A schema/rule count is inventory, not assertion coverage.
For operation semantics and runtime mismatches, read
reference/input-validation.md from the installed security-patterns skill.
Resolve the skill through the current client's catalog; directory names may
carry an adapter-specific prefix.
Language-Specific References
| Language |
Reference |
Key Topics |
| Python |
reference/python-pytest.md |
Fixtures, mocking, parametrize, markers, conftest, running tests |
| TypeScript |
reference/typescript-vitest.md |
Vitest/Jest, React Testing Library, mocking, running tests |
| PHP |
reference/php-phpunit.md |
PHPUnit test cases, mocking, running tests |
| Go |
reference/go-testing.md |
Table-driven tests, testify mocking, running tests |
| Flutter/Dart |
reference/flutter-testing.md |
Widget tests, unit tests, running tests |
For Python pytest patterns, see reference/python-pytest.md.
For TypeScript Vitest/Jest patterns, see reference/typescript-vitest.md.
For PHP PHPUnit patterns, see reference/php-phpunit.md.
For Go testing patterns, see reference/go-testing.md.
For Flutter/Dart testing patterns, see reference/flutter-testing.md.
Common Rationalizations
| Excuse |
Why It's Wrong |
| "It's too simple to test" |
Simple code breaks in integration — test the contract, not the complexity |
| "Tests slow down development" |
Tests slow down bugs reaching production — that's the point |
| "We'll add tests later" |
Untested code accumulates — later means never, and coverage gaps compound |
| "Mocking everything is fine" |
Over-mocking tests the mocks, not the code — mock at boundaries only |
| "100% coverage means no bugs" |
Coverage measures execution, not correctness — focus on behavior assertions |
Rules
- MUST follow Arrange-Act-Assert (AAA) structure in every test — unstructured tests degrade into procedural smoke tests
- MUST test behavior through the public interface, not internal implementation — tests coupled to internals break on every refactor
- NEVER test implementation details (private method return values, internal state flags) — they are not the contract
- NEVER hit real external services in unit tests — use fakes/stubs for boundaries; save real integration for integration tests
- CRITICAL: integration tests must hit real dependencies (database, message queue, external API) when mock-vs-prod divergence is a real risk. Mocked integration tests create false confidence.
- MANDATORY: flaky tests are bugs, not noise. Quarantine or delete them — a tolerated flaky test erodes the suite's credibility.
Gotchas
- Coverage numbers are easy to game: include generated code, test files that import but do not assert, or wide
# pragma: no cover usage. A 95% reported coverage with 60% real behavior assertion is common.
- Snapshot tests (Jest
.toMatchSnapshot(), pytest-regressions) accept any output as "correct" on first run. An incorrect initial snapshot becomes the accepted baseline — review snapshots as carefully as code.
- Mocks configured with
any matchers (e.g., .mock.calls[0][0] without a schema) pass even when the production call shape changes. Assert on specific arguments, not just "was called".
- Test isolation fails when globals leak (module-level mutable state, module-scoped fixtures, env vars set in one test). Flakiness that appears only under
pytest -n auto or jest --parallel is usually shared state.
- Property-based tests (Hypothesis, fast-check) shrink failing examples to minimal reproducers, but shrinking time can dominate the run. For complex generators, cap shrink deadlines or seed the failing example for next-run reproducibility.
- Test pyramid vs trophy: the "right" ratio depends on stack. Frontend apps with rendering concerns benefit from more integration tests (trophy); pure backend services align better with pyramid. Don't cargo-cult one model.
When NOT to Load
- For running the test suite — use
/test
- For test-first development workflow — use
/tdd
- For debugging a specific test failure — use
/debug on the failure output
- For test framework choice in a new project — use
/app-builder
- For performance/load testing — this skill covers correctness tests, not load
1---2name: testing-patterns3description: Testing strategy: pyramid, AAA, mocks/fakes/stubs, flaky tests, coverage. Triggers: test, fixture, mock, stub, e2e, TDD, Playwright, Cypress, flaky, coverage, property-based.4---56# Testing Patterns Skill78## Test Structure (AAA Pattern)910```python11def test_function_does_expected_thing():12 """Test description explaining what and why."""13 # Arrange - Set up test data and preconditions14 input_data = {"key": "value"}15 expected = "result"1617 # Act - Execute the code under test18 result = function_under_test(input_data)1920 # Assert - Verify the outcome21 assert result == expected22```2324---2526## Test Organization2728```29tests/30├── conftest.py # Shared fixtures31├── unit/ # Unit tests (isolated)32│ ├── test_search_core.py33│ └── test_utils.py34├── integration/ # Integration tests35│ └── test_api.py36└── e2e/ # End-to-end tests37 └── test_workflow.py38```3940---4142## Quality Targets4344| Metric | Target |45|--------|--------|46| Coverage | >70% overall |47| New code | 100% |48| Core modules | >80% |49| No flaky tests | 0 |5051---5253## Validation contract regressions5455Mirrored validators can agree on a generated snapshot while disagreeing on real56inputs. Execute the same accepted/rejected fixtures against the real backend57validator and the client evaluator. Include N/N+1 boundaries, collection/item58limits, omitted/null/empty/default values, nested paths, conditional branches,59hydrated updates versus fresh DTOs, Unicode units and previously valid inputs.6061Test the actual HTTP path for malformed input and unchanged persisted state62after a pre-write refusal. Test zero transport calls for a local refusal,63normal transport for a valid payload, and field correction/resubmission.64Run deterministic-generation and reviewed-exclusion checks separately from65behavioral parity. A schema/rule count is inventory, not assertion coverage.6667For operation semantics and runtime mismatches, read68`reference/input-validation.md` from the installed `security-patterns` skill.69Resolve the skill through the current client's catalog; directory names may70carry an adapter-specific prefix.7172## Language-Specific References7374| Language | Reference | Key Topics |75|----------|-----------|------------|76| Python | [reference/python-pytest.md](reference/python-pytest.md) | Fixtures, mocking, parametrize, markers, conftest, running tests |77| TypeScript | [reference/typescript-vitest.md](reference/typescript-vitest.md) | Vitest/Jest, React Testing Library, mocking, running tests |78| PHP | [reference/php-phpunit.md](reference/php-phpunit.md) | PHPUnit test cases, mocking, running tests |79| Go | [reference/go-testing.md](reference/go-testing.md) | Table-driven tests, testify mocking, running tests |80| Flutter/Dart | [reference/flutter-testing.md](reference/flutter-testing.md) | Widget tests, unit tests, running tests |8182For Python pytest patterns, see [reference/python-pytest.md](reference/python-pytest.md).8384For TypeScript Vitest/Jest patterns, see [reference/typescript-vitest.md](reference/typescript-vitest.md).8586For PHP PHPUnit patterns, see [reference/php-phpunit.md](reference/php-phpunit.md).8788For Go testing patterns, see [reference/go-testing.md](reference/go-testing.md).8990For Flutter/Dart testing patterns, see [reference/flutter-testing.md](reference/flutter-testing.md).9192## Common Rationalizations9394| Excuse | Why It's Wrong |95|--------|----------------|96| "It's too simple to test" | Simple code breaks in integration — test the contract, not the complexity |97| "Tests slow down development" | Tests slow down bugs reaching production — that's the point |98| "We'll add tests later" | Untested code accumulates — later means never, and coverage gaps compound |99| "Mocking everything is fine" | Over-mocking tests the mocks, not the code — mock at boundaries only |100| "100% coverage means no bugs" | Coverage measures execution, not correctness — focus on behavior assertions |101102## Rules103104- **MUST** follow Arrange-Act-Assert (AAA) structure in every test — unstructured tests degrade into procedural smoke tests105- **MUST** test behavior through the public interface, not internal implementation — tests coupled to internals break on every refactor106- **NEVER** test implementation details (private method return values, internal state flags) — they are not the contract107- **NEVER** hit real external services in unit tests — use fakes/stubs for boundaries; save real integration for integration tests108- **CRITICAL**: integration tests must hit real dependencies (database, message queue, external API) when mock-vs-prod divergence is a real risk. Mocked integration tests create false confidence.109- **MANDATORY**: flaky tests are bugs, not noise. Quarantine or delete them — a tolerated flaky test erodes the suite's credibility.110111## Gotchas112113- Coverage numbers are easy to game: include generated code, test files that import but do not assert, or wide `# pragma: no cover` usage. A 95% reported coverage with 60% real behavior assertion is common.114- Snapshot tests (Jest `.toMatchSnapshot()`, pytest-regressions) accept any output as "correct" on first run. An incorrect initial snapshot becomes the accepted baseline — review snapshots as carefully as code.115- Mocks configured with `any` matchers (e.g., `.mock.calls[0][0]` without a schema) pass even when the production call shape changes. Assert on specific arguments, not just "was called".116- Test isolation fails when globals leak (module-level mutable state, module-scoped fixtures, env vars set in one test). Flakiness that appears only under `pytest -n auto` or `jest --parallel` is usually shared state.117- Property-based tests (Hypothesis, fast-check) shrink failing examples to minimal reproducers, but shrinking time can dominate the run. For complex generators, cap shrink deadlines or seed the failing example for next-run reproducibility.118- Test pyramid vs trophy: the "right" ratio depends on stack. Frontend apps with rendering concerns benefit from more integration tests (trophy); pure backend services align better with pyramid. Don't cargo-cult one model.119120## When NOT to Load121122- For **running** the test suite — use `/test`123- For test-first development workflow — use `/tdd`124- For debugging a specific test failure — use `/debug` on the failure output125- For test framework choice in a new project — use `/app-builder`126- For performance/load testing — this skill covers correctness tests, not load