Skip to content

Conversation

Copy link
Contributor

Copilot AI commented Nov 3, 2025

Adds test coverage for #51398 (solution filters containing .shproj files) and #51481 (Platform configuration support in solution builds).

Tests Added

  • RunWithSolutionFilterContainingSharedProject_ShouldSkipSharedProjectAndSucceed: Verifies shared projects are automatically filtered out when running tests via solution filters, preventing build failures from non-buildable project types.

  • RunWithSolutionAndPlatformConfiguration_ShouldRespectPlatform: Validates that --property:Platform=<value> correctly filters projects based on solution platform configuration mappings (e.g., only projects configured for x86 run when Platform=x86 is specified).

Test Assets

  • MultiTestProjectSolutionWithSharedProject: Solution filter including a .shproj, TestProject (included), and OtherTestProject (excluded by filter)
  • MultiTestProjectSolutionWithPlatforms: Solution with x86/x64 platform configurations where OtherTestProject is excluded from x86 builds

Both assets include global.json with "test": { "runner": "Microsoft.Testing.Platform" } to enable MTP mode.


💡 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.

- Test for solution filter with shared projects (.shproj) being properly skipped
- Test for Platform configuration being respected in solution builds
- Both test assets include proper global.json to enable MTP mode

Co-authored-by: Youssef1313 <[email protected]>
Copilot AI changed the title [WIP] Make requested changes to #51411 Add tests for solution filter with shared projects and platform configuration support Nov 3, 2025
Copilot AI requested a review from Youssef1313 November 3, 2025 13:49
Copilot finished work on behalf of Youssef1313 November 3, 2025 13:49
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