Skip to content

Commit dec101b

Browse files
committed
only enable synchronizer once
1 parent 89f45aa commit dec101b

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

test/extensions/filters/network/ssh/reverse_tunnel_test.cc

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -10,6 +10,7 @@
1010
#include "envoy/config/endpoint/v3/endpoint.pb.validate.h"
1111
#include "api/extensions/filters/network/ssh/ssh.pb.h"
1212
#include "gtest/gtest.h"
13+
#include <mutex>
1314

1415
#include "source/extensions/filters/network/ssh/wire/messages.h"
1516
#include "test/extensions/filters/network/ssh/ssh_task.h"
@@ -1285,9 +1286,12 @@ TEST_P(DynamicPortForwardTest, DownstreamResetBeforeOpen) {
12851286
ASSERT_TRUE(driver->wait(th));
12861287
}
12871288

1288-
TEST_P(DynamicPortForwardTest, DownstreamResetBeforeInitialize) {
1289-
remote_stream_handler_sync.enable();
1289+
static std::once_flag enable_once;
12901290

1291+
TEST_P(DynamicPortForwardTest, DownstreamResetBeforeInitialize) {
1292+
std::call_once(enable_once, [] {
1293+
remote_stream_handler_sync.enable();
1294+
});
12911295
remote_stream_handler_sync.waitOn("initialize");
12921296
remote_stream_handler_sync.waitOn("downstream_closed");
12931297

0 commit comments

Comments
 (0)