Monitoring todo (#4007)

* Delete unneeded class

* Replace TODO with explanation

* Removed OWIN namespace and folder

src/ServiceControl.Audit/Infrastructure/OWIN/BodyUrlRouteFix.cs → src/ServiceControl.Audit/Infrastructure/WebApi/BodyUrlRouteFix.cs

@@ -1,4 +1,4 @@
-namespace ServiceControl.Audit.Infrastructure.OWIN
+namespace ServiceControl.Audit.Infrastructure.WebApi
 {
     using System.Threading.Tasks;
     using Microsoft.AspNetCore.Http;

src/ServiceControl.Audit/WebApplicationExtensions.cs

@@ -1,7 +1,7 @@
 namespace ServiceControl.Audit;
 
+using Infrastructure.WebApi;
 using Microsoft.AspNetCore.Builder;
-using Infrastructure.OWIN;
 
 public static class WebApplicationExtensions
 {

src/ServiceControl.Monitoring.AcceptanceTests/When_querying_queue_length_data.cs

@@ -82,7 +82,7 @@ await Define<TestContext>()
                          return false;
                      }
 
-                     // TODO this should be using camalCase (the same for a few other monitoring acceptance tests, something sounds off with the json options
+                     // Metric names are fixed names and therefore are not following the casing rules of the chosen serializer
                      if (monitoredEndpointDetails.Digest.Metrics.TryGetValue("QueueLength", out var queueLength) && queueLength.Average == 0.0)
                      {
                          return false;

src/ServiceControl/Infrastructure/OWIN/BodyUrlRouteFix.cs → src/ServiceControl/Infrastructure/WebApi/BodyUrlRouteFix.cs

@@ -1,4 +1,4 @@
-namespace ServiceControl.Infrastructure.OWIN
+namespace ServiceControl.Infrastructure.WebApi
 {
     using System.Threading.Tasks;
     using Microsoft.AspNetCore.Http;

src/ServiceControl/WebApplicationExtensions.cs

@@ -1,7 +1,7 @@
 namespace ServiceControl;
 
-using Infrastructure.OWIN;
 using Infrastructure.SignalR;
+using Infrastructure.WebApi;
 using Microsoft.AspNetCore.Builder;
 
 public static class WebApplicationExtensions