Commit 99f70a0c authored by Katie Dektar's avatar Katie Dektar Committed by Commit Bot

[Sheriff] Revert "[Kiosk] Fix flaky KioskTest.LaunchAppUserCancel"

This reverts commit 959b4db4.

Reason for revert: Failing consistently (2/2) since this change landed.
https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg/17292
https://ci.chromium.org/p/chromium/builders/ci/linux-chromeos-dbg/17293

Failed with status TIMEOUT

Original change's description:
> [Kiosk] Fix flaky KioskTest.LaunchAppUserCancel
> 
> Sometimes network timer was timing out and we were going into
> network configure screen. This screen is orthogonal to the purpose of
> this test. Let's disable network handling in this test.
> 
> Bug: 964333
> Change-Id: Ic93568970b7af07f4703c9474868aaefabccf1cd
> Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2075457
> Commit-Queue: Anatoliy Potapchuk <apotapchuk@chromium.org>
> Reviewed-by: Roman Sorokin [CET] <rsorokin@chromium.org>
> Cr-Commit-Position: refs/heads/master@{#745038}

TBR=rsorokin@chromium.org,apotapchuk@chromium.org

Change-Id: I39fa8e39a956c0780839eed42d4a43ca1264b3e4
No-Presubmit: true
No-Tree-Checks: true
No-Try: true
Bug: 964333
Reviewed-on: https://chromium-review.googlesource.com/c/chromium/src/+/2078999Reviewed-by: default avatarKatie Dektar <katie@chromium.org>
Commit-Queue: Katie Dektar <katie@chromium.org>
Cr-Commit-Position: refs/heads/master@{#745091}
parent 321d124a
......@@ -1021,10 +1021,8 @@ IN_PROC_BROWSER_TEST_F(KioskTest, LaunchAppNetworkPortal) {
WaitForAppLaunchSuccess();
}
IN_PROC_BROWSER_TEST_F(KioskTest, LaunchAppUserCancel) {
// Do not allow network configure UI to appear ensuring that we are at the
// splash screen.
ScopedCanConfigureNetwork can_configure_network(false, false);
// TODO(https://crbug.com/964333): Flakily seg faults.
IN_PROC_BROWSER_TEST_F(KioskTest, DISABLED_LaunchAppUserCancel) {
// Make fake_cws_ return empty update response.
set_test_app_version("");
OobeScreenWaiter splash_waiter(AppLaunchSplashScreenView::kScreenId);
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment