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

Unified Diff: ui/app_list/cocoa/app_list_view_controller_unittest.mm

Issue 309443002: Remove need to sign in to use the App Launcher. (Closed) Base URL: svn://svn.chromium.org/chrome/trunk/src
Patch Set: Rebase Created 6 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: ui/app_list/cocoa/app_list_view_controller_unittest.mm
diff --git a/ui/app_list/cocoa/app_list_view_controller_unittest.mm b/ui/app_list/cocoa/app_list_view_controller_unittest.mm
index 0a7d82b1faa1c86befec910d47a92e752bf9cdd2..60f83e7473123d71acffafe195914c1d9d50e839 100644
--- a/ui/app_list/cocoa/app_list_view_controller_unittest.mm
+++ b/ui/app_list/cocoa/app_list_view_controller_unittest.mm
@@ -20,7 +20,6 @@ class AppListViewControllerTest : public AppsGridControllerTestHelper {
virtual void SetUp() OVERRIDE {
app_list_view_controller_.reset([[AppListViewController alloc] init]);
scoped_ptr<AppListTestViewDelegate> delegate(new AppListTestViewDelegate);
- delegate->SetSignedIn(true);
[app_list_view_controller_
setDelegate:delegate.PassAs<app_list::AppListViewDelegate>()];
SetUpWithGridController([app_list_view_controller_ appsGridController]);
@@ -104,25 +103,5 @@ TEST_F(AppListViewControllerTest, PagerChangingPage) {
EXPECT_EQ(1.0, [apps_grid_controller_ visiblePortionOfPage:0]);
}
-// Test the view when the user is already signed in.
-TEST_F(AppListViewControllerTest, SignedIn) {
- // There should be just 1, visible subview when signed in.
- EXPECT_EQ(1u, [[[app_list_view_controller_ view] subviews] count]);
- EXPECT_FALSE([[app_list_view_controller_ backgroundView] isHidden]);
-}
-
-// Test the view when signin is required.
-TEST_F(AppListViewControllerTest, NeedsSignin) {
- // Begin the test with a signed out app list.
- delegate()->SetSignedIn(false);
- EXPECT_EQ(2u, [[[app_list_view_controller_ view] subviews] count]);
- EXPECT_TRUE([[app_list_view_controller_ backgroundView] isHidden]);
-
- // Simulate signing in, should enter the SignedIn state.
- delegate()->SetSignedIn(true);
- EXPECT_EQ(1u, [[[app_list_view_controller_ view] subviews] count]);
- EXPECT_FALSE([[app_list_view_controller_ backgroundView] isHidden]);
-}
-
} // namespace test
} // namespace app_list

Powered by Google App Engine
This is Rietveld 408576698