From 3dbddca6eff0bded529dd732f441d9851579083b Mon Sep 17 00:00:00 2001
From: Jeam_wang <jeam.wangwei@h-partners.com>
Date: Sat, 21 Jun 2025 18:42:06 +0800
Subject: [PATCH] modify test

Signed-off-by: Jeam_wang <jeam.wangwei@h-partners.com>
---
 .../app_id_mapping_config_manager_test.cpp    | 17 ++++++++
 .../service/cloud/cloud_service_impl.cpp      |  1 +
 .../service/test/BUILD.gn                     |  1 +
 .../service/test/cloud_data_test.cpp          | 42 ++++++++++++++++++-
 4 files changed, 60 insertions(+), 1 deletion(-)

diff --git a/services/distributeddataservice/framework/test/app_id_mapping_config_manager_test.cpp b/services/distributeddataservice/framework/test/app_id_mapping_config_manager_test.cpp
index c441ba910..16f41fdfc 100644
--- a/services/distributeddataservice/framework/test/app_id_mapping_config_manager_test.cpp
+++ b/services/distributeddataservice/framework/test/app_id_mapping_config_manager_test.cpp
@@ -14,6 +14,7 @@
 */
 
 #include "app_id_mapping/app_id_mapping_config_manager.h"
+#include "access_check/app_access_check_config_manager.h"
 
 #include <gtest/gtest.h>
 
@@ -22,6 +23,8 @@ using namespace OHOS::DistributedData;
 namespace OHOS::Test {
 class AppIdMappingConfigManagerTest : public testing::Test {};
 
+class AppAccessCheckConfigManagerTest : public testing::Test {};
+
 /**
 * @tc.name: Convert
 * @tc.desc: Generate a pair based on the input appId and accountId.
@@ -61,4 +64,18 @@ HWTEST_F(AppIdMappingConfigManagerTest, Convert02, TestSize.Level1)
     result = AppIdMappingConfigManager::GetInstance().Convert("src1");
     EXPECT_EQ(result, "dst1");
 }
+
+/**
+* @tc.name: IsTrust
+* @tc.desc: test IsTrust function.
+* @tc.type: FUNC
+*/
+HWTEST_F(AppAccessCheckConfigManagerTest, IsTrust, TestSize.Level1)
+{
+    ASSERT_FALSE(AppAccessCheckConfigManager::GetInstance().IsTrust("123"));
+    AppAccessCheckConfigManager::GetInstance().appMapper_.insert(std::make_pair("123", "321"));
+    ASSERT_FALSE(AppAccessCheckConfigManager::GetInstance().IsTrust("123"));
+    AppAccessCheckConfigManager::GetInstance().appMapper_.insert(std::make_pair("123", "123"));
+    ASSERT_TRUE(AppAccessCheckConfigManager::GetInstance().IsTrust("123"));
+}
 } // namespace OHOS::Test
\ No newline at end of file
diff --git a/services/distributeddataservice/service/cloud/cloud_service_impl.cpp b/services/distributeddataservice/service/cloud/cloud_service_impl.cpp
index 4404b855a..cef7b4b4c 100644
--- a/services/distributeddataservice/service/cloud/cloud_service_impl.cpp
+++ b/services/distributeddataservice/service/cloud/cloud_service_impl.cpp
@@ -150,6 +150,7 @@ int32_t CloudServiceImpl::EnableCloud(const std::string &id, const std::map<std:
     return SUCCESS;
 }
 
+
 void CloudServiceImpl::Report(
     const std::string &faultType, Fault errCode, const std::string &bundleName, const std::string &appendix)
 {
diff --git a/services/distributeddataservice/service/test/BUILD.gn b/services/distributeddataservice/service/test/BUILD.gn
index e6cf8c5d9..ced8b11f4 100644
--- a/services/distributeddataservice/service/test/BUILD.gn
+++ b/services/distributeddataservice/service/test/BUILD.gn
@@ -53,6 +53,7 @@ config("module_private_config") {
 }
 
 ohos_unittest("CloudDataTest") {
+  branch_protector_ret = "pac_ret"
   sanitize = {
     cfi = true
     cfi_cross_dso = true
diff --git a/services/distributeddataservice/service/test/cloud_data_test.cpp b/services/distributeddataservice/service/test/cloud_data_test.cpp
index ce7e8d8c8..09dc9de59 100644
--- a/services/distributeddataservice/service/test/cloud_data_test.cpp
+++ b/services/distributeddataservice/service/test/cloud_data_test.cpp
@@ -55,6 +55,7 @@
 #include "store/general_value.h"
 #include "store/store_info.h"
 #include "sync_manager.h"
+#include "sync_strategies/network_sync_strategy.h"
 #include "token_setproc.h"
 
 using namespace testing::ext;
@@ -841,7 +842,6 @@ HWTEST_F(CloudDataTest, UpdateStartSyncInfo, TestSize.Level0)
     MetaDataManager::GetInstance().LoadMeta(CloudLastSyncInfo::GetKey(user, TEST_CLOUD_BUNDLE,
          ""), lastSyncInfos, true);
     EXPECT_TRUE(!lastSyncInfos.empty());
-    printf("code: %d", lastSyncInfos[0].code);
     EXPECT_TRUE(lastSyncInfos[0].code == -1);
     EXPECT_TRUE(lastSyncInfos[0].startTime != 0);
     EXPECT_TRUE(lastSyncInfos[0].finishTime != 0);
@@ -3147,5 +3147,45 @@ HWTEST_F(CloudDataTest, GetValidGeneralCode, TestSize.Level0)
     ret = CloudData::SyncManager::ConvertValidGeneralCode(E_SYNC_TASK_MERGED);
     EXPECT_TRUE(ret == E_ERROR);
 }
+
+/**
+* @tc.name: CheckSyncAction
+* @tc.desc: test CheckSyncAction function.
+* @tc.type: FUNC
+*/
+HWTEST_F(CloudDataTest, CheckSyncAction, TestSize.Level1)
+{
+    delegate_.isNetworkAvailable_ = false;
+    CloudData::NetworkSyncStrategy strategy;
+    StoreInfo storeInfo;
+    auto result = strategy.CheckSyncAction(storeInfo);
+    EXPECT_EQ(result, E_NETWORK_ERROR);
+    storeInfo.user = 0;
+    delegate_.isNetworkAvailable_ = true;
+    result = strategy.CheckSyncAction(storeInfo);
+    EXPECT_EQ(result, E_OK);
+    storeInfo.user = 100;
+    strategy.user_ = 100;
+    result = strategy.CheckSyncAction(storeInfo);
+    EXPECT_EQ(result, E_BLOCKED_BY_NETWORK_STRATEGY);
+
+}
+
+/**
+* @tc.name: Check
+* @tc.desc: test Check function.
+* @tc.type: FUNC
+*/
+HWTEST_F(CloudDataTest, Check, TestSize.Level1)
+{
+    CloudData::NetworkSyncStrategy networkStrategy;
+    uint32_t strategy = 1;
+    auto result = networkStrategy.Check(strategy);
+    ASSERT_FALSE(result);
+    delegate_.networkType = NetworkDelegate::WIFI;
+    result = networkStrategy.Check(strategy);
+    ASSERT_TRUE(result);
+
+}
 } // namespace DistributedDataTest
 } // namespace OHOS::Test
\ No newline at end of file
-- 
Gitee