Copilot instructions

1---
2applyTo: "**/*.{ts,tsx,js,jsx}"
3---
4 
5# Test Coverage Review Instructions
6 
7Review code changes for adequate test coverage using freeCodeCamp's testing conventions.
8 
9## Testing Framework Reference
10 
11| Location | Framework | File Pattern | Notes |
12|----------|-----------|--------------|-------|
13| `api/` | Vitest | `*.test.ts` | Co-located with source |
14| `client/` | Vitest | `*.test.{ts,tsx}` | Co-located with source |
15| `e2e/` | Playwright | `*.spec.ts` | Dedicated directory |
16 
17## When to Flag Missing Tests
18 
19Comment on missing tests if ALL of these are true:
20 
21- PR modifies functional code (bug fix or new feature)
22- No corresponding test additions or modifications exist
23- Changes are not test-only or config-only
24 
25### Examples Requiring Tests
26 
27- New utility functions in `api/src/utils/` or `client/src/utils/`
28- New API route handlers in `api/src/routes/`
29- New React components with logic in `client/src/components/`
30- Bug fixes that change application behavior
31- New validation logic or data transformations
32 
33## When NOT to Flag
34 
35Do not comment on test coverage if:
36 
37- Changes are documentation, configuration, or non-functional
38- PR already includes appropriate test coverage
39- Changes are to test files themselves
40- Changes are trivial:
41  - Import reorganization
42  - Formatting changes
43  - Type-only changes (interfaces, type definitions)
44  - Comment updates
45- Changes are in areas without existing test patterns:
46  - Curriculum markdown files
47  - Configuration files
48  - Build scripts
49 
50## Detecting Outdated Tests
51 
52Flag if existing tests may be outdated:
53 
54- Test file exists for modified source but doesn't cover the changed functionality
55- Test assertions reference behavior that is being changed
56- Mock data doesn't reflect new data structures or API responses
57- Test descriptions no longer match actual test behavior
58 
59### Example Comment for Outdated Tests
60 
61```
62Tests in `src/utils/validate.test.ts` need updates for new validation rules.
63```
64 
65## Comment Format
66 
67### Missing Tests
68 
69Brief, actionable feedback only:
70 
71```
72Missing tests for `src/utils/validate.ts`:
73- Valid input case
74- Invalid input error
75- Empty string edge case
76```
77 
78### Sufficient Coverage
79 
80**DO NOT COMMENT.** Silence means approval.
81 
82### Outdated Tests
83 
84```
85Tests in `[test file]` need updates for `[source file]` changes:
86- [specific change]
87```
88 
89## Test Quality Indicators
90 
91When tests are present, briefly verify:
92 
93- Tests cover the happy path
94- Tests cover at least one error/edge case
95- Test descriptions are meaningful (not just "test 1", "test 2")
96- Mocks are appropriate (not mocking the thing being tested)
97 
98Do not block PRs for test style preferences if coverage is adequate.
99