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

Commit 3816c808 authored by Treehugger Robot's avatar Treehugger Robot Committed by Gerrit Code Review
Browse files

Merge "audioeffect_tests : Rename method to better reflect return type" into main

parents 636d8a60 240242ac

media/libaudioclient/tests/audioeffect_tests.cpp

+21 −24
Original line number Original line Diff line number Diff line
@@ -70,7 +70,7 @@ sp<AudioEffect> createEffect(const effect_uuid_t* type, const effect_uuid_t* uui 
    return effect;

    return effect;

}

}





status_t isEffectExistsOnAudioSession(const effect_uuid_t* type, const effect_uuid_t* uuid,

status_t createAndInitCheckEffect(const effect_uuid_t* type, const effect_uuid_t* uuid,

                                  int priority, audio_session_t sessionId) {

                                  int priority, audio_session_t sessionId) {

    sp<AudioEffect> effect = createEffect(type, uuid, priority, sessionId);

    sp<AudioEffect> effect = createEffect(type, uuid, priority, sessionId);

    return effect->initCheck();

    return effect->initCheck();
@@ -272,8 +272,7 @@ TEST(AudioEffectTest, ManageSourceDefaultEffects) { 
    EXPECT_FALSE(isEffectDefaultOnRecord(selectedEffectType, selectedEffectUuid,

    EXPECT_FALSE(isEffectDefaultOnRecord(selectedEffectType, selectedEffectUuid,

                                         capture->getAudioRecordHandle()))

                                         capture->getAudioRecordHandle()))

            << "Effect should not have been default on record. " << type;

            << "Effect should not have been default on record. " << type;

    EXPECT_EQ(NO_ERROR,

    EXPECT_EQ(NO_ERROR, createAndInitCheckEffect(selectedEffectType, selectedEffectUuid,

              isEffectExistsOnAudioSession(selectedEffectType, selectedEffectUuid,

                                                 kDefaultInputEffectPriority - 1,

                                                 kDefaultInputEffectPriority - 1,

                                                 capture->getAudioRecordHandle()->getSessionId()))

                                                 capture->getAudioRecordHandle()->getSessionId()))

            << "Effect should not have been added. " << type;

            << "Effect should not have been added. " << type;
@@ -296,7 +295,7 @@ TEST(AudioEffectTest, ManageSourceDefaultEffects) { 
                                        capture->getAudioRecordHandle()))

                                        capture->getAudioRecordHandle()))

            << "Effect should have been default on record. " << type;

            << "Effect should have been default on record. " << type;

    EXPECT_EQ(ALREADY_EXISTS,

    EXPECT_EQ(ALREADY_EXISTS,

              isEffectExistsOnAudioSession(selectedEffectType, selectedEffectUuid,

              createAndInitCheckEffect(selectedEffectType, selectedEffectUuid,

                                       kDefaultInputEffectPriority - 1,

                                       kDefaultInputEffectPriority - 1,

                                       capture->getAudioRecordHandle()->getSessionId()))

                                       capture->getAudioRecordHandle()->getSessionId()))

            << "Effect should have been added. " << type;

            << "Effect should have been added. " << type;
