Move test class organization guidance from usethis-python-test-full-coverage to new usethis-python-test skill#1704
Merged
nathanjmcdougall merged 2 commits intocopilot/improve-test-suite-instructionsfrom Apr 1, 2026
Conversation
…ce out of usethis-python-test-full-coverage Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/0336fce3-623f-4d2d-aad9-ced0c83edf6e Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] [WIP] Revise agent skills organization based on feedback for test class guidelines
Move test class organization guidance from usethis-python-test-full-coverage to new usethis-python-test skill
Apr 1, 2026
nathanjmcdougall
approved these changes
Apr 1, 2026
e2e461d
into
copilot/improve-test-suite-instructions
11 checks passed
nathanjmcdougall
added a commit
that referenced
this pull request
Apr 1, 2026
* Initial plan * Add test class organization guidelines to usethis-python-test-full-coverage skill Adds a "Test class organization" section covering: - Top-level classes: one per function or class under test - Nested classes: group by method, operation, or variant - Variant-specific tests must be nested inside existing test classes - Nesting depth guidance (2-3 levels max) - No docstrings on test classes or functions Resolves #1645 Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/93d2f601-5c64-4f12-8343-b34a2973b558 Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com> * Move test class organization guidance from usethis-python-test-full-coverage to new usethis-python-test skill (#1704) * Initial plan * Create usethis-python-test skill; move test class organization guidance out of usethis-python-test-full-coverage Agent-Logs-Url: https://github.com/usethis-python/usethis-python/sessions/0336fce3-623f-4d2d-aad9-ced0c83edf6e Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com> --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: nathanjmcdougall <18602289+nathanjmcdougall@users.noreply.github.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
The "Test class organization" section added to
usethis-python-test-full-coverageis general test guidance, not coverage-specific. It belongs in its own skill.Changes
usethis-python-test-full-coverage/SKILL.mdto v1.1 — removed the "Test class organization" section; this skill now focuses solely on coverage measurement and verification.agents/skills/usethis-python-test/SKILL.md(v1.0) — new skill for general test conventions (class organization, nesting patterns, variant-specific tests, nesting depth, no docstrings); referencesusethis-python-test-full-coveragefor coverage workusethis-python-testindocs/skills-directory.txtand the corresponding sync block inAGENTS.md