Repository navigation
[MWB][UITests] Fix StyleCop layout warnings (stack 3/4) - #51648
Merged
Clint Rutkas (crutkas) merged 2 commits intoNov 3, 2026
Merged
Clint Rutkas (crutkas) merged 2 commits into
Clint Rutkas (crutkas) merged 2 commits into
Conversation
Formatting only for SA1107, SA1116, SA1117, SA1118, SA1130, SA1501, SA1502, SA1513 and SA1515. The only token changes are delegate-to-lambda (SA1130) and hoisting a multi-line argument into a local (SA1118).
🧭 PR intakeVisual evidence: Not needed — The changed files do not indicate a visible UI change. No visual evidence is expected. Recommendation
✅ Ready for reviewThis PR passed the automated intake checks and is ready for maintainer review. Automated PR intake; PowerToys maintainers make final decisions. |
11 tasks
Member
Author
|
/azp run |
|
Azure Pipelines: Successfully started running 1 pipeline(s). |
The StyleCop layout fix expands the one-line delegating wrapper, which broke an exact single-line source match in mwbEndpointDiagnostics.Tests.ps1.
11 tasks
Gleb Khmyznikov (khmyznikov)
approved these changes
Nov 3, 2026
Clint Rutkas (crutkas)
deleted the
crutkas-fix-mwb-uitests-stylecop-warnings
branch
November 3, 2026 23:48
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary of the Pull Request
Mouse without border UI tests were added with allowing baseline warnings. i'm on a war against warnings. This is a stacked PR to keep the reviews small and tight.
PR Checklist
Detailed Description of the Pull Request / Additional comments
Validation Steps Performed
AI-generated summary
Stack 3/4 (base: #51647). Fixes StyleCop layout warnings in
MouseWithoutBorders.UITests: SA1107, SA1116, SA1117, SA1118, SA1130, SA1501, SA1502, SA1513, and SA1515.delegate-> lambda (SA1130). This is C# 3 syntax, so it still compiles under Windows PowerShell 5.1.provision["Configuration"] ?? throw ...argument into a local before its loop (SA1118) inTwoEndpointFixtureandEndpointRecoveryFixture.NativeSupport.cs/Receiver.cshere. The one-type-per-file move is layer 4, so that diff reviews as a pure move.Validation (local, x64 Debug): msbuild exit 0, with only SA1402 x6 and SA1649 x2 remaining (fixed in layer 4); the Windows PowerShell 5.1
Add-Typepayload compile passes;ClipboardObservationDistinguishesContentionFromReadableDataandStartMenuIdentityRequiresExactOsPathClassAndInteractiveSessionpass 2/2;Scripts\Test-SuitePreparation.ps1passes under PS 5.1 (89 checks). Sandbox end-to-end suites were not run locally.