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

Commit fe4b83f1 authored by Yifan Hong's avatar Yifan Hong
Browse files

binder: skip BinderRpc.Java test on non-debuggable builds. 

Test: TH
Fixes: 205512685
Change-Id: I1bedfbdf103faf139d54f9a4e6f28b70bcf3c9c9
parent 8347ab59

libs/binder/tests/binderRpcTest.cpp

+12 −1
Original line number Diff line number Diff line
@@ -22,6 +22,7 @@ 
#include <aidl/IBinderRpcTest.h>

#include <android-base/file.h>

#include <android-base/logging.h>

#include <android-base/properties.h>

#include <android/binder_auto_utils.h>

#include <android/binder_libbinder.h>

#include <binder/Binder.h>
@@ -1422,7 +1423,17 @@ TEST(BinderRpc, Java) { 
    auto socket = rpcServer->releaseServer();



    auto keepAlive = sp<BBinder>::make();

    ASSERT_EQ(OK, binder->setRpcClientDebug(std::move(socket), keepAlive));

    auto setRpcClientDebugStatus = binder->setRpcClientDebug(std::move(socket), keepAlive);



    if (!android::base::GetBoolProperty("ro.debuggable", false)) {

        ASSERT_EQ(INVALID_OPERATION, setRpcClientDebugStatus)

                << "setRpcClientDebug should return INVALID_OPERATION on non-debuggable builds, "

                   "but get "

                << statusToString(setRpcClientDebugStatus);

        GTEST_SKIP();

    }



    ASSERT_EQ(OK, setRpcClientDebugStatus);



    auto rpcSession = RpcSession::make();

    ASSERT_EQ(OK, rpcSession->setupInetClient("127.0.0.1", port));