Fix some analyzer issues (#753)

louis-z · Toa741 · web-flow · commit a4f9a83c9a19 · 2024-09-30T13:14:23.000-04:00
Co-authored-by: Thomas Cortes &lt;78750681+Toa741@users.noreply.github.com&gt;
diff --git a/NGitLab.Mock/Clients/PipelineClient.cs b/NGitLab.Mock/Clients/PipelineClient.cs
@@ -135,7 +135,7 @@ public IEnumerable<Bridge> GetBridges(PipelineBridgeQuery query)
         {
             var project = Context.Server.AllProjects.FindById(_projectId);
             var bridges = project.Jobs.Where(j => j.Pipeline.Id == query.PipelineId && j.DownstreamPipeline != null).Select(j => j.ToBridgeClient());
-            return (query.Scope == null || !query.Scope.Any()) ? bridges : bridges.Where(j => query.Scope.Contains(j.Status.ToString(), StringComparer.Ordinal));
+            return (query.Scope == null || query.Scope.Length == 0) ? bridges : bridges.Where(j => query.Scope.Contains(j.Status.ToString(), StringComparer.Ordinal));
         }
     }
 
@@ -152,7 +152,7 @@ public Models.Job[] GetJobs(int pipelineId)
         using (Context.BeginOperationScope())
         {
             var jobs = _jobClient.GetJobs(JobScopeMask.All).Where(j => j.Pipeline.Id == query.PipelineId);
-            return (query.Scope == null || !query.Scope.Any()) ? jobs : jobs.Where(j => query.Scope.Contains(j.Status.ToString(), StringComparer.Ordinal));
+            return (query.Scope == null || query.Scope.Length == 0) ? jobs : jobs.Where(j => query.Scope.Contains(j.Status.ToString(), StringComparer.Ordinal));
         }
     }
 
diff --git a/NGitLab.Mock/Internals/TemporaryDirectory.cs b/NGitLab.Mock/Internals/TemporaryDirectory.cs
@@ -60,7 +60,7 @@ public void MakeReadOnly()
         MakeReadOnly(new DirectoryInfo(FullPath));
     }
 
-    private void MakeReadOnly(FileSystemInfo fileSystemInfo)
+    private static void MakeReadOnly(FileSystemInfo fileSystemInfo)
     {
         if (fileSystemInfo is DirectoryInfo directoryInfo)
         {
diff --git a/NGitLab.Mock/UserCollection.cs b/NGitLab.Mock/UserCollection.cs
@@ -108,7 +108,7 @@ internal IEnumerable<User> Get(UserQuery query)
 
         if (query.ExcludeExternal == true)
         {
-            users = users.Where(u => !u.Identities.Any() || u.Identities.All(i => string.IsNullOrEmpty(i.ExternUid)));
+            users = users.Where(u => u.Identities.Length == 0 || u.Identities.All(i => string.IsNullOrEmpty(i.ExternUid)));
         }
 
         if (!string.IsNullOrEmpty(query.Search))
diff --git a/NGitLab.Tests/Docker/GitLabTestContext.cs b/NGitLab.Tests/Docker/GitLabTestContext.cs
@@ -25,7 +25,7 @@ public sealed class GitLabTestContext : IDisposable
     private static readonly Policy s_gitlabRetryPolicy = Policy.Handle<GitLabException>()
         .WaitAndRetry(MaxRetryCount, _ => TimeSpan.FromSeconds(1), (exception, timespan, retryCount, context) =>
         {
-            TestContext.WriteLine($"[{TestContext.CurrentContext.Test.FullName}] {exception.Message} -> Polly Retry {retryCount} of {MaxRetryCount}...");
+            TestContext.Out.WriteLine($"[{TestContext.CurrentContext.Test.FullName}] {exception.Message} -> Polly Retry {retryCount} of {MaxRetryCount}...");
         });
 
     private static readonly HashSet<string> s_generatedValues = new(StringComparer.Ordinal);
@@ -381,15 +381,15 @@ public async Task<IDisposable> StartRunnerForOneJobAsync(int projectId)
                 }
             }
 
