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

Commit 4aefd1c1 authored by Kevin Rocard's avatar Kevin Rocard
Browse files

Audio HAL VTS: Getter test assert logic was incorrect 



Test: vts-tradefed run vts --module VtsHalAudioV2_0Target
Test: call/play music/record/video...
Bug: 36311550
Change-Id: Iaf2d71829a15b12dcf56e825773c8a697896a264
Signed-off-by: default avatarKevin Rocard <krocard@google.com>
parent 5e5783da

audio/2.0/vts/functional/AudioPrimaryHidlHalTest.cpp

+4 −2
Original line number Diff line number Diff line
@@ -594,7 +594,7 @@ class OpenStreamTest : public AudioConfigPrimaryTest, 
                               returnIn(res, stream, suggestedConfigRetry)));

                // This time it must succeed

                ASSERT_OK(res);

                ASSERT_TRUE(stream == nullptr);

                ASSERT_TRUE(stream != nullptr);

                audioConfig = suggestedConfig;

                break;

            default:
@@ -895,7 +895,9 @@ static void checkGetNoParameter(IStream* stream, hidl_vec<hidl_string> keys, 
    ASSERT_OK(stream->getParameters(keys, returnIn(res, parameters)));

    ASSERT_RESULT(expectedResults, res);

    if (res == Result::OK) {

        ASSERT_EQ(0U, parameters.size());

        for (auto& parameter : parameters) {

            ASSERT_EQ(0U, parameter.value.size()) << toString(parameter);

        }

    }

}