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

Commit 55241f7c authored by Tomasz Wasilczyk's avatar Tomasz Wasilczyk
Browse files

Extend tune workaround time. 

Bug: 73963096
Test: atest VtsHalBroadcastradioV2_0TargetTest
Change-Id: I6b722fb4f82005fe468a17f0f29af9dc8bd0e8f7
parent 89802f74

broadcastradio/2.0/vts/functional/VtsHalBroadcastradioV2_0TargetTest.cpp

+5 −3
Original line number Diff line number Diff line
@@ -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 */);