From 4c7dc0989aaad0740d395649a58edac36f1f4254 Mon Sep 17 00:00:00 2001 From: wuminjie Date: Mon, 12 May 2025 11:11:09 +0800 Subject: [PATCH 1/2] =?UTF-8?q?=E7=94=A8=E4=BE=8B=E6=95=B4=E6=94=B9?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: wuminjie --- .../device_status_collect_manager_test.cpp | 6 +---- .../src/device_timed_collect_test.cpp | 22 +++++++++---------- .../system_ability_manager_dumper_test.cpp | 6 ++--- 3 files changed, 14 insertions(+), 20 deletions(-) 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 206af003..c93a999f 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 43d7649d..bce819ce 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 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 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->preferencesUtil_, 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 856df813..0f28ccb8 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 args; args.push_back("--ipc"); args.push_back("wifi_manager_service"); -- Gitee From 6aa829c72ec44bc33f10ac8f73cfbfcc094b83af Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E5=90=B4=E6=95=8F=E6=9D=B0?= Date: Mon, 12 May 2025 06:28:35 +0000 Subject: [PATCH 2/2] update services/samgr/native/test/unittest/src/device_timed_collect_test.cpp. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Signed-off-by: 吴敏杰 --- .../native/test/unittest/src/device_timed_collect_test.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) 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 bce819ce..66b045b2 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 @@ -800,7 +800,7 @@ HWTEST_F(DeviceTimedCollectTest, PostNonPersistenceTimedTaskLocked002, TestSize. std::string strTime = "timedevent"; int64_t timeGap = 0; deviceTimedCollect->PostNonPersistenceTimedTaskLocked(strTime, timeGap); - EXPECT_NE(deviceTimedCollect->preferencesUtil_, nullptr); + EXPECT_NE(deviceTimedCollect, nullptr); DTEST_LOG << " PostNonPersistenceTimedTaskLocked002 end" << std::endl; } -- Gitee