Test Trait Tagging
Analyze an existing test suite in any supported language and apply a standardized set of trait tags to each test method, giving teams visibility into their test distribution (positive vs. negative, critical-path coverage, smoke tests, etc.).
Language-specific guidance: Call the test-analysis-extensions skill to discover available extension files, then read the file matching the target codebase. The extension file documents framework-specific tag attributes and a "tag-support capability" (auto-edit, report-only, or convention-based) that drives whether this skill modifies source files or only emits a report.
When to Use
- Auditing a test project to understand the mix of test types
- Adding trait attributes to untagged tests
- Generating a summary report of trait distribution across a test suite
- Reviewing whether critical paths have sufficient coverage
When Not to Use
- Writing new tests from scratch (use
code-testing-agent for any language, or writing-mstest-tests for MSTest)
- Running or filtering tests (use
run-tests for .NET; equivalent native runners elsewhere)
- Migrating between test frameworks
- General quality, smell, flakiness, or assertion audits (use
test-anti-patterns or the matching analysis skill)
- Diagnostic .NET executed line/branch/Cobertura interpretation or project-wide CRAP risk (use
coverage-analysis); raw coverage collection (use run-tests for .NET, native tooling otherwise)
- CRAP analysis for a named method, class, or file (use
crap-score)
- Behavioral gaps where a test would survive broken production logic (use
test-gap-analysis)
Inputs
| Input |
Required |
Description |
| Test project or files |
Yes |
Path to the test project, folder, or specific test files to analyze |
| Scope |
No |
tag (apply canonical attributes, or a confirmed project convention), audit (report only), or both (default: both). Frameworks declared report-only always emit a report; convention-based frameworks edit only after the user confirms the convention. |
| Framework |
No |
Auto-detected. Override when detection fails. |
Trait Taxonomy
Use exactly these trait names and values. Do not invent new trait values outside this table.
| Trait Value |
Meaning |
Heuristics |
positive |
Verifies expected behavior under normal/valid conditions |
Asserts success, valid output, expected state, no exceptions for valid input |
negative |
Verifies correct handling of invalid input, errors, or edge cases |
Asserts exceptions, error codes, validation failures, rejects bad input |
boundary |
Tests limits, thresholds, empty/null/None/nil inputs, min/max values |
Operates on 0, -1, int.MaxValue / sys.maxsize / Number.MAX_SAFE_INTEGER / math.MaxInt64 / i32::MAX, empty string, null/None/nil/undefined, empty collection, boundary of valid range |
critical-path |
Core workflow that must never break; breakage blocks users |
Tests the primary success scenario of a key public API or user-facing feature |
smoke |
Quick sanity check that the system is operational |
Fast, no complex setup, verifies basic wiring (e.g., service resolves, endpoint returns 200) |
regression |
Reproduces a specific previously-reported bug |
References a bug ID, issue number, or describes a fix in its name or comments |
integration |
Crosses process, network, or persistence boundaries |
Uses real database, HTTP client, file system, external service, or multi-component setup |
end-to-end |
Full user workflow spanning the entire application stack |
Exercises a complete scenario from entry point to final result, distinct from single-boundary integration |
performance |
Validates timing, throughput, or resource consumption |
Asserts on elapsed time, memory, allocations, or uses benchmark harness (BenchmarkDotNet, pytest-benchmark, benchmark.js, JMH, go test -bench, criterion.rs, XCTMetric, kotlinx-benchmark, Google Benchmark) |
security |
Verifies authentication, authorization, input sanitization, or secrets handling |
Tests for SQL injection, XSS, CSRF, unauthorized access, token validation, permission checks |
concurrency |
Validates thread safety, parallelism, or async correctness |
Uses Task.WhenAll / Parallel.ForEach / SemaphoreSlim (.NET); asyncio.gather / threading.Lock / multiprocessing (Python); Promise.all / worker threads (JS/TS); CompletableFuture / ExecutorService / synchronized (Java); go func / sync.WaitGroup / sync.Mutex / chan (Go); Mutex / Thread.new (Ruby); tokio::spawn / Arc<Mutex<_>> / crossbeam (Rust); DispatchQueue / actor (Swift); coroutineScope / Mutex (Kotlin); Start-Job / RunspacePool (PowerShell); std::thread / std::mutex (C++); reproduces race conditions |
resilience |
Tests retry logic, timeouts, circuit breakers, or graceful degradation |
Asserts behavior under transient failures, network drops, or service unavailability (e.g., Polly, tenacity, p-retry, resilience4j, hystrix, opossum, retry-go) |
destructive |
Mutates shared or external state that is hard to roll back |
Deletes records, drops resources, modifies global config -- useful for CI isolation decisions |
configuration |
Verifies settings loading, defaults, environment behavior |
Tests missing config keys, invalid values, environment variable fallbacks, options validation |
flaky |
Known to intermittently fail (meta-tag for test health tracking) |
Mark tests the team knows are unreliable; used to quarantine or prioritize stabilization |
A single test may have multiple traits (e.g., both negative and boundary). At minimum, every test should receive one of positive or negative.
Workflow
Step 1: Detect the language, framework, and tagging capability
Identify the codebase's language and test framework. Call the test-analysis-extensions skill and read the matching extension file. The extension file declares a tag-support capability for each framework:
auto-edit — framework has canonical tag syntax this skill can safely insert (.NET [TestCategory] / [Trait] / [Category] / [Property], pytest @pytest.mark.<name>, JUnit 5 @Tag("..."), TestNG groups = {"..."}, RSpec metadata it "..." , :tag => true, Pester -Tag '...', Kotest @Tags(...), Swift Testing @Tag(.tagName), Catch2 [tag], doctest * doctest::test_suite("tag") decorator).
report-only — framework has no canonical, agreed-upon tag attribute; report tags in a Markdown table only and do not edit source (Go standard testing without build-tag conventions, Jest/Vitest without consistent describe-prefix convention, Rust without project-specific cfg conventions, XCTest without a test plan, GoogleTest without test-name prefix conventions, Mocha without describe-prefix conventions).
convention-based — framework uses naming or file conventions for tagging (Go //go:build integration build tags, file-name suffixes like *_integration_test.go, GoogleTest INTEGRATION_* filter prefix). Only emit canonical edits when the user has confirmed the project convention; otherwise treat as report-only.
Capture the capability before Step 4.
Step 2: Scan existing traits
Check which tests already have trait attributes. Use the loaded language extension as the source of truth — examples:
| Framework |
Existing Attribute |
Example |
| MSTest |
[TestCategory("...")] |
[TestCategory("positive")] |
| xUnit |
[Trait("Category", "...")] |
[Trait("Category", "positive")] |
| NUnit |
[Category("...")] |
[Category("positive")] |
| TUnit |
[Property("Category", "...")] |
[Property("Category", "positive")] |
| JUnit 5 |
@Tag("...") |
@Tag("positive") |
| TestNG |
@Test(groups = {"..."}) |
@Test(groups = {"positive"}) |
| pytest |
@pytest.mark.<name> |
@pytest.mark.positive |
| RSpec |
metadata after it |
it "...", :positive do |
| Pester |
-Tag '...' |
It '...' -Tag 'positive' |
| Kotest |
@Tags(...) |
@Tags(Positive) |
| Swift Testing |
@Tag(.<name>) |
@Test(.tags(.positive)) |
| Catch2 |
[tag] in name |
TEST_CASE("...", "[positive]") |
| doctest |
* doctest::test_suite("...") decorator |
TEST_CASE("..." *doctest::test_suite("positive")) |
Record which tests already have tags to avoid duplication.
Step 3: Classify each test method
Build one canonical inventory containing each discovered test exactly once.
Record the test identifier, behavioral classification, and traits in that
inventory; use the same rows for source edits, per-test reporting, totals, and
distribution counts. Do not hand-count a separate denominator. Before
publishing, reconcile the reported total with the number of inventory rows and
verify that every row contributes to each displayed trait count.
For each test method without traits, analyze:
- Method name -- names containing
Invalid, Fail, Error, Throw, Reject, BadInput, Null, None, Nil, Negative, raises_, _throws_, _returns_error suggest negative
- Assertion type --
Assert.ThrowsException / Assert.Throws / Should().Throw() / pytest.raises / expect(fn).toThrow / assertThrows / assert.Error(t, err) / expect { ... }.to raise_error / #[should_panic] / XCTAssertThrowsError / Should -Throw / EXPECT_THROW suggest negative
- Input values --
null / None / nil / undefined, "", 0, -1, int.MaxValue / sys.maxsize / Number.MAX_SAFE_INTEGER / math.MaxInt64 / i32::MAX, empty collections suggest boundary
- Setup complexity -- minimal setup with basic assertions suggests
smoke; external dependencies (file/db/net/env) suggest integration
- Comments and names -- references to issue numbers or "regression" / "bug" / "fix for #..." suggest
regression
- Timing assertions --
Stopwatch, BenchmarkDotNet, elapsed-time checks; pytest-benchmark fixtures; benchmark.js; JMH @Benchmark; go test -bench; criterion.rs; XCTMetric; Google Benchmark; kotlinx-benchmark suggest performance
- Feature centrality -- tests on primary public API entry points or critical user workflows suggest
critical-path
- Security patterns -- validates auth, checks permissions, sanitizes input, tests for injection, handles tokens/secrets suggest
security
- Parallel/async constructs -- per-language concurrency primitives (see Trait Taxonomy table) suggest
concurrency
- Fault injection -- simulates failures, tests retries, timeouts, or circuit breakers suggest
resilience
- State mutation -- deletes external records, drops resources, modifies shared/global state suggest
destructive
- Full-stack flow -- test spans entry point through data layer to final response, covering a complete user scenario suggest
end-to-end
- Config/settings -- loads configuration, tests missing keys, validates options, checks environment variables suggest
configuration
- Known instability -- test has skip / ignore annotations with comments about flakiness, or names contain "flaky" / "intermittent" suggest
flaky
- Default -- if the test verifies a normal success path, tag
positive
When in doubt between positive and negative, read the assertion: if it asserts success -> positive; if it asserts failure -> negative.
For a requested distribution or coverage-shape audit, use available production
code to map each test to the exact outcome it exercises before summarizing.
Call out duplicated boundary coverage and whether the test inventory represents
both sides of named thresholds and the observable collaborator outcomes on
business-critical paths. Keep these as concise distribution observations, not
new trait values. Do not perform mutation reasoning, prescribe new tests, or
expand into the behavioral-gap audit owned by test-gap-analysis.
Step 4: Apply trait attributes (or report only)
If the loaded language extension declares auto-edit for the framework, add the appropriate attribute to each test method. Place trait attributes adjacent to the existing test attribute. Examples:
Apply traits at the individual test-method/case level. Do not substitute one
class-level category for method-level classification: different methods usually
exercise different positive, negative, and boundary behavior.
MSTest:
[TestMethod]
[TestCategory("negative")]
[TestCategory("boundary")]
public void Parse_NullInput_ThrowsArgumentNullException() { ... }
xUnit:
[Fact]
[Trait("Category", "positive")]
[Trait("Category", "critical-path")]
public void CreateOrder_ValidItems_ReturnsConfirmation() { ... }
NUnit:
[Test]
[Category("regression")]
[Category("negative")]
public void Calculate_OverflowInput_ReturnsError() // Fix for #1234
{ ... }
pytest:
@pytest.mark.negative
@pytest.mark.boundary
def test_parse_none_input_raises_value_error():
...
JUnit 5:
@Test
@Tag("positive")
@Tag("critical-path")
void createOrder_validItems_returnsConfirmation() { ... }
TestNG:
@Test(groups = {"negative", "boundary"})
public void parse_nullInput_throwsIllegalArgumentException() { ... }
RSpec:
it "rejects null input", :negative, :boundary do
...
end
Pester:
It 'Rejects null input' -Tag 'negative','boundary' {
...
}
Kotest:
@Tags(Negative, Boundary)
class ParserSpec : StringSpec({
"rejects null input" { ... }
})
Swift Testing:
@Test(.tags(.negative, .boundary))
func parseNullInputThrows() throws { ... }
Catch2:
TEST_CASE("Parse null input throws", "[negative][boundary]") { ... }
If the loaded language extension declares report-only for the framework (Go standard testing, plain Jest/Vitest without convention, Rust without project-specific cfg, plain XCTest, plain GoogleTest, plain Mocha), do NOT modify source files. Instead emit a concise mapping from each test to its suggested tags. Recommend a project-wide convention only when the user asks how to persist or filter those tags; an analysis-only request should report and stop.
If the loaded language extension declares convention-based (e.g., Go //go:build integration, *_integration_test.go, GoogleTest INTEGRATION_* prefix), only emit canonical edits when the user has confirmed the project's convention. Otherwise treat as report-only.
Step 5: Generate trait summary
After tagging, produce a summary table. Include only traits with a non-zero
count unless the user asks for the full taxonomy; zero-filled rows obscure the
suite's actual shape. For a small report-only suite, keep the per-test mapping
and non-zero distribution together rather than expanding into a dashboard.
## Trait Distribution
| Trait | Count | % of Total |
|---------------|-------|------------|
| positive | 50 | 64.1% |
| negative | 28 | 35.9% |
| boundary | 8 | 10.3% |
| critical-path | 12 | 15.4% |
| **Total tests** | **78** | -- |
Note: Percentages exceed 100% because tests can have multiple traits.
Include observations such as:
- Ratio of positive to negative tests
- Whether critical-path tests exist for key public APIs
- Any tests that could not be confidently classified (list them for manual review)
boundary and every other specialized trait are additive. A boundary success
case still counts as positive; a rejected boundary still counts as negative.
Derive the positive/negative distribution after applying this rule.
Step 6: Verify edits before reporting
For every auto-edit framework, run the narrowest command that compiles the
edited attributes and confirms test discovery. This is required even when the
user asks only to add tags: syntactically plausible attributes are not a
completed edit.
| Framework |
Minimum verification |
| .NET |
Run dotnet build <test-project>, then confirm discovery with dotnet test <test-project> --list-tests --no-build. Do not execute the suite unless the user asks; route execution to run-tests. |
| pytest |
collect the edited suite with the repository's configured pytest command |
| JUnit/TestNG |
compile tests through the repository's Maven/Gradle test task |
| Other auto-edit frameworks |
Use the repository's narrowest compile or test-discovery command |
If an edit or patch application was uncertain, re-open the complete edited file
before verification and reconcile every inventory row with the actual
attribute next to that test. Do not report success from a partial diff or from
the intended patch. If verification fails, report the exact command and error;
never publish a successful distribution handoff for uncompiled edits.
Validation
Common Pitfalls
| Pitfall |
Solution |
| Guessing traits without reading the test body |
Always read assertions and setup to classify accurately |
Tagging a test only as boundary without positive/negative |
Every test should also be positive or negative -- boundary is additive |
| Using the wrong attribute syntax for the detected framework |
Match the attribute style to the loaded language extension (don't put [TestCategory] in an xUnit project or @pytest.mark.x in a unittest test) |
| Duplicating an existing category attribute |
Check for pre-existing traits in Step 2 before adding |
Over-tagging as critical-path |
Reserve for tests on primary public entry points, not every helper |
| Editing Go / plain Jest / plain Rust / plain XCTest / plain GoogleTest source |
These are report-only by default — emit a Markdown table instead. Only edit if the user confirms a project-wide convention (build tag, file suffix, describe-prefix, test-plan grouping). |
| Inventing tag prefixes for convention-based frameworks |
Confirm the project's existing convention before adopting one — don't guess between _integration_test.go, //go:build integration, or IntegrationTest prefix |
| Missing language-specific concurrency / async primitives |
Each language has its own primitives — read the loaded language extension and the Trait Taxonomy concurrency row before classifying as concurrency |
1---2name: test-tagging3description: Classifies existing tests by standard traits and reports their distribution. MUST USE to tag all tests with category attributes, categorize/tag/label each test, compare happy vs error paths, audit the test mix, describe coverage shape by test type, or tag then verify the project builds. Read bodies when names mislead. Apply canonical attributes; otherwise report only. DO NOT USE for test-quality audits, executed coverage or CRAP, behavioral gaps, writing tests, or migration.4license: MIT5---67# Test Trait Tagging89Analyze an existing test suite in any supported language and apply a standardized set of trait tags to each test method, giving teams visibility into their test distribution (positive vs. negative, critical-path coverage, smoke tests, etc.).1011> **Language-specific guidance**: Call the `test-analysis-extensions` skill to discover available extension files, then read the file matching the target codebase. The extension file documents framework-specific tag attributes and a "tag-support capability" (auto-edit, report-only, or convention-based) that drives whether this skill modifies source files or only emits a report.1213## When to Use1415- Auditing a test project to understand the mix of test types16- Adding trait attributes to untagged tests17- Generating a summary report of trait distribution across a test suite18- Reviewing whether critical paths have sufficient coverage1920## When Not to Use2122- Writing new tests from scratch (use `code-testing-agent` for any language, or `writing-mstest-tests` for MSTest)23- Running or filtering tests (use `run-tests` for .NET; equivalent native runners elsewhere)24- Migrating between test frameworks25- General quality, smell, flakiness, or assertion audits (use `test-anti-patterns` or the matching analysis skill)26- Diagnostic .NET executed line/branch/Cobertura interpretation or project-wide CRAP risk (use `coverage-analysis`); raw coverage collection (use `run-tests` for .NET, native tooling otherwise)27- CRAP analysis for a named method, class, or file (use `crap-score`)28- Behavioral gaps where a test would survive broken production logic (use `test-gap-analysis`)2930## Inputs3132| Input | Required | Description |33|-------|----------|-------------|34| Test project or files | Yes | Path to the test project, folder, or specific test files to analyze |35| Scope | No | `tag` (apply canonical attributes, or a confirmed project convention), `audit` (report only), or `both` (default: `both`). Frameworks declared `report-only` always emit a report; `convention-based` frameworks edit only after the user confirms the convention. |36| Framework | No | Auto-detected. Override when detection fails. |3738## Trait Taxonomy3940Use exactly these trait names and values. Do not invent new trait values outside this table.4142| Trait Value | Meaning | Heuristics |43|-------------|---------|------------|44| `positive` | Verifies expected behavior under normal/valid conditions | Asserts success, valid output, expected state, no exceptions for valid input |45| `negative` | Verifies correct handling of invalid input, errors, or edge cases | Asserts exceptions, error codes, validation failures, rejects bad input |46| `boundary` | Tests limits, thresholds, empty/null/None/nil inputs, min/max values | Operates on `0`, `-1`, `int.MaxValue` / `sys.maxsize` / `Number.MAX_SAFE_INTEGER` / `math.MaxInt64` / `i32::MAX`, empty string, null/None/nil/undefined, empty collection, boundary of valid range |47| `critical-path` | Core workflow that must never break; breakage blocks users | Tests the primary success scenario of a key public API or user-facing feature |48| `smoke` | Quick sanity check that the system is operational | Fast, no complex setup, verifies basic wiring (e.g., service resolves, endpoint returns 200) |49| `regression` | Reproduces a specific previously-reported bug | References a bug ID, issue number, or describes a fix in its name or comments |50| `integration` | Crosses process, network, or persistence boundaries | Uses real database, HTTP client, file system, external service, or multi-component setup |51| `end-to-end` | Full user workflow spanning the entire application stack | Exercises a complete scenario from entry point to final result, distinct from single-boundary `integration` |52| `performance` | Validates timing, throughput, or resource consumption | Asserts on elapsed time, memory, allocations, or uses benchmark harness (BenchmarkDotNet, pytest-benchmark, benchmark.js, JMH, `go test -bench`, criterion.rs, XCTMetric, kotlinx-benchmark, Google Benchmark) |53| `security` | Verifies authentication, authorization, input sanitization, or secrets handling | Tests for SQL injection, XSS, CSRF, unauthorized access, token validation, permission checks |54| `concurrency` | Validates thread safety, parallelism, or async correctness | Uses `Task.WhenAll` / `Parallel.ForEach` / `SemaphoreSlim` (.NET); `asyncio.gather` / `threading.Lock` / `multiprocessing` (Python); `Promise.all` / worker threads (JS/TS); `CompletableFuture` / `ExecutorService` / `synchronized` (Java); `go func` / `sync.WaitGroup` / `sync.Mutex` / `chan` (Go); `Mutex` / `Thread.new` (Ruby); `tokio::spawn` / `Arc<Mutex<_>>` / `crossbeam` (Rust); `DispatchQueue` / `actor` (Swift); `coroutineScope` / `Mutex` (Kotlin); `Start-Job` / `RunspacePool` (PowerShell); `std::thread` / `std::mutex` (C++); reproduces race conditions |55| `resilience` | Tests retry logic, timeouts, circuit breakers, or graceful degradation | Asserts behavior under transient failures, network drops, or service unavailability (e.g., Polly, tenacity, p-retry, resilience4j, hystrix, opossum, retry-go) |56| `destructive` | Mutates shared or external state that is hard to roll back | Deletes records, drops resources, modifies global config -- useful for CI isolation decisions |57| `configuration` | Verifies settings loading, defaults, environment behavior | Tests missing config keys, invalid values, environment variable fallbacks, options validation |58| `flaky` | Known to intermittently fail (meta-tag for test health tracking) | Mark tests the team knows are unreliable; used to quarantine or prioritize stabilization |5960A single test may have **multiple traits** (e.g., both `negative` and `boundary`). At minimum, every test should receive one of `positive` or `negative`.6162## Workflow6364### Step 1: Detect the language, framework, and tagging capability6566Identify the codebase's language and test framework. Call the `test-analysis-extensions` skill and read the matching extension file. The extension file declares a **tag-support capability** for each framework:6768- **`auto-edit`** — framework has canonical tag syntax this skill can safely insert (.NET `[TestCategory]` / `[Trait]` / `[Category]` / `[Property]`, pytest `@pytest.mark.<name>`, JUnit 5 `@Tag("...")`, TestNG `groups = {"..."}`, RSpec metadata `it "..." , :tag => true`, Pester `-Tag '...'`, Kotest `@Tags(...)`, Swift Testing `@Tag(.tagName)`, Catch2 `[tag]`, doctest `* doctest::test_suite("tag")` decorator).69- **`report-only`** — framework has no canonical, agreed-upon tag attribute; report tags in a Markdown table only and do not edit source (Go standard `testing` without build-tag conventions, Jest/Vitest without consistent describe-prefix convention, Rust without project-specific cfg conventions, XCTest without a test plan, GoogleTest without test-name prefix conventions, Mocha without describe-prefix conventions).70- **`convention-based`** — framework uses naming or file conventions for tagging (Go `//go:build integration` build tags, file-name suffixes like `*_integration_test.go`, GoogleTest `INTEGRATION_*` filter prefix). Only emit canonical edits when the user has confirmed the project convention; otherwise treat as `report-only`.7172Capture the capability before Step 4.7374### Step 2: Scan existing traits7576Check which tests already have trait attributes. Use the loaded language extension as the source of truth — examples:7778| Framework | Existing Attribute | Example |79|-----------|--------------------|---------|80| MSTest | `[TestCategory("...")]` | `[TestCategory("positive")]` |81| xUnit | `[Trait("Category", "...")]` | `[Trait("Category", "positive")]` |82| NUnit | `[Category("...")]` | `[Category("positive")]` |83| TUnit | `[Property("Category", "...")]` | `[Property("Category", "positive")]` |84| JUnit 5 | `@Tag("...")` | `@Tag("positive")` |85| TestNG | `@Test(groups = {"..."})` | `@Test(groups = {"positive"})` |86| pytest | `@pytest.mark.<name>` | `@pytest.mark.positive` |87| RSpec | metadata after `it` | `it "...", :positive do` |88| Pester | `-Tag '...'` | `It '...' -Tag 'positive'` |89| Kotest | `@Tags(...)` | `@Tags(Positive)` |90| Swift Testing | `@Tag(.<name>)` | `@Test(.tags(.positive))` |91| Catch2 | `[tag]` in name | `TEST_CASE("...", "[positive]")` |92| doctest | `* doctest::test_suite("...")` decorator | `TEST_CASE("..." *doctest::test_suite("positive"))` |9394Record which tests already have tags to avoid duplication.9596### Step 3: Classify each test method9798Build one canonical inventory containing each discovered test exactly once.99Record the test identifier, behavioral classification, and traits in that100inventory; use the same rows for source edits, per-test reporting, totals, and101distribution counts. Do not hand-count a separate denominator. Before102publishing, reconcile the reported total with the number of inventory rows and103verify that every row contributes to each displayed trait count.104105For each test method without traits, analyze:1061071. **Method name** -- names containing `Invalid`, `Fail`, `Error`, `Throw`, `Reject`, `BadInput`, `Null`, `None`, `Nil`, `Negative`, `raises_`, `_throws_`, `_returns_error` suggest `negative`1082. **Assertion type** -- `Assert.ThrowsException` / `Assert.Throws` / `Should().Throw()` / `pytest.raises` / `expect(fn).toThrow` / `assertThrows` / `assert.Error(t, err)` / `expect { ... }.to raise_error` / `#[should_panic]` / `XCTAssertThrowsError` / `Should -Throw` / `EXPECT_THROW` suggest `negative`1093. **Input values** -- `null` / `None` / `nil` / `undefined`, `""`, `0`, `-1`, `int.MaxValue` / `sys.maxsize` / `Number.MAX_SAFE_INTEGER` / `math.MaxInt64` / `i32::MAX`, empty collections suggest `boundary`1104. **Setup complexity** -- minimal setup with basic assertions suggests `smoke`; external dependencies (file/db/net/env) suggest `integration`1115. **Comments and names** -- references to issue numbers or "regression" / "bug" / "fix for #..." suggest `regression`1126. **Timing assertions** -- `Stopwatch`, `BenchmarkDotNet`, elapsed-time checks; pytest-benchmark fixtures; benchmark.js; JMH `@Benchmark`; `go test -bench`; criterion.rs; XCTMetric; Google Benchmark; kotlinx-benchmark suggest `performance`1137. **Feature centrality** -- tests on primary public API entry points or critical user workflows suggest `critical-path`1148. **Security patterns** -- validates auth, checks permissions, sanitizes input, tests for injection, handles tokens/secrets suggest `security`1159. **Parallel/async constructs** -- per-language concurrency primitives (see Trait Taxonomy table) suggest `concurrency`11610. **Fault injection** -- simulates failures, tests retries, timeouts, or circuit breakers suggest `resilience`11711. **State mutation** -- deletes external records, drops resources, modifies shared/global state suggest `destructive`11812. **Full-stack flow** -- test spans entry point through data layer to final response, covering a complete user scenario suggest `end-to-end`11913. **Config/settings** -- loads configuration, tests missing keys, validates options, checks environment variables suggest `configuration`12014. **Known instability** -- test has skip / ignore annotations with comments about flakiness, or names contain "flaky" / "intermittent" suggest `flaky`12115. **Default** -- if the test verifies a normal success path, tag `positive`122123When in doubt between `positive` and `negative`, read the assertion: if it asserts success -> `positive`; if it asserts failure -> `negative`.124125For a requested distribution or coverage-shape audit, use available production126code to map each test to the exact outcome it exercises before summarizing.127Call out duplicated boundary coverage and whether the test inventory represents128both sides of named thresholds and the observable collaborator outcomes on129business-critical paths. Keep these as concise distribution observations, not130new trait values. Do not perform mutation reasoning, prescribe new tests, or131expand into the behavioral-gap audit owned by `test-gap-analysis`.132133### Step 4: Apply trait attributes (or report only)134135**If the loaded language extension declares `auto-edit` for the framework**, add the appropriate attribute to each test method. Place trait attributes adjacent to the existing test attribute. Examples:136137Apply traits at the individual test-method/case level. Do not substitute one138class-level category for method-level classification: different methods usually139exercise different positive, negative, and boundary behavior.140141**MSTest:**142```csharp143[TestMethod]144[TestCategory("negative")]145[TestCategory("boundary")]146public void Parse_NullInput_ThrowsArgumentNullException() { ... }147```148149**xUnit:**150```csharp151[Fact]152[Trait("Category", "positive")]153[Trait("Category", "critical-path")]154public void CreateOrder_ValidItems_ReturnsConfirmation() { ... }155```156157**NUnit:**158```csharp159[Test]160[Category("regression")]161[Category("negative")]162public void Calculate_OverflowInput_ReturnsError() // Fix for #1234163{ ... }164```165166**pytest:**167```python168@pytest.mark.negative169@pytest.mark.boundary170def test_parse_none_input_raises_value_error():171 ...172```173174**JUnit 5:**175```java176@Test177@Tag("positive")178@Tag("critical-path")179void createOrder_validItems_returnsConfirmation() { ... }180```181182**TestNG:**183```java184@Test(groups = {"negative", "boundary"})185public void parse_nullInput_throwsIllegalArgumentException() { ... }186```187188**RSpec:**189```ruby190it "rejects null input", :negative, :boundary do191 ...192end193```194195**Pester:**196```powershell197It 'Rejects null input' -Tag 'negative','boundary' {198 ...199}200```201202**Kotest:**203```kotlin204@Tags(Negative, Boundary)205class ParserSpec : StringSpec({206 "rejects null input" { ... }207})208```209210**Swift Testing:**211```swift212@Test(.tags(.negative, .boundary))213func parseNullInputThrows() throws { ... }214```215216**Catch2:**217```cpp218TEST_CASE("Parse null input throws", "[negative][boundary]") { ... }219```220221**If the loaded language extension declares `report-only` for the framework** (Go standard `testing`, plain Jest/Vitest without convention, Rust without project-specific cfg, plain XCTest, plain GoogleTest, plain Mocha), do NOT modify source files. Instead emit a concise mapping from each test to its suggested tags. Recommend a project-wide convention only when the user asks how to persist or filter those tags; an analysis-only request should report and stop.222223**If the loaded language extension declares `convention-based`** (e.g., Go `//go:build integration`, `*_integration_test.go`, GoogleTest `INTEGRATION_*` prefix), only emit canonical edits when the user has confirmed the project's convention. Otherwise treat as `report-only`.224225### Step 5: Generate trait summary226227After tagging, produce a summary table. Include only traits with a non-zero228count unless the user asks for the full taxonomy; zero-filled rows obscure the229suite's actual shape. For a small report-only suite, keep the per-test mapping230and non-zero distribution together rather than expanding into a dashboard.231232```233## Trait Distribution234235| Trait | Count | % of Total |236|---------------|-------|------------|237| positive | 50 | 64.1% |238| negative | 28 | 35.9% |239| boundary | 8 | 10.3% |240| critical-path | 12 | 15.4% |241| **Total tests** | **78** | -- |242243Note: Percentages exceed 100% because tests can have multiple traits.244```245246Include observations such as:247- Ratio of positive to negative tests248- Whether critical-path tests exist for key public APIs249- Any tests that could not be confidently classified (list them for manual review)250251`boundary` and every other specialized trait are additive. A boundary success252case still counts as `positive`; a rejected boundary still counts as `negative`.253Derive the positive/negative distribution after applying this rule.254255### Step 6: Verify edits before reporting256257For every `auto-edit` framework, run the narrowest command that compiles the258edited attributes and confirms test discovery. This is required even when the259user asks only to add tags: syntactically plausible attributes are not a260completed edit.261262| Framework | Minimum verification |263|---|---|264| .NET | Run `dotnet build <test-project>`, then confirm discovery with `dotnet test <test-project> --list-tests --no-build`. Do not execute the suite unless the user asks; route execution to `run-tests`. |265| pytest | collect the edited suite with the repository's configured pytest command |266| JUnit/TestNG | compile tests through the repository's Maven/Gradle test task |267| Other auto-edit frameworks | Use the repository's narrowest compile or test-discovery command |268269If an edit or patch application was uncertain, re-open the complete edited file270before verification and reconcile every inventory row with the actual271attribute next to that test. Do not report success from a partial diff or from272the intended patch. If verification fails, report the exact command and error;273never publish a successful distribution handoff for uncompiled edits.274275## Validation276277- [ ] Every test method has at least one trait classification (`positive` or `negative` at minimum) — in the report for `report-only` frameworks, or as an attribute for `auto-edit` frameworks278- [ ] The total equals the per-test inventory count, and displayed trait counts were derived from that inventory279- [ ] No invented trait values outside the taxonomy table280- [ ] Existing trait attributes were preserved, not duplicated281- [ ] The trait summary table was generated282- [ ] For `auto-edit` frameworks, the project still builds / tests still discover after changes (`dotnet build` / `pytest --collect-only` / `mvn test-compile` / `go vet ./...` / `cargo check --tests` / `npm run test:list` / `Invoke-Pester -PassThru -Skip` / equivalent)283- [ ] For `report-only` frameworks, no source files were modified284- [ ] For `convention-based` frameworks, edits were applied ONLY when a project convention was confirmed285286## Common Pitfalls287288| Pitfall | Solution |289|---------|----------|290| Guessing traits without reading the test body | Always read assertions and setup to classify accurately |291| Tagging a test only as `boundary` without `positive`/`negative` | Every test should also be `positive` or `negative` -- `boundary` is additive |292| Using the wrong attribute syntax for the detected framework | Match the attribute style to the loaded language extension (don't put `[TestCategory]` in an xUnit project or `@pytest.mark.x` in a unittest test) |293| Duplicating an existing category attribute | Check for pre-existing traits in Step 2 before adding |294| Over-tagging as `critical-path` | Reserve for tests on primary public entry points, not every helper |295| Editing Go / plain Jest / plain Rust / plain XCTest / plain GoogleTest source | These are `report-only` by default — emit a Markdown table instead. Only edit if the user confirms a project-wide convention (build tag, file suffix, describe-prefix, test-plan grouping). |296| Inventing tag prefixes for convention-based frameworks | Confirm the project's existing convention before adopting one — don't guess between `_integration_test.go`, `//go:build integration`, or `IntegrationTest` prefix |297| Missing language-specific concurrency / async primitives | Each language has its own primitives — read the loaded language extension and the Trait Taxonomy concurrency row before classifying as `concurrency` |