fix: linting errors in testing package

Author: Christoph Bühler

src/KubeOps.Testing/KubernetesOperatorFactory.cs

@@ -12,104 +12,103 @@
 using Microsoft.Extensions.Hosting;
 using Microsoft.Extensions.Logging;
 
-namespace KubeOps.Testing
+namespace KubeOps.Testing;
+
+/// <summary>
+/// Operator factory for testing an operator created with the asp web server.
+/// </summary>
+/// <typeparam name="TTestStartup">Type of the Startup type (see asp.net).</typeparam>
+public class KubernetesOperatorFactory<TTestStartup> : WebApplicationFactory<TTestStartup>
+    where TTestStartup : class
 {
+    private string? _solutionRelativeContentRoot;
+
     /// <summary>
-    /// Operator factory for testing an operator created with the asp web server.
+    /// Return a mocked kubernetes client. This client defines
+    /// no-op for all related methods.
     /// </summary>
-    /// <typeparam name="TTestStartup">Type of the Startup type (see asp.net).</typeparam>
-    public class KubernetesOperatorFactory<TTestStartup> : WebApplicationFactory<TTestStartup>
-        where TTestStartup : class
-    {
-        private string? _solutionRelativeContentRoot;
+    public MockKubernetesClient MockedKubernetesClient =>
+        Services.GetRequiredService<IKubernetesClient>() as MockKubernetesClient ??
+        throw new ArgumentException("Wrong kubernetes client registered.");
 
-        /// <summary>
-        /// Return a mocked kubernetes client. This client defines
-        /// no-op for all related methods.
-        /// </summary>
-        public MockKubernetesClient MockedKubernetesClient =>
-            Services.GetRequiredService<IKubernetesClient>() as MockKubernetesClient ??
-            throw new ArgumentException("Wrong kubernetes client registered.");
-
-        /// <summary>
-        /// Set a specific content root path to the given factory.
-        /// </summary>
-        /// <param name="root">The solution relative content root path to configure.</param>
-        /// <returns>The <see cref="KubernetesOperatorFactory{TTestStartup}"/> for chaining.</returns>
-        public KubernetesOperatorFactory<TTestStartup> WithSolutionRelativeContentRoot(string root)
-        {
-            _solutionRelativeContentRoot = root;
-            return this;
-        }
-
-        /// <summary>
-        /// Start the server.
-        /// </summary>
-        public void Run()
-        {
-            // This triggers "EnsureServer()" in the base class.
-            var server = Server;
-        }
+    /// <summary>
+    /// Set a specific content root path to the given factory.
+    /// </summary>
+    /// <param name="root">The solution relative content root path to configure.</param>
+    /// <returns>The <see cref="KubernetesOperatorFactory{TTestStartup}"/> for chaining.</returns>
+    public KubernetesOperatorFactory<TTestStartup> WithSolutionRelativeContentRoot(string root)
+    {
+        _solutionRelativeContentRoot = root;
+        return this;
+    }
 
-        public Task EnqueueEvent<TEntity>(
-            ResourceEventType type,
-            TEntity resource,
-            int attempt = 0,
-            TimeSpan? delay = null)
-            where TEntity : class, IKubernetesObject<V1ObjectMeta>
-        {
-            var queue = Services.GetService<IEventQueue<TEntity>>();
+    /// <summary>
+    /// Start the server.
+    /// </summary>
+    public void Run()
+    {
+        // This triggers "EnsureServer()" in the base class.
+        var server = Server;
+    }
 
-            queue?.EnqueueLocal(new ResourceEvent<TEntity>(type, resource, attempt, delay));
+    public Task EnqueueEvent<TEntity>(
+        ResourceEventType type,
+        TEntity resource,
+        int attempt = 0,
+        TimeSpan? delay = null)
+        where TEntity : class, IKubernetesObject<V1ObjectMeta>
+    {
+        var queue = Services.GetService<IEventQueue<TEntity>>();
 
-            return Task.CompletedTask;
-        }
+        queue?.EnqueueLocal(new ResourceEvent<TEntity>(type, resource, attempt, delay));
 
-        public Task EnqueueFinalization<TEntity>(TEntity resource)
-            where TEntity : class, IKubernetesObject<V1ObjectMeta>
-        {
-            var queue = Services.GetService<IEventQueue<TEntity>>();
+        return Task.CompletedTask;
+    }
 
-            queue?.EnqueueLocal(new ResourceEvent<TEntity>(ResourceEventType.Finalizing, resource));
+    public Task EnqueueFinalization<TEntity>(TEntity resource)
+        where TEntity : class, IKubernetesObject<V1ObjectMeta>
+    {
+        var queue = Services.GetService<IEventQueue<TEntity>>();
 
-            return Task.CompletedTask;
-        }
+        queue?.EnqueueLocal(new ResourceEvent<TEntity>(ResourceEventType.Finalizing, resource));
 
-        /// <summary>
-        /// Create the host builder. Needed for the factory.
-        /// </summary>
-        /// <returns>The created <see cref="IHostBuilder"/>.</returns>
-        protected override IHostBuilder CreateHostBuilder() =>
-            Host.CreateDefaultBuilder()
-                .ConfigureWebHostDefaults(
-                    webBuilder => webBuilder
-                        .UseStartup<TTestStartup>());
+        return Task.CompletedTask;
+    }
 
-        /// <summary>
-        /// Configure the web-host.
-        /// This registers the mocked client as well as mocked
-        /// event queues.
-        /// </summary>
-        /// <param name="builder">The web host builder.</param>
-        protected override void ConfigureWebHost(IWebHostBuilder builder)
-        {
-            base.ConfigureWebHost(builder);
-            builder.ConfigureTestServices(
-                services =>
-                {
-                    var elector = services.First(
-                        d => d.ServiceType == typeof(IHostedService) && d.ImplementationType == typeof(LeaderElector));
-                    services.Remove(elector);
+    /// <summary>
+    /// Create the host builder. Needed for the factory.
+    /// </summary>
+    /// <returns>The created <see cref="IHostBuilder"/>.</returns>
+    protected override IHostBuilder CreateHostBuilder() =>
+        Host.CreateDefaultBuilder()
+            .ConfigureWebHostDefaults(
+                webBuilder => webBuilder
+                    .UseStartup<TTestStartup>());
 
-                    services.RemoveAll(typeof(IEventQueue<>));
-                    services.AddSingleton(typeof(IEventQueue<>), typeof(MockEventQueue<>));
-                });
-            if (_solutionRelativeContentRoot != null)
+    /// <summary>
+    /// Configure the web-host.
+    /// This registers the mocked client as well as mocked
+    /// event queues.
+    /// </summary>
+    /// <param name="builder">The web host builder.</param>
+    protected override void ConfigureWebHost(IWebHostBuilder builder)
+    {
+        base.ConfigureWebHost(builder);
+        builder.ConfigureTestServices(
+            services =>
             {
-                builder.UseSolutionRelativeContentRoot(_solutionRelativeContentRoot);
-            }
+                var elector = services.First(
+                    d => d.ServiceType == typeof(IHostedService) && d.ImplementationType == typeof(LeaderElector));
+                services.Remove(elector);
 
-            builder.ConfigureLogging(logging => logging.ClearProviders());
+                services.RemoveAll(typeof(IEventQueue<>));
+                services.AddSingleton(typeof(IEventQueue<>), typeof(MockEventQueue<>));
+            });
+        if (_solutionRelativeContentRoot != null)
+        {
+            builder.UseSolutionRelativeContentRoot(_solutionRelativeContentRoot);
         }
+
+        builder.ConfigureLogging(logging => logging.ClearProviders());
     }
 }