From d30c51157e77cf75f09aab7a3db1c274f1823ee6 Mon Sep 17 00:00:00 2001
From: Tom Cherry <tomcherry@google.com>
Date: Fri, 27 Mar 2020 13:36:58 -0700
Subject: [PATCH] init: disable oneshot_on test

Apparently relying on bootanim to restart or not is too flaky.

Bug: 152630580
Bug: 152637928
Test: this test no longer runs
Change-Id: If61b31bad276560a9f0db9811724bbc9efc51d28
---
 init/oneshot_on_test.cpp | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/init/oneshot_on_test.cpp b/init/oneshot_on_test.cpp
index 7e7cc3679..487e0ea69 100644
--- a/init/oneshot_on_test.cpp
+++ b/init/oneshot_on_test.cpp
@@ -25,6 +25,7 @@ using android::base::SetProperty;
 using android::base::WaitForProperty;
 using namespace std::literals;
 
+#if 0  // TODO(b/152637928): this test is flaky
 TEST(init, oneshot_on) {
     // Bootanim shouldn't be running once the device has booted.
     ASSERT_EQ("stopped", GetProperty("init.svc.bootanim", ""));
@@ -42,3 +43,4 @@ TEST(init, oneshot_on) {
     // Now that oneshot is enabled again, bootanim should transition into the 'stopped' state.
     EXPECT_TRUE(WaitForProperty("init.svc.bootanim", "stopped", 10s));
 }
+#endif