Merge pull request #41 from Lombiq/issue/OSOE-867

OSOE-867: Addressing analyzer warnings
Lombiq · Jun 11, 2024 · 9f1b5ee · 9f1b5ee
2 parents 2e9d8b7 + 0181902
commit 9f1b5ee
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 3 deletions.
diff --git a/Extensions/AutoMockerExtensions.cs b/Extensions/AutoMockerExtensions.cs
@@ -60,7 +60,7 @@ public static ListLoggerProvider MockLogging(this AutoMocker mocker)
     {
         var provider = new ListLoggerProvider();
 
-        mocker.Use<ILoggerFactory>(new LoggerFactory(new ILoggerProvider[] { provider }));
+        mocker.Use<ILoggerFactory>(new LoggerFactory([provider]));
         mocker.EnsureResolver<LoggerResolver>();
 
         return provider;

diff --git a/Integration/Controllers/AutoMockerController.cs b/Integration/Controllers/AutoMockerController.cs
@@ -33,6 +33,8 @@ public AutoMockerController(AutoMocker mocker)
 
     public void SetRequestUri(Uri uri)
     {
+        if (!ModelState.IsValid) return;
+
         HttpContext.Request.Host = HostString.FromUriComponent(uri);
         HttpContext.Request.IsHttps = uri.Scheme.Equals("HTTPS", StringComparison.OrdinalIgnoreCase);
 

diff --git a/Integration/Services/ListLogger.cs b/Integration/Services/ListLogger.cs
@@ -1,4 +1,4 @@
-using Microsoft.Extensions.Logging;
+using Microsoft.Extensions.Logging;
 using System;
 using System.Collections.Concurrent;
 using System.Collections.Generic;
@@ -9,7 +9,7 @@ namespace Lombiq.Tests.Integration.Services;
 public class ListLogger : ILogger
 {
     public string CategoryName { get; }
-    public IList<LogEntry> Logs { get; } = new List<LogEntry>();
+    public IList<LogEntry> Logs { get; } = [];
 
     public ListLogger(string categoryName) => CategoryName = categoryName;