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

Commit b7246642 authored by Elie Kheirallah's avatar Elie Kheirallah
Browse files

Removed unnecessary check. Replaced EXPECT_TRUE with EXPECT_NE. 

Test: binderLibTest
Change-Id: I8332dcd17a1e9ee66db6324f9c40cfc72468e279
parent 27590051

libs/binder/tests/binderLibTest.cpp

+2 −4
Original line number Original line Diff line number Diff line
@@ -247,13 +247,11 @@ class BinderLibTest : public ::testing::Test { 
        {

        {

            int32_t id;

            int32_t id;

            Parcel data, reply;

            Parcel data, reply;

            sp<IBinder> binder;





            EXPECT_THAT(m_server->transact(code, data, &reply), StatusEq(NO_ERROR));

            EXPECT_THAT(m_server->transact(code, data, &reply), StatusEq(NO_ERROR));





            EXPECT_FALSE(binder != nullptr);

            sp<IBinder> binder = reply.readStrongBinder();

            binder = reply.readStrongBinder();

            EXPECT_NE(nullptr, binder);

            EXPECT_TRUE(binder != nullptr);

            EXPECT_THAT(reply.readInt32(&id), StatusEq(NO_ERROR));

            EXPECT_THAT(reply.readInt32(&id), StatusEq(NO_ERROR));

            if (idPtr)

            if (idPtr)

                *idPtr = id;

                *idPtr = id;