Skip to content

Commit f7bc983

Browse files
committed
minor cleanup. introduce otel_cc_test_library
1 parent 32097fa commit f7bc983

File tree

7 files changed

+15
-41
lines changed

7 files changed

+15
-41
lines changed

bazel/otel_cc.bzl

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,10 @@ def otel_cc_library(**kwargs):
4343

4444
rules_cc_library(**kwargs)
4545

46+
def otel_cc_test_library(**kwargs):
47+
kwargs["testonly"] = kwargs.get("testonly", True)
48+
otel_cc_library(**kwargs)
49+
4650
def otel_cc_binary(**kwargs):
4751
rules_cc_binary(**kwargs)
4852

exporters/elasticsearch/BUILD

Lines changed: 1 addition & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# SPDX-License-Identifier: Apache-2.0
33

44
load("//:dll_deps.bzl", "dll_deps")
5-
load("//bazel:otel_cc.bzl", "otel_cc_binary", "otel_cc_import", "otel_cc_library", "otel_cc_shared_library", "otel_cc_test")
5+
load("//bazel:otel_cc.bzl", "otel_cc_library", "otel_cc_test")
66

77
package(default_visibility = ["//visibility:public"])
88

@@ -16,14 +16,6 @@ otel_cc_library(
1616
"include/opentelemetry/exporters/elasticsearch/es_log_record_exporter.h",
1717
"include/opentelemetry/exporters/elasticsearch/es_log_recordable.h",
1818
],
19-
linkopts = select({
20-
"//bazel:windows": [
21-
"-DEFAULTLIB:advapi32.lib",
22-
"-DEFAULTLIB:crypt32.lib",
23-
"-DEFAULTLIB:Normaliz.lib",
24-
],
25-
"//conditions:default": [],
26-
}),
2719
strip_include_prefix = "include",
2820
tags = ["es"],
2921
deps = [

exporters/ostream/BUILD

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,7 @@
22
# SPDX-License-Identifier: Apache-2.0
33

44
load("//:dll_deps.bzl", "dll_deps")
5-
load("//bazel:otel_cc.bzl", "otel_cc_binary", "otel_cc_import", "otel_cc_library", "otel_cc_shared_library", "otel_cc_test")
5+
load("//bazel:otel_cc.bzl", "otel_cc_library", "otel_cc_test_library", "otel_cc_test")
66

77
package(default_visibility = ["//visibility:public"])
88

@@ -89,9 +89,8 @@ otel_cc_library(
8989
],
9090
)
9191

92-
otel_cc_library(
92+
otel_cc_test_library(
9393
name = "ostream_capture",
94-
testonly = 1,
9594
hdrs = [
9695
"test/ostream_capture.h",
9796
],

exporters/prometheus/BUILD

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,7 @@
22
# SPDX-License-Identifier: Apache-2.0
33

44
load("//:dll_deps.bzl", "dll_deps")
5-
load("//bazel:otel_cc.bzl", "otel_cc_binary", "otel_cc_import", "otel_cc_library", "otel_cc_shared_library", "otel_cc_test")
6-
load("//bazel:otel_cc.bzl", "otel_cc_binary", "otel_cc_import", "otel_cc_library", "otel_cc_shared_library", "otel_cc_test")
5+
load("//bazel:otel_cc.bzl", "otel_cc_library", "otel_cc_test_library", "otel_cc_test")
76

87
package(default_visibility = ["//visibility:public"])
98

@@ -68,9 +67,8 @@ otel_cc_library(
6867
],
6968
)
7069

71-
otel_cc_library(
70+
otel_cc_test_library(
7271
name = "prometheus_test_helper",
73-
testonly = 1,
7472
hdrs = [
7573
"test/prometheus_test_helper.h",
7674
],
@@ -163,9 +161,8 @@ otel_cc_library(
163161
],
164162
)
165163

166-
otel_cc_library(
164+
otel_cc_test_library(
167165
name = "prometheus_push_test_helper",
168-
testonly = 1,
169166
hdrs = [
170167
"test/prometheus_push_test_helper.h",
171168
],

functional/otlp/BUILD

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,9 +1,8 @@
11
# Copyright The OpenTelemetry Authors
22
# SPDX-License-Identifier: Apache-2.0
33

4-
load("//bazel:otel_cc_benchmark.bzl", "otel_cc_benchmark")
4+
load("//bazel:otel_cc.bzl", "otel_cc_binary")
55
load("//:dll_deps.bzl", "dll_deps")
6-
load("//bazel:otel_cc.bzl", "otel_cc_binary", "otel_cc_import", "otel_cc_library", "otel_cc_shared_library", "otel_cc_test")
76

87
package(default_visibility = ["//visibility:public"])
98

sdk/test/common/BUILD

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33

44
load("//bazel:otel_cc_benchmark.bzl", "otel_cc_benchmark")
55
load("//:dll_deps.bzl", "dll_deps")
6-
load("//bazel:otel_cc.bzl", "otel_cc_binary", "otel_cc_import", "otel_cc_library", "otel_cc_shared_library", "otel_cc_test")
6+
load("//bazel:otel_cc.bzl", "otel_cc_test_library", "otel_cc_test")
77

88
otel_cc_test(
99
name = "random_test",
@@ -98,9 +98,8 @@ otel_cc_test(
9898
]),
9999
)
100100

101-
otel_cc_library(
101+
otel_cc_test_library(
102102
name = "baseline_circular_buffer",
103-
testonly = 1,
104103
hdrs = [
105104
"baseline_circular_buffer.h",
106105
],

sdk/test/metrics/BUILD

Lines changed: 2 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -2,12 +2,11 @@
22
# SPDX-License-Identifier: Apache-2.0
33

44
load("//:dll_deps.bzl", "dll_deps")
5-
load("//bazel:otel_cc.bzl", "otel_cc_library", "otel_cc_test")
5+
load("//bazel:otel_cc.bzl", "otel_cc_library", "otel_cc_test_library", "otel_cc_test")
66
load("//bazel:otel_cc_benchmark.bzl", "otel_cc_benchmark")
77

8-
otel_cc_library(
8+
otel_cc_test_library(
99
name = "metrics_common_test_utils",
10-
testonly = 1,
1110
srcs = [
1211
"common.cc",
1312
],
@@ -21,21 +20,6 @@ otel_cc_library(
2120
]),
2221
)
2322

24-
otel_cc_test(
25-
name = "meter_config_test",
26-
srcs = [
27-
"meter_config_test.cc",
28-
],
29-
tags = [
30-
"metrics",
31-
"test",
32-
],
33-
deps = dll_deps([
34-
"metrics_common_test_utils",
35-
"@googletest//:gtest_main",
36-
]),
37-
)
38-
3923
otel_cc_test(
4024
name = "all_tests",
4125
srcs = glob(["*_test.cc"]),

0 commit comments

Comments
 (0)