diff options
author | JackLivio <jack@livio.io> | 2018-10-11 15:10:36 -0400 |
---|---|---|
committer | JackLivio <jack@livio.io> | 2018-10-11 15:10:36 -0400 |
commit | 9785e1adda6f3abdf9714866dde9305b99332dd7 (patch) | |
tree | 352512254f9aecfe676520522af8508306ecdf0e | |
parent | c9697970a6ad6300936fe893efe7f3acae92a7c0 (diff) | |
download | sdl_core-9785e1adda6f3abdf9714866dde9305b99332dd7.tar.gz |
Bump base negotiated rpc version to 4.5.1
9 files changed, 13 insertions, 13 deletions
diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_button_event_notification.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_button_event_notification.cc index aeb53a115e..911d7faaff 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_button_event_notification.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_button_event_notification.cc @@ -166,7 +166,7 @@ void OnButtonEventNotification::SendButtonEvent(ApplicationConstSharedPtr app) { (*message_)[strings::msg_params][hmi_response::button_name].asInt()); if (btn_id == mobile_apis::ButtonName::PLAY_PAUSE && - app->msg_version() <= utils::version_4_5) { + app->msg_version() <= utils::base_rpc_version) { btn_id = mobile_apis::ButtonName::OK; } diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_button_press_notification.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_button_press_notification.cc index 56a87da305..9ca662389d 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_button_press_notification.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/on_button_press_notification.cc @@ -174,7 +174,7 @@ void OnButtonPressNotification::SendButtonPress(ApplicationConstSharedPtr app) { (*message_)[strings::msg_params][hmi_response::button_name].asInt()); if (btn_id == mobile_apis::ButtonName::PLAY_PAUSE && - app->msg_version() <= utils::version_4_5) { + app->msg_version() <= utils::base_rpc_version) { btn_id = mobile_apis::ButtonName::OK; } diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/register_app_interface_response.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/register_app_interface_response.cc index 31eb738357..3972aab3a8 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/register_app_interface_response.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/register_app_interface_response.cc @@ -63,7 +63,7 @@ void RegisterAppInterfaceResponse::Run() { application_manager::ApplicationSharedPtr app = application_manager_.application(connection_key()); - if (app && app->msg_version() <= utils::version_4_5 && + if (app && app->msg_version() <= utils::base_rpc_version && app->is_media_application() && (*message_)[strings::msg_params].keyExists( hmi_response::button_capabilities)) { diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subscribe_button_request.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subscribe_button_request.cc index 85cb26321b..7ee15d9119 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subscribe_button_request.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/subscribe_button_request.cc @@ -77,7 +77,7 @@ void SubscribeButtonRequest::Run() { return; } - if (app->msg_version() <= utils::version_4_5 && + if (app->msg_version() <= utils::base_rpc_version && btn_id == mobile_apis::ButtonName::OK && app->is_media_application()) { bool ok_supported = CheckHMICapabilities(mobile_apis::ButtonName::OK); bool play_pause_supported = diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/unsubscribe_button_request.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/unsubscribe_button_request.cc index ccfaef54cb..f914182e5b 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/unsubscribe_button_request.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/src/commands/mobile/unsubscribe_button_request.cc @@ -72,7 +72,7 @@ void UnsubscribeButtonRequest::Run() { static_cast<mobile_apis::ButtonName::eType>( (*message_)[str::msg_params][str::button_name].asInt()); - if (app->msg_version() <= utils::version_4_5 && + if (app->msg_version() <= utils::base_rpc_version && btn_id == mobile_apis::ButtonName::OK && app->is_media_application()) { bool ok_supported = CheckHMICapabilities(mobile_apis::ButtonName::OK); bool play_pause_supported = diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/subscribe_button_request_test.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/subscribe_button_request_test.cc index 0196b59b79..9e2e83e8f8 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/subscribe_button_request_test.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/subscribe_button_request_test.cc @@ -74,7 +74,7 @@ class SubscribeButtonRequestTest typedef SubscribeButtonRequestTest::MockHMICapabilities MockHMICapabilities; const utils::SemanticVersion mock_semantic_version(5, 0, 0); -const utils::SemanticVersion mock_semantic_version_4_5(4, 5, 0); +const utils::SemanticVersion mock_base_rpc_version(4, 5, 1); TEST_F(SubscribeButtonRequestTest, Run_AppNotRegistered_UNSUCCESS) { CommandPtr command(CreateCommand<SubscribeButtonRequest>()); @@ -202,7 +202,7 @@ TEST_F(SubscribeButtonRequestTest, Run_SUCCESS) { [am::strings::result_code].asInt())); } -TEST_F(SubscribeButtonRequestTest, Run_SUCCESS_App_Version_4_5) { +TEST_F(SubscribeButtonRequestTest, Run_SUCCESS_App_Base_RPC_Version) { const mobile_apis::ButtonName::eType kButtonName = mobile_apis::ButtonName::OK; @@ -213,7 +213,7 @@ TEST_F(SubscribeButtonRequestTest, Run_SUCCESS_App_Version_4_5) { MockAppPtr app(CreateMockApp()); ON_CALL(app_mngr_, application(_)).WillByDefault(Return(app)); ON_CALL(*app, msg_version()) - .WillByDefault(ReturnRef(mock_semantic_version_4_5)); + .WillByDefault(ReturnRef(mock_base_rpc_version)); ON_CALL(*app, is_media_application()).WillByDefault(Return(true)); ON_CALL(mock_hmi_capabilities_, is_ui_cooperating()) diff --git a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/unsubscribe_button_request_test.cc b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/unsubscribe_button_request_test.cc index d6bef42794..a848fff2b2 100644 --- a/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/unsubscribe_button_request_test.cc +++ b/src/components/application_manager/rpc_plugins/sdl_rpc_plugin/test/commands/mobile/unsubscribe_button_request_test.cc @@ -29,7 +29,7 @@ namespace { const uint32_t kConnectionKey = 1u; const mobile_apis::ButtonName::eType kButtonId = mobile_apis::ButtonName::OK; const utils::SemanticVersion mock_semantic_version(5, 0, 0); -const utils::SemanticVersion mock_semantic_version_4_5(4, 5, 0); +const utils::SemanticVersion mock_base_rpc_version(4, 5, 1); } // namespace class UnsubscribeButtonRequestTest @@ -151,7 +151,7 @@ TEST_F(UnsubscribeButtonRequestTest, Run_SUCCESS) { command->Run(); } -TEST_F(UnsubscribeButtonRequestTest, Run_SUCCESS_Version_4_5) { +TEST_F(UnsubscribeButtonRequestTest, Run_SUCCESS_Base_RPC_Version) { MessageSharedPtr command_msg(CreateMessage(smart_objects::SmartType_Map)); (*command_msg)[am::strings::params][am::strings::connection_key] = kConnectionKey; @@ -176,7 +176,7 @@ TEST_F(UnsubscribeButtonRequestTest, Run_SUCCESS_Version_4_5) { EXPECT_CALL(app_mngr_, application(kConnectionKey)) .WillRepeatedly(Return(mock_app)); ON_CALL(*mock_app, msg_version()) - .WillByDefault(ReturnRef(mock_semantic_version_4_5)); + .WillByDefault(ReturnRef(mock_base_rpc_version)); ON_CALL(*mock_app, is_media_application()).WillByDefault(Return(true)); EXPECT_CALL(*mock_app, diff --git a/src/components/include/utils/semantic_version.h b/src/components/include/utils/semantic_version.h index 244ae82177..ecd7a98de2 100644 --- a/src/components/include/utils/semantic_version.h +++ b/src/components/include/utils/semantic_version.h @@ -118,7 +118,7 @@ struct SemanticVersion { uint16_t patch_version_ = 0; }; -extern const SemanticVersion version_4_5; +extern const SemanticVersion base_rpc_version; } #endif // SRC_COMPONENTS_INCLUDE_UTILS_CALLABLE_H
\ No newline at end of file diff --git a/src/components/utils/src/semantic_version.cc b/src/components/utils/src/semantic_version.cc index 4b8d37b913..b7cf818791 100644 --- a/src/components/utils/src/semantic_version.cc +++ b/src/components/utils/src/semantic_version.cc @@ -33,6 +33,6 @@ namespace utils { -const SemanticVersion version_4_5(4, 5, 0); +const SemanticVersion base_rpc_version(4, 5, 1); } // namespace utils
\ No newline at end of file |