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

Unified Diff: chrome/browser/chromeos/login/wizard_controller_browsertest.cc

Issue 12213110: Implemented screen notifying users about malformed HWID. (Closed) Base URL: http://git.chromium.org/git/chromium.git@trunk
Patch Set: Implemented wizard controller test. Created 7 years, 10 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: chrome/browser/chromeos/login/wizard_controller_browsertest.cc
diff --git a/chrome/browser/chromeos/login/wizard_controller_browsertest.cc b/chrome/browser/chromeos/login/wizard_controller_browsertest.cc
index d79042ff9db8d2cc1294c0f3c16661b59bd72a8d..62b2b61f6731c40ec24ca43918826106c4bd70bd 100644
--- a/chrome/browser/chromeos/login/wizard_controller_browsertest.cc
+++ b/chrome/browser/chromeos/login/wizard_controller_browsertest.cc
@@ -24,6 +24,7 @@
#include "chrome/browser/chromeos/login/view_screen.h"
#include "chrome/browser/chromeos/login/wizard_controller.h"
#include "chrome/browser/chromeos/login/wizard_in_process_browser_test.h"
+#include "chrome/browser/chromeos/login/wrong_hwid_screen.h"
#include "chrome/test/base/ui_test_utils.h"
#include "grit/generated_resources.h"
#include "testing/gmock/include/gmock/gmock.h"
@@ -328,9 +329,30 @@ IN_PROC_BROWSER_TEST_F(WizardControllerFlowTest, ControlFlowResetScreen) {
EXPECT_FALSE(ExistingUserController::current_controller() == NULL);
}
+IN_PROC_BROWSER_TEST_F(WizardControllerFlowTest,
+ ControlFlowWrongHWIDScreenFromLogin) {
+ EXPECT_EQ(WizardController::default_controller()->GetNetworkScreen(),
+ WizardController::default_controller()->current_screen());
+
+ BaseLoginDisplayHost::default_host()->StartSignInScreen();
+ EXPECT_FALSE(ExistingUserController::current_controller() == NULL);
+ ExistingUserController::current_controller()->ShowWrongHWIDScreen();
+
+ WrongHWIDScreen* screen =
+ WizardController::default_controller()->GetWrongHWIDScreen();
+ EXPECT_EQ(screen, WizardController::default_controller()->current_screen());
+
+ // After warning is skipped, user returns to sign-in screen.
+ // And this destroys WizardController.
+ OnExit(ScreenObserver::WRONG_HWID_WARNING_SKIPPED);
+ EXPECT_FALSE(ExistingUserController::current_controller() == NULL);
+}
+
+// TODO(dzhioev): Add test emaulating device with wrong HWID.
+
// TODO(nkostylev): Add test for WebUI accelerators http://crosbug.com/22571
-COMPILE_ASSERT(ScreenObserver::EXIT_CODES_COUNT == 16,
+COMPILE_ASSERT(ScreenObserver::EXIT_CODES_COUNT == 17,
add_tests_for_new_control_flow_you_just_introduced);
} // namespace chromeos
« no previous file with comments | « chrome/browser/chromeos/login/wizard_controller.cc ('k') | chrome/browser/chromeos/login/wrong_hwid_screen.h » ('j') | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698