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

Commit c5784c14 authored by Mikhail Naganov's avatar Mikhail Naganov
Browse files

audio: Disable I/O tests for input MMap streams 

Due to practical difficulties with implementing MMap streams
according to the 202404 VSR, skip I/O tests for them.

Bug: 274456992
Bug: 350998390
Bug: 363139283
Bug: 364975627
Bug: 365876441
Test: atest VtsHalAudioCoreTargetTest
Change-Id: I2c9e08b29e6a12fb873a7629072c5ed808bba6e3
parent 704df561

audio/aidl/vts/VtsHalAudioCoreModuleTargetTest.cpp

+6 −11
Original line number Diff line number Diff line
@@ -3074,8 +3074,8 @@ std::shared_ptr<StateSequence> makeBurstCommands(bool isSync); 
static bool skipStreamIoTestForMixPortConfig(const AudioPortConfig& portConfig) {

    return (portConfig.flags.value().getTag() == AudioIoFlags::input &&

            isAnyBitPositionFlagSet(portConfig.flags.value().template get<AudioIoFlags::input>(),

                                    {AudioInputFlags::VOIP_TX, AudioInputFlags::HW_HOTWORD,

                                     AudioInputFlags::HOTWORD_TAP})) ||

                                    {AudioInputFlags::MMAP_NOIRQ, AudioInputFlags::VOIP_TX,

                                     AudioInputFlags::HW_HOTWORD, AudioInputFlags::HOTWORD_TAP})) ||

           (portConfig.flags.value().getTag() == AudioIoFlags::output &&

            isAnyBitPositionFlagSet(

                    portConfig.flags.value().template get<AudioIoFlags::output>(),
@@ -3133,11 +3133,8 @@ class StreamFixtureWithWorker { 
        EXPECT_FALSE(mWorker->hasError()) << mWorker->getError();

        EXPECT_EQ("", mWorkerDriver->getUnexpectedStateTransition());

        if (validatePosition) {

            if (IOTraits<Stream>::is_input &&

                !mStream->getStreamContext()->isMmapped() /*TODO(b/274456992) remove*/) {

            EXPECT_TRUE(mWorkerDriver->hasObservablePositionIncrease());

            EXPECT_TRUE(mWorkerDriver->hasHardwarePositionIncrease());

            }

            EXPECT_FALSE(mWorkerDriver->hasObservableRetrogradePosition());

            EXPECT_FALSE(mWorkerDriver->hasHardwareRetrogradePosition());

        }
@@ -4099,8 +4096,7 @@ class AudioStreamIo : public AudioCoreModuleBase, 
        EXPECT_FALSE(worker.hasError()) << worker.getError();

        EXPECT_EQ("", driver.getUnexpectedStateTransition());

        if (ValidatePosition(stream.getDevice())) {

            if (validatePositionIncrease &&

                !stream.getStreamContext()->isMmapped() /*TODO(b/274456992) remove*/) {

            if (validatePositionIncrease) {

                EXPECT_TRUE(driver.hasObservablePositionIncrease());

                EXPECT_TRUE(driver.hasHardwarePositionIncrease());

            }
@@ -4134,8 +4130,7 @@ class AudioStreamIo : public AudioCoreModuleBase, 
        EXPECT_FALSE(worker.hasError()) << worker.getError();

        EXPECT_EQ("", driver.getUnexpectedStateTransition());

        if (ValidatePosition(stream.getDevice())) {

            if (validatePositionIncrease &&

                !stream.getStreamContext()->isMmapped() /*TODO(b/274456992) remove*/) {

            if (validatePositionIncrease) {

                EXPECT_TRUE(driver.hasObservablePositionIncrease());

                EXPECT_TRUE(driver.hasHardwarePositionIncrease());

            }