Fix MetricTests style warnings

test/OpenTelemetry.Instrumentation.AspNetCore.Tests/MetricTests.cs

@@ -14,11 +14,7 @@
 // limitations under the License.
 // </copyright>
 
-using System;
-using System.Collections.Generic;
 using System.Diagnostics;
-using System.Linq;
-using System.Threading.Tasks;
 using Microsoft.AspNetCore.Hosting;
 using Microsoft.AspNetCore.Http;
 using Microsoft.AspNetCore.Mvc.Testing;
@@ -34,11 +30,11 @@ namespace OpenTelemetry.Instrumentation.AspNetCore.Tests
     public class MetricTests
         : IClassFixture<WebApplicationFactory<Program>>, IDisposable
     {
-        private readonly WebApplicationFactory<Program> factory;
-        private MeterProvider meterProvider = null;
-
         private const int StandardTagsCount = 6;
 
+        private readonly WebApplicationFactory<Program> factory;
+        private MeterProvider meterProvider;
+
         public MetricTests(WebApplicationFactory<Program> factory)
         {
             this.factory = factory;
@@ -197,6 +193,12 @@ void ConfigureTestServices(IServiceCollection services)
             Assert.Contains(tagsToAdd[1], tags);
         }
 
+        public void Dispose()
+        {
+            this.meterProvider?.Dispose();
+            GC.SuppressFinalize(this);
+        }
+
         private static List<MetricPoint> GetMetricPoints(Metric metric)
         {
             Assert.NotNull(metric);
@@ -210,7 +212,8 @@ private static List<MetricPoint> GetMetricPoints(Metric metric)
             return metricPoints;
         }
 
-        private static KeyValuePair<string, object>[] AssertMetricPoint(MetricPoint metricPoint,
+        private static KeyValuePair<string, object>[] AssertMetricPoint(
+            MetricPoint metricPoint,
             string expectedRoute = "api/Values",
             int expectedTagsCount = StandardTagsCount)
         {
@@ -220,16 +223,6 @@ private static KeyValuePair<string, object>[] AssertMetricPoint(MetricPoint metr
             Assert.Equal(1L, count);
             Assert.True(sum > 0);
 
-            /*
-            var bucket = metric.Buckets
-                .Where(b =>
-                    metric.PopulationSum > b.LowBoundary &&
-                    metric.PopulationSum <= b.HighBoundary)
-                .FirstOrDefault();
-            Assert.NotEqual(default, bucket);
-            Assert.Equal(1, bucket.Count);
-            */
-
             var attributes = new KeyValuePair<string, object>[metricPoint.Tags.Count];
             int i = 0;
             foreach (var tag in metricPoint.Tags)
@@ -253,11 +246,5 @@ private static KeyValuePair<string, object>[] AssertMetricPoint(MetricPoint metr
 
             return attributes;
         }
-
-        public void Dispose()
-        {
-            this.meterProvider?.Dispose();
-            GC.SuppressFinalize(this);
-        }
     }
 }