diff options
| author | 2019-11-25 05:43:47 -0800 | |
|---|---|---|
| committer | 2019-11-25 05:43:47 -0800 | |
| commit | 3a86bceed33a34e4a7bed314d4dcd0453c6f0e1e (patch) | |
| tree | 431ae1961af6cef69ff9a7058188c4c1663a80af | |
| parent | 4371b2a00e2d0eee85a6a8d635057894dbdb9f73 (diff) | |
| parent | 5a6df15108985d1aa1d67cc97aa54458f187927d (diff) | |
Increase timeout in testNativeWatchdogTriggersRollback am: 5f01cb9246
am: 5a6df15108
Change-Id: I6bbfa215974d24c46cd0dc0c717026ac8799bb13
| -rw-r--r-- | tests/RollbackTest/StagedRollbackTest/src/com/android/tests/rollback/host/StagedRollbackTest.java | 14 |
1 files changed, 11 insertions, 3 deletions
diff --git a/tests/RollbackTest/StagedRollbackTest/src/com/android/tests/rollback/host/StagedRollbackTest.java b/tests/RollbackTest/StagedRollbackTest/src/com/android/tests/rollback/host/StagedRollbackTest.java index fb5534549e9b..a590fc4e28d5 100644 --- a/tests/RollbackTest/StagedRollbackTest/src/com/android/tests/rollback/host/StagedRollbackTest.java +++ b/tests/RollbackTest/StagedRollbackTest/src/com/android/tests/rollback/host/StagedRollbackTest.java @@ -30,6 +30,8 @@ import org.junit.Ignore; import org.junit.Test; import org.junit.runner.RunWith; +import java.util.concurrent.TimeUnit; + /** * Runs the staged rollback tests. */ @@ -102,9 +104,15 @@ public class StagedRollbackTest extends BaseHostJUnit4Test { // crash system_server enough times to trigger a rollback crashProcess("system_server", NATIVE_CRASHES_THRESHOLD); - // Rollback should be committed automatically now - // Give time for rollback to be committed - assertTrue(getDevice().waitForDeviceNotAvailable(60000)); + // Rollback should be committed automatically now. + // Give time for rollback to be committed. This could take a while, + // because we need all of the following to happen: + // 1. system_server comes back up and boot completes. + // 2. Rollback health observer detects updatable crashing signal. + // 3. Staged rollback session becomes ready. + // 4. Device actually reboots. + // So we give a generous timeout here. + assertTrue(getDevice().waitForDeviceNotAvailable(TimeUnit.MINUTES.toMillis(5))); getDevice().waitForDeviceAvailable(); // verify rollback committed |