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

Unified Diff: ios/chrome/browser/ui/reading_list/reading_list_coordinator_unittest.mm

Issue 2879183002: Move the ReadingListModel in a Mediator (Closed)
Patch Set: Address comment Created 3 years, 7 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
Index: ios/chrome/browser/ui/reading_list/reading_list_coordinator_unittest.mm
diff --git a/ios/chrome/browser/ui/reading_list/reading_list_coordinator_unittest.mm b/ios/chrome/browser/ui/reading_list/reading_list_coordinator_unittest.mm
index bf3be9751a80aa0b506f085579193bb4a7dffd25..da2d45c13090cee64f058518e3ef7548c2c755f2 100644
--- a/ios/chrome/browser/ui/reading_list/reading_list_coordinator_unittest.mm
+++ b/ios/chrome/browser/ui/reading_list/reading_list_coordinator_unittest.mm
@@ -16,6 +16,7 @@
#import "ios/chrome/browser/ui/collection_view/collection_view_model.h"
#import "ios/chrome/browser/ui/reading_list/reading_list_collection_view_controller.h"
#import "ios/chrome/browser/ui/reading_list/reading_list_collection_view_item.h"
+#import "ios/chrome/browser/ui/reading_list/reading_list_mediator.h"
#import "ios/chrome/browser/ui/reading_list/reading_list_utils.h"
#import "ios/chrome/browser/ui/url_loader.h"
#include "ios/web/public/referrer.h"
@@ -98,6 +99,8 @@ class ReadingListCoordinatorTest : public web::WebTestWithWebState {
reading_list_model_.reset(new ReadingListModelImpl(
nullptr, nullptr, base::MakeUnique<base::DefaultClock>()));
+ mediator_.reset(
+ [[ReadingListMediator alloc] initWithModel:reading_list_model_.get()]);
large_icon_service_.reset(new favicon::LargeIconService(
&mock_favicon_service_, base::ThreadTaskRunnerHandle::Get(),
/*image_fetcher=*/nullptr));
@@ -121,14 +124,14 @@ class ReadingListCoordinatorTest : public web::WebTestWithWebState {
ReadingListCollectionViewController*
GetAReadingListCollectionViewController() {
return [[[ReadingListCollectionViewController alloc]
- initWithModel:reading_list_model_.get()
- largeIconService:large_icon_service_.get()
- readingListDownloadService:nil
- toolbar:nil] autorelease];
+ initWithDataSource:mediator_
+ largeIconService:large_icon_service_.get()
+ toolbar:nil] autorelease];
}
private:
base::scoped_nsobject<ReadingListCoordinator> coordinator_;
+ base::scoped_nsobject<ReadingListMediator> mediator_;
std::unique_ptr<ReadingListModelImpl> reading_list_model_;
base::scoped_nsobject<UrlLoaderStub> loader_mock_;
testing::StrictMock<favicon::MockFaviconService> mock_favicon_service_;

Powered by Google App Engine
This is Rietveld 408576698