-            TestContext.WriteLine("Test runner downloaded");
+            TestContext.Out.WriteLine("Test runner downloaded");
             if (OperatingSystem.IsLinux())
             {
                 using var chmodProcess = Process.Start("chmod", "+x \"" + path + "\"");
                 chmodProcess.WaitForExit();
                 if (chmodProcess.ExitCode != 0)
                     throw new InvalidOperationException("chmod failed");
 
-                TestContext.WriteLine("chmod run");
+                TestContext.Out.WriteLine("chmod run");
             }
 
             if (!IsContinuousIntegration())
@@ -401,7 +401,7 @@ public async Task<IDisposable> StartRunnerForOneJobAsync(int projectId)
                 if (gitConfigProcess.ExitCode != 0)
                     throw new InvalidOperationException("git config failed");
 
-                TestContext.WriteLine("git config changed");
+                TestContext.Out.WriteLine("git config changed");
             }
 
             var project = AdminClient.Projects[projectId];
@@ -412,7 +412,7 @@ public async Task<IDisposable> StartRunnerForOneJobAsync(int projectId)
             if (runner.Token == null)
                 throw new InvalidOperationException("Runner token is null");
 
-            TestContext.WriteLine($"Runner registered '{runner.Token}'");
+            TestContext.Out.WriteLine($"Runner registered '{runner.Token}'");
 
             // Use run-single, so we don't need to manage the configuration file.
             // Also, I don't think there is a need to run multiple jobs in a test
@@ -437,9 +437,7 @@ public async Task<IDisposable> StartRunnerForOneJobAsync(int projectId)
                 RedirectStandardError = true,
                 RedirectStandardOutput = true,
             };
-            var process = Process.Start(psi);
-            if (process == null)
-                throw new InvalidOperationException("Cannot start the runner");
+            var process = Process.Start(psi) ?? throw new InvalidOperationException("Cannot start the runner");
 
             // Give some time to ensure the runner doesn't stop immediately
             await Task.Delay(TimeSpan.FromSeconds(2)).ConfigureAwait(false);
@@ -452,7 +450,7 @@ public async Task<IDisposable> StartRunnerForOneJobAsync(int projectId)
             process.ErrorDataReceived += (sender, e) => Console.Error.WriteLine(e.Data);
             process.OutputDataReceived += (sender, e) => Console.Error.WriteLine(e.Data);
 
-            TestContext.WriteLine($"Runner started for project '{project.Id}' on '{DockerContainer.GitLabUrl}'");
+            TestContext.Out.WriteLine($"Runner started for project '{project.Id}' on '{DockerContainer.GitLabUrl}'");
             return new ProcessKill(process);
         }
         finally
@@ -474,7 +472,7 @@ public static async Task<T> RetryUntilAsync<T>(Func<T> action, Func<T, bool> pre
         while (!predicate(result))
         {
             cancellationToken.ThrowIfCancellationRequested();
-            TestContext.WriteLine($"[{TestContext.CurrentContext.Test.FullName}] RetryUntilAsync {retryCount++}...");
+            TestContext.Out.WriteLine($"[{TestContext.CurrentContext.Test.FullName}] RetryUntilAsync {retryCount++}...");
             await Task.Delay(1000, cancellationToken).ConfigureAwait(false);
 
             result = action();
diff --git a/NGitLab.Tests/Docker/GitLabTestContextRequestOptions.cs b/NGitLab.Tests/Docker/GitLabTestContextRequestOptions.cs
@@ -174,7 +174,7 @@ private WebResponse LogRequest(HttpWebRequest request, WebResponse response)
         }
 
         var logs = sb.ToString();
-        TestContext.WriteLine(new string('-', 100) + "\nGitLab request: " + logs);
+        TestContext.Out.WriteLine(new string('-', 100) + "\nGitLab request: " + logs);
         return response;
     }
 
diff --git a/NGitLab.Tests/JobTests.cs b/NGitLab.Tests/JobTests.cs
@@ -170,7 +170,7 @@ public async Task Test_get_job_trace()
                 var job = jobs.FirstOrDefault();
                 if (jobs.Any())
                 {
-                    TestContext.WriteLine("Job status: " + job.Status);
+                    TestContext.Out.WriteLine("Job status: " + job.Status);
                     return job.Status == JobStatus.Success || job.Status == JobStatus.Failed;
                 }
 
