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

Commit 45d9baa3 authored by Bernie Innocenti's avatar Bernie Innocenti
Browse files

Explicitly require root privs for tests requiring it

These tests currently require root privs to interact with netd:
  resolv_integration_test
  libnetdbpf_test
  netd_unit_test
  netd_integration_test

Bug: 134891412
Test: atest
Change-Id: I3e286fd7e7603e30b4f4d655fe311c35e05fbc78
parent ffe325ca
Loading
Loading
Loading
Loading
+1 −0
Original line number Diff line number Diff line
@@ -142,6 +142,7 @@ cc_library_static {
cc_test {
    name: "resolv_integration_test",
    test_suites: ["device-tests"],
    require_root: true,
    defaults: ["netd_defaults"],
    srcs: [
        "dns_responder/dns_responder.cpp",
+3 −0
Original line number Diff line number Diff line
@@ -156,6 +156,9 @@ void DnsResponderClient::SetUp() {
    // binder setup
    auto binder = android::defaultServiceManager()->getService(android::String16("netd"));
    mNetdSrv = android::interface_cast<android::net::INetd>(binder);
    if (mNetdSrv == nullptr) {
        LOG(FATAL) << "Can't connect to service 'netd'. Missing root privileges? uid=" << getuid();
    }

    auto resolvBinder =
            android::defaultServiceManager()->getService(android::String16("dnsresolver"));
+1 −0
Original line number Diff line number Diff line
@@ -67,6 +67,7 @@ class DnsResolverBinderTest : public ::testing::Test {
        if (binder != nullptr) {
            mDnsResolver = android::interface_cast<IDnsResolver>(binder);
        }
        // This could happen when the test isn't running as root, or if netd isn't running.
        assert(nullptr != mDnsResolver.get());
        // Create cache for test
        mDnsResolver->createNetworkCache(TEST_NETID);