bazel cleanup

Author: marcalff

api/include/opentelemetry/trace/provider.h

@@ -11,6 +11,10 @@
 #include "opentelemetry/trace/tracer_provider.h"
 #include "opentelemetry/version.h"
 
+// The friend declaration is sufficient,
+// we do not want a API -> SDK dependency.
+// IWYU pragma: no_include "opentelemetry/sdk/trace/provider.h"
+
 OPENTELEMETRY_BEGIN_NAMESPACE
 namespace sdk
 {

api/test/metrics/meter_provider_test.cc

@@ -4,7 +4,6 @@
 #include <gtest/gtest.h>
 
 #include "opentelemetry/metrics/provider.h"
-#include "opentelemetry/nostd/shared_ptr.h"
 
 using opentelemetry::metrics::MeterProvider;
 using opentelemetry::metrics::Provider;

sdk/src/common/BUILD

@@ -33,6 +33,18 @@ cc_library(
     ],
 )
 
+cc_library(
+    name = "disabled",
+    srcs = [
+        "disabled.cc",
+    ],
+    include_prefix = "src/common",
+    deps = [
+        "//api",
+        "//sdk:headers",
+    ],
+)
+
 cc_library(
     name = "env_variables",
     srcs = [

sdk/src/logs/BUILD

@@ -11,6 +11,7 @@ cc_library(
         "//api",
         "//sdk:headers",
         "//sdk/src/common:global_log_handler",
+        "//sdk/src/common:disabled",
         "//sdk/src/resource",
     ],
 )

sdk/src/metrics/BUILD

@@ -11,6 +11,7 @@ cc_library(
         "//api",
         "//sdk:headers",
         "//sdk/src/common:global_log_handler",
+        "//sdk/src/common:disabled",
         "//sdk/src/common:random",
         "//sdk/src/resource",
     ],

sdk/src/trace/BUILD

@@ -12,6 +12,7 @@ cc_library(
         "//api",
         "//sdk:headers",
         "//sdk/src/common:global_log_handler",
+        "//sdk/src/common:disabled",
         "//sdk/src/common:random",
         "//sdk/src/resource",
     ],

sdk/src/trace/provider.cc

@@ -6,7 +6,6 @@
 #include "opentelemetry/nostd/shared_ptr.h"
 #include "opentelemetry/sdk/common/disabled.h"
 #include "opentelemetry/sdk/trace/provider.h"
-#include "opentelemetry/trace/noop.h"
 #include "opentelemetry/trace/tracer_provider.h"
 #include "opentelemetry/version.h"
 

sdk/test/logs/BUILD

@@ -17,6 +17,22 @@ cc_test(
     ],
 )
 
+cc_test(
+    name = "logger_provider_set_test",
+    srcs = [
+        "logger_provider_set_test.cc",
+    ],
+    tags = [
+        "logs",
+        "test",
+    ],
+    deps = [
+        "//api",
+        "//sdk/src/logs",
+        "@com_google_googletest//:gtest_main",
+    ],
+)
+
 cc_test(
     name = "logger_sdk_test",
     srcs = [

sdk/test/singleton/BUILD

@@ -196,7 +196,7 @@ cc_test(
         "component_e",
         "component_f",
         "//api",
-        "//sdk/src/common",
+        "//sdk/src/common:disabled",
         "//sdk/src/trace",
         "@com_google_googletest//:gtest_main",
     ],

sdk/test/trace/BUILD

@@ -19,6 +19,22 @@ cc_test(
     ],
 )
 
+cc_test(
+    name = "tracer_provider_set_test",
+    srcs = [
+        "tracer_provider_set_test.cc",
+    ],
+    tags = [
+        "test",
+        "trace",
+    ],
+    deps = [
+        "//sdk/src/resource",
+        "//sdk/src/trace",
+        "@com_google_googletest//:gtest_main",
+    ],
+)
+
 cc_test(
     name = "span_data_test",
     srcs = [