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

Commit 2b89b423 authored by Treehugger Robot's avatar Treehugger Robot Committed by Automerger Merge Worker
Browse files

Merge "audio VTS: Exclude '1' from 'invalid' buffer size list" am: 6109a9c6 

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



Change-Id: I5a94268a573fac931287817a29746b6a1526a3ea
Signed-off-by: default avatarAutomerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>
parents a1b3286a 6109a9c6

audio/aidl/vts/VtsHalAudioCoreModuleTargetTest.cpp

+1 −3
Original line number Diff line number Diff line
@@ -1926,9 +1926,7 @@ class AudioStream : public AudioCoreModule { 
        }

        WithStream<Stream> stream(portConfig.value());

        ASSERT_NO_FATAL_FAILURE(stream.SetUpPortConfig(module.get()));

        // The buffer size of 1 frame should be impractically small, and thus

        // less than any minimum buffer size suggested by any HAL.

        for (long bufferSize : std::array<long, 4>{-1, 0, 1, std::numeric_limits<long>::max()}) {

        for (long bufferSize : std::array<long, 3>{-1, 0, std::numeric_limits<long>::max()}) {

            EXPECT_STATUS(EX_ILLEGAL_ARGUMENT, stream.SetUpNoChecks(module.get(), bufferSize))

                    << "for the buffer size " << bufferSize;

            EXPECT_EQ(nullptr, stream.get());