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

Unified Diff: components/image_fetcher/image_data_fetcher_unittest.cc

Issue 2761303002: Move common ImageFetcher component files to core/ (Closed)
Patch Set: Rebased. Created 3 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 side-by-side diff with in-line comments
Download patch
« no previous file with comments | « components/image_fetcher/image_data_fetcher.cc ('k') | components/image_fetcher/image_decoder.h » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: components/image_fetcher/image_data_fetcher_unittest.cc
diff --git a/components/image_fetcher/image_data_fetcher_unittest.cc b/components/image_fetcher/image_data_fetcher_unittest.cc
deleted file mode 100644
index d426c5012b38af4393560ea84314b4d99b468e15..0000000000000000000000000000000000000000
--- a/components/image_fetcher/image_data_fetcher_unittest.cc
+++ /dev/null
@@ -1,206 +0,0 @@
-// Copyright 2016 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 "components/image_fetcher/image_data_fetcher.h"
-
-#include <memory>
-
-#include "base/callback.h"
-#include "base/macros.h"
-#include "base/memory/ref_counted.h"
-#include "base/message_loop/message_loop.h"
-#include "net/base/load_flags.h"
-#include "net/http/http_response_headers.h"
-#include "net/http/http_status_code.h"
-#include "net/url_request/test_url_fetcher_factory.h"
-#include "net/url_request/url_request_status.h"
-#include "net/url_request/url_request_test_util.h"
-#include "testing/gmock/include/gmock/gmock.h"
-#include "testing/gtest/include/gtest/gtest.h"
-
-namespace {
-
-const char kImageURL[] = "http://www.example.com/image";
-const char kURLResponseData[] = "EncodedImageData";
-
-} // namespace
-
-namespace image_fetcher {
-
-class ImageDataFetcherTest : public testing::Test {
- public:
- ImageDataFetcherTest()
- : test_request_context_getter_(
- new net::TestURLRequestContextGetter(message_loop_.task_runner())),
- image_data_fetcher_(test_request_context_getter_.get()) {}
- ~ImageDataFetcherTest() override {}
-
- MOCK_METHOD2(OnImageDataFetched,
- void(const std::string&, const RequestMetadata&));
-
- MOCK_METHOD2(OnImageDataFetchedFailedRequest,
- void(const std::string&, const RequestMetadata&));
-
- MOCK_METHOD2(OnImageDataFetchedMultipleRequests,
- void(const std::string&, const RequestMetadata&));
-
- protected:
- base::MessageLoop message_loop_;
-
- scoped_refptr<net::URLRequestContextGetter> test_request_context_getter_;
-
- ImageDataFetcher image_data_fetcher_;
-
- net::TestURLFetcherFactory fetcher_factory_;
-
- private:
- DISALLOW_COPY_AND_ASSIGN(ImageDataFetcherTest);
-};
-
-TEST_F(ImageDataFetcherTest, FetchImageData) {
- image_data_fetcher_.FetchImageData(
- GURL(kImageURL),
- base::Bind(&ImageDataFetcherTest::OnImageDataFetched,
- base::Unretained(this)));
-
- RequestMetadata expected_metadata;
- expected_metadata.mime_type = std::string("image/png");
- expected_metadata.http_response_code = net::HTTP_OK;
- EXPECT_CALL(*this, OnImageDataFetched(std::string(kURLResponseData),
- expected_metadata));
-
- // Get and configure the TestURLFetcher.
- net::TestURLFetcher* test_url_fetcher = fetcher_factory_.GetFetcherByID(0);
- ASSERT_NE(nullptr, test_url_fetcher);
- EXPECT_TRUE(test_url_fetcher->GetLoadFlags() & net::LOAD_DO_NOT_SEND_COOKIES);
- EXPECT_TRUE(test_url_fetcher->GetLoadFlags() & net::LOAD_DO_NOT_SAVE_COOKIES);
- EXPECT_TRUE(test_url_fetcher->GetLoadFlags() &
- net::LOAD_DO_NOT_SEND_AUTH_DATA);
- test_url_fetcher->set_status(
- net::URLRequestStatus(net::URLRequestStatus::SUCCESS, net::OK));
- test_url_fetcher->SetResponseString(kURLResponseData);
- test_url_fetcher->set_response_code(net::HTTP_OK);
-
- std::string raw_header =
- "HTTP/1.1 200 OK\n"
- "Content-type: image/png\n\n";
- std::replace(raw_header.begin(), raw_header.end(), '\n', '\0');
- scoped_refptr<net::HttpResponseHeaders> headers(
- new net::HttpResponseHeaders(raw_header));
-
- test_url_fetcher->set_response_headers(headers);
-
- // Call the URLFetcher delegate to continue the test.
- test_url_fetcher->delegate()->OnURLFetchComplete(test_url_fetcher);
-}
-
-TEST_F(ImageDataFetcherTest, FetchImageData_FromCache) {
- image_data_fetcher_.FetchImageData(
- GURL(kImageURL), base::Bind(&ImageDataFetcherTest::OnImageDataFetched,
- base::Unretained(this)));
-
- RequestMetadata expected_metadata;
- expected_metadata.mime_type = std::string("image/png");
- expected_metadata.http_response_code = net::HTTP_OK;
- expected_metadata.from_http_cache = true;
- EXPECT_CALL(*this, OnImageDataFetched(std::string(kURLResponseData),
- expected_metadata));
-
- // Get and configure the TestURLFetcher.
- net::TestURLFetcher* test_url_fetcher = fetcher_factory_.GetFetcherByID(0);
- ASSERT_NE(nullptr, test_url_fetcher);
- test_url_fetcher->set_status(
- net::URLRequestStatus(net::URLRequestStatus::SUCCESS, net::OK));
- test_url_fetcher->SetResponseString(kURLResponseData);
- test_url_fetcher->set_response_code(net::HTTP_OK);
- test_url_fetcher->set_was_cached(true);
-
- std::string raw_header =
- "HTTP/1.1 200 OK\n"
- "Content-type: image/png\n\n";
- std::replace(raw_header.begin(), raw_header.end(), '\n', '\0');
- scoped_refptr<net::HttpResponseHeaders> headers(
- new net::HttpResponseHeaders(raw_header));
-
- test_url_fetcher->set_response_headers(headers);
-
- // Call the URLFetcher delegate to continue the test.
- test_url_fetcher->delegate()->OnURLFetchComplete(test_url_fetcher);
-}
-
-TEST_F(ImageDataFetcherTest, FetchImageData_NotFound) {
- image_data_fetcher_.FetchImageData(
- GURL(kImageURL), base::Bind(&ImageDataFetcherTest::OnImageDataFetched,
- base::Unretained(this)));
-
- RequestMetadata expected_metadata;
- expected_metadata.mime_type = std::string("image/png");
- expected_metadata.http_response_code = net::HTTP_NOT_FOUND;
- // For 404, expect an empty result even though correct image data is sent.
- EXPECT_CALL(*this, OnImageDataFetched(std::string(), expected_metadata));
-
- // Get and configure the TestURLFetcher.
- net::TestURLFetcher* test_url_fetcher = fetcher_factory_.GetFetcherByID(0);
- ASSERT_NE(nullptr, test_url_fetcher);
- test_url_fetcher->set_status(
- net::URLRequestStatus(net::URLRequestStatus::SUCCESS, net::OK));
- test_url_fetcher->SetResponseString(kURLResponseData);
-
- std::string raw_header =
- "HTTP/1.1 404 Not Found\n"
- "Content-type: image/png\n\n";
- std::replace(raw_header.begin(), raw_header.end(), '\n', '\0');
- scoped_refptr<net::HttpResponseHeaders> headers(
- new net::HttpResponseHeaders(raw_header));
-
- test_url_fetcher->set_response_headers(headers);
-
- // Call the URLFetcher delegate to continue the test.
- test_url_fetcher->delegate()->OnURLFetchComplete(test_url_fetcher);
-}
-
-TEST_F(ImageDataFetcherTest, FetchImageData_FailedRequest) {
- image_data_fetcher_.FetchImageData(
- GURL(kImageURL),
- base::Bind(&ImageDataFetcherTest::OnImageDataFetchedFailedRequest,
- base::Unretained(this)));
-
- RequestMetadata expected_metadata;
- expected_metadata.http_response_code = net::URLFetcher::RESPONSE_CODE_INVALID;
- EXPECT_CALL(
- *this, OnImageDataFetchedFailedRequest(std::string(), expected_metadata));
-
- // Get and configure the TestURLFetcher.
- net::TestURLFetcher* test_url_fetcher = fetcher_factory_.GetFetcherByID(0);
- ASSERT_NE(nullptr, test_url_fetcher);
- test_url_fetcher->set_status(
- net::URLRequestStatus(net::URLRequestStatus::FAILED,
- net::ERR_INVALID_URL));
-
- // Call the URLFetcher delegate to continue the test.
- test_url_fetcher->delegate()->OnURLFetchComplete(test_url_fetcher);
-}
-
-TEST_F(ImageDataFetcherTest, FetchImageData_MultipleRequests) {
- ImageDataFetcher::ImageDataFetcherCallback callback =
- base::Bind(&ImageDataFetcherTest::OnImageDataFetchedMultipleRequests,
- base::Unretained(this));
- EXPECT_CALL(*this, OnImageDataFetchedMultipleRequests(testing::_, testing::_))
- .Times(2);
-
- image_data_fetcher_.FetchImageData(GURL(kImageURL), callback);
- image_data_fetcher_.FetchImageData(GURL(kImageURL), callback);
-
- // Multiple calls to FetchImageData for the same URL will result in
- // multiple URLFetchers being created.
- net::TestURLFetcher* test_url_fetcher = fetcher_factory_.GetFetcherByID(0);
- ASSERT_NE(nullptr, test_url_fetcher);
- test_url_fetcher->delegate()->OnURLFetchComplete(test_url_fetcher);
-
- test_url_fetcher = fetcher_factory_.GetFetcherByID(1);
- ASSERT_NE(nullptr, test_url_fetcher);
- test_url_fetcher->delegate()->OnURLFetchComplete(test_url_fetcher);
-}
-
-} // namespace image_fetcher
« no previous file with comments | « components/image_fetcher/image_data_fetcher.cc ('k') | components/image_fetcher/image_decoder.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698