Merge pull request #1977 from ctiller/create-pull-request/patch-2073768

Automated fix for refs/heads/transport-refs-6
pull/36635/head
Craig Tiller 11 months ago committed by GitHub
commit 909f90dcd7
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
  1. 12
      CMakeLists.txt
  2. 20
      build_autogenerated.yaml
  3. 6
      test/core/call/yodel/yodel_test.h
  4. 89
      test/core/transport/test_suite/BUILD
  5. 1
      test/core/transport/test_suite/grpc_transport_test.bzl
  6. 10
      test/core/transport/test_suite/transport_test.h
  7. 40
      tools/distrib/fix_build_deps.py

12
CMakeLists.txt generated

@ -17521,15 +17521,15 @@ if(_gRPC_PLATFORM_LINUX OR _gRPC_PLATFORM_POSIX)
${_gRPC_PROTO_GENS_DIR}/test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.grpc.pb.cc
${_gRPC_PROTO_GENS_DIR}/test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.pb.h
${_gRPC_PROTO_GENS_DIR}/test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.grpc.pb.h
test/core/call/yodel/test_main.cc
test/core/call/yodel/yodel_test.cc
test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.cc
test/core/transport/test_suite/call_content.cc
test/core/transport/test_suite/call_shapes.cc
test/core/transport/test_suite/fixture.cc
test/core/transport/test_suite/inproc_fixture.cc
test/core/transport/test_suite/no_op.cc
test/core/transport/test_suite/stress.cc
test/core/transport/test_suite/test.cc
test/core/transport/test_suite/test_main.cc
test/core/transport/test_suite/transport_test.cc
)
if(WIN32 AND MSVC)
if(BUILD_SHARED_LIBS)
@ -30294,15 +30294,15 @@ if(_gRPC_PLATFORM_LINUX OR _gRPC_PLATFORM_POSIX)
src/core/ext/transport/chaotic_good/frame_header.cc
src/core/ext/transport/chaotic_good/server_transport.cc
src/core/lib/transport/promise_endpoint.cc
test/core/call/yodel/test_main.cc
test/core/call/yodel/yodel_test.cc
test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.cc
test/core/transport/test_suite/call_content.cc
test/core/transport/test_suite/call_shapes.cc
test/core/transport/test_suite/chaotic_good_fixture.cc
test/core/transport/test_suite/fixture.cc
test/core/transport/test_suite/no_op.cc
test/core/transport/test_suite/stress.cc
test/core/transport/test_suite/test.cc
test/core/transport/test_suite/test_main.cc
test/core/transport/test_suite/transport_test.cc
)
if(WIN32 AND MSVC)
if(BUILD_SHARED_LIBS)

@ -11433,20 +11433,20 @@ targets:
build: test
language: c++
headers:
- test/core/call/yodel/yodel_test.h
- test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.h
- test/core/transport/test_suite/fixture.h
- test/core/transport/test_suite/test.h
- test/core/transport/test_suite/transport_test.h
src:
- test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.proto
- test/core/call/yodel/test_main.cc
- test/core/call/yodel/yodel_test.cc
- test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.cc
- test/core/transport/test_suite/call_content.cc
- test/core/transport/test_suite/call_shapes.cc
- test/core/transport/test_suite/fixture.cc
- test/core/transport/test_suite/inproc_fixture.cc
- test/core/transport/test_suite/no_op.cc
- test/core/transport/test_suite/stress.cc
- test/core/transport/test_suite/test.cc
- test/core/transport/test_suite/test_main.cc
- test/core/transport/test_suite/transport_test.cc
deps:
- gtest
- protobuf
@ -19476,9 +19476,9 @@ targets:
- src/core/lib/promise/switch.h
- src/core/lib/promise/wait_set.h
- src/core/lib/transport/promise_endpoint.h
- test/core/call/yodel/yodel_test.h
- test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.h
- test/core/transport/test_suite/fixture.h
- test/core/transport/test_suite/test.h
- test/core/transport/test_suite/transport_test.h
src:
- test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.proto
- src/core/ext/transport/chaotic_good/chaotic_good_transport.cc
@ -19487,15 +19487,15 @@ targets:
- src/core/ext/transport/chaotic_good/frame_header.cc
- src/core/ext/transport/chaotic_good/server_transport.cc
- src/core/lib/transport/promise_endpoint.cc
- test/core/call/yodel/test_main.cc
- test/core/call/yodel/yodel_test.cc
- test/core/event_engine/fuzzing_event_engine/fuzzing_event_engine.cc
- test/core/transport/test_suite/call_content.cc
- test/core/transport/test_suite/call_shapes.cc
- test/core/transport/test_suite/chaotic_good_fixture.cc
- test/core/transport/test_suite/fixture.cc
- test/core/transport/test_suite/no_op.cc
- test/core/transport/test_suite/stress.cc
- test/core/transport/test_suite/test.cc
- test/core/transport/test_suite/test_main.cc
- test/core/transport/test_suite/transport_test.cc
deps:
- gtest
- protobuf

