Merge "Effect AIDL: Skipping vts test case if its not supported by effect" into main am: 4f2a709416 am: c39ed27778 am: 0fd7e58f72

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

Change-Id: I84bf447ac062dca950e76a735d5c91262ecbc976
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
This commit is contained in:
Treehugger Robot
2023-09-20 19:29:51 +00:00
committed by Automerger Merge Worker

View File

@@ -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));