From: Rishesh Agarwal Date: Mon, 15 Dec 2025 07:01:40 +0000 (-0800) Subject: Adding layering_check and parse_headers in test files (#41226) X-Git-Url: https://git.feebdaed.xyz/?a=commitdiff_plain;h=09218ae889c7d152b0de36ab5c1665b36629f519;p=0xmirror%2Fgrpc.git Adding layering_check and parse_headers in test files (#41226) Closes #41226 COPYBARA_INTEGRATE_REVIEW=https://github.com/grpc/grpc/pull/41226 from rishesh007:layering_test 02e735767f22e934850d5e150c5e3d577bdfbecb PiperOrigin-RevId: 844606259 --- diff --git a/CMakeLists.txt b/CMakeLists.txt index 8622033f36..ec8ea5d10c 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -11439,6 +11439,9 @@ if(_gRPC_PLATFORM_LINUX OR _gRPC_PLATFORM_MAC OR _gRPC_PLATFORM_POSIX) target_link_libraries(channelz_tool_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset grpcpp_channelz ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc_test_util @@ -11857,6 +11860,9 @@ target_include_directories(cli_call_test target_link_libraries(cli_call_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc++_test_util ) @@ -18865,6 +18871,9 @@ target_include_directories(grpc_cli target_link_libraries(grpc_cli ${_gRPC_ALLTARGETS_LIBRARIES} + absl::fixed_array + absl::overload + absl::charset grpc++ ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc++_test_config @@ -19611,6 +19620,9 @@ if(_gRPC_PLATFORM_LINUX OR _gRPC_PLATFORM_POSIX) target_link_libraries(grpc_tool_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset grpc++_reflection ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc++_test_config @@ -23527,6 +23539,9 @@ if(_gRPC_PLATFORM_LINUX OR _gRPC_PLATFORM_MAC OR _gRPC_PLATFORM_POSIX) target_link_libraries(latent_see_tool_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset grpc++ ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc_test_util @@ -38018,6 +38033,9 @@ target_include_directories(xds_audit_logger_registry_test target_link_libraries(xds_audit_logger_registry_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset grpc++ ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc_test_util @@ -40697,6 +40715,9 @@ target_include_directories(xds_common_types_test target_link_libraries(xds_common_types_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset grpc++ ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc_test_util @@ -45429,6 +45450,9 @@ target_include_directories(xds_http_filters_test target_link_libraries(xds_http_filters_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset grpc++ ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc_test_util @@ -45815,6 +45839,9 @@ target_include_directories(xds_lb_policy_registry_test target_link_libraries(xds_lb_policy_registry_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset grpc++ ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc_test_util @@ -46317,6 +46344,9 @@ target_include_directories(xds_listener_resource_type_test target_link_libraries(xds_listener_resource_type_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset grpc++ ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc_test_util @@ -50256,6 +50286,9 @@ target_include_directories(xds_route_config_resource_type_test target_link_libraries(xds_route_config_resource_type_test ${_gRPC_ALLTARGETS_LIBRARIES} gtest + absl::fixed_array + absl::overload + absl::charset grpc++ ${_gRPC_PROTOBUF_PROTOC_LIBRARIES} grpc_test_util diff --git a/build_autogenerated.yaml b/build_autogenerated.yaml index 17c6936429..f9aad6869a 100644 --- a/build_autogenerated.yaml +++ b/build_autogenerated.yaml @@ -8734,6 +8734,7 @@ targets: - test/cpp/util/proto_file_parser.h - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h + - third_party/utf8_range/utf8_validity.h src: - src/core/ext/transport/chaotic_good/chaotic_good_frame.proto - src/proto/grpc/channelz/v2/latent_see.proto @@ -8789,6 +8790,9 @@ targets: - third_party/googletest/googlemock/src/gmock_main.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpcpp_channelz - protoc - grpc_test_util @@ -9082,6 +9086,7 @@ targets: - test/cpp/util/proto_file_parser.h - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h + - third_party/utf8_range/utf8_validity.h src: - src/proto/grpc/reflection/v1alpha/reflection.proto - src/proto/grpc/testing/echo.proto @@ -9101,6 +9106,9 @@ targets: - test/cpp/util/service_describer.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - protoc - grpc++_test_util - name: client_auth_filter_test @@ -14364,6 +14372,7 @@ targets: - test/cpp/util/proto_file_parser.h - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h + - third_party/utf8_range/utf8_validity.h src: - src/proto/grpc/reflection/v1alpha/reflection.proto - test/cpp/util/cli_call.cc @@ -14374,6 +14383,9 @@ targets: - test/cpp/util/proto_reflection_descriptor_database.cc - test/cpp/util/service_describer.cc deps: + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpc++ - protoc - grpc++_test_config @@ -14555,6 +14567,7 @@ targets: - test/cpp/util/proto_file_parser.h - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h + - third_party/utf8_range/utf8_validity.h src: - src/proto/grpc/testing/echo.proto - src/proto/grpc/testing/echo_messages.proto @@ -14573,6 +14586,9 @@ targets: - test/cpp/util/service_describer.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpc++_reflection - protoc - grpc++_test_config @@ -18220,6 +18236,7 @@ targets: - test/cpp/util/proto_file_parser.h - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h + - third_party/utf8_range/utf8_validity.h src: - src/core/ext/transport/chaotic_good/chaotic_good_frame.proto - src/proto/grpc/channelz/v2/channelz.proto @@ -18272,6 +18289,9 @@ targets: - third_party/googletest/googlemock/src/gmock_main.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpc++ - protoc - grpc_test_util @@ -27816,6 +27836,7 @@ targets: - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h - third_party/protoc-gen-validate/validate/validate.h + - third_party/utf8_range/utf8_validity.h src: - src/proto/grpc/reflection/v1alpha/reflection.proto - third_party/envoy-api/envoy/annotations/deprecation.proto @@ -27911,6 +27932,9 @@ targets: - test/cpp/util/service_describer.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpc++ - protoc - grpc_test_util @@ -28705,6 +28729,7 @@ targets: - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h - third_party/protoc-gen-validate/validate/validate.h + - third_party/utf8_range/utf8_validity.h src: - src/proto/grpc/reflection/v1alpha/reflection.proto - third_party/envoy-api/envoy/annotations/deprecation.proto @@ -28866,6 +28891,9 @@ targets: - test/cpp/util/service_describer.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpc++ - protoc - grpc_test_util @@ -30064,6 +30092,7 @@ targets: - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h - third_party/protoc-gen-validate/validate/validate.h + - third_party/utf8_range/utf8_validity.h src: - src/proto/grpc/reflection/v1alpha/reflection.proto - third_party/envoy-api/envoy/annotations/deprecation.proto @@ -30179,6 +30208,9 @@ targets: - test/cpp/util/service_describer.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpc++ - protoc - grpc_test_util @@ -30196,6 +30228,7 @@ targets: - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h - third_party/protoc-gen-validate/validate/validate.h + - third_party/utf8_range/utf8_validity.h src: - src/proto/grpc/reflection/v1alpha/reflection.proto - third_party/envoy-api/envoy/annotations/deprecation.proto @@ -30289,6 +30322,9 @@ targets: - test/cpp/util/service_describer.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpc++ - protoc - grpc_test_util @@ -30305,6 +30341,7 @@ targets: - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h - third_party/protoc-gen-validate/validate/validate.h + - third_party/utf8_range/utf8_validity.h src: - src/proto/grpc/reflection/v1alpha/reflection.proto - third_party/envoy-api/envoy/annotations/deprecation.proto @@ -30427,6 +30464,9 @@ targets: - test/cpp/util/service_describer.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpc++ - protoc - grpc_test_util @@ -31520,6 +31560,7 @@ targets: - test/cpp/util/proto_reflection_descriptor_database.h - test/cpp/util/service_describer.h - third_party/protoc-gen-validate/validate/validate.h + - third_party/utf8_range/utf8_validity.h src: - src/proto/grpc/lookup/v1/rls_config.proto - src/proto/grpc/reflection/v1alpha/reflection.proto @@ -31617,6 +31658,9 @@ targets: - test/cpp/util/service_describer.cc deps: - gtest + - absl/container:fixed_array + - absl/functional:overload + - absl/strings:charset - grpc++ - protoc - grpc_test_util diff --git a/test/core/address_utils/BUILD b/test/core/address_utils/BUILD index ff86a13753..72a35488d3 100644 --- a/test/core/address_utils/BUILD +++ b/test/core/address_utils/BUILD @@ -19,6 +19,10 @@ licenses(["notice"]) grpc_package( name = "test/core/address_utils", + features = [ + "layering_check", + "parse_headers", + ], visibility = "private", ) diff --git a/test/core/bad_client/BUILD b/test/core/bad_client/BUILD index 26b0529063..16a3075743 100644 --- a/test/core/bad_client/BUILD +++ b/test/core/bad_client/BUILD @@ -15,7 +15,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_package") load(":generate_tests.bzl", "grpc_bad_client_tests") -grpc_package(name = "test/core/bad_client") +grpc_package( + name = "test/core/bad_client", + features = [ + "layering_check", + "parse_headers", + ], +) licenses(["notice"]) diff --git a/test/core/bad_connection/BUILD b/test/core/bad_connection/BUILD index 6b98a3378e..59dae7ccdb 100644 --- a/test/core/bad_connection/BUILD +++ b/test/core/bad_connection/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_binary", "grpc_package") licenses(["notice"]) -grpc_package(name = "test/core/bad_connection") +grpc_package( + name = "test/core/bad_connection", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_binary( name = "close_fd_test", diff --git a/test/core/bad_ssl/BUILD b/test/core/bad_ssl/BUILD index d0bbe83e87..6487a867a8 100644 --- a/test/core/bad_ssl/BUILD +++ b/test/core/bad_ssl/BUILD @@ -15,7 +15,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_package") load(":generate_tests.bzl", "grpc_bad_ssl_tests") -grpc_package(name = "test/core/bad_ssl") +grpc_package( + name = "test/core/bad_ssl", + features = [ + "layering_check", + "parse_headers", + ], +) licenses(["notice"]) diff --git a/test/core/call/BUILD b/test/core/call/BUILD index 4804200010..aac95626c1 100644 --- a/test/core/call/BUILD +++ b/test/core/call/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_library", "grpc_cc_test", "grpc_p load("//test/core/test_util:grpc_fuzzer.bzl", "grpc_fuzz_test") load("//test/cpp/microbenchmarks:grpc_benchmark_config.bzl", "HISTORY", "MONITORING", "grpc_cc_benchmark") -grpc_package(name = "test/core/call") +grpc_package( + name = "test/core/call", + features = [ + "layering_check", + "parse_headers", + ], +) GRPC_CALL_SPINE_BENCHMARK_LABELS = { "BM_ClientToServerStreaming_CallSpineFixture_:CPU(ns)": { diff --git a/test/core/call/yodel/BUILD b/test/core/call/yodel/BUILD index 8b74f876f7..8f38e5d02a 100644 --- a/test/core/call/yodel/BUILD +++ b/test/core/call/yodel/BUILD @@ -20,7 +20,13 @@ load( "grpc_package", ) -grpc_package(name = "test/core/call/yodel") +grpc_package( + name = "test/core/call/yodel", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_library( name = "yodel_test", diff --git a/test/core/channel/BUILD b/test/core/channel/BUILD index b7f24a36f7..6f5e29e32b 100644 --- a/test/core/channel/BUILD +++ b/test/core/channel/BUILD @@ -15,7 +15,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") load("//test/core/test_util:grpc_fuzzer.bzl", "grpc_fuzz_test") -grpc_package(name = "test/core/channel") +grpc_package( + name = "test/core/channel", + features = [ + "layering_check", + "parse_headers", + ], +) licenses(["notice"]) diff --git a/test/core/channelz/BUILD b/test/core/channelz/BUILD index 9caa0eaac4..6f3320ed84 100644 --- a/test/core/channelz/BUILD +++ b/test/core/channelz/BUILD @@ -15,7 +15,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") load("//test/core/test_util:grpc_fuzzer.bzl", "grpc_fuzz_test") -grpc_package(name = "test/core/channelz") +grpc_package( + name = "test/core/channelz", + features = [ + "layering_check", + "parse_headers", + ], +) licenses(["notice"]) diff --git a/test/core/channelz/v2tov1/BUILD b/test/core/channelz/v2tov1/BUILD index e880ba6541..3b16a665e9 100644 --- a/test/core/channelz/v2tov1/BUILD +++ b/test/core/channelz/v2tov1/BUILD @@ -21,7 +21,13 @@ load( "grpc_fuzz_test", ) -grpc_package(name = "test/core/channelz/v2tov1") +grpc_package( + name = "test/core/channelz/v2tov1", + features = [ + "layering_check", + "parse_headers", + ], +) licenses(["notice"]) diff --git a/test/core/channelz/zviz/BUILD b/test/core/channelz/zviz/BUILD index f286ab480d..269040cda7 100644 --- a/test/core/channelz/zviz/BUILD +++ b/test/core/channelz/zviz/BUILD @@ -15,7 +15,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_library", "grpc_cc_test", "grpc_package") load("//test/core/test_util:grpc_fuzzer.bzl", "grpc_fuzz_test") -grpc_package(name = "test/core/channelz/zviz") +grpc_package( + name = "test/core/channelz/zviz", + features = [ + "layering_check", + "parse_headers", + ], +) licenses(["notice"]) diff --git a/test/core/client_channel/BUILD b/test/core/client_channel/BUILD index ff02bdd585..7b7cd8d366 100644 --- a/test/core/client_channel/BUILD +++ b/test/core/client_channel/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") load("//test/core/test_util:grpc_fuzzer.bzl", "grpc_fuzz_test") load("//test/cpp/microbenchmarks:grpc_benchmark_config.bzl", "HISTORY", "grpc_cc_benchmark") -grpc_package(name = "test/core/client_channel") +grpc_package( + name = "test/core/client_channel", + features = [ + "layering_check", + "parse_headers", + ], +) licenses(["notice"]) diff --git a/test/core/client_idle/BUILD b/test/core/client_idle/BUILD index 4bffdc8db0..a3672b53f2 100644 --- a/test/core/client_idle/BUILD +++ b/test/core/client_idle/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") licenses(["notice"]) -grpc_package(name = "test/core/client_idle") +grpc_package( + name = "test/core/client_idle", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_test( name = "idle_filter_state_test", diff --git a/test/cpp/codegen/BUILD b/test/cpp/codegen/BUILD index ef3b7d7c6e..c52e35bf8e 100644 --- a/test/cpp/codegen/BUILD +++ b/test/cpp/codegen/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_binary", "grpc_cc_test", "grpc_pa licenses(["notice"]) -grpc_package(name = "test/cpp/codegen") +grpc_package( + name = "test/cpp/codegen", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_test( name = "codegen_test_full", diff --git a/test/cpp/common/BUILD b/test/cpp/common/BUILD index 355c95963b..2cc7a2dc51 100644 --- a/test/cpp/common/BUILD +++ b/test/cpp/common/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") licenses(["notice"]) -grpc_package(name = "test/cpp/common") +grpc_package( + name = "test/cpp/common", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_test( name = "alarm_test", diff --git a/test/cpp/end2end/BUILD b/test/cpp/end2end/BUILD index ebd4e4bfe8..6413a43602 100644 --- a/test/cpp/end2end/BUILD +++ b/test/cpp/end2end/BUILD @@ -228,6 +228,7 @@ grpc_cc_test( ], external_deps = [ "absl/log:check", + "absl/memory", "gtest", ], tags = [ @@ -240,6 +241,7 @@ grpc_cc_test( "//:gpr", "//:grpc", "//:grpc++", + "//src/core:grpc_check", "//src/proto/grpc/testing:echo_cc_grpc", "//src/proto/grpc/testing:echo_messages_cc_proto", "//src/proto/grpc/testing/duplicate:echo_duplicate_cc_grpc", @@ -447,6 +449,7 @@ grpc_cc_test( "//src/core:slice", "//src/core:wait_for_single_owner", "//src/proto/grpc/channelz:channelz_proto", + "//src/proto/grpc/channelz:channelz_proto_cc_proto", "//src/proto/grpc/testing:echo_cc_grpc", "//src/proto/grpc/testing:echo_messages_cc_proto", "//test/core/event_engine:event_engine_test_utils", @@ -935,7 +938,9 @@ grpc_cc_test( "//:grpc++", "//:grpc++_reflection", "//src/proto/grpc/reflection/v1:reflection_proto", + "//src/proto/grpc/reflection/v1:reflection_proto_cc_proto", "//src/proto/grpc/reflection/v1alpha:reflection_proto", + "//src/proto/grpc/reflection/v1alpha:reflection_proto_cc_proto", "//src/proto/grpc/testing:echo_cc_grpc", "//src/proto/grpc/testing:echo_messages_cc_proto", "//src/proto/grpc/testing/duplicate:echo_duplicate_cc_grpc", @@ -976,6 +981,8 @@ grpc_cc_test( ], external_deps = [ "absl/log:check", + "absl/log", + "absl/memory", "gtest", ], tags = [ @@ -988,6 +995,7 @@ grpc_cc_test( "//:gpr", "//:grpc", "//:grpc++", + "//src/core:grpc_check", "//src/proto/grpc/testing:echo_cc_grpc", "//src/proto/grpc/testing:echo_messages_cc_proto", "//src/proto/grpc/testing/duplicate:echo_duplicate_cc_grpc", @@ -1050,6 +1058,7 @@ grpc_cc_test( name = "server_load_reporting_end2end_test", srcs = ["server_load_reporting_end2end_test.cc"], external_deps = [ + "absl/log", "gtest", ], tags = [ @@ -1060,8 +1069,15 @@ grpc_cc_test( "no_windows", ], deps = [ + "//:config_vars", + "//:gpr", + "//:grpc", + "//:grpc++", "//:grpcpp_server_load_reporting", + "//src/core:client_channel_backup_poller", + "//src/proto/grpc/lb/v1:load_reporter_cc_grpc", "//src/proto/grpc/testing:echo_cc_grpc", + "//test/core/test_util:grpc_test_util", "//test/cpp/util:test_util", ], ) @@ -1202,6 +1218,7 @@ grpc_cc_test( srcs = ["message_allocator_end2end_test.cc"], external_deps = [ "absl/log:check", + "absl/log", "gtest", ], tags = [ @@ -1214,11 +1231,14 @@ grpc_cc_test( "//:gpr", "//:grpc", "//:grpc++", + "//:iomgr", + "//src/core:grpc_check", "//src/proto/grpc/testing:echo_cc_grpc", "//src/proto/grpc/testing:echo_messages_cc_proto", "//src/proto/grpc/testing:simple_messages_cc_proto", "//test/core/test_util:grpc_test_util", "//test/cpp/util:test_util", + "@com_google_protobuf//:protobuf_headers", ], ) diff --git a/test/cpp/ext/csm/BUILD b/test/cpp/ext/csm/BUILD index 7d59acf13f..01f927e4d4 100644 --- a/test/cpp/ext/csm/BUILD +++ b/test/cpp/ext/csm/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/ext/csm", + features = [ + "layering_check", + "parse_headers", + ], visibility = "tests", ) @@ -29,6 +33,7 @@ grpc_cc_test( external_deps = [ "google_cloud_cpp:opentelemetry", "gtest", + "otel/sdk:headers", "otel/sdk/src/metrics", ], tags = [ @@ -39,6 +44,7 @@ grpc_cc_test( ], deps = [ "//:grpc++", + "//src/core:xds_enabled_server", "//src/cpp/ext/csm:csm_observability", "//src/cpp/ext/otel:otel_plugin", "//test/core/test_util:grpc_test_util", @@ -51,8 +57,11 @@ grpc_cc_test( "metadata_exchange_test.cc", ], external_deps = [ + "absl/functional:any_invocable", "google_cloud_cpp:opentelemetry", "gtest", + "otel/api", + "otel/sdk:headers", "otel/sdk/src/metrics", ], tags = [ @@ -62,9 +71,15 @@ grpc_cc_test( "grpc:otel-namespace-calamity", ], deps = [ + "//:call_tracer", + "//:config", + "//:gpr", "//:grpc++", + "//src/core:env", "//src/cpp/ext/csm:csm_observability", + "//src/cpp/ext/otel:otel_plugin", "//test/core/test_util:grpc_test_util", + "//test/cpp/end2end:test_service_impl", "//test/cpp/ext/otel:otel_test_library", ], ) diff --git a/test/cpp/ext/filters/census/BUILD b/test/cpp/ext/filters/census/BUILD index 60e65c6bea..7d52cfd7b4 100644 --- a/test/cpp/ext/filters/census/BUILD +++ b/test/cpp/ext/filters/census/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/ext/filters/census", + features = [ + "layering_check", + "parse_headers", + ], visibility = "tests", ) diff --git a/test/cpp/ext/filters/logging/BUILD b/test/cpp/ext/filters/logging/BUILD index 2ec80578bf..fd9231cce2 100644 --- a/test/cpp/ext/filters/logging/BUILD +++ b/test/cpp/ext/filters/logging/BUILD @@ -18,7 +18,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_library", "grpc_cc_test", "grpc_p licenses(["notice"]) -grpc_package(name = "test/cpp/ext/filters/logging") +grpc_package( + name = "test/cpp/ext/filters/logging", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_library( name = "logging_test_library", diff --git a/test/cpp/ext/gcp/BUILD b/test/cpp/ext/gcp/BUILD index 744f6ee793..cb4739dda5 100644 --- a/test/cpp/ext/gcp/BUILD +++ b/test/cpp/ext/gcp/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") licenses(["notice"]) -grpc_package(name = "test/cpp/ext/gcp") +grpc_package( + name = "test/cpp/ext/gcp", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_test( name = "observability_test", diff --git a/test/cpp/ext/otel/BUILD b/test/cpp/ext/otel/BUILD index 5ef14eb51d..a6181b3662 100644 --- a/test/cpp/ext/otel/BUILD +++ b/test/cpp/ext/otel/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/ext/otel", + features = [ + "layering_check", + "parse_headers", + ], visibility = "tests", ) diff --git a/test/cpp/grpclb/BUILD b/test/cpp/grpclb/BUILD index 4b1a558d6c..a232042556 100644 --- a/test/cpp/grpclb/BUILD +++ b/test/cpp/grpclb/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/grpclb", + features = [ + "layering_check", + "parse_headers", + ], visibility = "public", ) # Allows external users to implement grpclb tests. diff --git a/test/cpp/interop/BUILD b/test/cpp/interop/BUILD index 70723db50f..166dd00c69 100644 --- a/test/cpp/interop/BUILD +++ b/test/cpp/interop/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/interop", + features = [ + "layering_check", + "parse_headers", + ], visibility = "public", ) @@ -364,18 +368,25 @@ grpc_cc_binary( "xds_interop_client.cc", ], external_deps = [ + "absl/algorithm:container", "absl/flags:flag", "absl/log:check", + "absl/log:log", + "absl/strings", "otel/exporters/prometheus:prometheus_exporter", "otel/sdk/src/metrics", + "otel/sdk:headers", ], deps = [ ":rpc_behavior_lb_policy", ":xds_stats_watcher", "//:grpc++", "//:grpc++_reflection", + "//:grpc_base", "//:grpcpp_admin", "//:grpcpp_csm_observability", + "//src/core:env", + "//src/core:grpc_check", "//src/proto/grpc/testing:empty_cc_proto", "//src/proto/grpc/testing:messages_cc_proto", "//src/proto/grpc/testing:test_cc_grpc", @@ -424,13 +435,17 @@ grpc_cc_binary( ], external_deps = [ "absl/flags:flag", + "absl/log:log", "otel/exporters/prometheus:prometheus_exporter", "otel/sdk/src/metrics", + "otel/sdk:headers", ], deps = [ ":xds_interop_server_lib", + "//:grpc", "//:grpc++", "//:grpcpp_csm_observability", + "//:iomgr", "//test/core/test_util:grpc_test_util", "//test/cpp/end2end:test_health_check_service_impl", "//test/cpp/util:test_config", @@ -616,12 +631,16 @@ grpc_cc_binary( ], external_deps = [ "absl/flags:flag", + "absl/log:log", ], tags = ["nobuilder"], deps = [ ":interop_server_lib", + ":server_helper_lib", "//:grpc++", "//:grpcpp_gcp_observability", + "//test/core/test_util:grpc_test_util", + "//test/cpp/util:test_config", "//test/cpp/util:test_util", ], ) @@ -633,8 +652,10 @@ grpc_cc_library( ], external_deps = [ "absl/flags:flag", + "absl/log:log", "otel/exporters/prometheus:prometheus_exporter", "otel/sdk/src/metrics", + "otel/sdk:headers", ], tags = [ "nobuilder", @@ -643,8 +664,13 @@ grpc_cc_library( ], deps = [ ":client_helper_lib", + "//:gpr", + "//:grpc", + "//:grpc++", "//:grpcpp_gcp_observability", "//src/cpp/ext/otel:otel_plugin", + "//test/core/test_util:grpc_test_util", + "//test/cpp/util:test_config", ], ) @@ -667,6 +693,8 @@ grpc_cc_binary( ], external_deps = [ "absl/flags:flag", + "absl/log:log", + "absl/strings", ], tags = [ "no_windows", @@ -676,6 +704,7 @@ grpc_cc_binary( "//:gpr", "//:grpc", "//:grpc++", + "//:iomgr", "//test/core/test_util:grpc_test_util", "//test/cpp/util:test_config", "//test/cpp/util:test_util", diff --git a/test/cpp/microbenchmarks/BUILD b/test/cpp/microbenchmarks/BUILD index c8d75f5876..78eee5aab1 100644 --- a/test/cpp/microbenchmarks/BUILD +++ b/test/cpp/microbenchmarks/BUILD @@ -17,7 +17,13 @@ load("//test/cpp/microbenchmarks:grpc_benchmark_config.bzl", "HISTORY", "grpc_cc licenses(["notice"]) -grpc_package(name = "test/cpp/microbenchmarks") +grpc_package( + name = "test/cpp/microbenchmarks", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_benchmark( name = "noop-benchmark", diff --git a/test/cpp/microbenchmarks/huffman_geometries/BUILD b/test/cpp/microbenchmarks/huffman_geometries/BUILD index a9cf5bc875..6607cd93ef 100644 --- a/test/cpp/microbenchmarks/huffman_geometries/BUILD +++ b/test/cpp/microbenchmarks/huffman_geometries/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/microbenchmarks/huffman_geometries", + features = [ + "layering_check", + "parse_headers", + ], visibility = "public", ) diff --git a/test/cpp/naming/BUILD b/test/cpp/naming/BUILD index 62ad3f1410..6033960a44 100644 --- a/test/cpp/naming/BUILD +++ b/test/cpp/naming/BUILD @@ -17,6 +17,10 @@ load(":generate_resolver_component_tests.bzl", "generate_resolver_component_test package( default_visibility = ["//visibility:public"], + features = [ + "layering_check", + "parse_headers", + ], ) licenses(["notice"]) @@ -67,6 +71,7 @@ grpc_cc_test( "//src/core:default_event_engine", "//src/core:env", "//src/core:experiments", + "//src/core:grpc_check", "//src/core:grpc_sockaddr", "//src/core:notification", "//src/core:pollset_set", diff --git a/test/cpp/naming/generate_resolver_component_tests.bzl b/test/cpp/naming/generate_resolver_component_tests.bzl index 2f0940af85..6c4e8abd95 100755 --- a/test/cpp/naming/generate_resolver_component_tests.bzl +++ b/test/cpp/naming/generate_resolver_component_tests.bzl @@ -35,6 +35,7 @@ def generate_resolver_component_tests(): "address_sorting_test.cc", ], external_deps = [ + "absl/log", "absl/log:check", "gtest", ], @@ -48,6 +49,16 @@ def generate_resolver_component_tests(): "//:gpr", "//test/cpp/util:test_config", "//third_party/address_sorting", + "//:config_vars", + "//:endpoint_addresses", + "//:exec_ctx", + "//:grpc_client_channel", + "//:grpc_resolver", + "//:grpc_resolver_dns_ares", + "//:iomgr", + "//:sockaddr_utils", + "//src/core:channel_args", + "//src/core:grpc_check", ], tags = [ "no_windows", @@ -64,7 +75,12 @@ def generate_resolver_component_tests(): "resolver_component_test.cc", ], external_deps = [ + "absl/flags:flag", "absl/log:check", + "absl/log:log", + "absl/memory", + "absl/strings", + "absl/strings:str_format", "gtest", ], deps = [ @@ -81,6 +97,22 @@ def generate_resolver_component_tests(): "//:gpr", "//src/core:ares_resolver", "//test/cpp/util:test_config", + "//:config", + "//:endpoint_addresses", + "//:grpc_client_channel", + "//:grpc_grpclb_balancer_addresses", + "//:grpc_resolver", + "//:grpc_resolver_dns_ares", + "//:iomgr", + "//:orphanable", + "//:parse_address", + "//:sockaddr_utils", + "//:work_serializer", + "//src/core:channel_args", + "//src/core:default_event_engine", + "//src/core:experiments", + "//src/core:grpc_check", + "//src/core:grpc_sockaddr", ], tags = ["no_windows", "grpc:broken-internally"], ) @@ -108,6 +140,7 @@ def generate_resolver_component_tests(): "//src/core:env", "//test/cpp/util:test_config", "//test/cpp/util/windows:manifest_file", + "//src/core:grpc_check", ], data = [ ":resolver_component_tests_runner", diff --git a/test/cpp/naming/utils/BUILD b/test/cpp/naming/utils/BUILD index a751e9fea3..6f7d3b3241 100644 --- a/test/cpp/naming/utils/BUILD +++ b/test/cpp/naming/utils/BUILD @@ -17,6 +17,10 @@ load("//bazel:grpc_build_system.bzl", "grpc_py_binary") package( default_visibility = ["//visibility:public"], + features = [ + "layering_check", + "parse_headers", + ], ) licenses(["notice"]) diff --git a/test/cpp/performance/BUILD b/test/cpp/performance/BUILD index 471bb4d421..42ba1cc451 100644 --- a/test/cpp/performance/BUILD +++ b/test/cpp/performance/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") licenses(["notice"]) -grpc_package(name = "test/cpp/performance") +grpc_package( + name = "test/cpp/performance", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_test( name = "writes_per_rpc_test", diff --git a/test/cpp/qps/BUILD b/test/cpp/qps/BUILD index 307e6a0cb8..0c9ae8d4a6 100644 --- a/test/cpp/qps/BUILD +++ b/test/cpp/qps/BUILD @@ -18,7 +18,13 @@ load("//test/cpp/qps:qps_benchmark_script.bzl", "json_run_localhost_batch", "qps licenses(["notice"]) -grpc_package(name = "test/cpp/qps") +grpc_package( + name = "test/cpp/qps", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_library( name = "parse_json", diff --git a/test/cpp/security/BUILD b/test/cpp/security/BUILD index 91eb4bda87..dbd72a5063 100644 --- a/test/cpp/security/BUILD +++ b/test/cpp/security/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") licenses(["notice"]) -grpc_package(name = "test/cpp/security") +grpc_package( + name = "test/cpp/security", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_test( name = "tls_certificate_verifier_test", diff --git a/test/cpp/server/BUILD b/test/cpp/server/BUILD index 5b15119337..ec62f89290 100644 --- a/test/cpp/server/BUILD +++ b/test/cpp/server/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") licenses(["notice"]) -grpc_package(name = "test/cpp/server") +grpc_package( + name = "test/cpp/server", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_test( name = "server_builder_test", diff --git a/test/cpp/server/load_reporter/BUILD b/test/cpp/server/load_reporter/BUILD index ccccfdda03..bee05b0dbf 100644 --- a/test/cpp/server/load_reporter/BUILD +++ b/test/cpp/server/load_reporter/BUILD @@ -16,7 +16,13 @@ load("//bazel:grpc_build_system.bzl", "grpc_cc_test", "grpc_package") licenses(["notice"]) -grpc_package(name = "test/cpp/server/load_reporter") +grpc_package( + name = "test/cpp/server/load_reporter", + features = [ + "layering_check", + "parse_headers", + ], +) grpc_cc_test( name = "lb_load_data_store_test", diff --git a/test/cpp/sleuth/BUILD b/test/cpp/sleuth/BUILD index 5863202e23..6c5dc51ffc 100644 --- a/test/cpp/sleuth/BUILD +++ b/test/cpp/sleuth/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/sleuth", + features = [ + "layering_check", + "parse_headers", + ], visibility = "public", ) diff --git a/test/cpp/test/BUILD b/test/cpp/test/BUILD index 6afbb8d8ec..e5dd1786b8 100644 --- a/test/cpp/test/BUILD +++ b/test/cpp/test/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/test", + features = [ + "layering_check", + "parse_headers", + ], visibility = "public", ) diff --git a/test/cpp/thread_manager/BUILD b/test/cpp/thread_manager/BUILD index 2d77a599a8..2f1ac6672d 100644 --- a/test/cpp/thread_manager/BUILD +++ b/test/cpp/thread_manager/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/thread_manager", + features = [ + "layering_check", + "parse_headers", + ], visibility = "public", ) diff --git a/test/cpp/util/BUILD b/test/cpp/util/BUILD index 07ffc54aa0..54fdc6a37e 100644 --- a/test/cpp/util/BUILD +++ b/test/cpp/util/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/util", + features = [ + "layering_check", + "parse_headers", + ], visibility = "public", ) @@ -124,6 +128,7 @@ grpc_cc_library( "//src/core:json_reader", "//src/core:json_writer", "//src/proto/grpc/channelz:channelz_proto", + "//src/proto/grpc/channelz:channelz_proto_cc_proto", ], ) @@ -133,10 +138,13 @@ grpc_cc_library( hdrs = GRPCXX_TESTUTIL_HDRS, external_deps = [ "protobuf", + "absl/memory", ], deps = [ + "//:grpc++_config_proto", "//:grpc++_unsecure", "//:subprocess", + "//src/core:grpc_fake_credentials", "//test/core/test_util:grpc_test_util_base", "//test/core/test_util:grpc_test_util_unsecure", ], @@ -176,6 +184,7 @@ grpc_cc_library( "//src/core:grpc_check", "//src/core:load_file", "//src/proto/grpc/reflection/v1alpha:reflection_proto", + "@com_google_protobuf//src/google/protobuf/compiler:importer", ], ) @@ -410,15 +419,23 @@ grpc_cc_binary( "absl/log:check", "absl/log:log", "absl/strings", + "absl/strings:str_format", + "protobuf", ], tags = [ "no_windows", # unistd.h ], deps = [ "//:gpr", + "//:grpc", "//:grpc++", "//:grpcpp_channelz", + "//src/core:grpc_check", + "//src/core:json", + "//src/core:json_writer", "//src/proto/grpc/channelz:channelz_proto", + "//src/proto/grpc/channelz:channelz_proto_cc_proto", + "//test/core/test_util:grpc_test_util", "//test/cpp/util:test_config", "//test/cpp/util:test_util", ], @@ -435,6 +452,7 @@ grpc_cc_test( external_deps = [ "absl/log:check", "absl/log:log", + "absl/strings", "gtest", ], flaky = True, @@ -452,6 +470,8 @@ grpc_cc_test( "//:grpc", "//:grpc++", "//:grpcpp_channelz", + "//src/core:env", + "//src/core:grpc_check", "//src/proto/grpc/channelz:channelz_proto", "//src/proto/grpc/testing:test_cc_grpc", "//test/core/test_util:grpc_test_util", diff --git a/test/cpp/util/windows/BUILD b/test/cpp/util/windows/BUILD index 7bf6080b21..d86f9e23dc 100644 --- a/test/cpp/util/windows/BUILD +++ b/test/cpp/util/windows/BUILD @@ -18,6 +18,10 @@ licenses(["notice"]) grpc_package( name = "test/cpp/util/windows", + features = [ + "layering_check", + "parse_headers", + ], visibility = "tests", )