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

Commit 0d192916 authored by Nathan Kulczak's avatar Nathan Kulczak
Browse files

vibrator/vts: Update tests for short durations 



Add minimum wait time for effect completion. Effects with short
durations were not completing in time with current wait time estimation.

Bug: 240946208
Test: Ran updated tests
Change-Id: I40f65244c3478fb2f0a3b84c3991313ad5e28cd9
Signed-off-by: default avatarNathan Kulczak <nathankulczak@google.com>
parent 7433ad73

vibrator/aidl/vts/VtsHalVibratorTargetTest.cpp

+5 −4
Original line number Diff line number Diff line
@@ -602,9 +602,10 @@ TEST_P(VibratorAidl, ComposeCallback) { 
            EXPECT_EQ(Status::EX_NONE, vibrator->compose(composite, callback).exceptionCode())

                << toString(primitive);



            //TODO(b/187207798): revert back to conservative timeout values once

            //latencies have been fixed

            EXPECT_EQ(completionFuture.wait_for(duration * 4), std::future_status::ready)

            // TODO(b/261130361): Investigate why latency from driver and hardware will cause test

            // to fail when wait duration is ~40ms or less.

            EXPECT_EQ(completionFuture.wait_for(duration + std::chrono::milliseconds(50)),

                      std::future_status::ready)

                    << toString(primitive);

            end = high_resolution_clock::now();