Skip to content

Conversation

@thomhurst
Copy link
Owner

Summary

  • Adds the class name to the Test column in the GitHub Reporter Details table
  • Format changed from MethodName(args) to ClassName.MethodName(args)
  • Makes it easier to identify which specific test is being referenced

Fixes #4323

Test plan

  • Verified build compiles successfully
  • Ran GitHubReporter unit tests - all 4 pass
  • Verify on a real GitHub Actions run that the format looks correct

🤖 Generated with Claude Code

Fixes #4323 - The GitHub Reporter now displays the class name alongside
the method name in the Test column of the Details table, making it
easier to identify which specific test is being referenced.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@thomhurst
Copy link
Owner Author

Summary: Adds class name to GitHub Reporter test output

Critical Issues: None found

Suggestions:

  • Consider consistency with JUnitXmlWriter.cs which uses similar pattern for TestMethodIdentifierProperty.TypeName
  • The null-safety fallback logic is correct and handles cases where class info may not be available

TUnit Rules Compliance:
✅ No dual-mode concerns
✅ No snapshot test updates needed
✅ No public API changes
✅ No reflection concerns
✅ No performance concerns
✅ No VSTest usage

Verdict: APPROVE - Clean change that improves test identification in GitHub Actions

@thomhurst thomhurst merged commit a0fb6fb into main Jan 12, 2026
12 of 13 checks passed
@thomhurst thomhurst deleted the feat/github-reporter-class-name branch January 12, 2026 19:29
This was referenced Jan 12, 2026
Sign up for free to 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.

[Feature]: add class name to Github Report

2 participants