diff options
author | Sho Amano <samano@xevo.com> | 2017-08-10 10:44:59 +0900 |
---|---|---|
committer | Sho Amano <samano@xevo.com> | 2017-08-11 10:44:34 +0900 |
commit | 407627232b3a527b63c44ef3b77bdf52a54c7688 (patch) | |
tree | fd0b031e0dd0f2d7db20b73f91bcadd9ed3bfdba /src/components/application_manager/test/message_helper | |
parent | 8d7330735ed5d6a5ea5a01a1e8e674e83c1de252 (diff) | |
download | sdl_core-407627232b3a527b63c44ef3b77bdf52a54c7688.tar.gz |
Use newly added SmartObject keys
Reflecting review comments.
Diffstat (limited to 'src/components/application_manager/test/message_helper')
-rw-r--r-- | src/components/application_manager/test/message_helper/message_helper_test.cc | 29 |
1 files changed, 15 insertions, 14 deletions
diff --git a/src/components/application_manager/test/message_helper/message_helper_test.cc b/src/components/application_manager/test/message_helper/message_helper_test.cc index 7b02d37b87..56f2b16145 100644 --- a/src/components/application_manager/test/message_helper/message_helper_test.cc +++ b/src/components/application_manager/test/message_helper/message_helper_test.cc @@ -1034,10 +1034,11 @@ TEST_F(MessageHelperTest, SendNaviSetVideoConfigRequest) { int32_t app_id = 123; smart_objects::SmartObject video_params(smart_objects::SmartType_Map); - video_params["protocol"] = hmi_apis::Common_VideoStreamingProtocol::RTP; - video_params["codec"] = hmi_apis::Common_VideoStreamingCodec::H264; - video_params["width"] = 640; - video_params["height"] = 480; + video_params[strings::protocol] = + hmi_apis::Common_VideoStreamingProtocol::RTP; + video_params[strings::codec] = hmi_apis::Common_VideoStreamingCodec::H264; + video_params[strings::width] = 640; + video_params[strings::height] = 480; MessageHelper::SendNaviSetVideoConfig( app_id, mock_application_manager, video_params); @@ -1046,16 +1047,16 @@ TEST_F(MessageHelperTest, SendNaviSetVideoConfigRequest) { (*result)[strings::params][strings::function_id].asInt()); smart_objects::SmartObject& msg_params = (*result)[strings::msg_params]; - EXPECT_TRUE(msg_params.keyExists("config")); - - EXPECT_TRUE(msg_params["config"].keyExists("protocol")); - EXPECT_EQ(1, msg_params["config"]["protocol"].asInt()); - EXPECT_TRUE(msg_params["config"].keyExists("codec")); - EXPECT_EQ(0, msg_params["config"]["codec"].asInt()); - EXPECT_TRUE(msg_params["config"].keyExists("width")); - EXPECT_EQ(640, msg_params["config"]["width"].asInt()); - EXPECT_TRUE(msg_params["config"].keyExists("height")); - EXPECT_EQ(480, msg_params["config"]["height"].asInt()); + EXPECT_TRUE(msg_params.keyExists(strings::config)); + + EXPECT_TRUE(msg_params[strings::config].keyExists(strings::protocol)); + EXPECT_EQ(1, msg_params[strings::config][strings::protocol].asInt()); + EXPECT_TRUE(msg_params[strings::config].keyExists(strings::codec)); + EXPECT_EQ(0, msg_params[strings::config][strings::codec].asInt()); + EXPECT_TRUE(msg_params[strings::config].keyExists(strings::width)); + EXPECT_EQ(640, msg_params[strings::config][strings::width].asInt()); + EXPECT_TRUE(msg_params[strings::config].keyExists(strings::height)); + EXPECT_EQ(480, msg_params[strings::config][strings::height].asInt()); } } // namespace application_manager_test |