Skip to content

fix: Update missed STAThread snapshot files with GeneratedCode attributes - #4693

Merged
thomhurst merged 4 commits into
mainfrom
fix/stathread-snapshot-update
Feb 6, 2026
Merged

fix: Update missed STAThread snapshot files with GeneratedCode attributes#4693
thomhurst merged 4 commits into
mainfrom
fix/stathread-snapshot-update

Conversation

@thomhurst

Copy link
Copy Markdown
Owner

Summary

  • PR Mark TestMetadataGenerator output with GeneratedCode attribute #4690 added [ExcludeFromCodeCoverage] and [GeneratedCode] attributes to TestMetadataGenerator output but missed the 4 STAThread snapshot files
  • This caused the Windows CI pipeline to fail with Verification failed for 'STAThreadTests.Test.DotNet10_0'
  • Updates all 4 STAThread .verified.txt files (DotNet10_0, DotNet9_0, DotNet8_0, Net4_7) with the same attribute additions

Test plan

  • Windows pipeline modularpipeline (windows-latest) passes (specifically RunSourceGeneratorTestsModule)
  • STAThreadTests snapshot verification succeeds on all TFMs

🤖 Generated with Claude Code

CopilotAIand others added 4 commits February 6, 2026 15:32
…ataGenerator output
Co-authored-by: thomhurst <30480171+thomhurst@users.noreply.github.com>
…dCode attributes
The original PR only updated DotNet9_0 variants but missed DotNet8_0,
DotNet10_0, and Net4_7 framework-specific snapshot files.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…utes
The PR #4690 missed updating the STAThreadTests snapshot files when adding
[ExcludeFromCodeCoverage] and [GeneratedCode] attributes to source generator
output, causing the Windows CI pipeline to fail on snapshot verification.
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@thomhurst
thomhurst enabled auto-merge (squash) February 6, 2026 19:11
@thomhurst
thomhurst merged commit bc42953 into mainFeb 6, 2026
12 of 13 checks passed
@thomhurst
thomhurst deleted the fix/stathread-snapshot-update branch February 6, 2026 19:32
This was referenced Feb 6, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@thomhurst