lib/mvfst
1
0
Fork 0
mirror of https://github.com/facebookincubator/mvfst.git synced 2025-11-09 10:00:57 +03:00
Code Activity

Fix CongestionControlRecreatedWithNewFactory test

Browse Source 
Summary: The new controller happens to be the exact same address

Reviewed By: lnicco

Differential Revision: D26566423

fbshipit-source-id: 88fa04d2cf8b02ef20e7979f0055ee84903d33e9
This commit is contained in:
Yang Chi
2021-02-20 12:05:37 -08:00
committed by Facebook GitHub Bot
parent 55b28f7fcd
commit cb0a4d34a9
1 changed files with 4 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
quic/fizz/client/test/QuicClientTransportTest.cpp
View File

@@ -5199,9 +5199,11 @@ TEST_F(
// Check Cubic CC instance is recreated with new CC factory // Check Cubic CC instance is recreated with new CC factory
auto factory = std::make_shared<TestCCFactory>(); auto factory = std::make_shared<TestCCFactory>();
client->setCongestionControllerFactory(factory); client->setCongestionControllerFactory(factory);
client->setCongestionControl(CongestionControlType::Cubic);
auto newCC = client->getConn().congestionController.get(); auto newCC = client->getConn().congestionController.get();
EXPECT_NE(cc, newCC); EXPECT_EQ(nullptr, newCC);
client->setCongestionControl(CongestionControlType::Cubic);
newCC = client->getConn().congestionController.get();
EXPECT_NE(nullptr, newCC);
EXPECT_EQ(factory->createdControllers, 1); EXPECT_EQ(factory->createdControllers, 1);
} }