Skip to content

Commit add021b

Browse files
CopilotMalcolmnixon
andcommitted
Fix parameter order for Assert.IsGreaterThanOrEqualTo
Co-authored-by: Malcolmnixon <1863707+Malcolmnixon@users.noreply.github.com>
1 parent 3790ee7 commit add021b

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

test/DemaConsulting.BuildMark.Tests/GitHubRepoConnectorTests.cs

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -195,14 +195,14 @@ public async Task GitHubRepoConnector_GetBuildInformationAsync_WithPullRequests_
195195
// PRs without linked issues are treated based on their labels
196196
// PR 100 with "bug" label should be in bugs
197197
Assert.IsNotNull(buildInfo.Bugs);
198-
Assert.IsGreaterThanOrEqualTo(buildInfo.Bugs.Count, 1, $"Expected at least 1 bug, got {buildInfo.Bugs.Count}");
198+
Assert.IsGreaterThanOrEqualTo(1, buildInfo.Bugs.Count, $"Expected at least 1 bug, got {buildInfo.Bugs.Count}");
199199
var bugPR = buildInfo.Bugs.FirstOrDefault(b => b.Index == 100);
200200
Assert.IsNotNull(bugPR, "PR 100 should be categorized as a bug");
201201
Assert.AreEqual("Fix critical bug", bugPR.Title);
202202

203203
// PR 101 with "feature" label should be in changes
204204
Assert.IsNotNull(buildInfo.Changes);
205-
Assert.IsGreaterThanOrEqualTo(buildInfo.Changes.Count, 1, $"Expected at least 1 change, got {buildInfo.Changes.Count}");
205+
Assert.IsGreaterThanOrEqualTo(1, buildInfo.Changes.Count, $"Expected at least 1 change, got {buildInfo.Changes.Count}");
206206
var featurePR = buildInfo.Changes.FirstOrDefault(c => c.Index == 101);
207207
Assert.IsNotNull(featurePR, "PR 101 should be categorized as a change");
208208
Assert.AreEqual("Add new feature", featurePR.Title);
@@ -258,7 +258,7 @@ public async Task GitHubRepoConnector_GetBuildInformationAsync_WithOpenIssues_Id
258258
// Known issues are open issues that aren't linked to any changes in this release
259259
Assert.IsNotNull(buildInfo.KnownIssues);
260260
// Since we have no PRs, all open issues should be known issues
261-
Assert.IsGreaterThanOrEqualTo(buildInfo.KnownIssues.Count, 1, $"Expected at least 1 known issue, got {buildInfo.KnownIssues.Count}");
261+
Assert.IsGreaterThanOrEqualTo(1, buildInfo.KnownIssues.Count, $"Expected at least 1 known issue, got {buildInfo.KnownIssues.Count}");
262262

263263
// Verify at least one known issue is present
264264
var knownIssueTitles = buildInfo.KnownIssues.Select(i => i.Title).ToList();

0 commit comments

Comments
 (0)