diff options
author | Mykola Korniichuk (GitHub) <42380041+mkorniichuk@users.noreply.github.com> | 2020-01-15 16:05:22 +0200 |
---|---|---|
committer | Shobhit Adlakha <ShobhitAd@users.noreply.github.com> | 2020-01-15 09:05:22 -0500 |
commit | e32a774e3b4d6a497855457436d65db169617e83 (patch) | |
tree | 1c8e53dbf62d521d1d669d5483f464002727a5fa /src/components/application_manager/test/application_impl_test.cc | |
parent | 5f86dabef5686bdbaf48993be6e82edab0097b41 (diff) | |
download | sdl_core-e32a774e3b4d6a497855457436d65db169617e83.tar.gz |
Fix misplaced OnDataVideoStreaming (#3154)
* Fix misplaced OnDataVideoStreaming
OnDataVideoStreaming notification was sent for each application
independently. Now one would be sent from application_manager with
respect to service status
* Fix OnDataStreamingNotification
* rename parameters
* fixup! rename parameters
Diffstat (limited to 'src/components/application_manager/test/application_impl_test.cc')
-rw-r--r-- | src/components/application_manager/test/application_impl_test.cc | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/components/application_manager/test/application_impl_test.cc b/src/components/application_manager/test/application_impl_test.cc index 7b45d387f7..11578f19d6 100644 --- a/src/components/application_manager/test/application_impl_test.cc +++ b/src/components/application_manager/test/application_impl_test.cc @@ -764,16 +764,16 @@ TEST_F(ApplicationImplTest, SuspendNaviStreaming) { protocol_handler::ServiceType type = protocol_handler::ServiceType::kMobileNav; EXPECT_CALL(mock_application_manager_, OnAppStreaming(app_id, type, false)); - EXPECT_CALL(*MockMessageHelper::message_helper_mock(), - SendOnDataStreaming(type, false, _)); + EXPECT_CALL(mock_application_manager_, + ProcessOnDataStreamingNotification(type, app_id, false)); app_impl->SuspendStreaming(type); } TEST_F(ApplicationImplTest, SuspendAudioStreaming) { protocol_handler::ServiceType type = protocol_handler::ServiceType::kAudio; EXPECT_CALL(mock_application_manager_, OnAppStreaming(app_id, type, false)); - EXPECT_CALL(*MockMessageHelper::message_helper_mock(), - SendOnDataStreaming(type, false, _)); + EXPECT_CALL(mock_application_manager_, + ProcessOnDataStreamingNotification(type, app_id, false)); app_impl->SuspendStreaming(type); } @@ -812,8 +812,8 @@ TEST_F(ApplicationImplTest, StopStreaming_StreamingApproved) { app_impl->set_video_streaming_approved(true); EXPECT_CALL(mock_application_manager_, OnAppStreaming(app_id, type, false)); - EXPECT_CALL(*MockMessageHelper::message_helper_mock(), - SendOnDataStreaming(type, false, _)); + EXPECT_CALL(mock_application_manager_, + ProcessOnDataStreamingNotification(type, app_id, false)); EXPECT_CALL(*MockMessageHelper::message_helper_mock(), SendNaviStopStream(app_id, _)); @@ -824,8 +824,8 @@ TEST_F(ApplicationImplTest, StopStreaming_StreamingApproved) { app_impl->set_audio_streaming_approved(true); type = protocol_handler::ServiceType::kAudio; EXPECT_CALL(mock_application_manager_, OnAppStreaming(app_id, type, false)); - EXPECT_CALL(*MockMessageHelper::message_helper_mock(), - SendOnDataStreaming(type, false, _)); + EXPECT_CALL(mock_application_manager_, + ProcessOnDataStreamingNotification(type, app_id, false)); EXPECT_CALL(*MockMessageHelper::message_helper_mock(), SendAudioStopStream(app_id, _)); |