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

Side by Side Diff: net/websockets/websocket_extension_parser_test.cc

Issue 23872029: Implement WebSocketExtensionParser (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Created 7 years, 3 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
(Empty)
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
3 // found in the LICENSE file.
4
5 #include "net/websockets/websocket_extension_parser.h"
6
7 #include <string>
8
9 #include "net/websockets/websocket_extension.h"
10 #include "testing/gtest/include/gtest/gtest.h"
11
12 namespace net {
13
14 namespace {
15
16 TEST(WebSocketExtensionParserTest, ParseEmpty) {
17 WebSocketExtensionParser parser;
18 parser.Parse("", 0);
19
20 EXPECT_TRUE(parser.has_error());
21 }
22
23 TEST(WebSocketExtensionParserTest, ParseSimple) {
24 WebSocketExtensionParser parser;
25 WebSocketExtension expected("foo");
26
27 parser.Parse("foo");
28
29 ASSERT_FALSE(parser.has_error());
30 EXPECT_TRUE(expected.Equals(parser.extension()));
31 }
32
33 TEST(WebSocketExtensionParserTest, ParseOneExtensionWithOneParamWithoutValue) {
34 WebSocketExtensionParser parser;
35 WebSocketExtension expected("foo");
36 expected.Add(WebSocketExtension::Parameter("bar"));
37
38 parser.Parse("\tfoo ; bar");
39
40 ASSERT_FALSE(parser.has_error());
41 EXPECT_TRUE(expected.Equals(parser.extension()));
42 }
43
44 TEST(WebSocketExtensionParserTest, ParseOneExtensionWithOneParamWithValue) {
45 WebSocketExtensionParser parser;
46 WebSocketExtension expected("foo");
47 expected.Add(WebSocketExtension::Parameter("bar", "baz"));
48
49 parser.Parse("foo ; bar= baz\t");
50
51 ASSERT_FALSE(parser.has_error());
52 EXPECT_TRUE(expected.Equals(parser.extension()));
53 }
54
55 TEST(WebSocketExtensionParserTest, ParseOneExtensionWithParams) {
56 WebSocketExtensionParser parser;
57 WebSocketExtension expected("foo");
58 expected.Add(WebSocketExtension::Parameter("bar", "baz"));
59 expected.Add(WebSocketExtension::Parameter("hoge", "fuga"));
60
61 parser.Parse("foo ; bar= baz;\t \thoge\t\t=fuga");
62
63 ASSERT_FALSE(parser.has_error());
64 EXPECT_TRUE(expected.Equals(parser.extension()));
65 }
66
67 TEST(WebSocketExtensionParserTest, InvalidPatterns) {
68 const char* patterns[] = {
69 "fo\ao", // control in extension name
70 "fo\x01o", // control in extension name
71 "fo<o", // separator in extension name
72 "foo/", // separator in extension name
73 ";bar", // empty extension name
74 "foo; b\ar", // control in parameter name
75 "foo; b\x7fr", // control in parameter name
76 "foo; b[r", // separator in parameter name
77 "foo; ba:", // separator in parameter name
78 "foo; =baz", // empty parameter name
79 "foo; bar=", // empty parameter value
80 "foo; =", // empty parameter name and value
81 "foo; bar=b\x02z", // control in parameter value
82 "foo; bar=b@z", // separator in parameter value
83 "foo; bar=b\\z", // separator in parameter value
84 "foo; bar=b?z", // separator in parameter value
85 "\"foo\"", // quoted extension name
86 "foo; \"bar\"", // quoted parameter name
87 "foo; bar=\"\a2\"", // control in quoted parameter value
88 "foo; bar=\"b@z\"", // separator in quoted parameter value
89 "foo; bar=\"b\\\\z\"", // separator in quoted parameter value
90 "foo; bar=\"\"", // quoted emptyparameter value
91 "foo; bar=\r\nbaz", // CRNL not followed by a space
92 "foo; bar=\r\n baz", // CRNL followed by a space
93 "foo, bar" // multiple extensions
Adam Rice 2013/09/17 12:41:36 Some other strings I think are invalid: "foo;",
yhirano 2013/09/18 01:15:43 Done.
94 };
95
96 for (size_t i = 0; i < arraysize(patterns); ++i) {
97 WebSocketExtensionParser parser;
98 parser.Parse(patterns[i]);
99 EXPECT_TRUE(parser.has_error());
100 }
101 }
102
103 TEST(WebSocketExtensionParserTest, QuotedParameterValue) {
104 WebSocketExtensionParser parser;
105 WebSocketExtension expected("foo");
106 expected.Add(WebSocketExtension::Parameter("bar", "baz"));
107
108 parser.Parse("foo; bar=\"ba\\z\"");
109
110 ASSERT_FALSE(parser.has_error());
111 EXPECT_TRUE(expected.Equals(parser.extension()));
112 }
113
114 } // namespace
115
116 } // namespace net
OLDNEW
« net/websockets/websocket_extension_parser.cc ('K') | « net/websockets/websocket_extension_parser.cc ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698