diff --git a/NGitLab.Tests/LintClientTests.cs b/NGitLab.Tests/LintClientTests.cs
@@ -1,5 +1,4 @@
-﻿using System.Linq;
-using System.Threading;
+﻿using System.Threading;
 using System.Threading.Tasks;
 using NGitLab.Models;
 using NGitLab.Tests.Docker;
@@ -38,8 +37,8 @@ public async Task LintValidCIYaml()
         var result = await context.Client.Lint.ValidateCIYamlContentAsync(project.Id.ToString(), ValidCIYaml, new(), CancellationToken.None);
 
         Assert.That(result.Valid, Is.True);
-        Assert.That(result.Errors.Any(), Is.False);
-        Assert.That(result.Warnings.Any(), Is.False);
+        Assert.That(result.Errors.Length != 0, Is.False);
+        Assert.That(result.Warnings.Length != 0, Is.False);
     }
 
     [Test]
@@ -53,8 +52,8 @@ public async Task LintInvalidCIYaml()
         var result = await context.Client.Lint.ValidateCIYamlContentAsync(project.Id.ToString(), InvalidCIYaml, new(), CancellationToken.None);
 
         Assert.That(result.Valid, Is.False);
-        Assert.That(result.Errors.Any(), Is.True);
-        Assert.That(result.Warnings.Any(), Is.False);
+        Assert.That(result.Errors.Length != 0, Is.True);
+        Assert.That(result.Warnings.Length != 0, Is.False);
     }
 
     [Test]
@@ -76,8 +75,8 @@ public async Task LintValidCIProjectYaml()
         var result = await context.Client.Lint.ValidateProjectCIConfigurationAsync(project.Id.ToString(), new(), CancellationToken.None);
 
         Assert.That(result.Valid, Is.True);
-        Assert.That(result.Errors.Any(), Is.False);
-        Assert.That(result.Warnings.Any(), Is.False);
+        Assert.That(result.Errors.Length != 0, Is.False);
+        Assert.That(result.Warnings.Length != 0, Is.False);
     }
 
     [Test]
@@ -99,7 +98,7 @@ public async Task LintInvalidProjectCIYaml()
         var result = await context.Client.Lint.ValidateProjectCIConfigurationAsync(project.Id.ToString(), new(), CancellationToken.None);
 
         Assert.That(result.Valid, Is.False);
-        Assert.That(result.Errors.Any(), Is.True);
-        Assert.That(result.Warnings.Any(), Is.False);
+        Assert.That(result.Errors.Length != 0, Is.True);
+        Assert.That(result.Warnings.Length != 0, Is.False);
     }
 }
diff --git a/NGitLab.Tests/MergeRequest/MergeRequestChangesClientTests.cs b/NGitLab.Tests/MergeRequest/MergeRequestChangesClientTests.cs
@@ -1,5 +1,4 @@
 using System;
-using System.Linq;
 using System.Threading.Tasks;
 using NGitLab.Tests.Docker;
 using NUnit.Framework;
@@ -19,7 +18,7 @@ public async Task GetChangesOnMergeRequest()
 
         var changes = await GitLabTestContext.RetryUntilAsync(
             () => mergeRequestChanges.MergeRequestChange.Changes,
-            changes => changes.Any(),
+            changes => changes.Length != 0,
             TimeSpan.FromSeconds(10));
 
         Assert.That(changes, Has.Length.EqualTo(1));
