From 465af0e6326f2d45165211f851a1f4f6fdeea84f Mon Sep 17 00:00:00 2001
From: Joao Grassi <joao.grassi@dynatrace.com>
Date: Fri, 8 Oct 2021 16:06:30 +0200
Subject: [PATCH] PR review fixes

---
 src/OpenTelemetry/Metrics/MeterProviderBuilderExtensions.cs | 4 ++--
 src/OpenTelemetry/Metrics/MeterProviderBuilderSdk.cs        | 4 ++--
 src/OpenTelemetry/Metrics/MeterProviderSdk.cs               | 4 ++--
 test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs         | 2 +-
 4 files changed, 7 insertions(+), 7 deletions(-)

diff --git a/src/OpenTelemetry/Metrics/MeterProviderBuilderExtensions.cs b/src/OpenTelemetry/Metrics/MeterProviderBuilderExtensions.cs
index 7e6a45bb54e..4e14eb3b276 100644
--- a/src/OpenTelemetry/Metrics/MeterProviderBuilderExtensions.cs
+++ b/src/OpenTelemetry/Metrics/MeterProviderBuilderExtensions.cs
@@ -53,7 +53,7 @@ public static MeterProviderBuilder AddReader(this MeterProviderBuilder meterProv
         public static MeterProviderBuilder AddView(this MeterProviderBuilder meterProviderBuilder, string instrumentName, string name)
         {
             // we only need to validate the custom view name in case something was actually provided
-            if (!string.IsNullOrWhiteSpace(name) && !MeterProviderBuilderSdk.ViewNameValid(name))
+            if (!string.IsNullOrWhiteSpace(name) && !MeterProviderBuilderSdk.IsViewNameValid(name))
             {
                 throw new ArgumentException($"Custom view name {name} is invalid.", nameof(name));
             }
@@ -78,7 +78,7 @@ public static MeterProviderBuilder AddView(this MeterProviderBuilder meterProvid
         public static MeterProviderBuilder AddView(this MeterProviderBuilder meterProviderBuilder, string instrumentName, MetricStreamConfiguration metricStreamConfiguration)
         {
             // we only need to validate the custom view name in case something was actually provided
-            if (!string.IsNullOrWhiteSpace(metricStreamConfiguration.Name) && !MeterProviderBuilderSdk.ViewNameValid(metricStreamConfiguration.Name))
+            if (!string.IsNullOrWhiteSpace(metricStreamConfiguration.Name) && !MeterProviderBuilderSdk.IsViewNameValid(metricStreamConfiguration.Name))
             {
                 throw new ArgumentException($"Custom view name {metricStreamConfiguration.Name} is invalid.", nameof(metricStreamConfiguration.Name));
             }
diff --git a/src/OpenTelemetry/Metrics/MeterProviderBuilderSdk.cs b/src/OpenTelemetry/Metrics/MeterProviderBuilderSdk.cs
index 0198ec5167b..58d993df746 100644
--- a/src/OpenTelemetry/Metrics/MeterProviderBuilderSdk.cs
+++ b/src/OpenTelemetry/Metrics/MeterProviderBuilderSdk.cs
@@ -29,7 +29,7 @@ internal class MeterProviderBuilderSdk : MeterProviderBuilderBase
         /// <remarks>See specification: <see href="https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/api.md#instrument"/>.</remarks>
         /// <param name="instrumentName">The instrument name.</param>
         /// <returns>Boolean indicating if the instrument is valid.</returns>
-        internal static bool InstrumentNameValid(string instrumentName)
+        internal static bool IsInstrumentNameValid(string instrumentName)
         {
             if (string.IsNullOrWhiteSpace(instrumentName))
             {
@@ -45,7 +45,7 @@ internal static bool InstrumentNameValid(string instrumentName)
         /// <remarks>See specification: <see href="https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/api.md#instrument"/>.</remarks>
         /// <param name="customViewName">The view name.</param>
         /// <returns>Boolean indicating if the instrument is valid.</returns>
-        internal static bool ViewNameValid(string customViewName)
+        internal static bool IsViewNameValid(string customViewName)
         {
             return InstrumentNameRegex.IsMatch(customViewName);
         }
diff --git a/src/OpenTelemetry/Metrics/MeterProviderSdk.cs b/src/OpenTelemetry/Metrics/MeterProviderSdk.cs
index 6cdbc23e510..9777564ecb9 100644
--- a/src/OpenTelemetry/Metrics/MeterProviderSdk.cs
+++ b/src/OpenTelemetry/Metrics/MeterProviderSdk.cs
@@ -139,7 +139,7 @@ internal MeterProviderSdk(
                                 var metricStreamConfig = metricStreamConfigs[i];
                                 var metricStreamName = metricStreamConfig?.Name ?? instrument.Name;
 
-                                if (!MeterProviderBuilderSdk.InstrumentNameValid(metricStreamName))
+                                if (!MeterProviderBuilderSdk.IsInstrumentNameValid(metricStreamName))
                                 {
                                     OpenTelemetrySdkEventSource.Log.MetricInstrumentIgnored(instrument.Name, instrument.Meter.Name, "Metric name is invalid.", "The name must comply with the OpenTelemetry specification: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/api.md#instrument");
                                     continue;
@@ -206,7 +206,7 @@ internal MeterProviderSdk(
                 {
                     try
                     {
-                        if (!MeterProviderBuilderSdk.InstrumentNameValid(instrument.Name))
+                        if (!MeterProviderBuilderSdk.IsInstrumentNameValid(instrument.Name))
                         {
                             OpenTelemetrySdkEventSource.Log.MetricInstrumentIgnored(instrument.Name, instrument.Meter.Name, "Metric name is invalid.", "The name must comply with the OpenTelemetry specification: https://github.com/open-telemetry/opentelemetry-specification/blob/main/specification/metrics/api.md#instrument");
                             return;
diff --git a/test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs b/test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs
index 7161d700556..493563f406d 100644
--- a/test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs
+++ b/test/OpenTelemetry.Tests/Metrics/MetricViewTests.cs
@@ -144,7 +144,7 @@ public void ViewToRenameMetricConditionally()
         }
 
         [Theory]
-        [MemberData(nameof(MetricsTestData.InvalidInstrumentNames), MemberType = typeof(MetricsTestData))]
+        [MemberData(nameof(MetricsTestData.InvalidViewNames), MemberType = typeof(MetricsTestData))]
         public void ViewWithInvalidNameIgnoredConditionally(string viewNewName)
         {
             using var meter1 = new Meter("ViewToRenameMetricConditionallyTest");