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

Side by Side Diff: net/quic/quic_server_test.cc

Issue 340433002: Port QuicServer to Chrome network stack (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Address review comments and fix blocked writers using callbacks Created 6 years, 6 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 unified diff | Download patch
OLDNEW
1 // Copyright 2013 The Chromium Authors. All rights reserved. 1 // Copyright 2013 The Chromium Authors. All rights reserved.
2 // Use of this source code is governed by a BSD-style license that can be 2 // Use of this source code is governed by a BSD-style license that can be
3 // found in the LICENSE file. 3 // found in the LICENSE file.
4 4
5 #include "net/tools/quic/quic_server.h" 5 #include "net/quic/quic_server.h"
6 6
7 #include "net/quic/crypto/quic_random.h" 7 #include "net/quic/crypto/quic_random.h"
8 #include "net/quic/quic_utils.h" 8 #include "net/quic/quic_utils.h"
9 #include "net/tools/quic/test_tools/mock_quic_dispatcher.h" 9 #include "net/quic/test_tools/mock_quic_dispatcher.h"
10 #include "net/quic/test_tools/quic_test_utils.h"
10 #include "testing/gtest/include/gtest/gtest.h" 11 #include "testing/gtest/include/gtest/gtest.h"
11 12
12 using ::testing::_; 13 using ::testing::_;
13 14
14 namespace net { 15 namespace net {
15 namespace tools {
16 namespace test { 16 namespace test {
17 17
18 namespace { 18 namespace {
19 19
20 class QuicServerDispatchPacketTest : public ::testing::Test { 20 class QuicChromeServerDispatchPacketTest : public ::testing::Test {
21 public: 21 public:
22 QuicServerDispatchPacketTest() 22 QuicChromeServerDispatchPacketTest()
23 : crypto_config_("blah", QuicRandom::GetInstance()), 23 : crypto_config_("blah", QuicRandom::GetInstance()),
24 dispatcher_(config_, crypto_config_, &eps_) { 24 dispatcher_(config_, crypto_config_, &helper_) {
25 dispatcher_.Initialize(1234); 25 dispatcher_.Initialize(NULL);
26 } 26 }
27 27
28 void DispatchPacket(const QuicEncryptedPacket& packet) { 28 void DispatchPacket(const QuicEncryptedPacket& packet) {
29 IPEndPoint client_addr, server_addr; 29 IPEndPoint client_addr, server_addr;
30 dispatcher_.ProcessPacket(server_addr, client_addr, packet); 30 dispatcher_.ProcessPacket(server_addr, client_addr, packet);
31 } 31 }
32 32
33 protected: 33 protected:
34 QuicConfig config_; 34 QuicConfig config_;
35 QuicCryptoServerConfig crypto_config_; 35 QuicCryptoServerConfig crypto_config_;
36 EpollServer eps_; 36 MockHelper helper_;
37 MockQuicDispatcher dispatcher_; 37 MockQuicDispatcher dispatcher_;
38 }; 38 };
39 39
40 TEST_F(QuicServerDispatchPacketTest, DispatchPacket) { 40 TEST_F(QuicChromeServerDispatchPacketTest, DispatchPacket) {
41 unsigned char valid_packet[] = { 41 unsigned char valid_packet[] = {
42 // public flags (8 byte connection_id) 42 // public flags (8 byte connection_id)
43 0x3C, 43 0x3C,
44 // connection_id 44 // connection_id
45 0x10, 0x32, 0x54, 0x76, 45 0x10, 0x32, 0x54, 0x76,
46 0x98, 0xBA, 0xDC, 0xFE, 46 0x98, 0xBA, 0xDC, 0xFE,
47 // packet sequence number 47 // packet sequence number
48 0xBC, 0x9A, 0x78, 0x56, 48 0xBC, 0x9A, 0x78, 0x56,
49 0x34, 0x12, 49 0x34, 0x12,
50 // private flags 50 // private flags
51 0x00 }; 51 0x00 };
52 QuicEncryptedPacket encrypted_valid_packet(QuicUtils::AsChars(valid_packet), 52 QuicEncryptedPacket encrypted_valid_packet(
53 arraysize(valid_packet), false); 53 QuicUtils::AsChars(valid_packet), arraysize(valid_packet), false);
Ryan Hamilton 2014/06/18 23:59:51 Bah! I like the old formatting better :>
dmz 2014/06/19 17:46:24 Ah, it seems one last 'git cl format' change slipp
54 54
55 EXPECT_CALL(dispatcher_, ProcessPacket(_, _, _)).Times(1); 55 EXPECT_CALL(dispatcher_, ProcessPacket(_, _, _)).Times(1);
56 DispatchPacket(encrypted_valid_packet); 56 DispatchPacket(encrypted_valid_packet);
57 } 57 }
58 58
59 } // namespace 59 } // namespace
60 } // namespace test 60 } // namespace test
61 } // namespace tools
62 } // namespace net 61 } // namespace net
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698