diff --git a/NGitLab.Tests/MergeRequest/MergeRequestClientTests.cs b/NGitLab.Tests/MergeRequest/MergeRequestClientTests.cs
@@ -36,9 +36,9 @@ await GitLabTestContext.RetryUntilAsync(
 
         Assert.That(context.Client.GetRepository(project.Id).Branches[mergeRequest.SourceBranch].Protected, Is.False, "The source branch is protected but should not be");
 
-        TestContext.WriteLine("MR is ready to be merged");
+        TestContext.Out.WriteLine("MR is ready to be merged");
         AcceptMergeRequest(mergeRequestClient, mergeRequest);
-        TestContext.WriteLine("MR is merged");
+        TestContext.Out.WriteLine("MR is merged");
 
         // Since GitLab 13.10, this part is flaky
         // await Task.Delay(TimeSpan.FromSeconds(5)).ConfigureAwait(false);
diff --git a/NGitLab.Tests/PipelineTests.cs b/NGitLab.Tests/PipelineTests.cs
@@ -71,8 +71,8 @@ public async Task Test_can_list_all_jobs_from_project()
         var pipelineClient = context.Client.GetPipelines(project.Id);
         JobTests.AddGitLabCiFile(context.Client, project);
 
-        var allJobs = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.AllJobs.ToList(), p => p.Any(), TimeSpan.FromSeconds(120));
-        Assert.That(allJobs.Any());
+        var allJobs = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.AllJobs.ToList(), p => p.Count != 0, TimeSpan.FromSeconds(120));
+        Assert.That(allJobs.Count != 0);
     }
 
     [Test]
@@ -91,8 +91,8 @@ public async Task Test_can_list_jobs_from_pipeline()
         {
             PipelineId = pipeline.Id, Scope = new[] { "success", "pending" },
         };
-        var allJobs = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.GetJobsAsync(pipelineJobQuery).ToList(), p => p.Any(), TimeSpan.FromSeconds(120));
-        Assert.That(allJobs.Any());
+        var allJobs = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.GetJobsAsync(pipelineJobQuery).ToList(), p => p.Count != 0, TimeSpan.FromSeconds(120));
+        Assert.That(allJobs.Count != 0);
     }
 
     [Test]
@@ -108,9 +108,9 @@ public async Task Test_search_for_pipeline()
         {
             Ref = project.DefaultBranch,
         };
-        var pipelinesFromQuery = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.Search(query).ToList(), p => p.Any(), TimeSpan.FromSeconds(120));
+        var pipelinesFromQuery = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.Search(query).ToList(), p => p.Count != 0, TimeSpan.FromSeconds(120));
 
-        Assert.That(pipelinesFromQuery.Any(), Is.True);
+        Assert.That(pipelinesFromQuery.Count != 0, Is.True);
     }
 
     [Test]
@@ -245,7 +245,7 @@ public async Task Test_retry()
             {
                 if (pipeline != null)
                 {
-                    TestContext.WriteLine("Pipeline status: " + pipeline.Status);
+                    TestContext.Out.WriteLine("Pipeline status: " + pipeline.Status);
                     return pipeline.Status is JobStatus.Failed;
                 }
 
diff --git a/NGitLab.Tests/ProjectVariableClientTests.cs b/NGitLab.Tests/ProjectVariableClientTests.cs
@@ -1,7 +1,6 @@
 ﻿using System.Linq;
 using System.Net;
 using System.Threading.Tasks;
-using NGitLab.Extensions;
 using NGitLab.Models;
 using NGitLab.Tests.Docker;
 using NuGet.Versioning;
diff --git a/NGitLab.Tests/ProjectsTests.cs b/NGitLab.Tests/ProjectsTests.cs
@@ -567,7 +567,7 @@ public async Task Test_get_by_project_query_projectQuery_MinAccessLevel_returns_
         var result = projectClient.Get(query).Take(10).ToArray();
 
         // Assert
-        Assert.That(result.Any(), Is.True);
+        Assert.That(result.Length != 0, Is.True);
     }
 
     [Test]
diff --git a/NGitLab.Tests/SetUpFixture.cs b/NGitLab.Tests/SetUpFixture.cs
@@ -9,6 +9,6 @@ public sealed class SetUpFixture
     [OneTimeSetUp]
     public void RunBeforeAnyTests()
     {
-        FunctionRetryExtensions.Logger = msg => TestContext.WriteLine($"[{TestContext.CurrentContext.Test.FullName}] {msg}");
+        FunctionRetryExtensions.Logger = msg => TestContext.Out.WriteLine($"[{TestContext.CurrentContext.Test.FullName}] {msg}");
     }
 }
diff --git a/NGitLab.Tests/UsersTests.cs b/NGitLab.Tests/UsersTests.cs
@@ -49,7 +49,7 @@ public async Task CreateUpdateDelete()
 
         users.Delete(addedUser.Id);
 
