Skip to content

Commit 9326bae

Browse files
azchohfiCopilot
andcommitted
Strip ANSI in the remaining PWA appId assertion
The previous commit made CleanResult ANSI-aware, but ProjectConfiguratorParsesPWASuccessfullyOnCIIfAppIdIsProvided never called it, so its 'AppId: <id>' assertion still compared plain text against Spectre-colored output and kept failing on all three CI runners. Reproduced locally by setting GITHUB_ACTIONS=true, which makes Spectre's default profile enrichers enable ANSI the same way CI does. Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com> Copilot-Session: 00c29930-8fdc-4c0c-80cc-d45ed0a5d077
1 parent 2786242 commit 9326bae

1 file changed

Lines changed: 2 additions & 0 deletions

File tree

MSStore.CLI.UnitTests/ProjectConfiguratorTests.cs

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -747,6 +747,8 @@ public async Task ProjectConfiguratorParsesPWASuccessfullyOnCIIfAppIdIsProvided(
747747
"--verbose"
748748
]);
749749

750+
result = CleanResult(result);
751+
750752
result.Error.Should().Contain("You've provided a URL, so we'll use");
751753
result.Error.Should().Contain($"AppId: {FakeApps[1].Id}");
752754
result.Error.Should().Contain("Submission commit success!");

0 commit comments

Comments
 (0)