Fix long/int type mismatches in tests

Test files still used int for IDs after the library changed them to long.
Added casts where the API still expects int parameters.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

Author: barnon-apiiro

test/GitLabApiClient.Test/GroupsClientTest.cs

@@ -21,7 +21,7 @@ namespace GitLabApiClient.Test
     public class GroupsClientTest
     {
         private readonly List<int> _groupIdsToClean = new List<int>();
-        private List<int> MilestoneIdsToClean { get; } = new List<int>();
+        private List<long> MilestoneIdsToClean { get; } = new List<long>();
         private List<string> VariableIdsToClean { get; } = new List<string>();
 
         private readonly GroupsClient _sut = new GroupsClient(
@@ -86,7 +86,7 @@ public async Task GroupMilestonesCanBeRetrieved()
 
             //act
             var milestones = await _sut.GetMilestonesAsync(TestGroupId);
-            var milestone = await _sut.GetMilestoneAsync(TestGroupId, createdMilestone.Id);
+            var milestone = await _sut.GetMilestoneAsync(TestGroupId, (int)createdMilestone.Id);
 
             //assert
             milestones.Should().NotBeEmpty();
@@ -207,7 +207,7 @@ public async Task CreatedGroupMilestoneCanBeUpdated()
             MilestoneIdsToClean.Add(createdMilestone.Id);
 
             //act
-            var updatedMilestone = await _sut.UpdateMilestoneAsync(TestGroupTextId, createdMilestone.Id, new UpdateGroupMilestoneRequest()
+            var updatedMilestone = await _sut.UpdateMilestoneAsync(TestGroupTextId, (int)createdMilestone.Id, new UpdateGroupMilestoneRequest()
             {
                 Title = "milestone22",
                 StartDate = "2018-11-05",
@@ -237,7 +237,7 @@ public async Task CreatedGroupMilestoneCanBeClosed()
             MilestoneIdsToClean.Add(createdMilestone.Id);
 
             //act
-            var updatedMilestone = await _sut.UpdateMilestoneAsync(TestGroupTextId, createdMilestone.Id, new UpdateGroupMilestoneRequest()
+            var updatedMilestone = await _sut.UpdateMilestoneAsync(TestGroupTextId, (int)createdMilestone.Id, new UpdateGroupMilestoneRequest()
             {
                 State = UpdatedMilestoneState.Close
             });
@@ -353,8 +353,8 @@ public Task DisposeAsync()
 
         private async Task CleanupGroups()
         {
-            foreach (int milestoneId in MilestoneIdsToClean)
-                await _sut.DeleteMilestoneAsync(TestGroupId, milestoneId);
+            foreach (long milestoneId in MilestoneIdsToClean)
+                await _sut.DeleteMilestoneAsync(TestGroupId, (int)milestoneId);
 
             foreach (int groupId in _groupIdsToClean)
                 await _sut.DeleteAsync(groupId.ToString());

test/GitLabApiClient.Test/IssuesClientTest.cs

@@ -37,7 +37,7 @@ public async Task CreatedIssueCanBeUpdated()
             });
 
             //act
-            var updatedIssue = await _sut.UpdateAsync(TestProjectTextId, createdIssue.Iid, new UpdateIssueRequest()
+            var updatedIssue = await _sut.UpdateAsync(TestProjectTextId, (int)createdIssue.Iid, new UpdateIssueRequest()
             {
                 Assignees = new List<int> { 11 },
                 Confidential = false,
@@ -63,7 +63,7 @@ public async Task CreatedIssueCanBeClosed()
             var createdIssue = await _sut.CreateAsync(TestProjectTextId, new CreateIssueRequest("Title1"));
 
             //act
-            var updatedIssue = await _sut.UpdateAsync(TestProjectTextId, createdIssue.Iid, new UpdateIssueRequest()
+            var updatedIssue = await _sut.UpdateAsync(TestProjectTextId, (int)createdIssue.Iid, new UpdateIssueRequest()
             {
                 State = UpdatedIssueState.Close
             });
@@ -104,8 +104,8 @@ public async Task CreatedIssueCanBeRetrieved()
             });
 
             //act
-            var issueById = await _sut.GetAsync(TestProjectId, issue.Iid);
-            var issueByProjectId = (await _sut.GetAllAsync(options: o => o.IssueIds = new[] { issue.Iid })).FirstOrDefault(i => i.Title == title);
+            var issueById = await _sut.GetAsync(TestProjectId, (int)issue.Iid);
+            var issueByProjectId = (await _sut.GetAllAsync(options: o => o.IssueIds = new[] { (int)issue.Iid })).FirstOrDefault(i => i.Title == title);
             var ownedIssue = (await _sut.GetAllAsync(options: o => o.Scope = Scope.CreatedByMe)).FirstOrDefault(i => i.Title == title);
 
             //assert
@@ -136,7 +136,7 @@ public async Task CreatedIssueNoteCanBeRetrieved()
                 CreatedAt = DateTime.Now
             });
             var issueNotes = (await _sut.GetNotesAsync(TestProjectId, issue.Iid)).FirstOrDefault(i => i.Body == body);
-            var issueNote = await _sut.GetNoteAsync(TestProjectId, issue.Iid, note.Id);
+            var issueNote = await _sut.GetNoteAsync(TestProjectId, (int)issue.Iid, (int)note.Id);
 
             //assert
             note.Should().Match<Note>(n =>
@@ -157,7 +157,7 @@ public async Task CreatedIssueNoteCanBeUpdated()
             var createdIssueNote = await _sut.CreateNoteAsync(TestProjectTextId, createdIssue.Iid, new CreateIssueNoteRequest("comment2"));
 
             //act
-            var updatedIssueNote = await _sut.UpdateNoteAsync(TestProjectTextId, createdIssue.Iid, createdIssueNote.Id, new UpdateIssueNoteRequest("comment22"));
+            var updatedIssueNote = await _sut.UpdateNoteAsync(TestProjectTextId, (int)createdIssue.Iid, (int)createdIssueNote.Id, new UpdateIssueNoteRequest("comment22"));
 
             //assert
             updatedIssueNote.Should().Match<Note>(n =>

test/GitLabApiClient.Test/MergeRequestClientTest.cs

@@ -96,7 +96,7 @@ public async Task MergeRequestWithoutCommitsCannotBeAccepted()
             });
 
             Func<Task<MergeRequest>> acceptAction = () =>
-                _sut.AcceptAsync(GitLabApiHelper.TestProjectTextId, createdMergeRequest.Iid, new AcceptMergeRequest());
+                _sut.AcceptAsync(GitLabApiHelper.TestProjectTextId, (int)createdMergeRequest.Iid, new AcceptMergeRequest());
 
             acceptAction.Should().Throw<GitLabException>().
                 WithMessage("{\"message\":\"405 Method Not Allowed\"}").
@@ -124,7 +124,7 @@ private async Task DeleteAllMergeRequests()
         {
             var mergeRequests = await _sut.GetAsync(GitLabApiHelper.TestProjectId);
             await Task.WhenAll(mergeRequests.Select(
-                m => _sut.DeleteAsync(GitLabApiHelper.TestProjectId, m.Iid)));
+                m => _sut.DeleteAsync(GitLabApiHelper.TestProjectId, (int)m.Iid)));
         }
     }
 }

test/GitLabApiClient.Test/ProjectsClientTest.cs

@@ -22,7 +22,7 @@ namespace GitLabApiClient.Test
     public class ProjectsClientTest : IAsyncLifetime
     {
         private List<int> ProjectIdsToClean { get; } = new List<int>();
-        private List<int> MilestoneIdsToClean { get; } = new List<int>();
+        private List<long> MilestoneIdsToClean { get; } = new List<long>();
         private List<string> VariableIdsToClean { get; } = new List<string>();
 
         private readonly ProjectsClient _sut = new ProjectsClient(
@@ -76,7 +76,7 @@ public async Task ProjectMilestonesRetrieved()
 
             //act
             var milestones = await _sut.GetMilestonesAsync(GitLabApiHelper.TestProjectId);
-            var milestone = await _sut.GetMilestoneAsync(GitLabApiHelper.TestProjectId, createdMilestone.Id);
+            var milestone = await _sut.GetMilestoneAsync(GitLabApiHelper.TestProjectId, (int)createdMilestone.Id);
 
             //assert
             milestones.Should().NotBeEmpty();
@@ -286,7 +286,7 @@ public async Task CreatedProjectMilestoneCanBeUpdated()
             MilestoneIdsToClean.Add(createdMilestone.Id);
 
             //act
-            var updatedMilestone = await _sut.UpdateMilestoneAsync(GitLabApiHelper.TestProjectTextId, createdMilestone.Id, new UpdateProjectMilestoneRequest()
+            var updatedMilestone = await _sut.UpdateMilestoneAsync(GitLabApiHelper.TestProjectTextId, (int)createdMilestone.Id, new UpdateProjectMilestoneRequest()
             {
                 Title = "milestone23",
                 StartDate = "2018-11-05",
@@ -353,7 +353,7 @@ public async Task CreatedProjectMilestoneCanBeClosed()
             MilestoneIdsToClean.Add(createdMilestone.Id);
 
             //act
-            var updatedMilestone = await _sut.UpdateMilestoneAsync(GitLabApiHelper.TestProjectTextId, createdMilestone.Id, new UpdateProjectMilestoneRequest()
+            var updatedMilestone = await _sut.UpdateMilestoneAsync(GitLabApiHelper.TestProjectTextId, (int)createdMilestone.Id, new UpdateProjectMilestoneRequest()
             {
                 State = UpdatedMilestoneState.Close
             });
@@ -411,8 +411,8 @@ public async Task ExportImportProject()
         private async Task CleanupProjects()
         {
 
-            foreach (int milestoneId in MilestoneIdsToClean)
-                await _sut.DeleteMilestoneAsync(GitLabApiHelper.TestProjectId, milestoneId);
+            foreach (long milestoneId in MilestoneIdsToClean)
+                await _sut.DeleteMilestoneAsync(GitLabApiHelper.TestProjectId, (int)milestoneId);
 
             foreach (int projectId in ProjectIdsToClean)
                 await _sut.DeleteAsync(projectId);

test/GitLabApiClient.Test/ToDoListClientTest.cs

@@ -140,7 +140,7 @@ private async Task DeleteAllMergeRequests()
         {
             var mergeRequests = await mergeRequestsClient.GetAsync(GitLabApiHelper.TestProjectId);
             await Task.WhenAll(mergeRequests.Select(
-                m => mergeRequestsClient.DeleteAsync(GitLabApiHelper.TestProjectId, m.Iid)));
+                m => mergeRequestsClient.DeleteAsync(GitLabApiHelper.TestProjectId, (int)m.Iid)));
         }
     }
 }