-        await GitLabTestContext.RetryUntilAsync(() => users.Get(addedUser.Username).ToList(), users => !users.Any(), TimeSpan.FromMinutes(2));
+        await GitLabTestContext.RetryUntilAsync(() => users.Get(addedUser.Username).ToList(), users => users.Count == 0, TimeSpan.FromMinutes(2));
     }
 
     [Test]
@@ -62,7 +62,7 @@ public async Task CreateAsync()
         var addedUser = await CreateNewUserAsync(context);
         Assert.That(addedUser.Bio, Is.EqualTo("bio"));
 
-        await GitLabTestContext.RetryUntilAsync(() => users.Get(addedUser.Username).ToList(), users => users.Any(), TimeSpan.FromMinutes(2));
+        await GitLabTestContext.RetryUntilAsync(() => users.Get(addedUser.Username).ToList(), users => users.Count != 0, TimeSpan.FromMinutes(2));
     }
 
     [Test]
@@ -106,7 +106,7 @@ public async Task DeactivatedAccountShouldBeAbleToActivate()
 
         users.Delete(addedUser.Id);
 
-        await GitLabTestContext.RetryUntilAsync(() => users.Get(addedUser.Username).ToList(), users => !users.Any(), TimeSpan.FromMinutes(2));
+        await GitLabTestContext.RetryUntilAsync(() => users.Get(addedUser.Username).ToList(), users => users.Count == 0, TimeSpan.FromMinutes(2));
     }
 
     [Test]
diff --git a/NGitLab/Impl/GroupsClient.cs b/NGitLab/Impl/GroupsClient.cs
@@ -1,6 +1,5 @@
 ﻿using System;
 using System.Collections.Generic;
-using System.Linq;
 using System.Threading;
 using System.Threading.Tasks;
 using NGitLab.Models;
@@ -52,7 +51,7 @@ private static string CreateGetUrl(GroupQuery query, int? page = null, int? perP
             return url;
         }
 
