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

Commit 1755cbba authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "Effect AIDL: Skipping vts test case if its not supported by effect"... 

Merge "Effect AIDL: Skipping vts test case if its not supported by effect" into main am: 4f2a7094 am: c39ed277 am: 0fd7e58f

Original change: https://android-review.googlesource.com/c/platform/hardware/interfaces/+/2755205



Change-Id: I84bf447ac062dca950e76a735d5c91262ecbc976
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents d9e285c0 0fd7e58f

audio/aidl/vts/VtsHalAudioEffectTargetTest.cpp

+20 −0
Original line number Original line Diff line number Diff line
@@ -504,6 +504,11 @@ TEST_P(AudioEffectTest, SetAndGetParameterAfterReset) { 




// Set and get AudioDeviceDescription in Parameter

// Set and get AudioDeviceDescription in Parameter

TEST_P(AudioEffectTest, SetAndGetParameterDeviceDescription) {

TEST_P(AudioEffectTest, SetAndGetParameterDeviceDescription) {

    if (!mDescriptor.common.flags.deviceIndication) {

        GTEST_SKIP() << "Skipping test as effect does not support deviceIndication"

                     << mDescriptor.common.flags.toString();

    }



    ASSERT_NO_FATAL_FAILURE(create(mFactory, mEffect, mDescriptor));

    ASSERT_NO_FATAL_FAILURE(create(mFactory, mEffect, mDescriptor));

    ASSERT_NO_FATAL_FAILURE(open(mEffect));

    ASSERT_NO_FATAL_FAILURE(open(mEffect));
@@ -527,6 +532,11 @@ TEST_P(AudioEffectTest, SetAndGetParameterDeviceDescription) { 




// Set and get AudioMode in Parameter

// Set and get AudioMode in Parameter

TEST_P(AudioEffectTest, SetAndGetParameterAudioMode) {

TEST_P(AudioEffectTest, SetAndGetParameterAudioMode) {

    if (!mDescriptor.common.flags.audioModeIndication) {

        GTEST_SKIP() << "Skipping test as effect does not support audioModeIndication"

                     << mDescriptor.common.flags.toString();

    }



    ASSERT_NO_FATAL_FAILURE(create(mFactory, mEffect, mDescriptor));

    ASSERT_NO_FATAL_FAILURE(create(mFactory, mEffect, mDescriptor));

    ASSERT_NO_FATAL_FAILURE(open(mEffect));

    ASSERT_NO_FATAL_FAILURE(open(mEffect));
@@ -547,6 +557,11 @@ TEST_P(AudioEffectTest, SetAndGetParameterAudioMode) { 




// Set and get AudioSource in Parameter

// Set and get AudioSource in Parameter

TEST_P(AudioEffectTest, SetAndGetParameterAudioSource) {

TEST_P(AudioEffectTest, SetAndGetParameterAudioSource) {

    if (!mDescriptor.common.flags.audioSourceIndication) {

        GTEST_SKIP() << "Skipping test as effect does not support audioSourceIndication"

                     << mDescriptor.common.flags.toString();

    }



    ASSERT_NO_FATAL_FAILURE(create(mFactory, mEffect, mDescriptor));

    ASSERT_NO_FATAL_FAILURE(create(mFactory, mEffect, mDescriptor));

    ASSERT_NO_FATAL_FAILURE(open(mEffect));

    ASSERT_NO_FATAL_FAILURE(open(mEffect));
@@ -567,6 +582,11 @@ TEST_P(AudioEffectTest, SetAndGetParameterAudioSource) { 




// Set and get VolumeStereo in Parameter

// Set and get VolumeStereo in Parameter

TEST_P(AudioEffectTest, SetAndGetParameterVolume) {

TEST_P(AudioEffectTest, SetAndGetParameterVolume) {

    if (mDescriptor.common.flags.volume == Flags::Volume::NONE) {

        GTEST_SKIP() << "Skipping test as effect does not support volume"

                     << mDescriptor.common.flags.toString();

    }



    ASSERT_NO_FATAL_FAILURE(create(mFactory, mEffect, mDescriptor));

    ASSERT_NO_FATAL_FAILURE(create(mFactory, mEffect, mDescriptor));

    ASSERT_NO_FATAL_FAILURE(open(mEffect));

    ASSERT_NO_FATAL_FAILURE(open(mEffect));