From 754c415d73b9e57a37c02a3e596866ba13cf4b8b Mon Sep 17 00:00:00 2001 From: "Adi (Suissa) Peleg" Date: Mon, 12 Aug 2024 13:24:21 -0400 Subject: [PATCH] test: update a broken test due to PRs race (#35671) Commit Message: test: update a broken test due to PRs race Additional Description: PR #35568 used an old `setup()` method call which was modified by PR #35209 that was merged just before it, causing a build failure. This PR update the test and fixes the build failure. Risk Level: low Testing: N/A Docs Changes: N/A Release Notes: N/A Platform Specific Features: N/A Signed-off-by: Adi Suissa-Peleg Signed-off-by: asingh-g --- test/common/http/conn_manager_impl_test_2.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/test/common/http/conn_manager_impl_test_2.cc b/test/common/http/conn_manager_impl_test_2.cc index 526a874ee330..8a37e53703ca 100644 --- a/test/common/http/conn_manager_impl_test_2.cc +++ b/test/common/http/conn_manager_impl_test_2.cc @@ -490,7 +490,7 @@ TEST_F(HttpConnectionManagerImplTest, DrainConnectionUponCompletionVsOnDrainTime Event::MockTimer* connection_duration_timer = setUpTimer(); EXPECT_CALL(*connection_duration_timer, enableTimer(_, _)); // Set up connection. - setup(false, ""); + setup(); // Create a filter so we can encode responses. MockStreamDecoderFilter* filter = new NiceMock();