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

Side by Side Diff: net/tools/quic/quic_reliable_client_stream_test.cc

Issue 25043005: Add better help information to quic_client and quic_server. Also, (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: fix comment Created 7 years, 2 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 | Annotate | Revision Log
« no previous file with comments | « net/tools/quic/quic_client_bin.cc ('k') | net/tools/quic/quic_server_bin.cc » ('j') | no next file with comments »
Toggle Intra-line Diffs ('i') | Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
OLDNEW
1 // Copyright (c) 2012 The Chromium Authors. All rights reserved. 1 // Copyright (c) 2012 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_reliable_client_stream.h" 5 #include "net/tools/quic/quic_reliable_client_stream.h"
6 6
7 #include "base/strings/string_number_conversions.h" 7 #include "base/strings/string_number_conversions.h"
8 #include "net/quic/quic_utils.h" 8 #include "net/quic/quic_utils.h"
9 #include "net/quic/test_tools/quic_test_utils.h" 9 #include "net/quic/test_tools/quic_test_utils.h"
10 #include "net/tools/epoll_server/epoll_server.h" 10 #include "net/tools/epoll_server/epoll_server.h"
(...skipping 48 matching lines...) Expand 10 before | Expand all | Expand 10 after
59 59
60 TEST_F(QuicClientStreamTest, TestFramingOnePacket) { 60 TEST_F(QuicClientStreamTest, TestFramingOnePacket) {
61 string message = headers_string_ + body_; 61 string message = headers_string_ + body_;
62 62
63 EXPECT_EQ(message.size(), stream_->ProcessData( 63 EXPECT_EQ(message.size(), stream_->ProcessData(
64 message.c_str(), message.size())); 64 message.c_str(), message.size()));
65 EXPECT_EQ(200u, stream_->headers().parsed_response_code()); 65 EXPECT_EQ(200u, stream_->headers().parsed_response_code());
66 EXPECT_EQ(body_, stream_->data()); 66 EXPECT_EQ(body_, stream_->data());
67 } 67 }
68 68
69 TEST_F(QuicClientStreamTest, TestFramingExtraData) { 69 TEST_F(QuicClientStreamTest, DISABLED_TestFramingExtraData) {
70 string large_body = "hello world!!!!!!"; 70 string large_body = "hello world!!!!!!";
71 71
72 EXPECT_EQ(headers_string_.size(), stream_->ProcessData( 72 EXPECT_EQ(headers_string_.size(), stream_->ProcessData(
73 headers_string_.c_str(), headers_string_.size())); 73 headers_string_.c_str(), headers_string_.size()));
74 // The headers should parse successfully. 74 // The headers should parse successfully.
75 EXPECT_EQ(QUIC_STREAM_NO_ERROR, stream_->stream_error()); 75 EXPECT_EQ(QUIC_STREAM_NO_ERROR, stream_->stream_error());
76 EXPECT_EQ(200u, stream_->headers().parsed_response_code()); 76 EXPECT_EQ(200u, stream_->headers().parsed_response_code());
77 77
78 stream_->ProcessData(large_body.c_str(), large_body.size()); 78 stream_->ProcessData(large_body.c_str(), large_body.size());
79 stream_->TerminateFromPeer(true); 79 stream_->TerminateFromPeer(true);
80 80
81 EXPECT_NE(QUIC_STREAM_NO_ERROR, stream_->stream_error()); 81 EXPECT_NE(QUIC_STREAM_NO_ERROR, stream_->stream_error());
82 } 82 }
83 83
84 TEST_F(QuicClientStreamTest, TestNoBidirectionalStreaming) { 84 TEST_F(QuicClientStreamTest, TestNoBidirectionalStreaming) {
85 QuicStreamFrame frame(3, false, 3, "asd"); 85 QuicStreamFrame frame(3, false, 3, "asd");
86 86
87 EXPECT_FALSE(stream_->write_side_closed()); 87 EXPECT_FALSE(stream_->write_side_closed());
88 EXPECT_TRUE(stream_->OnStreamFrame(frame)); 88 EXPECT_TRUE(stream_->OnStreamFrame(frame));
89 EXPECT_TRUE(stream_->write_side_closed()); 89 EXPECT_TRUE(stream_->write_side_closed());
90 } 90 }
91 91
92 } // namespace 92 } // namespace
93 } // namespace test 93 } // namespace test
94 } // namespace tools 94 } // namespace tools
95 } // namespace net 95 } // namespace net
96
OLDNEW
« no previous file with comments | « net/tools/quic/quic_client_bin.cc ('k') | net/tools/quic/quic_server_bin.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698