This is an automated email from the ASF dual-hosted git repository.

maskit pushed a commit to branch quic-latest
in repository https://gitbox.apache.org/repos/asf/trafficserver.git


The following commit(s) were added to refs/heads/quic-latest by this push:
     new e0cc477  Fix tests
e0cc477 is described below

commit e0cc477c91851cfc095ebf84f2ee8aff62f7693f
Author: Masakazu Kitajo <[email protected]>
AuthorDate: Wed Mar 6 15:55:08 2019 +0900

    Fix tests
---
 iocore/net/quic/test/test_QUICPacketFactory.cc     | 1 +
 iocore/net/quic/test/test_QUICVersionNegotiator.cc | 6 +++---
 2 files changed, 4 insertions(+), 3 deletions(-)

diff --git a/iocore/net/quic/test/test_QUICPacketFactory.cc 
b/iocore/net/quic/test/test_QUICPacketFactory.cc
index cd3bf33..5b3936c 100644
--- a/iocore/net/quic/test/test_QUICPacketFactory.cc
+++ b/iocore/net/quic/test/test_QUICPacketFactory.cc
@@ -53,6 +53,7 @@ 
TEST_CASE("QUICPacketFactory_Create_VersionNegotiationPacket", "[quic]")
     0x11, 0x12, 0x13, 0x14, 0x15, 0x16, 0x17, 0x18, // Destination Connection 
ID
     0x01, 0x02, 0x03, 0x04, 0x05, 0x06, 0x07, 0x08, // Source Connection ID
     0xff, 0x00, 0x00, 0x12,                         // Supported Version
+    0x1a, 0x2a, 0x3a, 0x4a,                         // Excercise Version
   };
   uint8_t buf[1024] = {0};
   size_t buf_len;
diff --git a/iocore/net/quic/test/test_QUICVersionNegotiator.cc 
b/iocore/net/quic/test/test_QUICVersionNegotiator.cc
index 2d3dbae..24b607c 100644
--- a/iocore/net/quic/test/test_QUICVersionNegotiator.cc
+++ b/iocore/net/quic/test/test_QUICVersionNegotiator.cc
@@ -68,7 +68,7 @@ TEST_CASE("QUICVersionNegotiator - Server Side", "[quic]")
     CHECK(vn.status() == QUICVersionNegotiationStatus::NEGOTIATED);
 
     // Validate version
-    QUICTransportParametersInClientHello tp(QUIC_EXERCISE_VERSIONS);
+    QUICTransportParametersInClientHello tp(QUIC_EXERCISE_VERSION);
     vn.validate(&tp);
     CHECK(vn.status() == QUICVersionNegotiationStatus::VALIDATED);
     CHECK(vn.negotiated_version() == QUIC_SUPPORTED_VERSIONS[0]);
@@ -80,7 +80,7 @@ TEST_CASE("QUICVersionNegotiator - Server Side", "[quic]")
     CHECK(vn.status() == QUICVersionNegotiationStatus::NOT_NEGOTIATED);
 
     // Negotiate version
-    packet_factory.set_version(QUIC_EXERCISE_VERSIONS);
+    packet_factory.set_version(QUIC_EXERCISE_VERSION);
     QUICPacketUPtr initial_packet =
       packet_factory.create_initial_packet({}, {}, 0, 
std::move(dummy_payload), dummy_payload_len, true, false, true, dummy_frames);
     vn.negotiate(initial_packet.get());
@@ -125,7 +125,7 @@ TEST_CASE("QUICVersionNegotiator - Client Side", "[quic]")
     CHECK(vn.status() == QUICVersionNegotiationStatus::NOT_NEGOTIATED);
 
     // Negotiate version
-    packet_factory.set_version(QUIC_EXERCISE_VERSIONS);
+    packet_factory.set_version(QUIC_EXERCISE_VERSION);
     QUICPacketUPtr initial_packet =
       packet_factory.create_initial_packet({}, {}, 0, 
std::move(dummy_payload), dummy_payload_len, true, false, true, dummy_frames);
 

Reply via email to