Solution clean up

src/ReportPortal.Client/Abstractions/IClientService.cs

@@ -35,7 +35,7 @@ public interface IClientService
         /// <summary>
         /// Gets the resource for managing asynchronous log items.
         /// </summary>
-        IAsyncLogItemResource  AsyncLogItem { get; }
+        IAsyncLogItemResource AsyncLogItem { get; }
 
         /// <summary>
         /// Gets the resource for managing users.

src/ReportPortal.Client/Abstractions/Models/AnalyzerMode.cs

@@ -12,13 +12,13 @@ public enum AnalyzerMode
         /// </summary>
         [JsonPropertyName("ALL")]
         All,
-        
+
         /// <summary>
         /// Analyzes the current launch.
         /// </summary>
         [JsonPropertyName("CURRENT_LAUNCH")]
         CurrentLaunch,
-        
+
         /// <summary>
         /// Analyzes launches by launch name.
         /// </summary>

src/ReportPortal.Client/Converters/DateTimeUnixEpochConverter.cs

@@ -1,6 +1,6 @@
 using System;
-using System.Text.Json.Serialization;
 using System.Text.Json;
+using System.Text.Json.Serialization;
 
 namespace ReportPortal.Client.Converters
 {

src/ReportPortal.Client/Extensions/UriExtensions.cs

@@ -18,7 +18,7 @@ public static Uri Normalize(this Uri uri)
 
             for (int i = 0; i < uri.Segments.Length; i++)
             {
-                if (!uri.Segments[i].Equals("v1/", StringComparison.OrdinalIgnoreCase) && 
+                if (!uri.Segments[i].Equals("v1/", StringComparison.OrdinalIgnoreCase) &&
                     !uri.Segments[i].Equals("v1", StringComparison.OrdinalIgnoreCase))
                 {
                     normalizedUriString += uri.Segments[i];

src/ReportPortal.Client/Properties/AssemblyInfo.cs

@@ -1,3 +1,3 @@
 using System.Runtime.CompilerServices;
 
-[assembly:InternalsVisibleTo("ReportPortal.Client.IntegrationTests")]
+[assembly: InternalsVisibleTo("ReportPortal.Client.IntegrationTests")]

src/ReportPortal.Client/ServiceException.cs

@@ -48,7 +48,7 @@ public ServiceException(string message, HttpStatusCode httpStatusCode, Uri reque
         /// Gets response body.
         /// </summary>
         public string ResponseBody { get; }
-        
+
         /// <inheritdoc/>
         public override string Message => _message;
     }

src/ReportPortal.Shared/Execution/Logging/LogMessageLevel.cs

@@ -9,27 +9,27 @@ public enum LogMessageLevel
         /// Trace level log message.
         /// </summary>
         Trace,
-        
+
         /// <summary>
         /// Debug level log message.
         /// </summary>
         Debug,
-        
+
         /// <summary>
         /// Information level log message.
         /// </summary>
         Info,
-        
+
         /// <summary>
         /// Warning level log message.
         /// </summary>
         Warning,
-        
+
         /// <summary>
         /// Error level log message.
         /// </summary>
         Error,
-        
+
         /// <summary>
         /// Fatal level log message.
         /// </summary>

src/ReportPortal.Shared/Extensibility/Embedded/Analytics/AnalyticsReportEventsObserver.cs

@@ -209,7 +209,7 @@ private void ReportEventsSource_OnBeforeLaunchStarting(Reporter.ILaunchReporter
                     }
 
                     var stringContent = new StringContent(content, Encoding.UTF8, "application/json");
-                    
+
                     try
                     {
                         using (var response = await httpClient.PostAsync(requestUri, stringContent))

src/ReportPortal.Shared/Extensibility/Embedded/Normalization/RequestNormalizer.cs

@@ -34,7 +34,7 @@ public void Initialize(IReportEventsSource reportEventsSource)
         private void ReportEventsSource_OnBeforeLaunchStarting(ILaunchReporter launchReporter, BeforeLaunchStartingEventArgs args)
         {
             args.StartLaunchRequest.Name = StringTrimmer.Trim(args.StartLaunchRequest.Name, MAX_LAUNCH_NAME_LENGTH);
-            
+
             NormalizeAttributes(args.StartLaunchRequest.Attributes);
         }
 

src/ReportPortal.Shared/Internal/Delegating/ExponentialRetryRequestExecuter.cs

@@ -64,7 +64,7 @@ public ExponentialRetryRequestExecuter(uint maxRetryAttempts, uint baseIndex, IR
         public HttpStatusCode[] HttpStatusCodes { get; private set; }
 
         /// <inheritdoc/>
-        public override async Task<T> ExecuteAsync<T>(Func<Task<T>> func, Action<Exception> beforeNextAttempt = null, IStatisticsCounter statisticsCounter = null, [CallerMemberName]string logicalOperationName = null)
+        public override async Task<T> ExecuteAsync<T>(Func<Task<T>> func, Action<Exception> beforeNextAttempt = null, IStatisticsCounter statisticsCounter = null, [CallerMemberName] string logicalOperationName = null)
         {
             T result = default;
             List<Exception> exceptions = new List<Exception>();

src/ReportPortal.Shared/Properties/AssemblyInfo.cs

@@ -1,3 +1,3 @@
 using System.Runtime.CompilerServices;
 
-[assembly:InternalsVisibleTo("ReportPortal.Shared.Tests")]
+[assembly: InternalsVisibleTo("ReportPortal.Shared.Tests")]

src/ReportPortal.Shared/Reporter/Http/HttpClientHandlerFactory.cs

@@ -37,15 +37,15 @@ public virtual HttpClientHandler Create()
             httpClientHandler.Proxy = GetProxy();
 
             var ignoreSslErrors = Configuration.GetValue<bool>("Server:IgnoreSslErrors", false);
-            
+
 #if NET462
             if (ignoreSslErrors)
             {
                 ServicePointManager.ServerCertificateValidationCallback += (sender, cert, chain, sslPolicyErrors) => true;
             }
             ServicePointManager.SecurityProtocol |= SecurityProtocolType.Tls12;
 #else
-            if (ignoreSslErrors)            
+            if (ignoreSslErrors)
             {
                 httpClientHandler.ServerCertificateCustomValidationCallback = (message, cert, chain, errors) => { return true; };
             }

src/ReportPortal.Shared/Reporter/ITestReporterInfo.cs

@@ -5,6 +5,6 @@
     /// </summary>
     public interface ITestReporterInfo : IReporterInfo
     {
-        
+
     }
 }

src/ReportPortal.Shared/Reporter/LaunchReporter.cs

@@ -1,14 +1,14 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-using ReportPortal.Client.Abstractions;
+using ReportPortal.Client.Abstractions;
 using ReportPortal.Client.Abstractions.Requests;
 using ReportPortal.Shared.Configuration;
 using ReportPortal.Shared.Extensibility;
 using ReportPortal.Shared.Extensibility.ReportEvents.EventArgs;
 using ReportPortal.Shared.Internal.Delegating;
 using ReportPortal.Shared.Reporter.Statistics;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Threading.Tasks;
 
 namespace ReportPortal.Shared.Reporter
 {

src/ReportPortal.Shared/Reporter/TestReporter.cs

@@ -1,15 +1,15 @@
-using System;
-using System.Collections.Generic;
-using System.Linq;
-using System.Threading.Tasks;
-using ReportPortal.Client.Abstractions;
+using ReportPortal.Client.Abstractions;
 using ReportPortal.Client.Abstractions.Requests;
 using ReportPortal.Shared.Configuration;
 using ReportPortal.Shared.Extensibility;
 using ReportPortal.Shared.Extensibility.ReportEvents.EventArgs;
 using ReportPortal.Shared.Internal.Delegating;
 using ReportPortal.Shared.Internal.Logging;
 using ReportPortal.Shared.Reporter.Statistics;
+using System;
+using System.Collections.Generic;
+using System.Linq;
+using System.Threading.Tasks;
 
 namespace ReportPortal.Shared.Reporter
 {

test/ReportPortal.Client.IntegrationTests/LaunchItem/AsyncLaunchItemFixture.cs

@@ -42,7 +42,7 @@ public async Task StartFinishDeleteLaunch()
             // wait until async launch will be processed
             Func<Task> getLaunchAction = async () => gotLaunch = await Service.Launch.GetAsync(launch.Uuid);
             await getLaunchAction.Should().NotThrowAfterAsync(TimeSpan.FromMinutes(1), TimeSpan.FromSeconds(1));
-            
+
             Assert.NotNull(gotLaunch);
             Assert.Equal("StartFinishDeleteAsyncLaunch", gotLaunch.Name);
             gotLaunch.StartTime.Should().BeCloseTo(startLaunchRequest.StartTime, precision: TimeSpan.FromMilliseconds(1));

test/ReportPortal.Client.IntegrationTests/LaunchItem/GetLaunchesItemFixture.cs

@@ -97,7 +97,7 @@ public async Task GetLaunchesSortedByDescendingDate()
 
         [Fact]
         public async Task GetInProgressLaunch()
-        {   
+        {
             var launch = await Service.Launch.StartAsync(new StartLaunchRequest
             {
                 Name = "StartForceFinishIncompleteLaunch",

test/ReportPortal.Client.IntegrationTests/LogItem/AsyncLogItemFixture.cs

@@ -54,7 +54,7 @@ public async Task CreateLogWithAttach()
                 Level = LogLevel.Info,
                 Attach = new LogItemAttach("application/octet-stream", data)
             });
-            
+
             Assert.NotNull(log.Uuid);
         }
 

test/ReportPortal.Shared.Benchmark/Program.cs

@@ -1,6 +1,5 @@
 using BenchmarkDotNet.Running;
 using ReportPortal.Shared.Benchmark.Reporter;
-using System;
 
 namespace ReportPortal.Shared.Benchmark
 {

test/ReportPortal.Shared.Tests/Execution/LogScopeFixture.cs

@@ -1,6 +1,5 @@
 using FluentAssertions;
 using Moq;
-using ReportPortal.Client.Abstractions.Models;
 using ReportPortal.Shared.Execution;
 using ReportPortal.Shared.Execution.Logging;
 using ReportPortal.Shared.Extensibility;

test/ReportPortal.Shared.Tests/Execution/Metadata/MetaAttributeFixture.cs

@@ -1,5 +1,4 @@
 using FluentAssertions;
-using Newtonsoft.Json.Linq;
 using ReportPortal.Client.Abstractions.Models;
 using ReportPortal.Shared.Execution.Metadata;
 using System;

test/ReportPortal.Shared.Tests/Extensibility/Embedded/Analytics/AnalyticsReportEventsObserverTest.cs

@@ -6,13 +6,10 @@
 using ReportPortal.Shared.Reporter;
 using ReportPortal.Shared.Tests.Helpers;
 using RichardSzalay.MockHttp;
-using RichardSzalay.MockHttp.Matchers;
 using System;
 using System.Linq;
 using System.Net;
 using System.Net.Http;
-using System.Text.RegularExpressions;
-using System.Threading.Tasks;
 using System.Web;
 using Xunit;
 

test/ReportPortal.Shared.Tests/Extensibility/Embedded/Normalization/RequestNormalizerTest.cs

@@ -1,5 +1,4 @@
 using FluentAssertions;
-using Moq;
 using ReportPortal.Client.Abstractions.Models;
 using ReportPortal.Client.Abstractions.Requests;
 using ReportPortal.Shared.Extensibility;
@@ -50,11 +49,11 @@ public void ShouldTrimLaunchAttributesDuringStarting()
                 Attributes = new[]
                 {
                     new ItemAttribute
-                    { 
+                    {
                         Key = new string('a', RequestNormalizer.MAX_ATTRIBUTE_KEY_LENGTH + 1),
                         Value = new string('b', RequestNormalizer.MAX_ATTRIBUTE_VALUE_LENGTH + 1)
                     },
-                    new ItemAttribute 
+                    new ItemAttribute
                     {
                         Key = new string('a', RequestNormalizer.MAX_ATTRIBUTE_KEY_LENGTH * 2),
                         Value = new string('b', RequestNormalizer.MAX_ATTRIBUTE_VALUE_LENGTH * 2)
@@ -81,7 +80,7 @@ public void ShouldTrimTestItemNameDuringStarting()
             var launchReporter = new LaunchReporter(service, null, null, _extensionManager);
             launchReporter.Start(new StartLaunchRequest());
 
-            var request = new StartTestItemRequest 
+            var request = new StartTestItemRequest
             {
                 Name = new string('a', RequestNormalizer.MAX_TEST_ITEM_NAME_LENGTH + 1)
             };
@@ -109,9 +108,9 @@ public void ShouldTrimTestItemAttributesDuringStarting()
                     new ItemAttribute
                     {
                         Key = new string('a', RequestNormalizer.MAX_ATTRIBUTE_KEY_LENGTH + 1),
-                        Value = new string('b', RequestNormalizer.MAX_ATTRIBUTE_VALUE_LENGTH + 1) 
+                        Value = new string('b', RequestNormalizer.MAX_ATTRIBUTE_VALUE_LENGTH + 1)
                     },
-                    new ItemAttribute 
+                    new ItemAttribute
                     {
                         Key = new string('a', RequestNormalizer.MAX_ATTRIBUTE_KEY_LENGTH * 2),
                         Value = new string('b', RequestNormalizer.MAX_ATTRIBUTE_VALUE_LENGTH * 2)

test/ReportPortal.Shared.Tests/Helpers/MockServiceBuilder.cs

@@ -28,7 +28,7 @@ public Mock<IClientService> Build()
             service.Setup(s => s.TestItem.FinishAsync(It.IsAny<string>(), It.IsAny<FinishTestItemRequest>(), default)).Returns(() => Task.FromResult(new MessageResponse()));
             service.Setup(s => s.AsyncTestItem.FinishAsync(It.IsAny<string>(), It.IsAny<FinishTestItemRequest>(), default)).Returns(() => Task.FromResult(new MessageResponse()));
 
-            service.Setup(s => s.Launch.FinishAsync(It.IsAny<string>(), It.IsAny<FinishLaunchRequest>(), default)).Returns(() => Task.FromResult(new LaunchFinishedResponse { Link = "http://server:80/path/to/launch"}));
+            service.Setup(s => s.Launch.FinishAsync(It.IsAny<string>(), It.IsAny<FinishLaunchRequest>(), default)).Returns(() => Task.FromResult(new LaunchFinishedResponse { Link = "http://server:80/path/to/launch" }));
             service.Setup(s => s.AsyncLaunch.FinishAsync(It.IsAny<string>(), It.IsAny<FinishLaunchRequest>(), default)).Returns(() => Task.FromResult(new LaunchFinishedResponse { Link = "http://server:80/path/to/launch" }));
 
             return service;

test/ReportPortal.Shared.Tests/Internal/Delegating/LinearRetryRequestExecuterTest.cs

@@ -62,7 +62,7 @@ public async Task ShouldRetryServiceExceptionAction()
             var action = new Mock<Func<Task<string>>>();
             action.Setup(a => a()).Throws(() => new ServiceException("", HttpStatusCode.BadGateway, new Uri("https://example.com"), HttpMethod.Post, ""));
 
-            var executer = new LinearRetryRequestExecuter(3, 0, null, new HttpStatusCode[] { HttpStatusCode.BadGateway} );
+            var executer = new LinearRetryRequestExecuter(3, 0, null, new HttpStatusCode[] { HttpStatusCode.BadGateway });
             await executer.Awaiting(e => e.ExecuteAsync(action.Object)).Should().ThrowAsync<RetryExecutionException>();
 
             action.Verify(a => a(), Times.Exactly(3));

test/ReportPortal.Shared.Tests/Internal/Delegating/RequestExecuterFactoryTest.cs

@@ -81,7 +81,7 @@ public void ConfigurableLinearRetry()
 
             executer.MaxRetryAttemps.Should().Be(5);
             executer.Delay.Should().Be(6000);
-            executer.HttpStatusCodes.Should().BeEquivalentTo(new HttpStatusCode[] { HttpStatusCode.InternalServerError});
+            executer.HttpStatusCodes.Should().BeEquivalentTo(new HttpStatusCode[] { HttpStatusCode.InternalServerError });
         }
 
         [Fact]

test/ReportPortal.Shared.Tests/Reporter/AsyncLogsReporterFixture.cs

@@ -82,7 +82,7 @@ public void ShouldSendBatchedRequests()
                 _extensionManager,
                 _requestExecuter,
                 _logRequestAmender.Object,
-                _reportEventsSource, 
+                _reportEventsSource,
                 20);
 
             for (int i = 0; i < 60; i++)
@@ -172,8 +172,8 @@ public void ShouldSendAsSeparateRequestPerLogWithAttachmentIncludingWithoutAttac
             var logsReporter = new LogsReporter(
                 _testReporter.Object,
                 service.Object,
-                _configuration, 
-                _extensionManager, 
+                _configuration,
+                _extensionManager,
                 _requestExecuter,
                 _logRequestAmender.Object,
                 _reportEventsSource,

test/ReportPortal.Shared.Tests/Reporter/AsyncReporterTest.cs

@@ -509,7 +509,7 @@ public void ShouldNotRerunOfLaunchOnlyIfRerunIsSet()
                 .Callback<StartLaunchRequest, CancellationToken>((r, t) => startLaunchRequest = r);
 
             var config = new ConfigurationBuilder().Build();
-    
+
             config.Properties[ConfigurationPath.AsyncReporting] = true;
             config.Properties["Launch:Rerun"] = "false";
             config.Properties["Launch:RerunOf"] = "any_uuid_of_existing_launch";

test/ReportPortal.Shared.Tests/Reporter/ReporterTest.cs

@@ -460,7 +460,7 @@ public void ShouldRerunOfLaunchOnlyIfRerunIsSet()
 
             service.Setup(s => s.Launch.StartAsync(It.IsAny<StartLaunchRequest>(), default))
                 .Returns(() => Task.FromResult(new LaunchCreatedResponse { Uuid = Guid.NewGuid().ToString() }))
-                .Callback<StartLaunchRequest, CancellationToken>((r,t) => startLaunchRequest = r);
+                .Callback<StartLaunchRequest, CancellationToken>((r, t) => startLaunchRequest = r);
 
             var config = new Shared.Configuration.ConfigurationBuilder().Build();
             config.Properties["Launch:Rerun"] = "true";