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

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

Issue 1006193003: Remove Balsa from QuicSpdyClientStream. (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: fix comments Created 5 years, 9 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
« no previous file with comments | « net/tools/quic/quic_spdy_client_stream.cc ('k') | net/tools/quic/quic_spdy_server_stream.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 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_spdy_client_stream.h" 5 #include "net/tools/quic/quic_spdy_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/quic/quic_client_session.h" 10 #include "net/tools/quic/quic_client_session.h"
(...skipping 47 matching lines...) Expand 10 before | Expand all | Expand 10 after
58 BalsaHeaders headers_; 58 BalsaHeaders headers_;
59 string headers_string_; 59 string headers_string_;
60 string body_; 60 string body_;
61 QuicCryptoClientConfig crypto_config_; 61 QuicCryptoClientConfig crypto_config_;
62 }; 62 };
63 63
64 INSTANTIATE_TEST_CASE_P(Tests, QuicSpdyClientStreamTest, 64 INSTANTIATE_TEST_CASE_P(Tests, QuicSpdyClientStreamTest,
65 ::testing::ValuesIn(QuicSupportedVersions())); 65 ::testing::ValuesIn(QuicSupportedVersions()));
66 66
67 TEST_P(QuicSpdyClientStreamTest, TestFraming) { 67 TEST_P(QuicSpdyClientStreamTest, TestFraming) {
68 EXPECT_EQ(headers_string_.size(), stream_->ProcessData( 68 stream_->OnStreamHeaders(headers_string_);
69 headers_string_.c_str(), headers_string_.size())); 69 stream_->OnStreamHeadersComplete(false, headers_string_.size());
70 EXPECT_EQ(body_.size(), 70 EXPECT_EQ(body_.size(), stream_->ProcessData(body_.c_str(), body_.size()));
71 stream_->ProcessData(body_.c_str(), body_.size())); 71 EXPECT_EQ("200 Ok", stream_->headers().find(":status")->second);
72 EXPECT_EQ(200u, stream_->headers().parsed_response_code()); 72 EXPECT_EQ(200, stream_->response_code());
73 EXPECT_EQ(body_, stream_->data()); 73 EXPECT_EQ(body_, stream_->data());
74 } 74 }
75 75
76 TEST_P(QuicSpdyClientStreamTest, TestFramingOnePacket) { 76 TEST_P(QuicSpdyClientStreamTest, TestFramingOnePacket) {
77 string message = headers_string_ + body_; 77 stream_->OnStreamHeaders(headers_string_);
78 78 stream_->OnStreamHeadersComplete(false, headers_string_.size());
79 EXPECT_EQ(message.size(), stream_->ProcessData( 79 EXPECT_EQ(body_.size(), stream_->ProcessData(body_.c_str(), body_.size()));
80 message.c_str(), message.size())); 80 EXPECT_EQ("200 Ok", stream_->headers().find(":status")->second);
81 EXPECT_EQ(200u, stream_->headers().parsed_response_code()); 81 EXPECT_EQ(200, stream_->response_code());
82 EXPECT_EQ(body_, stream_->data()); 82 EXPECT_EQ(body_, stream_->data());
83 } 83 }
84 84
85 TEST_P(QuicSpdyClientStreamTest, DISABLED_TestFramingExtraData) { 85 TEST_P(QuicSpdyClientStreamTest, DISABLED_TestFramingExtraData) {
86 string large_body = "hello world!!!!!!"; 86 string large_body = "hello world!!!!!!";
87 87
88 EXPECT_EQ(headers_string_.size(), stream_->ProcessData( 88 stream_->OnStreamHeaders(headers_string_);
89 headers_string_.c_str(), headers_string_.size())); 89 stream_->OnStreamHeadersComplete(false, headers_string_.size());
90 // The headers should parse successfully. 90 // The headers should parse successfully.
91 EXPECT_EQ(QUIC_STREAM_NO_ERROR, stream_->stream_error()); 91 EXPECT_EQ(QUIC_STREAM_NO_ERROR, stream_->stream_error());
92 EXPECT_EQ(200u, stream_->headers().parsed_response_code()); 92 EXPECT_EQ("200 Ok", stream_->headers().find(":status")->second);
93 EXPECT_EQ(200, stream_->response_code());
93 94
94 EXPECT_CALL(*connection_, 95 EXPECT_CALL(*connection_,
95 SendRstStream(stream_->id(), QUIC_BAD_APPLICATION_PAYLOAD, 0)); 96 SendRstStream(stream_->id(), QUIC_BAD_APPLICATION_PAYLOAD, 0));
96 stream_->ProcessData(large_body.c_str(), large_body.size()); 97 stream_->ProcessData(large_body.c_str(), large_body.size());
97 98
98 EXPECT_NE(QUIC_STREAM_NO_ERROR, stream_->stream_error()); 99 EXPECT_NE(QUIC_STREAM_NO_ERROR, stream_->stream_error());
99 } 100 }
100 101
101 TEST_P(QuicSpdyClientStreamTest, TestNoBidirectionalStreaming) { 102 TEST_P(QuicSpdyClientStreamTest, TestNoBidirectionalStreaming) {
102 QuicStreamFrame frame(3, false, 3, MakeIOVector("asd")); 103 QuicStreamFrame frame(3, false, 3, MakeIOVector("asd"));
103 104
104 EXPECT_FALSE(stream_->write_side_closed()); 105 EXPECT_FALSE(stream_->write_side_closed());
105 stream_->OnStreamFrame(frame); 106 stream_->OnStreamFrame(frame);
106 EXPECT_TRUE(stream_->write_side_closed()); 107 EXPECT_TRUE(stream_->write_side_closed());
107 } 108 }
108 109
109 } // namespace 110 } // namespace
110 } // namespace test 111 } // namespace test
111 } // namespace tools 112 } // namespace tools
112 } // namespace net 113 } // namespace net
OLDNEW
« no previous file with comments | « net/tools/quic/quic_spdy_client_stream.cc ('k') | net/tools/quic/quic_spdy_server_stream.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698