x

ctiller · Jan 18, 2024 · cb45fe1 · cb45fe1
1 parent 167411e
commit cb45fe1
Show file tree

Hide file tree

Showing 3 changed files with 26 additions and 27 deletions.
diff --git a/test/core/transport/test_suite/fixture.h b/test/core/transport/test_suite/fixture.h
@@ -51,27 +51,27 @@ class TransportFixtureRegistry {
 
 }  // namespace grpc_core
 
-#define TRANSPORT_FIXTURE(name)                                                \
-  class TransportFixture_##name : public grpc_core::TransportFixture {         \
-   public:                                                                     \
-    using TransportFixture::TransportFixture;                                  \
-    ClientAndServerTransportPair CreateTransportPair(                          \
-        std::shared_ptr<grpc_event_engine::experimental::FuzzingEventEngine>   \
-            event_engine) override;                                            \
-                                                                               \
-   private:                                                                    \
-    static grpc_core::TransportFixture* Create() {                             \
-      return new TransportFixture_##name();                                    \
-    }                                                                          \
-    static int registered_;                                                    \
-  };                                                                           \
-  int TransportFixture_##name::registered_ =                                   \
-      (grpc_core::TransportFixtureRegistry::Get().RegisterFixture(             \
-           #name, &TransportFixture_##name::Create),                           \
-       0);                                                                     \
-  grpc_core::TransportFixture::ClientAndServerTransportPair                    \
-      TransportFixture_##name::CreateTransportPair(                            \
-          std::shared_ptr<grpc_event_engine::experimental::FuzzingEventEngine> \
-              event_engine)
+#define TRANSPORT_FIXTURE(name)                                              \
+  class TransportFixture_##name : public grpc_core::TransportFixture {       \
+   public:                                                                   \
+    using TransportFixture::TransportFixture;                                \
+    ClientAndServerTransportPair CreateTransportPair(                        \
+        std::shared_ptr<grpc_event_engine::experimental::FuzzingEventEngine> \
+            event_engine) override;                                          \
+                                                                             \
+   private:                                                                  \
+    static grpc_core::TransportFixture* Create() {                           \
+      return new TransportFixture_##name();                                  \
+    }                                                                        \
+    static int registered_;                                                  \
+  };                                                                         \
+  int TransportFixture_##name::registered_ =                                 \
+      (grpc_core::TransportFixtureRegistry::Get().RegisterFixture(           \
+           #name, &TransportFixture_##name::Create),                         \
+       0);                                                                   \
+  grpc_core::TransportFixture::ClientAndServerTransportPair                  \
+      TransportFixture_##name::CreateTransportPair(                          \
+          std::shared_ptr<grpc_event_engine::experimental::                  \
+                              FuzzingEventEngine> event_engine GRPC_UNUSED)
 
 #endif  // GRPC_TEST_CORE_TRANSPORT_TEST_SUITE_FIXTURE_H
diff --git a/test/core/transport/test_suite/test.cc b/test/core/transport/test_suite/test.cc
@@ -231,7 +231,7 @@ Arena* TransportTest::Acceptor::CreateArena() {
 }
 
 absl::StatusOr<CallInitiator> TransportTest::Acceptor::CreateCall(
-    ClientMetadata& client_initial_metadata, Arena* arena) {
+    ClientMetadata&, Arena* arena) {
   auto call = MakeCall(event_engine_, arena);
   handlers_.push(std::move(call.handler));
   return std::move(call.initiator);

diff --git a/test/core/transport/test_suite/test.h b/test/core/transport/test_suite/test.h
@@ -149,9 +149,8 @@ Promise<Empty> WrapPromiseAndNext(std::shared_ptr<ActionState> action_state,
 }
 
 template <typename Arg>
-NextSpawner<Arg> WrapFollowUps(NameAndLocation loc,
-                               ActionStateFactory action_state_factory,
-                               PromiseSpawner spawner) {
+NextSpawner<Arg> WrapFollowUps(NameAndLocation, ActionStateFactory,
+                               PromiseSpawner) {
   return [](Empty) {};
 }
 
@@ -216,7 +215,7 @@ class TransportTest : public ::testing::Test {
                   options;
               return options;
             }(),
-            fuzzing_event_engine::Actions())),
+            actions)),
         fixture_(std::move(fixture)),
         rng_(rng) {}