summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorElie Kheirallah <khei@google.com>2022-05-03 18:01:43 +0000
committerElie Kheirallah <khei@google.com>2022-05-03 18:18:47 +0000
commitb7246642bee083cbd415ce731c14955735fafcee (patch)
tree0e713498ecf8f0b93e94da1b7289eb30d075008f
parent27590051170905e2d1792e8958d3a6b72e47acf5 (diff)
downloadnative-b7246642bee083cbd415ce731c14955735fafcee.tar.gz
Removed unnecessary check. Replaced EXPECT_TRUE with EXPECT_NE.
Test: binderLibTest Change-Id: I8332dcd17a1e9ee66db6324f9c40cfc72468e279
-rw-r--r--libs/binder/tests/binderLibTest.cpp6
1 files changed, 2 insertions, 4 deletions
diff --git a/libs/binder/tests/binderLibTest.cpp b/libs/binder/tests/binderLibTest.cpp
index b1e17b7892..57a1fda95f 100644
--- a/libs/binder/tests/binderLibTest.cpp
+++ b/libs/binder/tests/binderLibTest.cpp
@@ -247,13 +247,11 @@ class BinderLibTest : public ::testing::Test {
{
int32_t id;
Parcel data, reply;
- sp<IBinder> binder;
EXPECT_THAT(m_server->transact(code, data, &reply), StatusEq(NO_ERROR));
- EXPECT_FALSE(binder != nullptr);
- binder = reply.readStrongBinder();
- EXPECT_TRUE(binder != nullptr);
+ sp<IBinder> binder = reply.readStrongBinder();
+ EXPECT_NE(nullptr, binder);
EXPECT_THAT(reply.readInt32(&id), StatusEq(NO_ERROR));
if (idPtr)
*idPtr = id;