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

Side by Side Diff: chrome/browser/search/suggestions/image_fetcher_impl_browsertest.cc

Issue 1409163006: Migrating tests to use EmbeddedTestServer (/chrome/browser misc) (Closed) Base URL: https://chromium.googlesource.com/chromium/src.git@master
Patch Set: Rebase. 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 2014 The Chromium Authors. All rights reserved. 1 // Copyright 2014 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 "chrome/browser/search/suggestions/image_fetcher_impl.h" 5 #include "chrome/browser/search/suggestions/image_fetcher_impl.h"
6 6
7 #include "base/bind.h" 7 #include "base/bind.h"
8 #include "base/files/file_path.h" 8 #include "base/files/file_path.h"
9 #include "base/memory/scoped_ptr.h" 9 #include "base/memory/scoped_ptr.h"
10 #include "base/run_loop.h" 10 #include "base/run_loop.h"
11 #include "chrome/browser/profiles/profile.h" 11 #include "chrome/browser/profiles/profile.h"
12 #include "chrome/browser/ui/browser.h" 12 #include "chrome/browser/ui/browser.h"
13 #include "chrome/test/base/in_process_browser_test.h" 13 #include "chrome/test/base/in_process_browser_test.h"
14 #include "components/suggestions/image_fetcher_delegate.h" 14 #include "components/suggestions/image_fetcher_delegate.h"
15 #include "net/test/spawned_test_server/spawned_test_server.h" 15 #include "net/test/embedded_test_server/embedded_test_server.h"
16 #include "testing/gtest/include/gtest/gtest.h" 16 #include "testing/gtest/include/gtest/gtest.h"
17 #include "url/gurl.h" 17 #include "url/gurl.h"
18 18
19 class SkBitmap; 19 class SkBitmap;
20 20
21 namespace suggestions { 21 namespace suggestions {
22 22
23 namespace { 23 namespace {
24 24
25 const char kTestUrl[] = "http://go.com/"; 25 const char kTestUrl[] = "http://go.com/";
26 const char kTestImagePath[] = "files/image_decoding/droids.png"; 26 const char kTestImagePath[] = "/image_decoding/droids.png";
27 const char kInvalidImagePath[] = "files/DOESNOTEXIST"; 27 const char kInvalidImagePath[] = "/DOESNOTEXIST";
28 28
29 const base::FilePath::CharType kDocRoot[] = 29 const base::FilePath::CharType kDocRoot[] =
30 FILE_PATH_LITERAL("chrome/test/data"); 30 FILE_PATH_LITERAL("chrome/test/data");
31 31
32 class TestImageFetcherDelegate : public ImageFetcherDelegate { 32 class TestImageFetcherDelegate : public ImageFetcherDelegate {
33 public: 33 public:
34 TestImageFetcherDelegate() 34 TestImageFetcherDelegate()
35 : num_delegate_valid_called_(0), 35 : num_delegate_valid_called_(0),
36 num_delegate_null_called_(0) {} 36 num_delegate_null_called_(0) {}
37 ~TestImageFetcherDelegate() override{}; 37 ~TestImageFetcherDelegate() override{};
(...skipping 13 matching lines...) Expand all
51 private: 51 private:
52 int num_delegate_valid_called_; 52 int num_delegate_valid_called_;
53 int num_delegate_null_called_; 53 int num_delegate_null_called_;
54 }; 54 };
55 55
56 } // end namespace 56 } // end namespace
57 57
58 class ImageFetcherImplBrowserTest : public InProcessBrowserTest { 58 class ImageFetcherImplBrowserTest : public InProcessBrowserTest {
59 protected: 59 protected:
60 ImageFetcherImplBrowserTest() 60 ImageFetcherImplBrowserTest()
61 : num_callback_valid_called_(0), 61 : num_callback_valid_called_(0), num_callback_null_called_(0) {
62 num_callback_null_called_(0), 62 test_server_.ServeFilesFromSourceDirectory(base::FilePath(kDocRoot));
63 test_server_(net::SpawnedTestServer::TYPE_HTTP, 63 }
64 net::SpawnedTestServer::kLocalhost,
65 base::FilePath(kDocRoot)) {}
66 64
67 void SetUpInProcessBrowserTestFixture() override { 65 void SetUpInProcessBrowserTestFixture() override {
68 ASSERT_TRUE(test_server_.Start()); 66 ASSERT_TRUE(test_server_.Start());
69 InProcessBrowserTest::SetUpInProcessBrowserTestFixture(); 67 InProcessBrowserTest::SetUpInProcessBrowserTestFixture();
70 } 68 }
71 69
72 void TearDownInProcessBrowserTestFixture() override { test_server_.Stop(); }
73
74 ImageFetcherImpl* CreateImageFetcher() { 70 ImageFetcherImpl* CreateImageFetcher() {
75 ImageFetcherImpl* fetcher = 71 ImageFetcherImpl* fetcher =
76 new ImageFetcherImpl(browser()->profile()->GetRequestContext()); 72 new ImageFetcherImpl(browser()->profile()->GetRequestContext());
77 fetcher->SetImageFetcherDelegate(&delegate_); 73 fetcher->SetImageFetcherDelegate(&delegate_);
78 return fetcher; 74 return fetcher;
79 } 75 }
80 76
81 void OnImageAvailable(base::RunLoop* loop, 77 void OnImageAvailable(base::RunLoop* loop,
82 const GURL& url, 78 const GURL& url,
83 const SkBitmap* bitmap) { 79 const SkBitmap* bitmap) {
(...skipping 13 matching lines...) Expand all
97 GURL(kTestUrl), 93 GURL(kTestUrl),
98 image_url, 94 image_url,
99 base::Bind(&ImageFetcherImplBrowserTest::OnImageAvailable, 95 base::Bind(&ImageFetcherImplBrowserTest::OnImageAvailable,
100 base::Unretained(this), &run_loop)); 96 base::Unretained(this), &run_loop));
101 run_loop.Run(); 97 run_loop.Run();
102 } 98 }
103 99
104 int num_callback_valid_called_; 100 int num_callback_valid_called_;
105 int num_callback_null_called_; 101 int num_callback_null_called_;
106 102
107 net::SpawnedTestServer test_server_; 103 net::EmbeddedTestServer test_server_;
108 TestImageFetcherDelegate delegate_; 104 TestImageFetcherDelegate delegate_;
109 105
110 DISALLOW_COPY_AND_ASSIGN(ImageFetcherImplBrowserTest); 106 DISALLOW_COPY_AND_ASSIGN(ImageFetcherImplBrowserTest);
111 }; 107 };
112 108
113 IN_PROC_BROWSER_TEST_F(ImageFetcherImplBrowserTest, NormalFetch) { 109 IN_PROC_BROWSER_TEST_F(ImageFetcherImplBrowserTest, NormalFetch) {
114 GURL image_url(test_server_.GetURL(kTestImagePath).spec()); 110 GURL image_url(test_server_.GetURL(kTestImagePath).spec());
115 StartOrQueueNetworkRequestHelper(image_url); 111 StartOrQueueNetworkRequestHelper(image_url);
116 112
117 EXPECT_EQ(1, num_callback_valid_called_); 113 EXPECT_EQ(1, num_callback_valid_called_);
(...skipping 19 matching lines...) Expand all
137 GURL invalid_image_url(test_server_.GetURL(kInvalidImagePath).spec()); 133 GURL invalid_image_url(test_server_.GetURL(kInvalidImagePath).spec());
138 StartOrQueueNetworkRequestHelper(invalid_image_url); 134 StartOrQueueNetworkRequestHelper(invalid_image_url);
139 135
140 EXPECT_EQ(0, num_callback_valid_called_); 136 EXPECT_EQ(0, num_callback_valid_called_);
141 EXPECT_EQ(1, num_callback_null_called_); 137 EXPECT_EQ(1, num_callback_null_called_);
142 EXPECT_EQ(0, delegate_.num_delegate_valid_called()); 138 EXPECT_EQ(0, delegate_.num_delegate_valid_called());
143 EXPECT_EQ(1, delegate_.num_delegate_null_called()); 139 EXPECT_EQ(1, delegate_.num_delegate_null_called());
144 } 140 }
145 141
146 } // namespace suggestions 142 } // namespace suggestions
OLDNEW
« no previous file with comments | « chrome/browser/safe_browsing/safe_browsing_test.cc ('k') | chrome/browser/sessions/session_restore_browsertest.cc » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698