commit 7de36aaef8c5fd507a5bbd7ba923f8a3dc2b1913
author Treehugger Robot <treehugger-gerrit@google.com> Tue Dec 20 06:09:42 2022 +0000
committer Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com> Tue Dec 20 06:09:42 2022 +0000
tree 6ace5a7e47a6613b5d0ee9f4fe9c46eebc523517
parent 0ec3ca466655e87126167e6e348e57381a62889f
parent 2b89b4230842aa681f689b2640308aed443001de

Merge "audio VTS: Exclude '1' from 'invalid' buffer size list" am: 6109a9c6bf am: 2b89b42308

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

Change-Id: I69eaffe623a394aceebd75b5b859deb23daf73de
Signed-off-by: Automerger Merge Worker <android-build-automerger-merge-worker@system.gserviceaccount.com>

audio/aidl/vts/VtsHalAudioCoreModuleTargetTest.cpp

diff --git a/audio/aidl/vts/VtsHalAudioCoreModuleTargetTest.cpp b/audio/aidl/vts/VtsHalAudioCoreModuleTargetTest.cpp
index 4e3f525..2508afd 100644
--- a/audio/aidl/vts/VtsHalAudioCoreModuleTargetTest.cpp
+++ b/audio/aidl/vts/VtsHalAudioCoreModuleTargetTest.cpp
@@ -1926,9 +1926,7 @@
         }
         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());