generic proxy: fix test in debug mode (#30632)

Signed-off-by: wbpcode <wbphub@live.com>
envoyproxy · Oct 31, 2023 · 9fb9844 · 9fb9844
1 parent 03fdc59
commit 9fb9844
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 2 deletions.
diff --git a/contrib/generic_proxy/filters/network/source/BUILD b/contrib/generic_proxy/filters/network/source/BUILD
@@ -171,4 +171,6 @@ envoy_cc_library(
         ":file_access_log_lib",
         "//contrib/generic_proxy/filters/network/source/interface:stream_interface",
     ],
+    # Ensure this factory in the source is always linked in.
+    alwayslink = 1,
 )
diff --git a/contrib/generic_proxy/filters/network/test/proxy_test.cc b/contrib/generic_proxy/filters/network/test/proxy_test.cc
@@ -764,8 +764,6 @@ TEST_F(FilterTest, NewStreamAndReplyNormallyWithMultipleFrames) {
   EXPECT_EQ(1, filter_->activeStreamsForTest().size());
   EXPECT_EQ(0, filter_->frameHandlersForTest().size());
 
-  std::cout << "OK decoding" << std::endl;
-
   auto active_stream = filter_->activeStreamsForTest().begin()->get();
 
   EXPECT_CALL(
@@ -913,6 +911,8 @@ TEST_F(FilterTest, TestStats) {
   auto active_stream = filter_->activeStreamsForTest().begin()->get();
   Buffer::OwnedImpl buffer;
   buffer.add("123");
+  // Mock response.
+  active_stream->onResponseStart(std::make_unique<FakeStreamCodecFactory::FakeResponse>());
   active_stream->onEncodingSuccess(buffer, true);
   EXPECT_EQ(1, filter_config_->stats().response_.value());
   EXPECT_EQ(0, filter_config_->stats().request_active_.value());

diff --git a/source/common/upstream/BUILD b/source/common/upstream/BUILD
@@ -619,4 +619,6 @@ envoy_cc_library(
         "//source/common/network:resolver_lib",
         "@envoy_api//envoy/config/upstream/local_address_selector/v3:pkg_cc_proto",
     ],
+    # Ensure this factory in the source is always linked in.
+    alwayslink = 1,
 )