summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
author Yifan Hong <elsk@google.com> 2022-01-14 04:19:23 +0000
committer Gerrit Code Review <noreply-gerritcodereview@google.com> 2022-01-14 04:19:23 +0000
commitbec007aad982c278fd655645559d28e9038ab0e1 (patch)
treebfd6f3d0ded6b93fae0246263edb22445da12392
parent48165b9c32c70d9bed1769d6d33173ae7a1bbda4 (diff)
parente3caaf2dca61716c450cc651b9c75bcfc87c3e0c (diff)
Merge "binderRpcTest: BinderRpc.Java skip on user builds."
-rw-r--r--libs/binder/tests/binderRpcTest.cpp7
1 files changed, 4 insertions, 3 deletions
diff --git a/libs/binder/tests/binderRpcTest.cpp b/libs/binder/tests/binderRpcTest.cpp
index ca68b99e35..c2639e7c67 100644
--- a/libs/binder/tests/binderRpcTest.cpp
+++ b/libs/binder/tests/binderRpcTest.cpp
@@ -1517,10 +1517,11 @@ TEST(BinderRpc, Java) {
auto keepAlive = sp<BBinder>::make();
auto setRpcClientDebugStatus = binder->setRpcClientDebug(std::move(socket), keepAlive);
- if (!android::base::GetBoolProperty("ro.debuggable", false)) {
+ if (!android::base::GetBoolProperty("ro.debuggable", false) ||
+ android::base::GetProperty("ro.build.type", "") == "user") {
ASSERT_EQ(INVALID_OPERATION, setRpcClientDebugStatus)
- << "setRpcClientDebug should return INVALID_OPERATION on non-debuggable builds, "
- "but get "
+ << "setRpcClientDebug should return INVALID_OPERATION on non-debuggable or user "
+ "builds, but get "
<< statusToString(setRpcClientDebugStatus);
GTEST_SKIP();
}