skills.rest
jsco2t
jsco2tCOMMUNITY
Explore all skills by this author →
GitHub

test-reviewer

Review test code for value, reliability, and craftsmanship.

1|Updated Jun 7, 2024
Search Tags:#qa#reliability#code-review#test-quality#test-architecture#software-testing#test-review
One-click install
npx skills add https://github.com/jsco2t/dotfiles --skill test-reviewer-jsco2t
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: test-reviewer
Source: https://github.com/jsco2t/dotfiles/tree/main/.claude/skills/test-reviewer
Command: npx skills add https://github.com/jsco2t/dotfiles --skill test-reviewer-jsco2t

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

Reviews test code to determine value, reliability, and craftsmanship.

Core Features & Use Cases

  • Value assessment: detects tautologies, redundant coverage, and missing critical paths; ensures tests truly protect the codebase.
  • Reliability and determinism: flags flaky tests, non-deterministic setup, and environment coupling; promotes stable CI results.
  • Test architecture guidance: enforces proper pyramid balance, clear unit vs integration responsibilities, and maintainable test structures.
  • Readability and maintainability: emphasizes clear names, arrange-act-assert clarity, minimal shared fixtures, and actionable feedback for developers.

Quick Start

Provide a test-diff and relevant files to receive a structured critique.

Frequently Asked Questions about test-reviewer

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I review test code for reliability and maintainability?▼

Test code review evaluates reliability and maintainability by flagging flaky tests, non-deterministic setup, and environment coupling, while enforcing clear names and arrange-act-assert clarity to ensure stable CI results.

What is the best way to detect flaky tests and non-deterministic behavior in my test suite?▼

Detect flaky tests and non-deterministic behavior by reviewing test code for environment coupling and unstable setup. Structured critiques identify these issues to promote deterministic behavior and reliable CI results.

How do I enforce the testing pyramid alignment for unit and integration tests?▼

Enforce testing pyramid alignment by reviewing test architecture to ensure proper balance between unit, integration, and end-to-end tests, clarifying responsibilities to maintain a structured and scalable test suite.

Can I review test code that includes end-to-end and integration testing scopes?▼

Yes, test code review applies to codebases with test suites of varying quality and scopes, specifically including unit, integration, and end-to-end testing, to evaluate overall value and craftsmanship.

Why do my tests have redundant coverage and missing critical paths?▼

Tests have redundant coverage and missing critical paths when value assessment is lacking. Reviewing test code detects tautologies and identifies missing critical paths to ensure tests truly protect the codebase.

What do I need to provide to get a structured critique of my test code?▼

To receive a structured critique of your test code, you need to provide a test-diff and relevant files, which are evaluated for value, reliability, and craftsmanship.

Related Skills

Explore More Software Engineering Skills→
kinncjkinncj
23

rubber-duck

Critique plans, code, and tests with structured CRITICAL, WARN, and INFO verdicts.

Community
Intermediate
jamesaphoenixjamesaphoenix
6

test-quiet

Hide passing vitest output and surface failures with jq and bunx.

Community
Intermediate
idiaz01idiaz01
3

springboot-verification

Orchestrates build, static analysis, tests with Jacoco coverage, and security scans for Spring Boot projects in CI pipelines.

Community
Advanced
Chang-Shih-YungChang-Shih-Yung

health

Aggregate type checker, linter, and test results into a 0-10 composite score.

Community
Advanced