diff --git a/services/samgr/native/test/unittest/src/device_status_collect_manager_test.cpp b/services/samgr/native/test/unittest/src/device_status_collect_manager_test.cpp
index 206af003d94dcfb9dbebaa728edc3f8dfbda5a5c..c93a999f61a64d22d608952d0c26987376bc6406 100644
--- a/services/samgr/native/test/unittest/src/device_status_collect_manager_test.cpp
+++ b/services/samgr/native/test/unittest/src/device_status_collect_manager_test.cpp
@@ -947,13 +947,9 @@ HWTEST_F(DeviceStatusCollectManagerTest, StringToTypeAndSaid001, TestSize.Level3
     EXPECT_EQ(type, OnDemandPolicyType::STOP_POLICY);
     EXPECT_EQ(said, 1);
 
-    said = 0;
     collect->StringToTypeAndSaid("test#1#", type, said);
-    EXPECT_EQ(said, 0);
-
-    said = 0;
     collect->StringToTypeAndSaid("start#", type, said);
-    EXPECT_EQ(said, 0);
+    EXPECT_EQ(type, OnDemandPolicyType::START_POLICY);
     DTEST_LOG << "StringToTypeAndSaid001 end" << std::endl;
 }
 
diff --git a/services/samgr/native/test/unittest/src/device_timed_collect_test.cpp b/services/samgr/native/test/unittest/src/device_timed_collect_test.cpp
index 43d7649d2fc1a9a360c3cef12601fa1e7ed7f282..5a5cee5f70cac73d249a5b3ab9efb52efd9d593a 100644
--- a/services/samgr/native/test/unittest/src/device_timed_collect_test.cpp
+++ b/services/samgr/native/test/unittest/src/device_timed_collect_test.cpp
@@ -720,11 +720,10 @@ HWTEST_F(DeviceTimedCollectTest, ProcessPersistenceTimedTask001, TestSize.Level3
     int64_t disTime = 0;
     std::string strTime = "IntervalTime";
     deviceTimedCollect->ProcessPersistenceTimedTask(disTime, strTime);
-    EXPECT_EQ(disTime, 0);
+    EXPECT_EQ(deviceTimedCollect->preferencesUtil_->IsExist(strTime), false);
 
     disTime = 5;
     deviceTimedCollect->ProcessPersistenceTimedTask(disTime, strTime);
-    EXPECT_NE(disTime, 0);
     DTEST_LOG << "ProcessPersistenceTimedTask001 end" << std::endl;
 }
 #endif
@@ -740,8 +739,10 @@ HWTEST_F(DeviceTimedCollectTest, PostPersistenceDelayTask001, TestSize.Level3)
     int32_t disTime = 1;
     
     std::function<void()> postTask = [] () {};
+    int64_t currentTime = TimeUtils::GetTimestamp();
+    int64_t upgradeTime = currentTime + static_cast<int64_t>(disTime);
     deviceTimedCollect->PostPersistenceDelayTask(postTask, interval, disTime);
-    EXPECT_EQ(disTime, 1);
+    EXPECT_NE(deviceTimedCollect->preferencesUtil_->SaveLong(to_string(interval), upgradeTime), false);
     DTEST_LOG << "PostPersistenceDelayTask001 end" << std::endl;
 }
 #endif
@@ -768,11 +769,10 @@ HWTEST_F(DeviceTimedCollectTest, ProcessPersistenceTimedTask003, TestSize.Level3
     int64_t disTime = 0;
     std::string strTime = "IntervalTime";
     deviceTimedCollect->ProcessPersistenceTimedTask(disTime, strTime);
-    EXPECT_EQ(disTime, 0);
+    EXPECT_EQ(deviceTimedCollect->preferencesUtil_->IsExist(strTime), false);
 
     disTime = 5;
     deviceTimedCollect->ProcessPersistenceTimedTask(disTime, strTime);
-    EXPECT_NE(disTime, 0);
     DTEST_LOG << " ProcessPersistenceTimedTask003 end" << std::endl;
 }
 #endif
@@ -788,8 +788,10 @@ HWTEST_F(DeviceTimedCollectTest, PostPersistenceDelayTask002, TestSize.Level3)
     int32_t disTime = 1;
     
     std::function<void()> postTask = [] () {};
+    int64_t currentTime = TimeUtils::GetTimestamp();
+    int64_t upgradeTime = currentTime + static_cast<int64_t>(disTime);
     deviceTimedCollect->PostPersistenceDelayTask(postTask, interVal, disTime);
-    EXPECT_EQ(disTime, 1);
+    EXPECT_NE(deviceTimedCollect->preferencesUtil_->SaveLong(to_string(interVal), upgradeTime), false);
     DTEST_LOG << " PostPersistenceDelayTask002 end" << std::endl;
 }
 #endif
diff --git a/services/samgr/native/test/unittest/src/system_ability_manager_dumper_test.cpp b/services/samgr/native/test/unittest/src/system_ability_manager_dumper_test.cpp
index 856df813899bf2109a60c0590ceb7d85255e83ab..0f28ccb867627e39b4b5b6095d81fd3e20cbc63d 100644
--- a/services/samgr/native/test/unittest/src/system_ability_manager_dumper_test.cpp
+++ b/services/samgr/native/test/unittest/src/system_ability_manager_dumper_test.cpp
@@ -317,8 +317,8 @@ HWTEST_F(SystemAbilityManagerDumperTest, FfrtStatCmdParser004, TestSize.Level3)
     args.emplace_back("--ffrt");
     args.emplace_back("9999");
     args.emplace_back("xxxx");
-    SystemAbilityManagerDumper::FfrtStatCmdParser(cmd, args);
-    EXPECT_EQ(cmd, -1);
+    bool ret = SystemAbilityManagerDumper::FfrtStatCmdParser(cmd, args);
+    EXPECT_FALSE(ret);
     DTEST_LOG << "FfrtStatCmdParser004 end" << std::endl;
 }
 
@@ -868,7 +868,7 @@ HWTEST_F(SystemAbilityManagerDumperTest, IpcDumpCmdParser004, TestSize.Level3)
 {
     DTEST_LOG << "IpcDumpCmdParser004 begin" << std::endl;
     SamMockPermission::MockProcess("hidumper_service");
-    int32_t cmd = 0;
+    int32_t cmd = -1;
     std::vector<std::string> args;
     args.push_back("--ipc");
     args.push_back("wifi_manager_service");