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

Commit 4cdc4f20 authored by Treehugger Robot's avatar Treehugger Robot Committed by Hongguang Chen
Browse files

Merge "VTS: fix some logic errors for tuner HAL vts [1/1]" into android11-tests-dev am: 0638fc1e 

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

Bug: 208191849
Change-Id: Ida7b10a22c9c72ac9bc0fffc97750aaf27860fd7
parent 0a5a431d

tv/tuner/1.0/vts/functional/FrontendTests.cpp

100644 → 100755
+2 −0
Original line number Diff line number Diff line
@@ -377,6 +377,7 @@ AssertionResult FrontendTests::tuneFrontend(FrontendConfig config, bool testWith 
        result &= getDvrTests()->getDvrPlaybackMQDescriptor() == success();

        getDvrTests()->startPlaybackInputThread(mDvrConfig.playbackInputFile,

                                                mDvrConfig.settings.playback());

        getDvrTests()->startDvrPlayback();

        if (!result) {

            ALOGW("[vts] Software frontend dvr configure failed.");

            return failure();
@@ -400,6 +401,7 @@ AssertionResult FrontendTests::stopTuneFrontend(bool testWithDemux) { 
    status = mFrontend->stopTune();

    if (mIsSoftwareFe && testWithDemux) {

        getDvrTests()->stopPlaybackThread();

        getDvrTests()->stopDvrPlayback();

        getDvrTests()->closeDvrPlayback();

    }

    return AssertionResult(status == Result::SUCCESS);

tv/tuner/1.1/vts/functional/FrontendTests.cpp

+2 −0
Original line number Diff line number Diff line
@@ -444,6 +444,7 @@ AssertionResult FrontendTests::tuneFrontend(FrontendConfig1_1 config, bool testW 
        result &= getDvrTests()->getDvrPlaybackMQDescriptor() == success();

        getDvrTests()->startPlaybackInputThread(mDvrConfig.playbackInputFile,

                                                mDvrConfig.settings.playback());

        getDvrTests()->startDvrPlayback();

        if (!result) {

            ALOGW("[vts] Software frontend dvr configure failed.");

            return failure();
@@ -459,6 +460,7 @@ AssertionResult FrontendTests::stopTuneFrontend(bool testWithDemux) { 
    status = mFrontend->stopTune();

    if (mIsSoftwareFe && testWithDemux) {

        getDvrTests()->stopPlaybackThread();

        getDvrTests()->stopDvrPlayback();

        getDvrTests()->closeDvrPlayback();

    }

    return AssertionResult(status == Result::SUCCESS);

tv/tuner/aidl/vts/functional/FrontendTests.cpp

+2 −0
Original line number Diff line number Diff line
@@ -437,6 +437,7 @@ AssertionResult FrontendTests::tuneFrontend(FrontendConfig config, bool testWith 
        getDvrTests()->startPlaybackInputThread(

                mDvrConfig.playbackInputFile,

                mDvrConfig.settings.get<DvrSettings::Tag::playback>());

        getDvrTests()->startDvrPlayback();

    }

    mFrontendCallback->tuneTestOnLock(mFrontend, config.settings);

    return AssertionResult(true);
@@ -448,6 +449,7 @@ AssertionResult FrontendTests::stopTuneFrontend(bool testWithDemux) { 
    status = mFrontend->stopTune();

    if (mIsSoftwareFe && testWithDemux) {

        getDvrTests()->stopPlaybackThread();

        getDvrTests()->stopDvrPlayback();

        getDvrTests()->closeDvrPlayback();

    }

    return AssertionResult(status.isOk());