Skip to content

Commit 741c53a

Browse files
authored
Fix AzDo tests broken in #4440 (#4447)
<!-- Link the GitHub or AzDO issue this pull request is associated with. Please copy and paste the full URL rather than using the dotnet/arcade-services# syntax --> #4425
1 parent 8aa3faf commit 741c53a

2 files changed

Lines changed: 3 additions & 3 deletions

File tree

eng/templates/jobs/darc-and-api-tests.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -77,7 +77,7 @@ jobs:
7777
7878
.\darc\darc.exe add-build-to-channel `
7979
--id $darcBuild[0].id `
80-
--channel "General Testing"
80+
--channel "General Testing" `
8181
--ci `
8282
--azdev-pat $(AzdoToken)
8383

test/ProductConstructionService.ScenarioTests/ScenarioTestBase.cs

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -376,8 +376,8 @@ protected static async Task<string> CheckAzDoPullRequest(
376376
trimmedTitle.Should().Be(expectedPRTitle);
377377

378378
PrStatus expectedPRState = isCompleted ? PrStatus.Closed : PrStatus.Open;
379-
var prStatus = await AzDoClient.GetPullRequestStatusAsync(GetAzDoApiRepoUrl(targetRepoName) + $"/pullRequests/{pullRequestId}");
380-
prStatus.Should().Be(expectedPRState);
379+
var prInfo = await AzDoClient.GetPullRequestStatusAsync(GetAzDoApiRepoUrl(targetRepoName) + $"/pullRequests/{pullRequestId}");
380+
prInfo.Status.Should().Be(expectedPRState);
381381

382382
using (ChangeDirectory(repoDirectory))
383383
{

0 commit comments

Comments
 (0)