diff --git a/audio/core/all-versions/vts/functional/AudioPrimaryHidlHalTest.h b/audio/core/all-versions/vts/functional/AudioPrimaryHidlHalTest.h index 6fc9339c54..38a06c4740 100644 --- a/audio/core/all-versions/vts/functional/AudioPrimaryHidlHalTest.h +++ b/audio/core/all-versions/vts/functional/AudioPrimaryHidlHalTest.h @@ -717,12 +717,6 @@ INSTANTIATE_TEST_CASE_P( ::testing::ValuesIn(ConfigHelper::getRequiredSupportCaptureAudioConfig()), ::testing::Values(AudioInputFlag::NONE)), &DeviceConfigParameterToString); -INSTANTIATE_TEST_CASE_P( - SupportedInputBufferSize, RequiredInputBufferSizeTest, - ::testing::Combine(::testing::ValuesIn(getDeviceParameters()), - ::testing::ValuesIn(ConfigHelper::getSupportedCaptureAudioConfig()), - ::testing::Values(AudioInputFlag::NONE)), - &DeviceConfigParameterToString); INSTANTIATE_TEST_CASE_P( RecommendedCaptureAudioConfigSupport, OptionalInputBufferSizeTest, ::testing::Combine( @@ -930,12 +924,6 @@ INSTANTIATE_TEST_CASE_P( ::testing::ValuesIn(ConfigHelper::getRequiredSupportPlaybackAudioConfig()), ::testing::Values(AudioOutputFlag::NONE)), &DeviceConfigParameterToString); -INSTANTIATE_TEST_CASE_P( - SupportedOutputStreamConfig, OutputStreamTest, - ::testing::Combine(::testing::ValuesIn(getDeviceParameters()), - ::testing::ValuesIn(ConfigHelper::getSupportedPlaybackAudioConfig()), - ::testing::Values(AudioOutputFlag::NONE)), - &DeviceConfigParameterToString); INSTANTIATE_TEST_CASE_P( RecommendedOutputStreamConfigSupport, OutputStreamTest, ::testing::Combine( @@ -946,7 +934,7 @@ INSTANTIATE_TEST_CASE_P( #elif MAJOR_VERSION >= 6 // For V6 and above test according to the audio policy manager configuration. // This is more correct as CDD is written from the apps perspective. -// Audio system provides necessary format conversions for the missing configurations. +// Audio system provides necessary format conversions for missing configurations. INSTANTIATE_TEST_CASE_P(DeclaredOutputStreamConfigSupport, OutputStreamTest, ::testing::ValuesIn(getOutputDeviceConfigParameters()), &DeviceConfigParameterToString); @@ -991,12 +979,6 @@ INSTANTIATE_TEST_CASE_P( ::testing::ValuesIn(ConfigHelper::getRequiredSupportCaptureAudioConfig()), ::testing::Values(AudioInputFlag::NONE)), &DeviceConfigParameterToString); -INSTANTIATE_TEST_CASE_P( - SupportedInputStreamConfig, InputStreamTest, - ::testing::Combine(::testing::ValuesIn(getDeviceParameters()), - ::testing::ValuesIn(ConfigHelper::getSupportedCaptureAudioConfig()), - ::testing::Values(AudioInputFlag::NONE)), - &DeviceConfigParameterToString); INSTANTIATE_TEST_CASE_P( RecommendedInputStreamConfigSupport, InputStreamTest, ::testing::Combine( @@ -1007,7 +989,7 @@ INSTANTIATE_TEST_CASE_P( #elif MAJOR_VERSION >= 6 // For V6 and above test according to the audio policy manager configuration. // This is more correct as CDD is written from the apps perspective. -// Audio system provides necessary format conversions for the missing configurations. +// Audio system provides necessary format conversions for missing configurations. INSTANTIATE_TEST_CASE_P(DeclaredInputStreamConfigSupport, InputStreamTest, ::testing::ValuesIn(getInputDeviceConfigParameters()), &DeviceConfigParameterToString); diff --git a/audio/core/all-versions/vts/functional/ConfigHelper.h b/audio/core/all-versions/vts/functional/ConfigHelper.h index 48aae8c5b3..a2f4116baa 100644 --- a/audio/core/all-versions/vts/functional/ConfigHelper.h +++ b/audio/core/all-versions/vts/functional/ConfigHelper.h @@ -57,12 +57,6 @@ struct ConfigHelper { {24000, 48000}, {AudioFormat::PCM_16_BIT}); } - static const vector getSupportedPlaybackAudioConfig() { - // TODO: retrieve audio config supported by the platform - // as declared in the policy configuration - return {}; - } - static const vector getRequiredSupportCaptureAudioConfig() { if (!primaryHasMic()) return {}; return combineAudioConfig({AudioChannelMask::IN_MONO}, {8000, 11025, 16000, 44100}, @@ -73,11 +67,6 @@ struct ConfigHelper { return combineAudioConfig({AudioChannelMask::IN_STEREO}, {22050, 48000}, {AudioFormat::PCM_16_BIT}); } - static const vector getSupportedCaptureAudioConfig() { - // TODO: retrieve audio config supported by the platform - // as declared in the policy configuration - return {}; - } static vector combineAudioConfig(vector channelMasks, vector sampleRates,