Add eval coverage for dotnet-test/filter-syntax#824
Conversation
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Skill Coverage Report
|
There was a problem hiding this comment.
Pull request overview
Adds skill-validator eval coverage for the dotnet-test/filter-syntax reference skill by introducing a new eval.yaml with scenarios targeting xUnit v3 trait filtering and xUnit v3 query filter language usage.
Changes:
- Added a new
tests/dotnet-test/filter-syntax/eval.yamlwith two scenarios focused on xUnit v3 trait filtering ([Trait]+--filter-trait) and query filtering (--filter-query+[Category=Smoke]). - Introduced deterministic
output_matchesassertions and rubric items to validate that the scenarios cover the intended teaching points.
Show a summary per file
| File | Description |
|---|---|
| tests/dotnet-test/filter-syntax/eval.yaml | Adds new eval scenarios intended to cover the filter-syntax skill’s xUnit v3 trait and query filter guidance. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 4
…ertions Match known-good MTP xUnit v3 fixtures, enforce the real Trait annotation and SDK 9 separator, and tighten the query-filter scenario with a realistic fixture. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
/evaluate |
Skill Validation Results
[1] Model: claude-opus-4.6 | Judge: claude-opus-4.6 🔍 Full Results - additional metrics and failure investigation steps
|
|
👋 @Evangelink — this PR has 1 unresolved review thread(s). When you're ready, please address the feedback and push an update; the triage bot will pick up the next state automatically. (Add the |
|
✅ Approved by @YuliiaKovalova. cc @dotnet/skills-merge-approvers — ready to merge. |
Adds
tests/dotnet-test/filter-syntax/eval.yamlto give the previously-untesteddotnet-test/filter-syntaxskill eval coverage.What this covers
The skill-coverage tool (
eng/skill-coverage/Measure-SkillCoverage.ps1) reported one uncovered teaching point:[trait](SKILL.md ~line 114) — xUnit's trait attribute / trait-based--filterusage.A new scenario ("Filter xUnit v3 tests by trait and show how to tag them") produces output that genuinely references the xUnit trait attribute and
--filter-trait, with a deterministicoutput_matchesassertion (\[[Tt]rait) that makes the point covered. A second scenario exercises the xUnit v3 query filter language (combined class pattern +[Category=Smoke]trait qualifier).Prompts are written as natural developer requests (no skill references); rubric items test outcomes rather than techniques.
Verification
Measure-SkillCoverage.ps1 -PluginName dotnet-test -SkillName filter-syntax -Format Json→ 100% (1/1 points covered);[trait]no longer inuncovered, no regressions.SkillValidator -- check --plugin ./plugins/dotnet-test→ ✅ All checks passed (remaining warnings are pre-existing and unrelated).Co-authored-by: Copilot 223556219+Copilot@users.noreply.github.com