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..66b045b282b2e727a8ba39e003156df8b55eb721 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
@@ -741,7 +740,7 @@ HWTEST_F(DeviceTimedCollectTest, PostPersistenceDelayTask001, TestSize.Level3)
     
     std::function<void()> postTask = [] () {};
     deviceTimedCollect->PostPersistenceDelayTask(postTask, interval, disTime);
-    EXPECT_EQ(disTime, 1);
+    EXPECT_NE(deviceTimedCollect->preferencesUtil_, nullptr);
     DTEST_LOG << "PostPersistenceDelayTask001 end" << std::endl;
 }
 #endif
@@ -754,7 +753,7 @@ HWTEST_F(DeviceTimedCollectTest, PostNonPersistenceTimedTaskLocked001, TestSize.
     std::string strTime = "timedevent";
     int64_t timeGap = 0;
     deviceTimedCollect->PostNonPersistenceTimedTaskLocked(strTime, timeGap);
-    EXPECT_EQ(timeGap, 0);
+    EXPECT_NE(deviceTimedCollect, nullptr);
     DTEST_LOG << "PostNonPersistenceTimedTaskLocked001 end" << std::endl;
 }
 
@@ -768,11 +767,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
@@ -789,7 +787,7 @@ HWTEST_F(DeviceTimedCollectTest, PostPersistenceDelayTask002, TestSize.Level3)
     
     std::function<void()> postTask = [] () {};
     deviceTimedCollect->PostPersistenceDelayTask(postTask, interVal, disTime);
-    EXPECT_EQ(disTime, 1);
+    EXPECT_NE(deviceTimedCollect->preferencesUtil_, nullptr);
     DTEST_LOG << " PostPersistenceDelayTask002 end" << std::endl;
 }
 #endif
@@ -802,7 +800,7 @@ HWTEST_F(DeviceTimedCollectTest, PostNonPersistenceTimedTaskLocked002, TestSize.
     std::string strTime = "timedevent";
     int64_t timeGap = 0;
     deviceTimedCollect->PostNonPersistenceTimedTaskLocked(strTime, timeGap);
-    EXPECT_EQ(timeGap, 0);
+    EXPECT_NE(deviceTimedCollect, nullptr);
     DTEST_LOG << " PostNonPersistenceTimedTaskLocked002 end" << std::endl;
 }
 
@@ -836,10 +834,10 @@ HWTEST_F(DeviceTimedCollectTest, ProcessPersistenceLoopTask001, TestSize.Level3)
     int triggerTime = 1729909174;
     std::string strInterval = "1729909174";
     deviceTimedCollect->ProcessPersistenceLoopTask(disTime, triggerTime, strInterval);
-    EXPECT_EQ(0, disTime);
+    EXPECT_NE(deviceTimedCollect->preferencesUtil_, nullptr);
     disTime = 100;
     deviceTimedCollect->ProcessPersistenceLoopTask(disTime, triggerTime, strInterval);
-    EXPECT_NE(0, disTime);
+    EXPECT_NE(deviceTimedCollect->preferencesUtil_, nullptr);
     DTEST_LOG << " ProcessPersistenceLoopTask001 end" << std::endl;
 }
 
@@ -888,10 +886,10 @@ HWTEST_F(DeviceTimedCollectTest, PostNonPersistenceTimedTaskLocked003, TestSize.
     std::string strTime = "10";
     int64_t timeGap = 0;
     deviceTimedCollect->PostNonPersistenceTimedTaskLocked(strTime, timeGap);
-    EXPECT_EQ(0, timeGap);
+    EXPECT_NE(deviceTimedCollect, nullptr);
     timeGap = 5;
     deviceTimedCollect->PostNonPersistenceTimedTaskLocked(strTime, timeGap);
-    EXPECT_NE(0, timeGap);
+    EXPECT_NE(deviceTimedCollect, nullptr);
     DTEST_LOG << " PostNonPersistenceTimedTaskLocked003 end" << std::endl;
 }
 
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");