Replace int IDs by long ones + a few more changes (#811)

* Replace int IDs by long ones

Remove redundant stuff

* Remove obsolete code

* Tweak

* Address issue 797

* review + a few more changes related to runners

Author: louis-z

Directory.Build.props

@@ -8,7 +8,7 @@
     <Authors>$(Company), NGitLab contributors</Authors>
 
     <!-- Compile Options -->
-    <LangVersion>11.0</LangVersion>
+    <LangVersion>12</LangVersion>
     <Deterministic>true</Deterministic>
     <Features>strict</Features>
     <TreatWarningsAsErrors Condition="'$(Configuration)' != 'Debug'">true</TreatWarningsAsErrors>

NGitLab.Mock.Tests/MergeRequestsMockTests.cs

@@ -57,7 +57,7 @@ public void Test_merge_requests_approvable_by_me_can_be_listed()
             .BuildServer();
 
         var client = server.CreateClient("user1");
-        var mergeRequests = client.MergeRequests.Get(new MergeRequestQuery { ApproverIds = new[] { 1 } }).ToArray();
+        var mergeRequests = client.MergeRequests.Get(new MergeRequestQuery { ApproverIds = [1L] }).ToArray();
 
         Assert.That(mergeRequests, Has.Length.EqualTo(1), "Merge requests count is invalid");
         Assert.That(mergeRequests[0].Title, Is.EqualTo("Merge request 2"), "Merge request found is invalid");

NGitLab.Mock.Tests/MilestonesMockTests.cs

@@ -103,8 +103,8 @@ public void Test_milestones_can_be_closed_and_activated_from_project()
     [Test]
     public void Test_projects_merge_request_can_be_found_from_milestone()
     {
-        const int ProjectId = 1;
-        const int MilestoneId = 1;
+        const long ProjectId = 1;
+        const long MilestoneId = 1;
         using var server = new GitLabConfig()
             .WithUser("user1", isDefault: true)
             .WithProject("Test", id: ProjectId, addDefaultUserAsMaintainer: true, configure: project => project
@@ -122,8 +122,8 @@ public void Test_projects_merge_request_can_be_found_from_milestone()
     [Test]
     public void Test_groups_merge_request_can_be_found_from_milestone()
     {
-        const int projectId = 1;
-        const int milestoneId = 1;
+        const long projectId = 1;
+        const long milestoneId = 1;
         using var server = new GitLabConfig()
             .WithUser("user1", isDefault: true)
             .WithGroup("parentGroup", id: projectId, configure: group => group

NGitLab.Mock.Tests/ProjectsMockTests.cs

@@ -1,6 +1,7 @@
 using System;
 using System.Globalization;
 using System.IO;
+using System.Linq;
 using System.Net;
 using System.Threading.Tasks;
 using FluentAssertions;
@@ -213,7 +214,7 @@ public async Task CreateAsync_WhenMockCreatedWithSupportedOptions_TheyAreAvailab
         {
             Name = "MyProject",
             Path = "my-project",
-            NamespaceId = "my-group",
+            NamespaceId = server.Groups.First(g => string.Equals(g.Name, "MyGroup", StringComparison.Ordinal)).Id,
             Description = "Description",
             DefaultBranch = "foo",
             InitializeWithReadme = true,
@@ -228,7 +229,7 @@ public async Task CreateAsync_WhenMockCreatedWithSupportedOptions_TheyAreAvailab
         actual.Name.Should().Be(expected.Name);
         actual.Path.Should().Be(expected.Path);
         actual.Description.Should().Be(expected.Description);
-        actual.Namespace.FullPath.Should().Be(expected.NamespaceId);
+        actual.Namespace.Id.Should().Be(expected.NamespaceId);
         actual.NameWithNamespace.Should().Be($"MyGroup / {expected.Name}");
         actual.PathWithNamespace.Should().Be($"my-group/{expected.Path}");
         actual.DefaultBranch.Should().Be(expected.DefaultBranch);

NGitLab.Mock/Badge.cs

@@ -4,7 +4,7 @@ namespace NGitLab.Mock;
 
 public sealed class Badge : GitLabObject
 {
-    public int Id { get; set; }
+    public long Id { get; set; }
 
     public string LinkUrl { get; set; }
 

NGitLab.Mock/BadgeCollection.cs

@@ -10,7 +10,7 @@ public BadgeCollection(GitLabObject parent)
     {
     }
 
-    public Badge GetById(int id)
+    public Badge GetById(long id)
     {
         return this.FirstOrDefault(badge => badge.Id == id);
     }

NGitLab.Mock/Clients/BranchClient.cs

@@ -8,9 +8,9 @@ namespace NGitLab.Mock.Clients;
 
 internal sealed class BranchClient : ClientBase, IBranchClient
 {
-    private readonly int _projectId;
+    private readonly long _projectId;
 
-    public BranchClient(ClientContext context, int projectId)
+    public BranchClient(ClientContext context, long projectId)
         : base(context)
     {
         _projectId = projectId;

NGitLab.Mock/Clients/ClientBase.cs

@@ -30,7 +30,7 @@ protected Group GetGroup(object id, GroupPermission permissions)
 
         var group = id switch
         {
-            int idInt => Server.AllGroups.FindById(idInt),
+            long idLong => Server.AllGroups.FindById(idLong),
             string idStr => Server.AllGroups.FindGroup(idStr),
             IIdOrPathAddressable gid when gid.Path != null => Server.AllGroups.FindByNamespacedPath(gid.Path),
             IIdOrPathAddressable gid => Server.AllGroups.FindById(gid.Id),
@@ -78,7 +78,7 @@ protected Project GetProject(object id, ProjectPermission permissions)
 
         var project = id switch
         {
-            int idInt => Server.AllProjects.FindById(idInt),
+            long idLong => Server.AllProjects.FindById(idLong),
             string idStr => Server.AllProjects.FindProject(idStr),
             IIdOrPathAddressable pid when pid.Path != null => Server.AllProjects.FindByNamespacedPath(pid.Path),
             IIdOrPathAddressable pid => Server.AllProjects.FindById(pid.Id),
@@ -116,7 +116,7 @@ protected Project GetProject(object id, ProjectPermission permissions)
         return project;
     }
 
-    protected User GetUser(int userId)
+    protected User GetUser(long userId)
     {
         var user = Server.Users.GetById(userId);
         if (user == null)
@@ -125,7 +125,7 @@ protected User GetUser(int userId)
         return user;
     }
 
-    protected Issue GetIssue(int projectId, int issueId)
+    protected Issue GetIssue(long projectId, long issueId)
     {
         var project = GetProject(projectId, ProjectPermission.View);
         var issue = project.Issues.GetByIid(issueId);
@@ -135,7 +135,7 @@ protected Issue GetIssue(int projectId, int issueId)
         return issue;
     }
 
-    protected Milestone GetMilestone(int projectId, int milestoneId)
+    protected Milestone GetMilestone(long projectId, long milestoneId)
     {
         var project = GetProject(projectId, ProjectPermission.View);
         var milestone = project.Milestones.GetByIid(milestoneId);
@@ -145,7 +145,7 @@ protected Milestone GetMilestone(int projectId, int milestoneId)
         return milestone;
     }
 
-    protected MergeRequest GetMergeRequest(int projectId, int mergeRequestIid)
+    protected MergeRequest GetMergeRequest(long projectId, long mergeRequestIid)
     {
         var project = GetProject(projectId, ProjectPermission.View);
         var mergeRequest = project.MergeRequests.GetByIid(mergeRequestIid);

NGitLab.Mock/Clients/ClusterClient.cs

@@ -6,7 +6,7 @@ namespace NGitLab.Mock.Clients;
 
 internal sealed class ClusterClient : ClientBase, IClusterClient
 {
-    private readonly int _projectId;
+    private readonly long _projectId;
 
     public ClusterClient(ClientContext context, ProjectId projectId)
         : base(context)

NGitLab.Mock/Clients/CommitClient.cs

@@ -7,7 +7,7 @@ namespace NGitLab.Mock.Clients;
 
 internal sealed class CommitClient : ClientBase, ICommitClient
 {
-    private readonly int _projectId;
+    private readonly long _projectId;
 
     public CommitClient(ClientContext context, ProjectId projectId)
         : base(context)