-        if (query.SkipGroups != null && query.SkipGroups.Any())
+        if (query.SkipGroups != null && query.SkipGroups.Length != 0)
         {
             foreach (var skipGroup in query.SkipGroups)
             {
@@ -116,7 +115,7 @@ private static string CreateSubgroupGetUrl(GroupId id, SubgroupQuery query, int?
             return url;
         }
 
-        if (query.SkipGroups != null && query.SkipGroups.Any())
+        if (query.SkipGroups != null && query.SkipGroups.Length != 0)
         {
             foreach (var skipGroup in query.SkipGroups)
             {
diff --git a/NGitLab/Impl/PipelineClient.cs b/NGitLab/Impl/PipelineClient.cs
@@ -175,7 +175,7 @@ private string CreateSearchUrl(PipelineQuery query)
             queryEntries.Add("per_page", query.PerPage.Value.ToString());
 
         var stringQuery = string.Join("&", queryEntries.Select(kp => $"{kp.Key}={kp.Value}"));
-        var url = $"{_projectPath}/pipelines{(queryEntries.Any() ? $"?{stringQuery}" : string.Empty)}";
+        var url = $"{_projectPath}/pipelines{(queryEntries.Count != 0 ? $"?{stringQuery}" : string.Empty)}";
         return url;
     }
 
diff --git a/NGitLab/Impl/ProjectClient.cs b/NGitLab/Impl/ProjectClient.cs
@@ -244,7 +244,7 @@ public Dictionary<string, double> GetLanguages(string id)
         // So now we wait for the languages to be returned with a max wait time of 10 s.
         // The waiting logic should be removed once GitLab fix the issue in a version > 11.10.4-ee.
         var started = DateTime.UtcNow;
-        while (!languages.Any() && (DateTime.UtcNow - started) < TimeSpan.FromSeconds(10))
+        while (languages.Count == 0 && (DateTime.UtcNow - started) < TimeSpan.FromSeconds(10))
         {
             Thread.Sleep(1000);
             languages = DoGetLanguages(id);
diff --git a/NGitLab/Impl/VariableClient.cs b/NGitLab/Impl/VariableClient.cs
@@ -9,7 +9,7 @@ internal abstract class VariableClient
     private readonly string _urlPrefix;
     private readonly API _api;
 
-    private string EnvironmentScopeFilter(string environmentScope = null) => !string.IsNullOrWhiteSpace(environmentScope) ? $"?filter[environment_scope]={Uri.EscapeDataString(environmentScope)}" : string.Empty;
+    private static string EnvironmentScopeFilter(string environmentScope = null) => !string.IsNullOrWhiteSpace(environmentScope) ? $"?filter[environment_scope]={Uri.EscapeDataString(environmentScope)}" : string.Empty;
 
     protected VariableClient(API api, string urlPrefix)
     {
diff --git a/NGitLab/Models/VariableCreate.cs b/NGitLab/Models/VariableCreate.cs
@@ -1,5 +1,4 @@
-﻿using System.ComponentModel;
-using System.Text.Json.Serialization;
+﻿using System.Text.Json.Serialization;
 
 namespace NGitLab.Models;
 

Original file line number	Diff line number	Diff line change
`@@ -135,7 +135,7 @@ public IEnumerable<Bridge> GetBridges(PipelineBridgeQuery query)`
`135`	`135`	`{`
`136`	`136`	`var project = Context.Server.AllProjects.FindById(_projectId);`
`137`	`137`	`var bridges = project.Jobs.Where(j => j.Pipeline.Id == query.PipelineId && j.DownstreamPipeline != null).Select(j => j.ToBridgeClient());`
`138`		`- return (query.Scope == null \|\| !query.Scope.Any()) ? bridges : bridges.Where(j => query.Scope.Contains(j.Status.ToString(), StringComparer.Ordinal));`
	`138`	`+ return (query.Scope == null \|\| query.Scope.Length == 0) ? bridges : bridges.Where(j => query.Scope.Contains(j.Status.ToString(), StringComparer.Ordinal));`
`139`	`139`	`}`
`140`	`140`	`}`
`141`	`141`
`@@ -152,7 +152,7 @@ public Models.Job[] GetJobs(int pipelineId)`
`152`	`152`	`using (Context.BeginOperationScope())`
`153`	`153`	`{`
`154`	`154`	`var jobs = _jobClient.GetJobs(JobScopeMask.All).Where(j => j.Pipeline.Id == query.PipelineId);`
`155`		`- return (query.Scope == null \|\| !query.Scope.Any()) ? jobs : jobs.Where(j => query.Scope.Contains(j.Status.ToString(), StringComparer.Ordinal));`
	`155`	`+ return (query.Scope == null \|\| query.Scope.Length == 0) ? jobs : jobs.Where(j => query.Scope.Contains(j.Status.ToString(), StringComparer.Ordinal));`
`156`	`156`	`}`
`157`	`157`	`}`
`158`	`158`
Original file line number	Diff line number	Diff line change
`@@ -60,7 +60,7 @@ public void MakeReadOnly()`
`60`	`60`	`MakeReadOnly(new DirectoryInfo(FullPath));`
`61`	`61`	`}`
`62`	`62`
`63`		`- private void MakeReadOnly(FileSystemInfo fileSystemInfo)`
	`63`	`+ private static void MakeReadOnly(FileSystemInfo fileSystemInfo)`
`64`	`64`	`{`
`65`	`65`	`if (fileSystemInfo is DirectoryInfo directoryInfo)`
`66`	`66`	`{`
Original file line number	Diff line number	Diff line change
`@@ -108,7 +108,7 @@ internal IEnumerable<User> Get(UserQuery query)`
`108`	`108`
`109`	`109`	`if (query.ExcludeExternal == true)`
`110`	`110`	`{`
`111`		`- users = users.Where(u => !u.Identities.Any() \|\| u.Identities.All(i => string.IsNullOrEmpty(i.ExternUid)));`
	`111`	`+ users = users.Where(u => u.Identities.Length == 0 \|\| u.Identities.All(i => string.IsNullOrEmpty(i.ExternUid)));`
`112`	`112`	`}`
`113`	`113`
`114`	`114`	`if (!string.IsNullOrEmpty(query.Search))`
Original file line number	Diff line number	Diff line change
`@@ -174,7 +174,7 @@ private WebResponse LogRequest(HttpWebRequest request, WebResponse response)`
`174`	`174`	`}`
`175`	`175`
`176`	`176`	`var logs = sb.ToString();`
`177`		`- TestContext.WriteLine(new string('-', 100) + "\nGitLab request: " + logs);`
	`177`	`+ TestContext.Out.WriteLine(new string('-', 100) + "\nGitLab request: " + logs);`
`178`	`178`	`return response;`
`179`	`179`	`}`
`180`	`180`
Original file line number	Diff line number	Diff line change
`@@ -170,7 +170,7 @@ public async Task Test_get_job_trace()`
`170`	`170`	`var job = jobs.FirstOrDefault();`
`171`	`171`	`if (jobs.Any())`
`172`	`172`	`{`
`173`		`- TestContext.WriteLine("Job status: " + job.Status);`
	`173`	`+ TestContext.Out.WriteLine("Job status: " + job.Status);`
`174`	`174`	`return job.Status == JobStatus.Success \|\| job.Status == JobStatus.Failed;`
`175`	`175`	`}`
`176`	`176`
Original file line number	Diff line number	Diff line change
`@@ -71,8 +71,8 @@ public async Task Test_can_list_all_jobs_from_project()`
`71`	`71`	`var pipelineClient = context.Client.GetPipelines(project.Id);`
`72`	`72`	`JobTests.AddGitLabCiFile(context.Client, project);`
`73`	`73`
`74`		`- var allJobs = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.AllJobs.ToList(), p => p.Any(), TimeSpan.FromSeconds(120));`
`75`		`- Assert.That(allJobs.Any());`
	`74`	`+ var allJobs = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.AllJobs.ToList(), p => p.Count != 0, TimeSpan.FromSeconds(120));`
	`75`	`+ Assert.That(allJobs.Count != 0);`
`76`	`76`	`}`
`77`	`77`
`78`	`78`	`[Test]`
`@@ -91,8 +91,8 @@ public async Task Test_can_list_jobs_from_pipeline()`
`91`	`91`	`{`
`92`	`92`	`PipelineId = pipeline.Id, Scope = new[] { "success", "pending" },`
`93`	`93`	`};`
`94`		`- var allJobs = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.GetJobsAsync(pipelineJobQuery).ToList(), p => p.Any(), TimeSpan.FromSeconds(120));`
`95`		`- Assert.That(allJobs.Any());`
	`94`	`+ var allJobs = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.GetJobsAsync(pipelineJobQuery).ToList(), p => p.Count != 0, TimeSpan.FromSeconds(120));`
	`95`	`+ Assert.That(allJobs.Count != 0);`
`96`	`96`	`}`
`97`	`97`
`98`	`98`	`[Test]`
`@@ -108,9 +108,9 @@ public async Task Test_search_for_pipeline()`
`108`	`108`	`{`
`109`	`109`	`Ref = project.DefaultBranch,`
`110`	`110`	`};`
`111`		`- var pipelinesFromQuery = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.Search(query).ToList(), p => p.Any(), TimeSpan.FromSeconds(120));`
	`111`	`+ var pipelinesFromQuery = await GitLabTestContext.RetryUntilAsync(() => pipelineClient.Search(query).ToList(), p => p.Count != 0, TimeSpan.FromSeconds(120));`
`112`	`112`
`113`		`- Assert.That(pipelinesFromQuery.Any(), Is.True);`
	`113`	`+ Assert.That(pipelinesFromQuery.Count != 0, Is.True);`
`114`	`114`	`}`
`115`	`115`
`116`	`116`	`[Test]`
`@@ -245,7 +245,7 @@ public async Task Test_retry()`
`245`	`245`	`{`
`246`	`246`	`if (pipeline != null)`
`247`	`247`	`{`
`248`		`- TestContext.WriteLine("Pipeline status: " + pipeline.Status);`
	`248`	`+ TestContext.Out.WriteLine("Pipeline status: " + pipeline.Status);`
`249`	`249`	`return pipeline.Status is JobStatus.Failed;`
`250`	`250`	`}`
`251`	`251`