diff options
author | Andrey Oleynik <aoleynik@luxoft.com> | 2017-06-02 09:51:34 +0300 |
---|---|---|
committer | Andrey Oleynik <aoleynik@luxoft.com> | 2017-06-02 09:58:00 +0300 |
commit | 41c6008f7ab14afc979a69e233c9549ddbfd99d6 (patch) | |
tree | a622de8420f8c9d2344f81990dc0955f85917f29 /src/components/policy/policy_external/test | |
parent | 79b7f6b99eacf79979e72aa721ff08e34297199e (diff) | |
download | sdl_core-41c6008f7ab14afc979a69e233c9549ddbfd99d6.tar.gz |
Fixes unit test for external user consent
Fixed typo, usage of wrong variable and OnPermissionsUpdated notification
expectation as application permissions are changed by external consents.
Diffstat (limited to 'src/components/policy/policy_external/test')
-rw-r--r-- | src/components/policy/policy_external/test/policy_manager_impl_test.cc | 24 |
1 files changed, 16 insertions, 8 deletions
diff --git a/src/components/policy/policy_external/test/policy_manager_impl_test.cc b/src/components/policy/policy_external/test/policy_manager_impl_test.cc index f2fc875021..f4e922e647 100644 --- a/src/components/policy/policy_external/test/policy_manager_impl_test.cc +++ b/src/components/policy/policy_external/test/policy_manager_impl_test.cc @@ -1075,7 +1075,7 @@ TEST_F( EXPECT_TRUE(policy_manager_->SetExternalConsentStatus(status_on)); // Checking ExternalConsent consents after setting new ExternalConsent status - const ConsentGroups& external_consent_statuss = + const ConsentGroups& external_consent_status = *updated_consent_records->second.external_consent_status_groups; ApplicationPolicies::const_iterator app_parameters = @@ -1085,17 +1085,17 @@ TEST_F( app_parameters); EXPECT_EQ(app_parameters->second.groups.size(), - external_consent_statuss.size()); + external_consent_status.size()); ConsentGroups::const_iterator updated_group_1 = - external_consent_statuss.find(group_name_1_); + external_consent_status.find(group_name_1_); - EXPECT_TRUE(external_consent_statuss.end() != updated_group_1); + EXPECT_TRUE(external_consent_status.end() != updated_group_1); ConsentGroups::const_iterator updated_group_2 = - external_consent_statuss.find(group_name_2_); + external_consent_status.find(group_name_2_); - EXPECT_TRUE(external_consent_statuss.end() != updated_group_2); + EXPECT_TRUE(external_consent_status.end() != updated_group_2); EXPECT_EQ(Boolean(false), updated_group_1->second); EXPECT_EQ(Boolean(true), updated_group_2->second); @@ -1106,9 +1106,17 @@ TEST_F( status_off.insert(ExternalConsentStatusItem(type_2_, id_2_, kStatusOff)); status_off.insert(ExternalConsentStatusItem(type_3_, id_3_, kStatusOff)); - EXPECT_CALL(listener_, OnPermissionsUpdated(app_id_1_, _)).Times(0); + EXPECT_CALL(listener_, OnPermissionsUpdated(app_id_1_, _)).Times(1); - EXPECT_TRUE(policy_manager_->SetExternalConsentStatus(status_on)); + EXPECT_TRUE(policy_manager_->SetExternalConsentStatus(status_off)); + + updated_group_1 = external_consent_status.find(group_name_1_); + + EXPECT_TRUE(external_consent_status.end() != updated_group_1); + + updated_group_2 = external_consent_status.find(group_name_2_); + + EXPECT_TRUE(external_consent_status.end() != updated_group_2); EXPECT_EQ(Boolean(true), updated_group_1->second); EXPECT_EQ(Boolean(false), updated_group_2->second); |