From 55241f7c7699084ffa9d77b1c84991783d69f153 Mon Sep 17 00:00:00 2001 From: Tomasz Wasilczyk Date: Mon, 30 Apr 2018 08:53:24 -0700 Subject: [PATCH] Extend tune workaround time. Bug: 73963096 Test: atest VtsHalBroadcastradioV2_0TargetTest Change-Id: I6b722fb4f82005fe468a17f0f29af9dc8bd0e8f7 --- .../vts/functional/VtsHalBroadcastradioV2_0TargetTest.cpp | 8 +++++--- 1 file changed, 5 insertions(+), 3 deletions(-) diff --git a/broadcastradio/2.0/vts/functional/VtsHalBroadcastradioV2_0TargetTest.cpp b/broadcastradio/2.0/vts/functional/VtsHalBroadcastradioV2_0TargetTest.cpp index 6877f0745a..598926f95b 100644 --- a/broadcastradio/2.0/vts/functional/VtsHalBroadcastradioV2_0TargetTest.cpp +++ b/broadcastradio/2.0/vts/functional/VtsHalBroadcastradioV2_0TargetTest.cpp @@ -66,6 +66,8 @@ static constexpr auto programListScan = 5min; } // namespace timeout +static constexpr auto gTuneWorkaround = 200ms; + static const ConfigFlag gConfigFlagValues[] = { ConfigFlag::FORCE_MONO, ConfigFlag::FORCE_ANALOG, @@ -414,7 +416,7 @@ TEST_F(BroadcastRadioHalTest, FmTune) { * This sleep workaround will fix default implementation, but the real HW tests will still be * flaky. We probably need to implement egmock alternative based on actions. */ - std::this_thread::sleep_for(100ms); + std::this_thread::sleep_for(gTuneWorkaround); // try tuning ProgramInfo infoCb = {}; @@ -500,7 +502,7 @@ TEST_F(BroadcastRadioHalTest, Scan) { ASSERT_TRUE(openSession()); // TODO(b/69958777): see FmTune workaround - std::this_thread::sleep_for(100ms); + std::this_thread::sleep_for(gTuneWorkaround); EXPECT_TIMEOUT_CALL(*mCallback, onCurrentProgramInfoChanged_, _); auto result = mSession->scan(true /* up */, true /* skip subchannel */); @@ -525,7 +527,7 @@ TEST_F(BroadcastRadioHalTest, Step) { ASSERT_TRUE(openSession()); // TODO(b/69958777): see FmTune workaround - std::this_thread::sleep_for(100ms); + std::this_thread::sleep_for(gTuneWorkaround); EXPECT_TIMEOUT_CALL(*mCallback, onCurrentProgramInfoChanged_, _).Times(AnyNumber()); auto result = mSession->step(true /* up */);