From f234e45b208a33132fb34092a42c7d4ec32a1b0f Mon Sep 17 00:00:00 2001 From: Pierre Lee Date: Mon, 14 Jan 2019 22:49:04 +0800 Subject: [PATCH] The test failed when one of the three cameras was configured with a maximum android.scaler.availableStreamConfiguration size of 1600*1200 VTS gets the name of all the cameras in the machine and check each camera, When traversal to the configuration reference of 1600*1200 camera, the outputPreviewStreams variable does not clear, save the value of the Previous camera The outputPreviewStreams variable needs to be cleared when check next camera Bug: 122806546 Test: ConfigureStreamsWithSessionParameters can pass when one of the three cameras was configured with a maximum android.scaler.availableStreamConfiguration size of 1600*1200 Change-Id: Ia4845d43871730e14b5ba1411ce72f8c4bb69042 --- .../2.4/vts/functional/VtsHalCameraProviderV2_4TargetTest.cpp | 2 ++ 1 file changed, 2 insertions(+) diff --git a/camera/provider/2.4/vts/functional/VtsHalCameraProviderV2_4TargetTest.cpp b/camera/provider/2.4/vts/functional/VtsHalCameraProviderV2_4TargetTest.cpp index 233c5d22cf..22b738256d 100644 --- a/camera/provider/2.4/vts/functional/VtsHalCameraProviderV2_4TargetTest.cpp +++ b/camera/provider/2.4/vts/functional/VtsHalCameraProviderV2_4TargetTest.cpp @@ -2896,6 +2896,8 @@ TEST_F(CameraHidlTest, configureStreamsWithSessionParameters) { continue; } + outputPreviewStreams.clear(); + ASSERT_EQ(Status::OK, getAvailableOutputStreams(staticMetaBuffer, outputPreviewStreams, &previewThreshold)); ASSERT_NE(0u, outputPreviewStreams.size());