Fix linting issues (#393)

pascalberger · web-flow · commit f8ad7852ac1c · 2023-02-09T22:51:12.000+01:00
diff --git a/src/Cake.AzureDevOps.Tests/ArgumentChecksTests.cs b/src/Cake.AzureDevOps.Tests/ArgumentChecksTests.cs
@@ -2,6 +2,7 @@
 {
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class ArgumentChecksTests
     {
         public sealed class TheNotNullExtension
diff --git a/src/Cake.AzureDevOps.Tests/Authentication/AuthenticationProviderTests.cs b/src/Cake.AzureDevOps.Tests/Authentication/AuthenticationProviderTests.cs
@@ -4,6 +4,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AuthenticationProviderTests
     {
         public sealed class TheAuthenticationNtlmMethod
diff --git a/src/Cake.AzureDevOps.Tests/Authentication/AzureDevOpsAadCredentialsTests.cs b/src/Cake.AzureDevOps.Tests/Authentication/AzureDevOpsAadCredentialsTests.cs
@@ -4,6 +4,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsAadCredentialsTests
     {
         public sealed class TheCtor
diff --git a/src/Cake.AzureDevOps.Tests/Authentication/AzureDevOpsBasicCredentialsTests.cs b/src/Cake.AzureDevOps.Tests/Authentication/AzureDevOpsBasicCredentialsTests.cs
@@ -4,6 +4,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsBasicCredentialsTests
     {
         public sealed class TheCtor
diff --git a/src/Cake.AzureDevOps.Tests/Authentication/AzureDevOpsNtlmCredentialsTests.cs b/src/Cake.AzureDevOps.Tests/Authentication/AzureDevOpsNtlmCredentialsTests.cs
@@ -4,6 +4,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsNtlmCredentialsTests
     {
         public sealed class TheCtor
diff --git a/src/Cake.AzureDevOps.Tests/Authentication/AzureDevOpsOAuthCredentialsTests.cs b/src/Cake.AzureDevOps.Tests/Authentication/AzureDevOpsOAuthCredentialsTests.cs
@@ -4,6 +4,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsOAuthCredentialsTests
     {
         public sealed class TheCtor
diff --git a/src/Cake.AzureDevOps.Tests/BaseAzureDevOpsCollectionSettingsTests.cs b/src/Cake.AzureDevOps.Tests/BaseAzureDevOpsCollectionSettingsTests.cs
@@ -5,8 +5,10 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class BaseAzureDevOpsCollectionSettingsTests
     {
+        // ReSharper disable once ClassNeverInstantiated.Global
         public sealed class TheCtor
         {
             public sealed class WithCollectionUrlAndCredentialsParameter
diff --git a/src/Cake.AzureDevOps.Tests/BaseAzureDevOpsCredentialsSettingsTests.cs b/src/Cake.AzureDevOps.Tests/BaseAzureDevOpsCredentialsSettingsTests.cs
@@ -4,8 +4,10 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class BaseAzureDevOpsCredentialsSettingsTests
     {
+        // ReSharper disable once ClassNeverInstantiated.Global
         public sealed class TheCtor
         {
             public sealed class WithCredentialsParameter
diff --git a/src/Cake.AzureDevOps.Tests/BaseAzureDevOpsProjectSettingsTests.cs b/src/Cake.AzureDevOps.Tests/BaseAzureDevOpsProjectSettingsTests.cs
@@ -5,8 +5,10 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class BaseAzureDevOpsProjectSettingsTests
     {
+        // ReSharper disable once ClassNeverInstantiated.Global
         public sealed class TheCtor
         {
             public sealed class WithCollectionUrlAndProjectGuidAndCredentialsParameter
diff --git a/src/Cake.AzureDevOps.Tests/Boards/WorkItemTracking/AzureDevOpsWorkItemSettingsTests.cs b/src/Cake.AzureDevOps.Tests/Boards/WorkItemTracking/AzureDevOpsWorkItemSettingsTests.cs
@@ -6,6 +6,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsWorkItemSettingsTests
     {
         public sealed class TheCtorForProjectGuid
diff --git a/src/Cake.AzureDevOps.Tests/Fakes/FakeAllSetBuildClientFactory.cs b/src/Cake.AzureDevOps.Tests/Fakes/FakeAllSetBuildClientFactory.cs
@@ -16,23 +16,23 @@ public override BuildHttpClient CreateBuildClient(Uri collectionUrl, IAzureDevOp
             var mock = new Mock<BuildHttpClient>(MockBehavior.Loose, collectionUrl, credentials.ToVssCredentials());
 
             mock.Setup(arg => arg.GetBuildAsync(It.IsAny<Guid>(), It.IsAny<int>(), null, null, default))
-                .ReturnsAsync((Guid projectId, int buildId, string propertyFilters, object userState, CancellationToken token) => new Build
+                .ReturnsAsync((Guid projectId, int buildId, string _, object _, CancellationToken _) => new Build
                 {
                     Id = buildId,
                     BuildNumber = buildId.ToString(),
                     Project = new TeamProjectReference { Id = projectId },
                 });
 
             mock.Setup(arg => arg.GetBuildAsync(It.IsAny<string>(), It.IsAny<int>(), null, null, default))
-                .ReturnsAsync((string projectName, int buildId, string propertyFilters, object userState, CancellationToken token) => new Build
+                .ReturnsAsync((string projectName, int buildId, string _, object _, CancellationToken _) => new Build
                 {
                     Id = buildId,
                     BuildNumber = buildId.ToString(),
                     Project = new TeamProjectReference { Name = projectName },
                 });
 
             mock.Setup(arg => arg.GetBuildWorkItemsRefsAsync(It.IsAny<string>(), It.IsAny<int>(), It.IsAny<int?>(), null, default))
-               .ReturnsAsync((string projectName, int buildId, int? top, object userState, CancellationToken token) => new List<ResourceRef>
+               .ReturnsAsync((string _, int _, int? _, object _, CancellationToken _) => new List<ResourceRef>
                {
                   new () { Id = "42" },
                });
diff --git a/src/Cake.AzureDevOps.Tests/Fakes/FakeAllSetGitClientFactory.cs b/src/Cake.AzureDevOps.Tests/Fakes/FakeAllSetGitClientFactory.cs
@@ -15,7 +15,7 @@ public override GitHttpClient CreateGitClient(Uri collectionUrl, IAzureDevOpsCre
             var mock = new Mock<GitHttpClient>(MockBehavior.Loose, collectionUrl, credentials.ToVssCredentials());
 
             mock.Setup(arg => arg.GetPullRequestAsync(It.IsAny<string>(), It.IsAny<string>(), It.IsAny<int>(), null, null, null, null, null, null, default))
-                .ReturnsAsync((string project1, string repoId1, int prId, int i1, int i2, int i3, bool b1, bool b2, object o1, CancellationToken c1) => new GitPullRequest
+                .ReturnsAsync((string _, string repoId1, int prId, int _, int _, int _, bool _, bool _, object _, CancellationToken _) => new GitPullRequest
                 {
                     PullRequestId = prId,
                     Status = PullRequestStatus.Active,
@@ -40,7 +40,7 @@ public override GitHttpClient CreateGitClient(Uri collectionUrl, IAzureDevOpsCre
                     1,
                     null,
                     default))
-                .ReturnsAsync((string project2, string repoId2, GitPullRequestSearchCriteria sc, int j1, int j2, int top, object o2, CancellationToken c2)
+                .ReturnsAsync((string _, string repoId2, GitPullRequestSearchCriteria sc, int _, int _, int _, object _, CancellationToken _)
                     => new List<GitPullRequest>(new[]
                     {
                         new GitPullRequest
@@ -74,7 +74,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                     It.IsAny<string>(),
                     It.IsAny<object>(),
                     default))
-             .ReturnsAsync((IdentityRefWithVote identity, Guid project, int prId, string reviewerId, object o, CancellationToken c)
+             .ReturnsAsync((IdentityRefWithVote identity, Guid _, int _, string _, object _, CancellationToken _)
                     => new IdentityRefWithVote
                     {
                         Vote = identity.Vote,
@@ -95,7 +95,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                     It.IsAny<int>(),
                     It.IsAny<object>(),
                     It.IsAny<CancellationToken>()))
-             .ReturnsAsync((GitPullRequestStatus status, Guid repoId, int prId, object o, CancellationToken c)
+             .ReturnsAsync((GitPullRequestStatus status, Guid _, int _, object _, CancellationToken _)
                     => new GitPullRequestStatus
                     {
                         Context = status.Context,
@@ -136,7 +136,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                     It.IsAny<GitTargetVersionDescriptor>(),
                     null,
                     CancellationToken.None))
-             .ReturnsAsync((string prj, Guid rId, bool? b, int? t, int? s, GitBaseVersionDescriptor bvd, GitTargetVersionDescriptor tvd, object o, CancellationToken c)
+             .ReturnsAsync((string _, Guid _, bool? _, int? _, int? _, GitBaseVersionDescriptor _, GitTargetVersionDescriptor _, object _, CancellationToken _)
                     => gitCommitDiffs);
 
             m.Setup(arg => arg.UpdateThreadAsync(
@@ -146,7 +146,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                 It.IsAny<int>(),
                 null,
                 CancellationToken.None))
-             .ReturnsAsync((GitPullRequestCommentThread prct, Guid g, int prId, int thId, object o, CancellationToken c)
+             .ReturnsAsync((GitPullRequestCommentThread prct, Guid _, int _, int thId, object _, CancellationToken _)
                     => new GitPullRequestCommentThread { Id = thId, Status = prct.Status });
 
             // Setup GitPullRequestCommentThread collection
@@ -182,7 +182,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                 null,
                 null,
                 CancellationToken.None))
-             .ReturnsAsync((Guid rId, int prId, int? it, int? baseIt, object o, CancellationToken c)
+             .ReturnsAsync((Guid _, int _, int? _, int? _, object _, CancellationToken _)
                     => commentThreads);
 
             m.Setup(arg => arg.CreateThreadAsync(
@@ -191,7 +191,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                 It.IsAny<int>(),
                 null,
                 CancellationToken.None))
-             .ReturnsAsync((GitPullRequestCommentThread prct, Guid g, int i, object o, CancellationToken c)
+             .ReturnsAsync((GitPullRequestCommentThread prct, Guid _, int _, object _, CancellationToken _)
                     => prct);
 
             m.Setup(arg => arg.GetPullRequestIterationsAsync(
@@ -200,7 +200,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                 null,
                 null,
                 CancellationToken.None))
-             .ReturnsAsync((Guid repoId, int prId, bool? b, object o, CancellationToken c)
+             .ReturnsAsync((Guid _, int _, bool? _, object _, CancellationToken _)
                     => new List<GitPullRequestIteration>
                     {
                         new () { Id = 42, CreatedDate = DateTime.Today.AddDays(-3) },
@@ -213,7 +213,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                     null,
                     null,
                     CancellationToken.None))
-                .ReturnsAsync((Guid repoId, int prId, bool? b, object o, CancellationToken c)
+                .ReturnsAsync((Guid _, int _, bool? _, object _, CancellationToken _)
                     => new List<GitPullRequestIteration>
                     {
                         new () { Id = null },
@@ -238,7 +238,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                 null,
                 null,
                 CancellationToken.None))
-             .ReturnsAsync((Guid repoId, int prId, int iterId, int? t, int? s, int? ct, object o, CancellationToken c)
+             .ReturnsAsync((Guid _, int _, int _, int? _, int? _, int? _, object _, CancellationToken _)
                     => changes);
 
             // Setup pull request creation
@@ -288,11 +288,11 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
              .ReturnsAsync(
                 (
                     GitPullRequest gitPullRequestToCreate,
-                    string project,
+                    string _,
                     string repositoryId,
-                    bool? supportsIterations,
-                    object userState,
-                    CancellationToken cancellationToken) =>
+                    bool? _,
+                    object _,
+                    CancellationToken _) =>
                 {
                     gitPullRequestToCreate.PullRequestId = 777;
                     gitPullRequestToCreate.Repository = new GitRepository
@@ -315,7 +315,7 @@ protected override Mock<GitHttpClient> Setup(Mock<GitHttpClient> m)
                    It.IsAny<int>(),
                    null,
                    CancellationToken.None))
-               .ReturnsAsync((Comment comment, Guid repositoryId, int prId, int threadId, int commentId, object o, CancellationToken c)
+               .ReturnsAsync((Comment comment, Guid _, int _, int _, int _, object _, CancellationToken _)
                    => new Comment
                    {
                         Id = comment.Id,
diff --git a/src/Cake.AzureDevOps.Tests/Fakes/FakeAllSetTestManagementClientFactory.cs b/src/Cake.AzureDevOps.Tests/Fakes/FakeAllSetTestManagementClientFactory.cs
@@ -35,12 +35,23 @@ public override TestManagementHttpClient CreateTestManagementClient(Uri collecti
                 });
 
             mock.Setup(arg => arg.GetTestResultsAsync(It.IsAny<Guid>(), It.IsAny<int>(), null, It.IsAny<int?>(), It.IsAny<int?>(), It.IsAny<IEnumerable<TestOutcome>>(), null, default))
-                .ReturnsAsync((Guid projectId, int testRunId, ResultDetails? details, int? skip, int? top, IEnumerable<TestOutcome> outcomes, object userState, CancellationToken token) => new List<TestCaseResult>()
+                .ReturnsAsync((Guid _, int _, ResultDetails? _, int? _, int? top, IEnumerable<TestOutcome> _, object _, CancellationToken _) =>
                 {
-                    new () { AutomatedTestName = "t1", Outcome = "Passed", ErrorMessage = string.Empty },
-                    new () { AutomatedTestName = "t2", Outcome = "Failed", ErrorMessage = "Error" },
-                    new () { AutomatedTestName = "t3", Outcome = "Passed", ErrorMessage = string.Empty },
-                }.Take(top.Value).ToList());
+                    var result =
+                        new List<TestCaseResult>
+                        {
+                            new () { AutomatedTestName = "t1", Outcome = "Passed", ErrorMessage = string.Empty },
+                            new () { AutomatedTestName = "t2", Outcome = "Failed", ErrorMessage = "Error" },
+                            new () { AutomatedTestName = "t3", Outcome = "Passed", ErrorMessage = string.Empty },
+                        };
+
+                    if (top.HasValue)
+                    {
+                        result = result.Take(top.Value).ToList();
+                    }
+
+                    return result;
+                });
 
             mock = this.Setup(mock);
 
diff --git a/src/Cake.AzureDevOps.Tests/Fakes/FakeAllSetWorkItemTrackingClientFactory.cs b/src/Cake.AzureDevOps.Tests/Fakes/FakeAllSetWorkItemTrackingClientFactory.cs
@@ -16,7 +16,7 @@ public override WorkItemTrackingHttpClient CreateWorkItemTrackingClient(Uri coll
             var mock = new Mock<WorkItemTrackingHttpClient>(MockBehavior.Strict, collectionUrl, credentials.ToVssCredentials());
 
             mock.Setup(arg => arg.GetWorkItemsAsync(It.IsAny<IEnumerable<int>>(), It.IsAny<IEnumerable<string>>(), It.IsAny<DateTime?>(), It.IsAny<WorkItemExpand?>(), It.IsAny<WorkItemErrorPolicy?>(), null, default))
-                .ReturnsAsync((IEnumerable<int> workItemIds, IEnumerable<string> fields, DateTime? asOf, WorkItemExpand? expand, WorkItemErrorPolicy? errorPolicy, object userState, CancellationToken token) =>
+                .ReturnsAsync((IEnumerable<int> workItemIds, IEnumerable<string> _, DateTime? _, WorkItemExpand? _, WorkItemErrorPolicy? _, object _, CancellationToken _) =>
                 {
                     return workItemIds.Select(x => new WorkItem { Id = x }).ToList();
                 });
diff --git a/src/Cake.AzureDevOps.Tests/Pipelines/AzureDevOpsBuildSettingsTests.cs b/src/Cake.AzureDevOps.Tests/Pipelines/AzureDevOpsBuildSettingsTests.cs
@@ -6,6 +6,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsBuildSettingsTests
     {
         public sealed class TheCtorForProjectGuid
diff --git a/src/Cake.AzureDevOps.Tests/Pipelines/AzureDevOpsBuildTests.cs b/src/Cake.AzureDevOps.Tests/Pipelines/AzureDevOpsBuildTests.cs
@@ -7,6 +7,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsBuildTests
     {
         public sealed class TheGetTestRunsMethod
diff --git a/src/Cake.AzureDevOps.Tests/Pipelines/AzureDevOpsBuildsSettingsTests.cs b/src/Cake.AzureDevOps.Tests/Pipelines/AzureDevOpsBuildsSettingsTests.cs
@@ -6,6 +6,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsBuildsSettingsTests
     {
         public sealed class TheCtorForProjectGuid
diff --git a/src/Cake.AzureDevOps.Tests/Repos/PullRequest/AzureDevOpsCreatePullRequestSettingsTests.cs b/src/Cake.AzureDevOps.Tests/Repos/PullRequest/AzureDevOpsCreatePullRequestSettingsTests.cs
@@ -6,6 +6,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsCreatePullRequestSettingsTests
     {
         public sealed class TheCtor
diff --git a/src/Cake.AzureDevOps.Tests/Repos/PullRequest/AzureDevOpsPullRequestSettingsTests.cs b/src/Cake.AzureDevOps.Tests/Repos/PullRequest/AzureDevOpsPullRequestSettingsTests.cs
@@ -6,6 +6,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsPullRequestSettingsTests
     {
         public sealed class TheCtorForSourceRefName
diff --git a/src/Cake.AzureDevOps.Tests/Repos/PullRequest/AzureDevOpsPullRequestTests.cs b/src/Cake.AzureDevOps.Tests/Repos/PullRequest/AzureDevOpsPullRequestTests.cs
@@ -12,6 +12,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsPullRequestTests
     {
         public sealed class TheCtor
diff --git a/src/Cake.AzureDevOps.Tests/Repos/PullRequest/CommentThread/AzureDevOpsCommentTests.cs b/src/Cake.AzureDevOps.Tests/Repos/PullRequest/CommentThread/AzureDevOpsCommentTests.cs
@@ -5,6 +5,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsCommentTests
     {
         public sealed class TheCtor
diff --git a/src/Cake.AzureDevOps.Tests/Repos/PullRequest/CommentThread/AzureDevOpsPullRequestCommentThreadTests.cs b/src/Cake.AzureDevOps.Tests/Repos/PullRequest/CommentThread/AzureDevOpsPullRequestCommentThreadTests.cs
@@ -8,6 +8,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class AzureDevOpsPullRequestCommentThreadTests
     {
         public sealed class TheCtor
diff --git a/src/Cake.AzureDevOps.Tests/Repos/PullRequest/PullRequestStatusExtensionsTests.cs b/src/Cake.AzureDevOps.Tests/Repos/PullRequest/PullRequestStatusExtensionsTests.cs
@@ -5,6 +5,7 @@
     using Shouldly;
     using Xunit;
 
+    // ReSharper disable once ClassNeverInstantiated.Global
     public sealed class PullRequestStatusExtensionsTests
     {
         public sealed class TheToAzureDevOpsPullRequestState
diff --git a/src/Cake.AzureDevOps.sln.DotSettings b/src/Cake.AzureDevOps.sln.DotSettings
@@ -1,6 +1,8 @@
 ﻿<wpf:ResourceDictionary xml:space="preserve" xmlns:x="http://schemas.microsoft.com/winfx/2006/xaml" xmlns:s="clr-namespace:System;assembly=mscorlib" xmlns:ss="urn:shemas-jetbrains-com:settings-storage-xaml" xmlns:wpf="http://schemas.microsoft.com/winfx/2006/xaml/presentation">
 	<s:String x:Key="/Default/CodeInspection/Highlighting/InspectionSeverities/=ArrangeThisQualifier/@EntryIndexedValue">DO_NOT_SHOW</s:String>
 	<s:String x:Key="/Default/CodeInspection/Highlighting/InspectionSeverities/=ConvertToUsingDeclaration/@EntryIndexedValue">DO_NOT_SHOW</s:String>
+	<s:String x:Key="/Default/CodeInspection/Highlighting/InspectionSeverities/=IntroduceOptionalParameters_002EGlobal/@EntryIndexedValue">DO_NOT_SHOW</s:String>
+	<s:String x:Key="/Default/CodeInspection/Highlighting/InspectionSeverities/=IntroduceOptionalParameters_002ELocal/@EntryIndexedValue">DO_NOT_SHOW</s:String>
 	<s:String x:Key="/Default/CodeInspection/Highlighting/InspectionSeverities/=MemberCanBePrivate_002EGlobal/@EntryIndexedValue">DO_NOT_SHOW</s:String>
 	<s:String x:Key="/Default/CodeInspection/Highlighting/InspectionSeverities/=ParameterHidesMember/@EntryIndexedValue">DO_NOT_SHOW</s:String>
 	<s:String x:Key="/Default/CodeInspection/Highlighting/InspectionSeverities/=RedundantNameQualifier/@EntryIndexedValue">DO_NOT_SHOW</s:String>
diff --git a/src/Cake.AzureDevOps/ITestManagementClientFactory.cs b/src/Cake.AzureDevOps/ITestManagementClientFactory.cs
@@ -25,6 +25,7 @@ internal interface ITestManagementClientFactory
         /// <param name="credentials">The credentials to connect to Azure DevOps.</param>
         /// <param name="authorizedIdentity">Returns identity which is authorized.</param>
         /// <returns>The instance of <see cref="TestManagementHttpClient"/> class.</returns>
+        // ReSharper disable once UnusedMemberInSuper.Global
         TestManagementHttpClient CreateTestManagementClient(Uri collectionUrl, IAzureDevOpsCredentials credentials, out Identity authorizedIdentity);
     }
 }
diff --git a/src/Cake.AzureDevOps/Repos/PullRequest/AzureDevOpsPullRequest.cs b/src/Cake.AzureDevOps/Repos/PullRequest/AzureDevOpsPullRequest.cs
@@ -507,8 +507,7 @@ public IEnumerable<FilePath> GetModifiedFiles()
                         null, // int? top
                         null, // int? skip
                         baseVersionDescriptor,
-                        targetVersionDescriptor,
-                        null) // object userState
+                        targetVersionDescriptor)
                     .ConfigureAwait(false)
                     .GetAwaiter()
                     .GetResult();
diff --git a/src/Cake.AzureDevOps/Repos/PullRequest/AzureDevOpsPullRequestSettings.cs b/src/Cake.AzureDevOps/Repos/PullRequest/AzureDevOpsPullRequestSettings.cs

Original file line number	Diff line number	Diff line change
`@@ -2,6 +2,7 @@`
`2`	`2`	`{`
`3`	`3`	`using Xunit;`
`4`	`4`
	`5`	`+ // ReSharper disable once ClassNeverInstantiated.Global`
`5`	`6`	`public sealed class ArgumentChecksTests`
`6`	`7`	`{`
`7`	`8`	`public sealed class TheNotNullExtension`
Original file line number	Diff line number	Diff line change
`@@ -4,6 +4,7 @@`
`4`	`4`	`using Shouldly;`
`5`	`5`	`using Xunit;`
`6`	`6`
	`7`	`+ // ReSharper disable once ClassNeverInstantiated.Global`
`7`	`8`	`public sealed class AuthenticationProviderTests`
`8`	`9`	`{`
`9`	`10`	`public sealed class TheAuthenticationNtlmMethod`
Original file line number	Diff line number	Diff line change
`@@ -5,8 +5,10 @@`
`5`	`5`	`using Shouldly;`
`6`	`6`	`using Xunit;`
`7`	`7`
	`8`	`+ // ReSharper disable once ClassNeverInstantiated.Global`
`8`	`9`	`public sealed class BaseAzureDevOpsCollectionSettingsTests`
`9`	`10`	`{`
	`11`	`+ // ReSharper disable once ClassNeverInstantiated.Global`
`10`	`12`	`public sealed class TheCtor`
`11`	`13`	`{`
`12`	`14`	`public sealed class WithCollectionUrlAndCredentialsParameter`
Original file line number	Diff line number	Diff line change
`@@ -4,8 +4,10 @@`
`4`	`4`	`using Shouldly;`
`5`	`5`	`using Xunit;`
`6`	`6`
	`7`	`+ // ReSharper disable once ClassNeverInstantiated.Global`
`7`	`8`	`public sealed class BaseAzureDevOpsCredentialsSettingsTests`
`8`	`9`	`{`
	`10`	`+ // ReSharper disable once ClassNeverInstantiated.Global`
`9`	`11`	`public sealed class TheCtor`
`10`	`12`	`{`
`11`	`13`	`public sealed class WithCredentialsParameter`
Original file line number	Diff line number	Diff line change
`@@ -6,6 +6,7 @@`
`6`	`6`	`using Shouldly;`
`7`	`7`	`using Xunit;`
`8`	`8`
	`9`	`+ // ReSharper disable once ClassNeverInstantiated.Global`
`9`	`10`	`public sealed class AzureDevOpsWorkItemSettingsTests`
`10`	`11`	`{`
`11`	`12`	`public sealed class TheCtorForProjectGuid`