CtrlK
BlogDocsLog inGet started
Tessl Logo
SkillPublic

testing-expert

Inspect test quality: assertion strength, coverage gaps, tests that pass while the feature is broken.

57

Quality

72%

Does it follow best practices?

Run evals on this skill

Adds up to 20 points to the overall score

View guide
SecuritybySnyk

Passed

No findings from the security scan

Fix and improve this skill with Tessl

tessl review fix ./ywai/skills/testing-expert/SKILL.md
SKILL.md
Quality
Evals
Security

Quality

Content

72%Weight 40%Scale 1-5

Reviews the quality of instructions and guidance provided to agents. Good implementation is clear, handles edge cases, and produces reliable results.

The body is an efficient, well-structured overview that establishes a clear evaluation philosophy and defers detail appropriately to four real reference files. The main weakness is workflow clarity: the inspection procedure is described in implicit prose rather than explicit, checkpointed steps.

Suggestions

Turn the 'Inspecting' paragraph into a short numbered sequence with explicit checkpoints (e.g., 1. Read the test name and state the behavior it claims; 2. Check the assertions pin it; 3. Ask what change would make it fail; 4. Check error paths and boundaries).

Include one inline weak-vs-strong assertion example in the body so the core heuristic is usable without opening references/assertion-quality.md.

DimensionReasoningScore

Conciseness

The body is lean and each section contributes skill-specific framing ('A test suite is only worth the failures it would catch', the two failure modes, consequence-oriented reporting). Minor over-explanation: the causes of flakiness (timing guesses, shared state, real clocks) restate what Claude already knows.

4 / 5

Actionability

For an instruction-only skill, the guidance is concrete and executable: start from the test's name, check 'whether removing the feature would make it fail', check error paths, and the weak-vs-strong report phrasing example ('an empty cart throws in checkout and nothing would catch it'). Not a 5 because no inline example of a weak vs. strong assertion is given — that detail is deferred entirely to references.

4 / 5

Workflow Clarity

The 'Inspecting' section does give a rough sequence (name → do assertions pin the behavior → would removal fail → error paths and boundaries), but it is prose rather than checkpointed steps, and the actual staged procedure (the 2-minute pass and deep pass) lives in references/inspection-checklist.md rather than the body. Sequence present but checkpoints implicit.

3 / 5

Progressive Disclosure

The body is a concise overview with a dedicated Reference section pointing to four real, existing files in references/ (inspection-checklist.md, red-flags.md, assertion-quality.md, example-report.md), each clearly signaled with a one-line description. References are one level deep and easy to navigate, matching the top anchor.

5 / 5

Total

16

/

20

Passed

Description

58%Weight 40%Scale 1-5

Based on the skill's description, can an agent find and select it at the right time? Clear, specific descriptions lead to better discovery.

The description is concise and specific about what the skill inspects, but it lacks any 'Use when...' trigger guidance and misses the most natural user-facing trigger terms (flaky tests, unit tests, test suite). Adding an explicit when-to-use clause with those synonyms would raise both completeness and trigger term quality.

Suggestions

Add a 'Use when...' clause, e.g. 'Use when reviewing a test suite, auditing test coverage, or when tests pass but the feature is still broken.'

Include natural trigger synonyms users would say: 'flaky tests', 'unit tests', 'test suite', 'false positives'.

Keep the third-person imperative voice (it already uses 'Inspect...'), while extending the action list to cover flaky-test detection to round out coverage.

DimensionReasoningScore

Specificity

Lists several concrete inspection targets — 'assertion strength', 'coverage gaps', 'tests that pass while the feature is broken' — which matches the anchor for several specific actions with minor gaps. Not a 5 because it omits other natural test-quality actions (flaky detection, suite triage); not a 3 because it names three distinct actions rather than 1-2.

4 / 5

Completeness

Has a clear 'what' (inspect test quality across assertion strength, coverage gaps, false-green tests) but no 'Use when...' clause or equivalent explicit trigger guidance, which caps completeness at 3 per the judging guidelines.

3 / 5

Trigger Term Quality

Includes relevant keywords like 'test quality' and 'coverage gaps', but misses the common variations and synonyms users would naturally say — most notably 'flaky tests', plus 'unit tests', 'test suite', and 'false positives'. Falls between anchors 3 and 4, closer to 3 because the most natural trigger phrase for half the skill's scope (flakiness) is absent.

3 / 5

Distinctiveness Conflict Risk

Occupies a fairly distinct niche (test-quality inspection) with specific triggers (assertions, coverage, passing-while-broken), so it is mostly distinct. Not a 5 because terms like 'coverage gaps' and 'test quality' could overlap with a general code-review or testing skill.

4 / 5

Total

14

/

20

Passed

Validation

100%

Checks the skill against the spec for correct structure and formatting. All validation checks must pass before discovery and implementation can be scored.

Validation — 16 / 16 Passed

Validation for skill structure

No warnings or errors.

Repository

YoizenSA/dev-ai-workflow

Path

ywai/skills/testing-expert/SKILL.md

Reviewed

Table of Contents

Is this your skill?

If you maintain this skill, you can claim it as your own. Once claimed, you can manage eval scenarios, bundle related skills, attach documentation or rules, and ensure cross-agent compatibility.