Added chunked upload support for embedded_test_server

net/test/embedded_test_server/http_request_unittest.cc

 // Copyright (c) 2012 The Chromium Authors. All rights reserved.
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
 #include "net/test/embedded_test_server/http_request.h"
 
 #include "testing/gtest/include/gtest/gtest.h"
 
 namespace net {
 namespace test_server {
@@ skipping 58 matching lines @@
   parser.ProcessChunk("Content-Length: 0\r\n\r\n");
   ASSERT_EQ(HttpRequestParser::ACCEPTED, parser.ParseRequest());
 
   scoped_ptr<HttpRequest> request = parser.GetRequest();
   EXPECT_EQ("", request->content);
   EXPECT_TRUE(request->has_content);
   EXPECT_EQ(1u, request->headers.count("Content-Length"));
   EXPECT_EQ("0", request->headers["Content-Length"]);
 }
 
+TEST(HttpRequestTest, ParseRequestWithChunkedBody) {
+  HttpRequestParser parser;
+
+  parser.ProcessChunk("POST /foobar.html HTTP/1.1\r\n");
+  parser.ProcessChunk("Transfer-Encoding: chunked\r\n\r\n");
+  parser.ProcessChunk("5\r\nhello\r\n");
+  parser.ProcessChunk("1\r\n \r\n");
+  parser.ProcessChunk("5\r\nworld\r\n");
+  parser.ProcessChunk("0\r\n\r\n");
+  ASSERT_EQ(HttpRequestParser::ACCEPTED, parser.ParseRequest());
+
+  scoped_ptr<HttpRequest> request = parser.GetRequest();
+  EXPECT_EQ("hello world", request->content);
+  EXPECT_TRUE(request->has_content);
+  EXPECT_EQ(1u, request->headers.count("Transfer-Encoding"));
+  EXPECT_EQ("chunked", request->headers["Transfer-Encoding"]);
+}

[mmenke, 2015/02/12 18:22:48]

Another suggested test:

  std::string data =
      "POST /foobar.html HTTP/1.1\r\n"
      "Transfer-Encoding: chunked\r\n\r\n"
      "5\r\nhello\r\n"
      ...

  for (int i = 0; i < data.length; ++i) {
    parser.ProcessChunk(data[i]);
  }

This will hit the problem with your code.

[xunjieli, 2015/02/12 22:17:22]

Done.

+
 TEST(HttpRequestTest, ParseRequestWithoutBody) {
   HttpRequestParser parser;
 
   parser.ProcessChunk("POST /foobar.html HTTP/1.1\r\n\r\n");
   ASSERT_EQ(HttpRequestParser::ACCEPTED, parser.ParseRequest());
 
   scoped_ptr<HttpRequest> request = parser.GetRequest();
   EXPECT_EQ("", request->content);
   EXPECT_FALSE(request->has_content);
 }
 
 TEST(HttpRequestTest, ParseGet) {
   HttpRequestParser parser;
 
   parser.ProcessChunk("GET /foobar.html HTTP/1.1\r\n\r\n");
   ASSERT_EQ(HttpRequestParser::ACCEPTED, parser.ParseRequest());
 
   scoped_ptr<HttpRequest> request = parser.GetRequest();
   EXPECT_EQ("/foobar.html", request->relative_url);
   EXPECT_EQ("GET", request->method_string);
   EXPECT_EQ(METHOD_GET, request->method);
   EXPECT_EQ("", request->content);
   EXPECT_FALSE(request->has_content);
 }
 
 }  // namespace test_server
 }  // namespace net