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

Commit af898821 authored by Tomasz Wasilczyk's avatar Tomasz Wasilczyk
Browse files

Fix program info call over-saturation. 

It was an omission to not allow onCurrentProgramInfoChanged
to be called more than once.

Bug: 120186193
Test: atest VtsHalBroadcastradioV2_0TargetTest
Change-Id: I667c552fb88e8fa3be3931901f7ba86ee7743baa
(cherry picked from commit 3f71e6c33ae34065cbf40ce1a2e4f9914383f969)
parent 128d7a80

broadcastradio/2.0/vts/functional/VtsHalBroadcastradioV2_0TargetTest.cpp

+3 −3
Original line number Diff line number Diff line
@@ -512,12 +512,12 @@ TEST_F(BroadcastRadioHalTest, Seek) { 
    // TODO(b/69958777): see FmTune workaround

    std::this_thread::sleep_for(gTuneWorkaround);



    EXPECT_TIMEOUT_CALL(*mCallback, onCurrentProgramInfoChanged_, _);

    EXPECT_TIMEOUT_CALL(*mCallback, onCurrentProgramInfoChanged_, _).Times(AnyNumber());

    auto result = mSession->scan(true /* up */, true /* skip subchannel */);

    EXPECT_EQ(Result::OK, result);

    EXPECT_TIMEOUT_CALL_WAIT(*mCallback, onCurrentProgramInfoChanged_, timeout::tune);



    EXPECT_TIMEOUT_CALL(*mCallback, onCurrentProgramInfoChanged_, _);

    EXPECT_TIMEOUT_CALL(*mCallback, onCurrentProgramInfoChanged_, _).Times(AnyNumber());

    result = mSession->scan(false /* down */, false /* don't skip subchannel */);

    EXPECT_EQ(Result::OK, result);

    EXPECT_TIMEOUT_CALL_WAIT(*mCallback, onCurrentProgramInfoChanged_, timeout::tune);
@@ -546,7 +546,7 @@ TEST_F(BroadcastRadioHalTest, Step) { 
    EXPECT_EQ(Result::OK, result);

    EXPECT_TIMEOUT_CALL_WAIT(*mCallback, onCurrentProgramInfoChanged_, timeout::tune);



    EXPECT_TIMEOUT_CALL(*mCallback, onCurrentProgramInfoChanged_, _);

    EXPECT_TIMEOUT_CALL(*mCallback, onCurrentProgramInfoChanged_, _).Times(AnyNumber());

    result = mSession->step(false /* down */);

    EXPECT_EQ(Result::OK, result);

    EXPECT_TIMEOUT_CALL_WAIT(*mCallback, onCurrentProgramInfoChanged_, timeout::tune);