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

Side by Side Diff: google_apis/drive/files_list_request_runner_unittest.cc

Issue 1431653003: Migrating tests to use EmbeddedTestServer (misc) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Fixing comments. Created 5 years, 1 month 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 2015 The Chromium Authors. All rights reserved. 1 // Copyright 2015 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 "google_apis/drive/files_list_request_runner.h" 5 #include "google_apis/drive/files_list_request_runner.h"
6 6
7 #include "base/memory/scoped_ptr.h" 7 #include "base/memory/scoped_ptr.h"
8 #include "base/message_loop/message_loop.h" 8 #include "base/message_loop/message_loop.h"
9 #include "base/run_loop.h" 9 #include "base/run_loop.h"
10 #include "base/sequenced_task_runner.h" 10 #include "base/sequenced_task_runner.h"
(...skipping 55 matching lines...) Expand 10 before | Expand all | Expand 10 after
66 request_context_getter_ = 66 request_context_getter_ =
67 new net::TestURLRequestContextGetter(message_loop_.task_runner()); 67 new net::TestURLRequestContextGetter(message_loop_.task_runner());
68 68
69 request_sender_.reset( 69 request_sender_.reset(
70 new RequestSender(new DummyAuthService, request_context_getter_.get(), 70 new RequestSender(new DummyAuthService, request_context_getter_.get(),
71 message_loop_.task_runner(), kTestUserAgent)); 71 message_loop_.task_runner(), kTestUserAgent));
72 72
73 test_server_.RegisterRequestHandler( 73 test_server_.RegisterRequestHandler(
74 base::Bind(&FilesListRequestRunnerTest::OnFilesListRequest, 74 base::Bind(&FilesListRequestRunnerTest::OnFilesListRequest,
75 base::Unretained(this), test_server_.base_url())); 75 base::Unretained(this), test_server_.base_url()));
76 ASSERT_TRUE(test_server_.InitializeAndWaitUntilReady()); 76 ASSERT_TRUE(test_server_.Start());
77 77
78 runner_.reset(new FilesListRequestRunner( 78 runner_.reset(new FilesListRequestRunner(
79 request_sender_.get(), 79 request_sender_.get(),
80 google_apis::DriveApiUrlGenerator(test_server_.base_url(), 80 google_apis::DriveApiUrlGenerator(test_server_.base_url(),
81 test_server_.GetURL("/download/")))); 81 test_server_.GetURL("/download/"))));
82 } 82 }
83 83
84 void TearDown() override { 84 void TearDown() override {
85 on_completed_callback_ = base::Closure(); 85 on_completed_callback_ = base::Closure();
86 http_request_.reset(); 86 http_request_.reset();
(...skipping 23 matching lines...) Expand all
110 // Handles a HTTP request to the Drive API server and returns a fake response. 110 // Handles a HTTP request to the Drive API server and returns a fake response.
111 scoped_ptr<net::test_server::HttpResponse> OnFilesListRequest( 111 scoped_ptr<net::test_server::HttpResponse> OnFilesListRequest(
112 const GURL& base_url, 112 const GURL& base_url,
113 const net::test_server::HttpRequest& request) { 113 const net::test_server::HttpRequest& request) {
114 http_request_.reset(new net::test_server::HttpRequest(request)); 114 http_request_.reset(new net::test_server::HttpRequest(request));
115 return fake_server_response_.Pass(); 115 return fake_server_response_.Pass();
116 } 116 }
117 117
118 base::MessageLoopForIO message_loop_; // Test server needs IO thread. 118 base::MessageLoopForIO message_loop_; // Test server needs IO thread.
119 scoped_ptr<RequestSender> request_sender_; 119 scoped_ptr<RequestSender> request_sender_;
120 net::test_server::EmbeddedTestServer test_server_; 120 net::EmbeddedTestServer test_server_;
121 scoped_ptr<FilesListRequestRunner> runner_; 121 scoped_ptr<FilesListRequestRunner> runner_;
122 scoped_refptr<net::TestURLRequestContextGetter> request_context_getter_; 122 scoped_refptr<net::TestURLRequestContextGetter> request_context_getter_;
123 base::Closure on_completed_callback_; 123 base::Closure on_completed_callback_;
124 124
125 // Response set by test cases to be returned from the HTTP server. 125 // Response set by test cases to be returned from the HTTP server.
126 scoped_ptr<net::test_server::BasicHttpResponse> fake_server_response_; 126 scoped_ptr<net::test_server::BasicHttpResponse> fake_server_response_;
127 127
128 // A requests and a response stored for verification in test cases. 128 // A requests and a response stored for verification in test cases.
129 scoped_ptr<net::test_server::HttpRequest> http_request_; 129 scoped_ptr<net::test_server::HttpRequest> http_request_;
130 scoped_ptr<DriveApiErrorCode> response_error_; 130 scoped_ptr<DriveApiErrorCode> response_error_;
(...skipping 133 matching lines...) Expand 10 before | Expand all | Expand 10 after
264 http_request_->relative_url); 264 http_request_->relative_url);
265 265
266 // There must be no backoff in case of an error different than 266 // There must be no backoff in case of an error different than
267 // DRIVE_RESPONSE_TOO_LARGE. 267 // DRIVE_RESPONSE_TOO_LARGE.
268 ASSERT_TRUE(response_error_.get()); 268 ASSERT_TRUE(response_error_.get());
269 EXPECT_EQ(DRIVE_NO_SPACE, *response_error_); 269 EXPECT_EQ(DRIVE_NO_SPACE, *response_error_);
270 EXPECT_FALSE(response_entry_.get()); 270 EXPECT_FALSE(response_entry_.get());
271 } 271 }
272 272
273 } // namespace google_apis 273 } // namespace google_apis
OLDNEW

Powered by Google App Engine
This is Rietveld 408576698