@ -437,7 +437,7 @@ class YodelTest : public ::testing::Test {
int YodelTest_##name::registered_ = \
(grpc_core::NoDestructSingleton< \
grpc_core::yodel_detail::ParameterizedTestRegistry< \
grpc_core::test_type, parameter_type>>::Get() \
grpc_core::test_type, (parameter_type)>>::Get() \
->RegisterTest(#name, &Create), \
0); \
void YodelTest_##name::TestImpl()
@ -446,8 +446,8 @@ class YodelTest : public ::testing::Test {
int YodelTestParam_##name = \
(grpc_core::NoDestructSingleton< \
grpc_core::yodel_detail::ParameterizedTestRegistry< \
grpc_core::test_type, parameter_type>>::Get() \
grpc_core::test_type, (parameter_type)>>::Get() \
->RegisterParameter(#name, value), \
0)
#endif
#endif // GRPC_TEST_CORE_CALL_YODEL_YODEL_TEST_H

@ -16,32 +16,31 @@ load(
"//bazel:grpc_build_system.bzl",
"grpc_cc_library",
"grpc_package",
"grpc_proto_library",
)
load("grpc_transport_test.bzl", "grpc_transport_test")
grpc_package(name="test/core/transport/test_suite")
grpc_package(name = "test/core/transport/test_suite")
grpc_cc_library(
name="inproc_fixture",
testonly=1,
srcs=["inproc_fixture.cc"],
deps=[
name = "inproc_fixture",
testonly = 1,
srcs = ["inproc_fixture.cc"],
deps = [
"test",
"//src/core:grpc_transport_inproc",
],
alwayslink=1,
alwayslink = 1,
)
grpc_cc_library(
name="chaotic_good_fixture",
testonly=1,
srcs=["chaotic_good_fixture.cc"],
external_deps=[
name = "chaotic_good_fixture",
testonly = 1,
srcs = ["chaotic_good_fixture.cc"],
external_deps = [
"absl/log:check",
"gtest",
],
deps=[
deps = [
"test",
"//src/core:chaotic_good_client_transport",
"//src/core:chaotic_good_server_transport",
@ -50,61 +49,61 @@ grpc_cc_library(
"//src/core:grpc_promise_endpoint",
"//src/core:resource_quota",
],
alwayslink=1,
alwayslink = 1,
)
grpc_cc_library(
name="test",
testonly=1,
srcs=["transport_test.cc"],
hdrs=["transport_test.h"],
external_deps=[
name = "test",
testonly = 1,
srcs = ["transport_test.cc"],
hdrs = ["transport_test.h"],
external_deps = [
"absl/random",
"absl/random:bit_gen_ref",
"gtest",
],
deps=[
deps = [
"//test/core/call/yodel:yodel_test",
],
)
grpc_cc_library(
name="call_content",
testonly=1,
srcs=["call_content.cc"],
external_deps=["gtest"],
deps=["test"],
alwayslink=1,
name = "call_content",
testonly = 1,
srcs = ["call_content.cc"],
external_deps = ["gtest"],
deps = ["test"],
alwayslink = 1,
)
grpc_cc_library(
name="call_shapes",
testonly=1,
srcs=["call_shapes.cc"],
deps=["test"],
alwayslink=1,
name = "call_shapes",
testonly = 1,
srcs = ["call_shapes.cc"],
deps = ["test"],
alwayslink = 1,
)
grpc_cc_library(
name="no_op",
testonly=1,
srcs=["no_op.cc"],
deps=["test"],
alwayslink=1,
name = "no_op",
testonly = 1,
srcs = ["no_op.cc"],
deps = ["test"],
alwayslink = 1,
)
grpc_cc_library(
name="stress",
testonly=1,
srcs=["stress.cc"],
external_deps=["absl/random"],
deps=["test"],
alwayslink=1,
name = "stress",
testonly = 1,
srcs = ["stress.cc"],
external_deps = ["absl/random"],
deps = ["test"],
alwayslink = 1,
)
grpc_transport_test(
name="inproc",
deps=[
name = "inproc",
deps = [
":call_content",
":call_shapes",
":inproc_fixture",
@ -114,8 +113,8 @@ grpc_transport_test(
)
grpc_transport_test(
name="chaotic_good",
deps=[
name = "chaotic_good",
deps = [
":call_content",
":call_shapes",
":chaotic_good_fixture",

@ -16,7 +16,6 @@
Generate one transport test & associated fuzzer
"""
load("//bazel:grpc_build_system.bzl", "grpc_cc_test")
load("//test/core/call/yodel:grpc_yodel_test.bzl", "grpc_yodel_test")
def grpc_transport_test(name, deps):

@ -12,8 +12,8 @@
// See the License for the specific language governing permissions and
// limitations under the License.
#ifndef GRPC_TEST_CORE_TRANSPORT_TEST_SUITE_TEST_H
#define GRPC_TEST_CORE_TRANSPORT_TEST_SUITE_TEST_H
#ifndef GRPC_TEST_CORE_TRANSPORT_TEST_SUITE_TRANSPORT_TEST_H
#define GRPC_TEST_CORE_TRANSPORT_TEST_SUITE_TRANSPORT_TEST_H
#include <memory>
#include <queue>
@ -75,12 +75,12 @@ class TransportTest : public YodelTest {
#define TRANSPORT_TEST(name) YODEL_TEST_P(TransportTest, TransportFixture, name)
#define TRANSPORT_FIXTURE(name) \
static ::grpc_core::ClientAndServerTransportPair name( \
static grpc_core::ClientAndServerTransportPair name( \
std::shared_ptr<grpc_event_engine::experimental::FuzzingEventEngine> \
event_engine); \
YODEL_TEST_PARAM(TransportTest, TransportFixture, name, name); \
static ::grpc_core::ClientAndServerTransportPair name( \
static grpc_core::ClientAndServerTransportPair name( \
std::shared_ptr<grpc_event_engine::experimental::FuzzingEventEngine> \
event_engine)
#endif // GRPC_TEST_CORE_TRANSPORT_TEST_SUITE_TEST_H
#endif // GRPC_TEST_CORE_TRANSPORT_TEST_SUITE_TRANSPORT_TEST_H

@ -116,7 +116,9 @@ EXTERNAL_DEPS = {
"opentelemetry/sdk/resource/semantic_conventions.h": "otel/sdk:headers",
"ares.h": "cares",
"fuzztest/fuzztest.h": ["fuzztest", "fuzztest_main"],
"google/api/monitored_resource.pb.h": ("google/api:monitored_resource_cc_proto"),
"google/api/monitored_resource.pb.h": (
"google/api:monitored_resource_cc_proto"
),
"google/devtools/cloudtrace/v2/tracing.grpc.pb.h": (
"googleapis_trace_grpc_service"
),
@ -135,7 +137,9 @@ EXTERNAL_DEPS = {
"opencensus-trace-stackdriver_exporter"
),
"opencensus/trace/context_util.h": "opencensus-trace-context_util",
"opencensus/trace/propagation/grpc_trace_bin.h": ("opencensus-trace-propagation"),
"opencensus/trace/propagation/grpc_trace_bin.h": (
"opencensus-trace-propagation"
),
"opencensus/tags/context_util.h": "opencensus-tags-context_util",
"opencensus/trace/span_context.h": "opencensus-trace-span_context",
"openssl/base.h": "libssl",
@ -225,7 +229,11 @@ parsing_path = None
# Convert the source or header target to a relative path.
def _get_filename(name, parsing_path):
filename = "%s%s" % (
(parsing_path + "/" if (parsing_path and not name.startswith("//")) else ""),
(
parsing_path + "/"
if (parsing_path and not name.startswith("//"))
else ""
),
name,
)
filename = filename.replace("//:", "")
@ -311,7 +319,9 @@ def buildozer_set_list(name, values, target, via=""):
buildozer("remove %s" % name, target)
return
adjust = via if via else name
buildozer("set %s %s" % (adjust, " ".join('"%s"' % s for s in values)), target)
buildozer(
"set %s %s" % (adjust, " ".join('"%s"' % s for s in values)), target
)
if via:
buildozer("remove %s" % name, target)
buildozer("rename %s %s" % (via, name), target)
@ -477,11 +487,19 @@ class Choices:
def add_one_of(self, choices, trigger):
if not choices:
return
choices = sum([self.apply_substitutions(choice) for choice in choices], [])
choices = sum(
[self.apply_substitutions(choice) for choice in choices], []
)
if args.explain and (args.why is None or args.why in choices):
print("{}: Adding one of {} for {}".format(self.library, choices, trigger))
print(
"{}: Adding one of {} for {}".format(
self.library, choices, trigger
)
)
self.to_add.append(
tuple(make_relative_path(choice, self.library) for choice in choices)
tuple(
make_relative_path(choice, self.library) for choice in choices
)
)
def add(self, choice, trigger):
@ -628,12 +646,16 @@ def make_library(library):
# assume a system include
continue
print("# ERROR: can't categorize header: %s used by %s" % (hdr, library))
print(
"# ERROR: can't categorize header: %s used by %s" % (hdr, library)
)
error = True
deps.remove(library)
deps = sorted(deps.best(lambda x: SCORERS[args.score](x, original_deps[library])))
deps = sorted(
deps.best(lambda x: SCORERS[args.score](x, original_deps[library]))
)
external_deps = sorted(
external_deps.best(
lambda x: SCORERS[args.score](x, original_external_deps[library])

Loading…
Cancel
Save