Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit fb3e64ab authored by Bart Van Assche's avatar Bart Van Assche
Browse files

service_test: Only run the ServiceStopTest as root 



Despite 'require_root: true' in Android.bp for CtsInitTestCases, it can
happen that this test is run without root privileges. If that happens,
skip the ServiceStopTest.

Bug: 313551148
Change-Id: I45f29369915a1f61e08ffd9bf5601a83a147597c
Signed-off-by: default avatarBart Van Assche <bvanassche@google.com>
parent 58beb4a6

init/service_test.cpp

+5 −0
Original line number Diff line number Diff line
@@ -236,6 +236,11 @@ class ServiceStopTest : public testing::TestWithParam<bool> {}; 
// Service::Stop() if their uid_%d/pid_%d cgroup directory got removed. This test, if run with the

// parameter set to 'true', verifies that such services are stopped.

TEST_P(ServiceStopTest, stop) {

    if (getuid() != 0) {

        GTEST_SKIP() << "Must be run as root.";

        return;

    }



    static constexpr std::string_view kServiceName = "ServiceA";

    static constexpr std::string_view kScriptTemplate = R"init(

service $name /system/bin/yes