Update Testing framework to v4 - #1649
renovate[bot] wants to merge 5 commits into
Conversation
|
Azure Pipelines: There may be pipelines that require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
Pull request overview
This PR updates the centralized test runner package versions used by the repository’s xUnit v3 + Microsoft.Testing.Platform (MTP v2) setup, moving to the new major release of the xunit.v3.*mtp-v2 packages.
Changes:
- Bump
xunit.v3.core.mtp-v2from3.2.2to4.0.0. - Bump
xunit.v3.mtp-v2from3.2.2to4.0.0.
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
|
One or more custom setup steps configured for this repository failed during this Copilot code review run: Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review. Note You can configure setup steps for Copilot code review separately from Copilot cloud agent with a |
1c16295 to
fbbda73
Compare
fbbda73 to
1337e9d
Compare
There was a problem hiding this comment.
🔵 Needs a closer look
A major-version test framework runner upgrade can introduce breaking behavioral changes that should be validated by a full test run/CI before merging.
Review details
- Files reviewed: 1/1 changed files
- Comments generated: 0 new
- Review effort level: Lite
1337e9d to
7179327
Compare
7179327 to
10e0874
Compare
There was a problem hiding this comment.
Copilot review overview
🟡 Changes recommended
Critical framework compatibility issues must be resolved before approval.
Get a fresh assessment by requesting another Copilot review.
Review effort: Lite
Findings: 4
Open (6)
xUnit v4 breaks net472 test project compatibility · New xUnit v4 is incompatible with Xunit.StaFact 3 · New Version 4.0.0 ofxunit.v3.mtp-v2does not provide anet472asset, but both test projects add… Version 4.0.0 ofxunit.v3.core.mtp-v2does not provide anet472asset, while… These two package versions are duplicated and must remain in sync. Consider introducing a single… These two package versions are duplicated and must remain in sync. Consider introducing a single…
|
One or more custom setup steps configured for this repository failed during this Copilot code review run: Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review. Note You can configure setup steps for Copilot code review separately from Copilot cloud agent with a |
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
/azp run |
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The dependency upgrade and configuration migration have no unresolved blocking issues.
Review effort: Lite
Findings: None
Resolved since last review (6)
xUnit v4 is incompatible with Xunit.StaFact 3 xUnit v4 breaks net472 test project compatibility Version 4.0.0 ofxunit.v3.mtp-v2does not provide anet472asset, but both test projects add… Version 4.0.0 ofxunit.v3.core.mtp-v2does not provide anet472asset, while… These two package versions are duplicated and must remain in sync. Consider introducing a single… These two package versions are duplicated and must remain in sync. Consider introducing a single…
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
There was a problem hiding this comment.
Copilot review overview
🔵 Needs a closer look
The available review evidence does not establish sufficient confidence for automated approval.
Review effort: Lite
Findings: None
Resolved since last review (7)
Update Xunit.Combinatorial for xUnit v4 compatibility xUnit v4 is incompatible with Xunit.StaFact 3 xUnit v4 breaks net472 test project compatibility Version 4.0.0 ofxunit.v3.mtp-v2does not provide anet472asset, but both test projects add… Version 4.0.0 ofxunit.v3.core.mtp-v2does not provide anet472asset, while… These two package versions are duplicated and must remain in sync. Consider introducing a single… These two package versions are duplicated and must remain in sync. Consider introducing a single…


This PR contains the following updates:
3.2.2→4.0.13.2.2→4.0.1Configuration
📅 Schedule: (in timezone America/Los_Angeles)
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about these updates again.
This PR was generated by Mend Renovate. View the repository job log.