test-coverage

MUST be used whenever fixing test coverage for a Flows app to meet the 80% line coverage hard gate. This skill finds AND fixes coverage gaps — it configures tooling, writes missing tests, covers untested paths, and refactors code for testability. It does not just report. Triggers: test coverage, fix

By cognitedata · 1,906 installs

npx skills add cognitedata/builder-skills --skill test-coverage

Source repository · Upstream listing

Test Coverage Fix Fix test coverage for $ARGUMENTS (or the whole app if no argument is given). This skill enforces the 80% line coverage hard gate required for Flows app approval by finding AND fixing coverage gaps. Work through every step in order. Step 1 — Verify test framework and coverage tooling Check that the project has a working test framework with coverage configured: Verify: A test framework (Vitest or Jest) is installed and configured The config file has a coverage section (e.g. coverage: { provider: 'v8', ... } in vitest.config.ts) A coverage reporter is configured (at least text and lcov or json summary ) If coverage tooling is not configured, fix it now: 1. Install the coverage provider: 2. Add the coverage configuration to vitest.config.ts . Read the existing config file, then add the coverage section inside test : Write the updated config file. If no vitest.config.ts exists at all, create one with the full defineConfig wrapper. Step 2 — Validate coverage scope The 80% threshold applies to all .ts and .tsx files under src/ , excluding only: Test files ( .test.ts , .test.tsx , .spec.ts , .spec.tsx ) Type declaration files ( vite env.d.ts ) The entry point ( main.tsx ) Apps must not exclude pages, components, hooks, or other production code from coverage measurement. If the config excludes production files, fix it now: Remove any exclusion that hides production code from coverage measurement. Only test files, type declarations, and the entry point should be excluded. Rewrite the exclude array to contain only: Specifically remove any exclusions for: src/pages/ or src/components/ or src/hooks/ — NOT allowed Specific feature files — NOT allowed unless they are generated code src/ / .tsx (all components) — NOT allowed , this hides the majority of the app Write the corrected config file. Step 3 — Run tests and collect coverage Record the coverage summary: Statements: X% Branches: X% Functions: X% Lines: X% Hard gate: Overall line coverage must be at least 80% . Apps below this threshold are listed as must fix . If tests fail to run, fix them now: Common fixes: Missing imports: Read the failing test file, add the missing import statement, write the fixed file. Broken mocks: Read the test to understand what is being mocked. Fix the mock to match the current API of the mocked module. Outdated snapshots: Run npx vitest run update to update snapshots, then review the diff to ensure correctness. Missing dependencies: Run pnpm add D <missing package for any test utilities not yet installed. Config errors: Read the config file, fix syntax or option errors, write the corrected file. Re run tests after each fix until all tests pass. Then record the coverage summary. Step 4 — Find and write missing test files For every non trivial .ts / .tsx file under src/ , check whether a corresponding test file exists: Categorize each file without a test: Services, hooks, utils, contexts, ViewModel hooks — Write the test file now (see below) Pure presentational components with no logic — Mark as N/A (no test required) Barrel exports ( index.ts that only re exports) — Mark as N/A Type only files ( .d.ts , files with only type/interface exports) — Mark as N/A For each file missing a test, create a comprehensive test file. Use context injection for dependency mocking where the production code supports it. If the production code uses hard coded imports, note this as a testability concern but still write the test using vi.mock with a justification comment. Follow this process for each: 1. Read the source file to understand its exports, dependencies, and logic. 2. Create a .test.ts or .test.tsx file in the same directory as the source file. 3. Write tests covering: happy path, error path, empty state, and edge cases. Use the right testing pattern for each file type: For hooks: Test with renderHook from @testing library/react Wrap with necessary providers (QueryClientProvider, custom context providers, etc.) Test initial state, loading state, success state, and error state Example structure: For services/utils: Test with direct function calls Mock CDF SDK responses where needed Test return values, side effects, and thrown errors Example structure: For components with logic: Test with render from @testing library/react Verify loading, error, and data states Test user interactions that trigger state changes Example structure: Dependency mocking guidelines: Use context injection (not vi.mock ) where possible — provide test dependencies via the hook's context. If the production code uses hard coded imports that prevent context injection, use vi.mock with a justification comment explaining why (e.g., // vi.mock required: useDataSource uses direct import, not context injection ). Ensure mocks are type safe — no as unknown as T casts. Define proper mock objects that satisfy the interface. After writing each test file, run npx vitest run <test file to verify it passes. Step 5 — Fix low coverage files If the coverage tool produces per file metrics, list files below 80% line coverage: For each file below 80% coverage, read the uncovered lines from the coverage report, then add test cases that exercise those specific code paths: 1. Read the uncovered lines from the coverage report. Check the coverage/ directory for detailed per file reports (lcov or html) that show which lines are uncovered. 2. Read the source file to understand what those uncovered lines do. 3. Add test cases that exercise those specific code paths: Uncovered error paths: Add tests that trigger error conditions (network failures, invalid input, null values, permission errors). Force errors by providing bad input or mocking dependencies to throw. Uncovered branches: Add tests for each conditional branch. If an if/else has only the true branch tested, write a test that triggers the false branch. Uncovered functions: Add tests that call each exported function that lacks coverage. Verify return values and side effects. Uncovered catch blocks: Mock the upstream call to reject/throw, verify the catch block behavior. 4. Run coverage again after adding tests to verify the file now meets 80%: Repeat until each file reaches at least 80% line coverage or you have covered all feasible paths. Step 6 — Fix testability patterns Assess and fix testability issues in the codebase: For each testability issue found, refactor the production code to support better testing patterns. Then update the corresponding test to use the improved pattern. Issue Fix Hook imports dependencies directly instead of using context Add a context type with default dependencies. Create a context with createContext , provide defaults that use the real implementation. In the hook, use useContext to get dependencies. Tests can then provide mock dependencies via the context provider without vi.mock . Service has no interface Extract a TypeScript interface describing the service's public API. Have the class/object implement the interface. Tests mock against the interface, not the concrete implementation. Page component mixes data fetching with rendering Extract data logic into a use ViewModel hook. The page component calls the ViewModel hook and renders based on its return value. Test the ViewModel hook separately with renderHook , test the page component with a mocked ViewModel. Tests use vi.mock for modules that could use context injection After refactoring the production code to use context injection (above), update the test to provide mock dependencies via the context provider. Remove the vi.mock call. Add a comment explaining the pattern. Tests use as unknown as T casts for mocks Define a proper mock type or object that satisfies the required interface. Replace the cast with a properly typed mock. If the interface is large, create a helper function that returns a partial mock with only the methods used, typed correctly. For each refactored file: 1. Read the source file and its test file. 2. Refactor the production code (add context, extract interface, extract ViewModel hook). 3. Update the test to use the improved pattern. 4. Run npx vitest run <test file to verify the test still passes. Step 7 — Report remaining gaps Re run the full test suite with coverage to get final numbers: Produce a summary of what was done and what remains: Done Summarize: Overall coverage before and after fixes, vs the 80% gate (PASS or FAIL) Number of test files written and tests added Number of files refactored for testability Any remaining must fix items that need human review