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

Commit 29db860b authored by Devin Moore's avatar Devin Moore
Browse files

Change binderRpcTest to use 'activity' for a Java test service 

The service just needs to exist on all devices that this test runs on.
We recently hit an issue with 'batteryproperties' not existing. The
'activity' service is likely on more devices.

Test: atest binderRpcTest
Bug: 359783809
Change-Id: I71fea4daeb929971add9eefa96e3d48f815e2212
parent 187efe18

libs/binder/tests/binderRpcTest.cpp

+2 −2
Original line number Original line Diff line number Diff line
@@ -1384,8 +1384,8 @@ TEST(BinderRpc, Java) { 
    sp<IServiceManager> sm = defaultServiceManager();

    sp<IServiceManager> sm = defaultServiceManager();

    ASSERT_NE(nullptr, sm);

    ASSERT_NE(nullptr, sm);

    // Any Java service with non-empty getInterfaceDescriptor() would do.

    // Any Java service with non-empty getInterfaceDescriptor() would do.

    // Let's pick batteryproperties.

    // Let's pick activity.

    auto binder = sm->checkService(String16("batteryproperties"));

    auto binder = sm->checkService(String16("activity"));

    ASSERT_NE(nullptr, binder);

    ASSERT_NE(nullptr, binder);

    auto descriptor = binder->getInterfaceDescriptor();

    auto descriptor = binder->getInterfaceDescriptor();

    ASSERT_GE(descriptor.size(), 0u);

    ASSERT_GE(descriptor.size(), 0u);