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

Commit e1723a49 authored by Harpreet \"Eli\" Sangha's avatar Harpreet \"Eli\" Sangha
Browse files

vibrator: vts: Utilize New toString() Support 



Use the new toString() support for AIDL enums for improved logging.

Test: atest VtsHalVibratorTargetTest
Change-Id: Ie9d98f6b9f367dec9e6f0b9b0ed18dab0a32c742
Signed-off-by: default avatarHarpreet \"Eli\" Sangha <eliptus@google.com>
parent f78d7734

vibrator/aidl/vts/VtsHalVibratorTargetTest.cpp

+4 −5
Original line number Diff line number Diff line
@@ -135,13 +135,12 @@ TEST_P(VibratorAidl, ValidateEffect) { 
            Status status = vibrator->perform(effect, strength, nullptr /*callback*/, &lengthMs);



            if (isEffectSupported) {

                EXPECT_TRUE(status.isOk())

                        << static_cast<int>(effect) << " " << static_cast<int>(strength);

                EXPECT_TRUE(status.isOk()) << toString(effect) << " " << toString(strength);

                EXPECT_GT(lengthMs, 0);

                usleep(lengthMs * 1000);

            } else {

                EXPECT_EQ(status.exceptionCode(), Status::EX_UNSUPPORTED_OPERATION)

                        << static_cast<int>(effect) << " " << static_cast<int>(strength);

                        << toString(effect) << " " << toString(strength);

                EXPECT_EQ(lengthMs, 0);

            }

        }
@@ -202,7 +201,7 @@ TEST_P(VibratorAidl, InvalidEffectsUnsupported) { 
            int32_t lengthMs;

            Status status = vibrator->perform(effect, strength, nullptr /*callback*/, &lengthMs);

            EXPECT_EQ(status.exceptionCode(), Status::EX_UNSUPPORTED_OPERATION)

                    << static_cast<int>(effect) << " " << static_cast<int>(strength);

                    << toString(effect) << " " << toString(strength);

        }

    }

    for (Effect effect : kEffects) {
@@ -210,7 +209,7 @@ TEST_P(VibratorAidl, InvalidEffectsUnsupported) { 
            int32_t lengthMs;

            Status status = vibrator->perform(effect, strength, nullptr /*callback*/, &lengthMs);

            EXPECT_EQ(status.exceptionCode(), Status::EX_UNSUPPORTED_OPERATION)

                    << static_cast<int>(effect) << " " << static_cast<int>(strength);

                    << toString(effect) << " " << toString(strength);

        }

    }

}