test: add edge-case tests for TestClassShouldBeValidAnalyzer static-class guard

[Amaury Levé (Evangelink)]

Summary

Fixes#9951.

The TestClassShouldBeValidAnalyzer (MSTEST0002) static-class validation branch had two untested paths. This adds two focused tests to TestClassShouldBeValidAnalyzerTests:

• WhenStaticTestClassContainsDerivedTestMethodAttribute_Diagnostic — a static [TestClass] with [DataTestMethod] (a TestMethodAttribute subclass) fires MSTEST0002, exercising the attribute.AttributeClass.Inherits(testMethodAttributeSymbol) branch of the static-class guard.
• WhenStaticTestClassContainsGlobalTestInitialize_NoDiagnostic — a static [TestClass] with only [GlobalTestInitialize] does NOT fire, confirming the guard intentionally omits GlobalTestInitialize from its checked attribute set (TestInitialize, TestCleanup, ClassInitialize, ClassCleanup, and TestMethod-derived).

Test status

Full MSTest.Analyzers.UnitTests suite passes on net472 and net8.0.

[Copilot]

Pull request overview

Adds focused MSTEST0002 coverage for static test-class edge cases.

Changes:

• Verifies derived TestMethodAttribute usage triggers the diagnostic.
• Verifies GlobalTestInitialize alone does not trigger it.

Show a summary per file

File	Description
test/UnitTests/MSTest.Analyzers.UnitTests/TestClassShouldBeValidAnalyzerTests.cs	Adds diagnostic and no-diagnostic edge-case tests.

Review details

• Files reviewed: 1/1 changed files
• Comments generated: 0
• Review effort level: Medium

[github-actions]

🧪 Test quality grade — PR #9967

Grade	Test	Notes
B (80–89)	new TestClassShouldBeValidAnalyzerTests. WhenStaticTestClassContainsDerivedTestMethodAttribute_ Diagnostic	Strong multi-facet assertion via VerifyCodeFixAsync; body is slightly over 30 lines — consider trimming unnecessary boilerplate in the code/fixedCode strings.
A (90–100)	new TestClassShouldBeValidAnalyzerTests. WhenStaticTestClassContainsGlobalTestInitialize_ NoDiagnostic	No issues found.

_{This advisory comment was generated automatically. Grades are heuristic
and informational — they do not block merging. Re-run with
/grade-tests.}

Warning

Firewall blocked 1 domain

The following domain was blocked by the firewall during workflow execution:

• awmgmcpg

To allow these domains, add them to the network.allowed list in your workflow frontmatter:

network:
allowed:
- defaults
- "awmgmcpg"

See Network Configuration for more information.

🤖 Automated content by GitHub Copilot. Generated by the Grade Tests on PR (on open / sync) workflow. · 33.1 AIC · ⌖ 6.07 AIC · ⊞ 8.9K · [◷]( · ◷)