Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(683)

Unified Diff: net/quic/test_tools/simulator/quic_endpoint_test.cc

Issue 2808273006: Landing Recent QUIC changes until Sun Apr 9 16:12:55 (Closed)
Patch Set: increment enabled_options in e2e test Created 3 years, 8 months ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « net/quic/test_tools/quic_test_utils.cc ('k') | net/tools/quic/chlo_extractor_test.cc » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: net/quic/test_tools/simulator/quic_endpoint_test.cc
diff --git a/net/quic/test_tools/simulator/quic_endpoint_test.cc b/net/quic/test_tools/simulator/quic_endpoint_test.cc
index 9962b55a5f6110b8802531bb6b16ff4cf0927cc3..5a136733138db583d9faf4447e0d2f5a401218a8 100644
--- a/net/quic/test_tools/simulator/quic_endpoint_test.cc
+++ b/net/quic/test_tools/simulator/quic_endpoint_test.cc
@@ -16,6 +16,7 @@
using ::testing::_;
using ::testing::NiceMock;
using ::testing::Return;
+using net::test::GetPeerInMemoryConnectionId;
namespace net {
namespace simulator {
@@ -48,7 +49,8 @@ TEST_F(QuicEndpointTest, OneWayTransmission) {
QuicEndpoint endpoint_a(&simulator_, "Endpoint A", "Endpoint B",
Perspective::IS_CLIENT, 42);
QuicEndpoint endpoint_b(&simulator_, "Endpoint B", "Endpoint A",
- Perspective::IS_SERVER, 42);
+ Perspective::IS_SERVER,
+ GetPeerInMemoryConnectionId(42));
auto link_a = Link(&endpoint_a, switch_.port(1));
auto link_b = Link(&endpoint_b, switch_.port(2));
@@ -83,7 +85,8 @@ TEST_F(QuicEndpointTest, WriteBlocked) {
QuicEndpoint endpoint_a(&simulator_, "Endpoint A", "Endpoint B",
Perspective::IS_CLIENT, 42);
QuicEndpoint endpoint_b(&simulator_, "Endpoint B", "Endpoint A",
- Perspective::IS_SERVER, 42);
+ Perspective::IS_SERVER,
+ GetPeerInMemoryConnectionId(42));
auto link_a = Link(&endpoint_a, switch_.port(1));
auto link_b = Link(&endpoint_b, switch_.port(2));
@@ -123,7 +126,8 @@ TEST_F(QuicEndpointTest, TwoWayTransmission) {
QuicEndpoint endpoint_a(&simulator_, "Endpoint A", "Endpoint B",
Perspective::IS_CLIENT, 42);
QuicEndpoint endpoint_b(&simulator_, "Endpoint B", "Endpoint A",
- Perspective::IS_SERVER, 42);
+ Perspective::IS_SERVER,
+ GetPeerInMemoryConnectionId(42));
auto link_a = Link(&endpoint_a, switch_.port(1));
auto link_b = Link(&endpoint_b, switch_.port(2));
@@ -153,11 +157,14 @@ TEST_F(QuicEndpointTest, Competition) {
auto endpoint_c = QuicMakeUnique<QuicEndpoint>(
&simulator_, "Endpoint C", "Endpoint D (C)", Perspective::IS_CLIENT, 44);
auto endpoint_d_a = QuicMakeUnique<QuicEndpoint>(
- &simulator_, "Endpoint D (A)", "Endpoint A", Perspective::IS_SERVER, 42);
+ &simulator_, "Endpoint D (A)", "Endpoint A", Perspective::IS_SERVER,
+ GetPeerInMemoryConnectionId(42));
auto endpoint_d_b = QuicMakeUnique<QuicEndpoint>(
- &simulator_, "Endpoint D (B)", "Endpoint B", Perspective::IS_SERVER, 43);
+ &simulator_, "Endpoint D (B)", "Endpoint B", Perspective::IS_SERVER,
+ GetPeerInMemoryConnectionId(43));
auto endpoint_d_c = QuicMakeUnique<QuicEndpoint>(
- &simulator_, "Endpoint D (C)", "Endpoint C", Perspective::IS_SERVER, 44);
+ &simulator_, "Endpoint D (C)", "Endpoint C", Perspective::IS_SERVER,
+ GetPeerInMemoryConnectionId(44));
QuicEndpointMultiplexer endpoint_d(
"Endpoint D",
{endpoint_d_a.get(), endpoint_d_b.get(), endpoint_d_c.get()});
« no previous file with comments | « net/quic/test_tools/quic_test_utils.cc ('k') | net/tools/quic/chlo_extractor_test.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698