From 0a0e4793e3c46752dc31c2d5c730655217add0bb Mon Sep 17 00:00:00 2001 From: Nikita Ioffe Date: Wed, 30 Jun 2021 00:20:01 +0100 Subject: [PATCH] Only run RebootTest under root This test requires running test services, which causes test to crash (and still incorrectly be reported as passing) when running on non-rooted device. Ignore-AOSP-First: reboot_test is not in AOSP yet Bug: 190958734 Test: atest CtsInitTestCases Merged-In: I3c5c9917d0a787d66272ccf4aefc57e6573841bc Change-Id: I3c5c9917d0a787d66272ccf4aefc57e6573841bc --- init/reboot_test.cpp | 10 ++++++++++ 1 file changed, 10 insertions(+) diff --git a/init/reboot_test.cpp b/init/reboot_test.cpp index 06702e3a6..b3d038d14 100644 --- a/init/reboot_test.cpp +++ b/init/reboot_test.cpp @@ -113,6 +113,11 @@ service $name /system/bin/yes } TEST_F(RebootTest, StopServicesSIGTERM) { + if (getuid() != 0) { + GTEST_SKIP() << "Skipping test, must be run as root."; + return; + } + AddTestService("A"); AddTestService("B"); @@ -148,6 +153,11 @@ TEST_F(RebootTest, StopServicesSIGTERM) { } TEST_F(RebootTest, StopServicesSIGKILL) { + if (getuid() != 0) { + GTEST_SKIP() << "Skipping test, must be run as root."; + return; + } + AddTestService("A"); AddTestService("B");