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

Commit 1e7e982a authored by Kevin Rocard's avatar Kevin Rocard Committed by Keun Soo Yim
Browse files

Audio VTS: HAL can support more than the native sampling rates 



getSupportedSampleRate should return the native sampling rates,
(IE. the sampling rates that can be played without resampling)
but other sampling rates can be supported by the HAL.

The test was too strict as it was failing if HALs were supporting more
sample rates than there native (optimized) ones.

For example, a HAL might have its best performance (no resampling)
on 48kHz but still support 16kHz through resampling.

Note: getSupportedSampleRate might be renamed to getNativeSampleRate in
the next major HAL revision to avoid ambiguity.

Test: vts-tradefed run commandAndExit vts --module VtsHalAudioV2_0Target -t CheckConfig.audioPolicyConfigurationValidation
Bug: 69811500
Merged-In: I1ec1ce422bc5039637463c6641060508f4ee892b
Change-Id: I1ec1ce422bc5039637463c6641060508f4ee892b
Signed-off-by: default avatarKevin Rocard <krocard@google.com>
(cherry picked from commit 02025847)
parent eb488fad

audio/2.0/vts/functional/AudioPrimaryHidlHalTest.cpp

+17 −13
Original line number Diff line number Diff line
@@ -768,11 +768,12 @@ TEST_IO_STREAM(GetBufferSize, 
               ASSERT_GE(extract(stream->getBufferSize()),

                         extract(stream->getFrameSize())));



template <class Property, class CapabilityGetter, class Getter, class Setter>

template <class Property, class CapabilityGetter>

static void testCapabilityGetter(const string& name, IStream* stream,

                                 Property currentValue,

                                 CapabilityGetter capablityGetter,

                                 Getter getter, Setter setter) {

                                 Return<Property> (IStream::*getter)(),

                                 Return<Result> (IStream::*setter)(Property),

                                 bool currentMustBeSupported = true) {

    hidl_vec<Property> capabilities;

    ASSERT_OK((stream->*capablityGetter)(returnIn(capabilities)));

    if (capabilities.size() == 0) {
@@ -783,12 +784,14 @@ static void testCapabilityGetter(const string& name, IStream* stream, 
        doc::partialTest(name + " is not supported");

        return;

    };

    // TODO: This code has never been tested on a hal that supports

    // getSupportedSampleRates



    if (currentMustBeSupported) {

        Property currentValue = extract((stream->*getter)());

        EXPECT_NE(std::find(capabilities.begin(), capabilities.end(), currentValue),

                  capabilities.end())

            << "current " << name << " is not in the list of the supported ones "

            << toString(capabilities);

    }



    // Check that all declared supported values are indeed supported

    for (auto capability : capabilities) {
@@ -800,15 +803,17 @@ static void testCapabilityGetter(const string& name, IStream* stream, 
TEST_IO_STREAM(SupportedSampleRate,

               "Check that the stream sample rate is declared as supported",

               testCapabilityGetter("getSupportedSampleRate", stream.get(),

                                    extract(stream->getSampleRate()),

                                    &IStream::getSupportedSampleRates,

                                    &IStream::getSampleRate,

                                    &IStream::setSampleRate))

                                    &IStream::setSampleRate,

                                    // getSupportedSampleRate returns the native sampling rates,

                                    // (the sampling rates that can be played without resampling)

                                    // but other sampling rates can be supported by the HAL.

                                    false))



TEST_IO_STREAM(SupportedChannelMask,

               "Check that the stream channel mask is declared as supported",

               testCapabilityGetter("getSupportedChannelMask", stream.get(),

                                    extract(stream->getChannelMask()),

                                    &IStream::getSupportedChannelMasks,

                                    &IStream::getChannelMask,

                                    &IStream::setChannelMask))
@@ -816,7 +821,6 @@ TEST_IO_STREAM(SupportedChannelMask, 
TEST_IO_STREAM(SupportedFormat,

               "Check that the stream format is declared as supported",

               testCapabilityGetter("getSupportedFormat", stream.get(),

                                    extract(stream->getFormat()),

                                    &IStream::getSupportedFormats,

                                    &IStream::getFormat, &IStream::setFormat))