@@ -313,8 +312,7 @@ TEST(AudioEffectTest, ManageSourceDefaultEffects) { 
    EXPECT_FALSE(isEffectDefaultOnRecord(selectedEffectType, selectedEffectUuid,

    EXPECT_FALSE(isEffectDefaultOnRecord(selectedEffectType, selectedEffectUuid,

                                         capture->getAudioRecordHandle()))

                                         capture->getAudioRecordHandle()))

            << "Effect should not have been default on record. " << type;

            << "Effect should not have been default on record. " << type;

    EXPECT_EQ(NO_ERROR,

    EXPECT_EQ(NO_ERROR, createAndInitCheckEffect(selectedEffectType, selectedEffectUuid,

              isEffectExistsOnAudioSession(selectedEffectType, selectedEffectUuid,

                                                 kDefaultInputEffectPriority - 1,

                                                 kDefaultInputEffectPriority - 1,

                                                 capture->getAudioRecordHandle()->getSessionId()))

                                                 capture->getAudioRecordHandle()->getSessionId()))

            << "Effect should not have been added. " << type;

            << "Effect should not have been added. " << type;
@@ -421,7 +419,7 @@ TEST_P(AudioPlaybackEffectTest, StreamDefaultEffectTest) { 
    EXPECT_EQ(NO_ERROR, playback->create());

    EXPECT_EQ(NO_ERROR, playback->create());

    EXPECT_EQ(NO_ERROR, playback->start());

    EXPECT_EQ(NO_ERROR, playback->start());

    EXPECT_EQ(compatCheck ? NO_ERROR : NO_INIT,

    EXPECT_EQ(compatCheck ? NO_ERROR : NO_INIT,

              isEffectExistsOnAudioSession(&mType, &mUuid, kDefaultOutputEffectPriority - 1,

              createAndInitCheckEffect(&mType, &mUuid, kDefaultOutputEffectPriority - 1,

                                       playback->getAudioTrackHandle()->getSessionId()))

                                       playback->getAudioTrackHandle()->getSessionId()))

            << "Effect should not have been added. " << mTypeStr;

            << "Effect should not have been added. " << mTypeStr;

    EXPECT_EQ(NO_ERROR, playback->waitForConsumption());

    EXPECT_EQ(NO_ERROR, playback->waitForConsumption());
@@ -445,7 +443,7 @@ TEST_P(AudioPlaybackEffectTest, StreamDefaultEffectTest) { 
    EXPECT_EQ(NO_ERROR, playback->start());

    EXPECT_EQ(NO_ERROR, playback->start());

    // If effect chosen is not compatible with the session, then effect won't be applied

    // If effect chosen is not compatible with the session, then effect won't be applied

    EXPECT_EQ(compatCheck ? ALREADY_EXISTS : NO_INIT,

    EXPECT_EQ(compatCheck ? ALREADY_EXISTS : NO_INIT,

              isEffectExistsOnAudioSession(&mType, &mUuid, kDefaultOutputEffectPriority - 1,

              createAndInitCheckEffect(&mType, &mUuid, kDefaultOutputEffectPriority - 1,

                                       playback->getAudioTrackHandle()->getSessionId()))

                                       playback->getAudioTrackHandle()->getSessionId()))

            << "Effect should have been added. " << mTypeStr;

            << "Effect should have been added. " << mTypeStr;

    EXPECT_EQ(NO_ERROR, playback->waitForConsumption());

    EXPECT_EQ(NO_ERROR, playback->waitForConsumption());
@@ -467,7 +465,7 @@ TEST_P(AudioPlaybackEffectTest, StreamDefaultEffectTest) { 
    EXPECT_EQ(NO_ERROR, playback->create());

    EXPECT_EQ(NO_ERROR, playback->create());

    EXPECT_EQ(NO_ERROR, playback->start());

    EXPECT_EQ(NO_ERROR, playback->start());

    EXPECT_EQ(compatCheck ? NO_ERROR : NO_INIT,

    EXPECT_EQ(compatCheck ? NO_ERROR : NO_INIT,

              isEffectExistsOnAudioSession(&mType, &mUuid, kDefaultOutputEffectPriority - 1,

              createAndInitCheckEffect(&mType, &mUuid, kDefaultOutputEffectPriority - 1,

                                       playback->getAudioTrackHandle()->getSessionId()))

                                       playback->getAudioTrackHandle()->getSessionId()))

            << "Effect should not have been added. " << mTypeStr;

            << "Effect should not have been added. " << mTypeStr;

    EXPECT_EQ(NO_ERROR, playback->waitForConsumption());

    EXPECT_EQ(NO_ERROR, playback->waitForConsumption());
@@ -502,8 +500,8 @@ TEST_P(AudioPlaybackEffectTest, CheckOutputFlagCompatibility) { 
    EXPECT_EQ(NO_ERROR, playback->create());

    EXPECT_EQ(NO_ERROR, playback->create());

    EXPECT_EQ(NO_ERROR, playback->start());

    EXPECT_EQ(NO_ERROR, playback->start());





    EXPECT_EQ(ALREADY_EXISTS, isEffectExistsOnAudioSession(

    EXPECT_EQ(ALREADY_EXISTS,

                                      &mType, &mUuid, kDefaultOutputEffectPriority - 1, sessionId))

              createAndInitCheckEffect(&mType, &mUuid, kDefaultOutputEffectPriority - 1, sessionId))

            << "Effect should have been added. " << mTypeStr;

            << "Effect should have been added. " << mTypeStr;

    if (mSelectFastMode) {

    if (mSelectFastMode) {

        EXPECT_EQ(mIsFastCompatibleEffect ? AUDIO_OUTPUT_FLAG_FAST : 0,

        EXPECT_EQ(mIsFastCompatibleEffect ? AUDIO_OUTPUT_FLAG_FAST : 0,
@@ -556,8 +554,7 @@ TEST(AudioEffectTest, TestHapticEffect) { 
    ASSERT_EQ(NO_ERROR, playback->loadResource("/data/local/tmp/bbb_2ch_24kHz_s16le.raw"));

    ASSERT_EQ(NO_ERROR, playback->loadResource("/data/local/tmp/bbb_2ch_24kHz_s16le.raw"));

    EXPECT_EQ(NO_ERROR, playback->create());

    EXPECT_EQ(NO_ERROR, playback->create());

    EXPECT_EQ(NO_ERROR, playback->start());

    EXPECT_EQ(NO_ERROR, playback->start());

    ASSERT_EQ(ALREADY_EXISTS,

    ASSERT_EQ(ALREADY_EXISTS, createAndInitCheckEffect(selectedEffectType, selectedEffectUuid,

              isEffectExistsOnAudioSession(selectedEffectType, selectedEffectUuid,

                                                       kDefaultOutputEffectPriority - 1, sessionId))

                                                       kDefaultOutputEffectPriority - 1, sessionId))

            << "Effect should have been added. " << type;

            << "Effect should have been added. " << type;

    EXPECT_EQ(NO_ERROR, playback->waitForConsumption());

    EXPECT_EQ(NO_ERROR, playback->waitForConsumption());