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

Unified Diff: ios/clean/chrome/browser/ui/tab_grid/tab_grid_mediator_unittest.mm

Issue 2885983003: [ios] TabCollectionItem (Closed)
Patch Set: Address comments 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/clean/chrome/browser/ui/tab_grid/tab_grid_mediator_unittest.mm
diff --git a/ios/clean/chrome/browser/ui/tab_grid/tab_grid_mediator_unittest.mm b/ios/clean/chrome/browser/ui/tab_grid/tab_grid_mediator_unittest.mm
index 792a6cac41a7139625f74fd48ff95402a0aae516..2f514f0e290a05c2eedf87648e5b3219818ced5c 100644
--- a/ios/clean/chrome/browser/ui/tab_grid/tab_grid_mediator_unittest.mm
+++ b/ios/clean/chrome/browser/ui/tab_grid/tab_grid_mediator_unittest.mm
@@ -41,7 +41,7 @@ class TabGridMediatorTest : public PlatformTest {
}
void SetConsumer() {
- consumer_ = [OCMockObject mockForProtocol:@protocol(TabGridConsumer)];
+ consumer_ = OCMProtocolMock(@protocol(TabGridConsumer));
mediator_.consumer = consumer_;
}
@@ -55,20 +55,16 @@ class TabGridMediatorTest : public PlatformTest {
// the list is empty.
TEST_F(TabGridMediatorTest, TestRemoveNoTabsOverlay) {
SetConsumer();
- [[consumer_ expect] insertItemAtIndex:0];
- [[consumer_ expect] removeNoTabsOverlay];
InsertWebStateAt(0);
- EXPECT_OCMOCK_VERIFY(consumer_);
+ [[consumer_ verify] removeNoTabsOverlay];
}
// Tests that the noTabsOverlay is added when the web state list becomes empty.
TEST_F(TabGridMediatorTest, TestAddNoTabsOverlay) {
- InsertWebStateAt(0);
SetConsumer();
- [[consumer_ expect] deleteItemAtIndex:0];
- [[consumer_ expect] addNoTabsOverlay];
+ InsertWebStateAt(0);
web_state_list_->CloseWebStateAt(0);
- EXPECT_OCMOCK_VERIFY(consumer_);
+ [[consumer_ verify] addNoTabsOverlay];
}
} // namespace

Powered by Google App Engine
This is Rietveld 408576698