Move TracerBuilderExtensions to the same namespace (#280)

src/OpenTelemetry.Collector.AspNetCore/TracerBuilderExtensions.cs

@@ -13,10 +13,11 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 // </copyright>
-using System;
-using OpenTelemetry.Trace.Configuration;
 
-namespace OpenTelemetry.Collector.AspNetCore
+using System;
+using OpenTelemetry.Collector.AspNetCore;
+
+namespace OpenTelemetry.Trace.Configuration
 {
     public static class TracerBuilderExtensions
     {

src/OpenTelemetry.Collector.Dependencies/TracerBuilderExtensions.cs

@@ -13,10 +13,11 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 // </copyright>
-using System;
-using OpenTelemetry.Trace.Configuration;
 
-namespace OpenTelemetry.Collector.Dependencies
+using System;
+using OpenTelemetry.Collector.Dependencies;
+
+namespace OpenTelemetry.Trace.Configuration
 {
     public static class TracerBuilderExtensions
     {

src/OpenTelemetry.Exporter.ApplicationInsights/TracerBuilderExtensions.cs

@@ -13,11 +13,12 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 // </copyright>
+
 using System;
 using Microsoft.ApplicationInsights.Extensibility;
-using OpenTelemetry.Trace.Configuration;
+using OpenTelemetry.Exporter.ApplicationInsights;
 
-namespace OpenTelemetry.Exporter.ApplicationInsights
+namespace OpenTelemetry.Trace.Configuration
 {
     public static class TracerBuilderExtensions
     {

src/OpenTelemetry.Exporter.Zipkin/TracerBuilderExtensions.cs

@@ -13,10 +13,11 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 // </copyright>
-using System;
-using OpenTelemetry.Trace.Configuration;
 
-namespace OpenTelemetry.Exporter.Zipkin
+using System;
+using OpenTelemetry.Exporter.Zipkin;
+
+namespace OpenTelemetry.Trace.Configuration
 {
     public static class TracerBuilderExtensions
     {

test/OpenTelemetry.Collector.AspNetCore.Tests/BasicTests.cs

@@ -48,7 +48,8 @@ namespace OpenTelemetry.Collector.AspNetCore.Tests
         [Fact]
         public void AddRequestCollector_BadArgs()
         {
-            Assert.Throws<ArgumentNullException>(() => TracerBuilderExtensions.AddRequestCollector(null));
+            TracerBuilder builder = null;
+            Assert.Throws<ArgumentNullException>(() => builder.AddRequestCollector());
             Assert.Throws<ArgumentNullException>(() => TracerFactory.Create(b => b.AddRequestCollector(null)));
         }
 

test/OpenTelemetry.Collector.Dependencies.Tests/BasicTests.cs

@@ -50,7 +50,8 @@ namespace OpenTelemetry.Collector.Dependencies.Tests
         [Fact]
         public void AddDependencyCollector_BadArgs()
         {
-            Assert.Throws<ArgumentNullException>(() => TracerBuilderExtensions.AddDependencyCollector(null));
+            TracerBuilder builder = null;
+            Assert.Throws<ArgumentNullException>(() => builder.AddDependencyCollector());
             Assert.Throws<ArgumentNullException>(() => TracerFactory.Create(b => b.AddDependencyCollector(null)));
         }
 

test/OpenTelemetry.Exporter.ApplicationInsights.Tests/TracerBuilderConfigurationTests.cs

@@ -51,7 +51,8 @@ namespace OpenTelemetry.Exporter.ApplicationInsights.Tests
         [Fact]
         public void UseApplicationInsights_BadArgs()
         {
-            Assert.Throws<ArgumentNullException>(() => TracerBuilderExtensions.UseApplicationInsights(null, _ => { }));
+            TracerBuilder builder = null;
+            Assert.Throws<ArgumentNullException>(() => builder.UseApplicationInsights(_ => { }));
             Assert.Throws<ArgumentNullException>(() => TracerFactory.Create(b => b.UseApplicationInsights(null)));
         }
     }