Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Jan 12, 2026

Ports the changes from AArnott/Nerdbank.MessagePack#855 to update copilot instructions for Microsoft.Testing.Platform v2 / xunit v3.

Changes Made

Updated .github/copilot-instructions.md to replace outdated VSTest --filter syntax with correct MTP v2/xunit v3 test commands.

Added detailed testing section with working examples:

  • Running all tests: dotnet test --no-build -c Release
  • Running individual test methods: -- --filter-method ClassName.MethodName
  • Running test classes: -- --filter-class ClassName
  • Wildcard filtering: -- --filter-method "*Pattern*"
  • Trait-based filtering: -- --filter-trait and -- --filter-not-trait
  • Framework targeting: --framework net9.0
  • Listing tests: dotnet run -- --list-tests

Key syntax differences from VSTest:

  • Options pass to test runner via -- separator, not directly to dotnet test
  • Traditional --filter "TestCategory!=FailsInCloudTest" replaced with -- --filter-not-trait "FailsInCloudTest=true"
  • Filter options are --filter-method, --filter-class, --filter-namespace, --filter-trait, not query expressions

Validation

All commands were verified working with the Library.Tests project before documentation.

Original prompt

Port the change from this pull request to the Library.Template repo.


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Copilot AI changed the title [WIP] Port changes from existing pull request to Library.Template repo Clarify which PR needs to be ported to Library.Template Jan 12, 2026
Copilot AI requested a review from AArnott January 12, 2026 04:27
@AArnott
Copy link
Owner

AArnott commented Jan 12, 2026

@copilot the PR I want you to port to this repo is AArnott/Nerdbank.MessagePack#855

Co-authored-by: AArnott <3548+AArnott@users.noreply.github.com>
Copy link
Contributor Author

Copilot AI commented Jan 12, 2026

@copilot the PR I want you to port to this repo is AArnott/Nerdbank.MessagePack#855

Done. Ported the MTP v2/xunit v3 test command documentation from that PR to this repository's copilot instructions. All commands were verified working with the Library.Tests project. (181d9f1)

Copilot AI changed the title Clarify which PR needs to be ported to Library.Template Port MTP v2/xunit v3 test commands to copilot instructions Jan 12, 2026
@AArnott AArnott marked this pull request as ready for review January 12, 2026 12:52
@AArnott AArnott enabled auto-merge (squash) January 12, 2026 12:54
@codecov
Copy link

codecov bot commented Jan 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (82fb1dd) to head (181d9f1).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #469   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            1         1           
  Lines            2         2           
=========================================
  Hits             2         2           
Flag Coverage Δ
Linux 50.00% <ø> (ø)
Windows 100.00% <ø> (ø)
macOS 50.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AArnott AArnott disabled auto-merge January 12, 2026 12:59
@AArnott AArnott merged commit d9b1245 into main Jan 12, 2026
10 checks passed
@AArnott AArnott deleted the copilot/port-change-to-library-template branch January 12, 2026 12:59
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.

2 participants