diff --git a/test/unittest/phone/concurrent_task_client_test.cpp b/test/unittest/phone/concurrent_task_client_test.cpp index c92d6caab1ba0e754c6a6d48b9984101124c4731..7cd0ab80b99dd9fdcf7b66677ff513b590125386 100644 --- a/test/unittest/phone/concurrent_task_client_test.cpp +++ b/test/unittest/phone/concurrent_task_client_test.cpp @@ -61,6 +61,7 @@ HWTEST_F(ConcurrentTaskClientTest, ReportDataTest, TestSize.Level1) payload["pid"] = "12345"; payload["type"] = "appStart"; ConcurrentTaskClient::GetInstance().ReportData(resType, value, payload); + EXPECT_FALSE(payload.empty()); } /** @@ -75,6 +76,7 @@ HWTEST_F(ConcurrentTaskClientTest, ReportSceneInfoTest, TestSize.Level1) payload["uid"] = "3587"; payload["value"] = "12345"; ConcurrentTaskClient::GetInstance().ReportSceneInfo(type, payload); + EXPECT_FALSE(payload.empty()); } /** @@ -87,6 +89,7 @@ HWTEST_F(ConcurrentTaskClientTest, QueryIntervalTest, TestSize.Level1) int queryItem = 3; IntervalReply queryRs = {87, 657, 357, 214}; ConcurrentTaskClient::GetInstance().QueryInterval(queryItem, queryRs); + EXPECT_TRUE(queryRs.tid != -1); } /** @@ -96,7 +99,10 @@ HWTEST_F(ConcurrentTaskClientTest, QueryIntervalTest, TestSize.Level1) */ HWTEST_F(ConcurrentTaskClientTest, StopRemoteObjectTest, TestSize.Level1) { + std::unordered_map payload; ConcurrentTaskClient::GetInstance().StopRemoteObject(); + payload["uid"] = "3587"; + EXPECT_FALSE(payload.empty()); } /** @@ -112,6 +118,7 @@ HWTEST_F(ConcurrentTaskClientTest, QueryDeadlineTest, TestSize.Level1) payload[1111] = 60; payload[2222] = 90; ConcurrentTaskClient::GetInstance().QueryDeadline(queryItem, ddlReply, payload); + EXPECT_FALSE(payload.empty()); } /** @@ -125,6 +132,7 @@ HWTEST_F(ConcurrentTaskClientTest, RequestAuthTest, TestSize.Level1) payload["uid"] = "3587"; payload["pid"] = "12345"; ConcurrentTaskClient::GetInstance().RequestAuth(payload); + EXPECT_FALSE(payload.empty()); } } } \ No newline at end of file diff --git a/test/unittest/phone/concurrent_task_controller_interface_test.cpp b/test/unittest/phone/concurrent_task_controller_interface_test.cpp index 09688289d24e0f64bebe6b36cf2eb615bebd6f35..c0d723bb97162aa8cfa5679430fd242c46b0798a 100644 --- a/test/unittest/phone/concurrent_task_controller_interface_test.cpp +++ b/test/unittest/phone/concurrent_task_controller_interface_test.cpp @@ -59,6 +59,7 @@ HWTEST_F(ConcurrentTaskControllerInterfaceTest, RequestAuthTest, TestSize.Level1 const Json::Value payload; TaskControllerInterface repData; repData.RequestAuth(payload); + EXPECT_TRUE(payload.empty()); } /** @@ -73,6 +74,7 @@ HWTEST_F(ConcurrentTaskControllerInterfaceTest, ReportDataTest, TestSize.Level1) const Json::Value payload; TaskControllerInterface repData; repData.ReportData(resType, value, payload); + EXPECT_TRUE(payload.empty()); } /** @@ -86,6 +88,7 @@ HWTEST_F(ConcurrentTaskControllerInterfaceTest, ReportSceneInfoTest, TestSize.Le const Json::Value payload; TaskControllerInterface repData; repData.ReportSceneInfo(resType, payload); + EXPECT_TRUE(payload.empty()); } /** @@ -99,6 +102,7 @@ HWTEST_F(ConcurrentTaskControllerInterfaceTest, QueryDeadlineTest, TestSize.Leve DeadlineReply ddlReply = {false}; const Json::Value payload; TaskControllerInterface::GetInstance().QueryDeadline(queryItem, ddlReply, payload); + EXPECT_TRUE(payload.empty()); } /** @@ -120,6 +124,7 @@ HWTEST_F(ConcurrentTaskControllerInterfaceTest, QueryIntervalTest, TestSize.Leve queInt.QueryInterval(queryItem, queryRs); queryItem = QURRY_TYPE_MAX; queInt.QueryInterval(queryItem, queryRs); + EXPECT_FALSE(queInt.inited_); } /** @@ -130,6 +135,7 @@ HWTEST_F(ConcurrentTaskControllerInterfaceTest, QueryIntervalTest, TestSize.Leve HWTEST_F(ConcurrentTaskControllerInterfaceTest, InitTest, TestSize.Level1) { TaskControllerInterface::GetInstance().Init(); + EXPECT_FALSE(TaskControllerInterface::GetInstance().inited_); } /** @@ -140,6 +146,7 @@ HWTEST_F(ConcurrentTaskControllerInterfaceTest, InitTest, TestSize.Level1) HWTEST_F(ConcurrentTaskControllerInterfaceTest, ReleaseTest, TestSize.Level1) { TaskControllerInterface::GetInstance().Release(); + EXPECT_FALSE(TaskControllerInterface::GetInstance().inited_); } } // namespace FFRT_TEST } // namespace OHOS \ No newline at end of file diff --git a/test/unittest/phone/concurrent_task_service_ability_test.cpp b/test/unittest/phone/concurrent_task_service_ability_test.cpp index e1047f5d35ed4185d8fb68ed6bfd9213a5bd09bd..f6d09998b1ad616ef1875f3f198959613c076d69 100644 --- a/test/unittest/phone/concurrent_task_service_ability_test.cpp +++ b/test/unittest/phone/concurrent_task_service_ability_test.cpp @@ -62,6 +62,7 @@ HWTEST_F(ConcurrentTaskServiceAbilityTest, OnStartTest, TestSize.Level1) concurrenttaskserviceability.OnStart(); concurrenttaskserviceability.OnStart(); concurrenttaskserviceability.OnStop(); + EXPECT_TRUE(concurrenttaskserviceability.service_ != nullptr); } /** @@ -79,6 +80,7 @@ HWTEST_F(ConcurrentTaskServiceAbilityTest, OnAddSystemAbilityTest, TestSize.Leve concurrenttaskserviceability.OnStart(); concurrenttaskserviceability.OnAddSystemAbility(systemAbilityId, deviceId); concurrenttaskserviceability.OnRemoveSystemAbility(systemAbilityId, deviceId); + EXPECT_TRUE(concurrenttaskserviceability.service_ != nullptr); concurrenttaskserviceability.OnStop(); } } diff --git a/test/unittest/phone/concurrent_task_service_test.cpp b/test/unittest/phone/concurrent_task_service_test.cpp index 32ee825b058a6385eb3d7abc06b6920a86de9721..7ab705378706fc412175d1c19f328da224a6423b 100644 --- a/test/unittest/phone/concurrent_task_service_test.cpp +++ b/test/unittest/phone/concurrent_task_service_test.cpp @@ -58,6 +58,7 @@ HWTEST_F(ConcurrentTaskServiceTest, QueryIntervalTest, TestSize.Level1) IntervalReply queryRs = {87, 657, 357, 214}; ConcurrentTaskService queInt; queInt.QueryInterval(queryItem, queryRs); + EXPECT_NE(queryRs.tid, -1); } /** @@ -74,6 +75,7 @@ HWTEST_F(ConcurrentTaskServiceTest, QueryDeadlineTest, TestSize.Level1) payload["2222"] = "90"; ConcurrentTaskService queInt; queInt.QueryDeadline(queryItem, ddlReply, payload); + EXPECT_FALSE(payload.empty()); } } } \ No newline at end of file diff --git a/test/unittest/phone/func_loader_test.cpp b/test/unittest/phone/func_loader_test.cpp index 0943205916733a5b11a8a667552c616c66a42ba2..e90ab8ae12cd9e50529bf9f1dc45c119a98ad783 100644 --- a/test/unittest/phone/func_loader_test.cpp +++ b/test/unittest/phone/func_loader_test.cpp @@ -58,6 +58,7 @@ HWTEST_F(FuncLoaderTest, LoadFileTest, TestSize.Level1) { FuncLoader funcLoader("111"); funcLoader.LoadFile("222"); + EXPECT_FALSE(funcLoader.enable_); } /** @@ -75,7 +76,8 @@ HWTEST_F(FuncLoaderTest, LoadSymbolTest, TestSize.Level1) funcLoader.LoadSymbol("ReportSceneInfo"); funcLoader.LoadSymbol("QueryInterval"); funcLoader.LoadSymbol("QueryDeadline"); - funcLoader.LoadSymbol("RequestAuth"); + void* funcSym = funcLoader.LoadSymbol("RequestAuth"); + EXPECT_TRUE(funcSym == nullptr); } /** diff --git a/test/unittest/phone/qos_interface_test.cpp b/test/unittest/phone/qos_interface_test.cpp index 1fa0ca682d48651435d868625cb10c4e50165fb6..308033642904d8ed7e3b07c6e5511b80cde876b0 100644 --- a/test/unittest/phone/qos_interface_test.cpp +++ b/test/unittest/phone/qos_interface_test.cpp @@ -169,11 +169,6 @@ HWTEST_F(QosInterfaceTest, QosPolicyTest, TestSize.Level1) struct QosPolicyDatas *policyDatas = nullptr; ret = QosPolicySet(policyDatas); EXPECT_EQ(ret, -1); -#if defined(ARM64_TEST) && ARM64_TEST - EXPECT_EQ(ret, 0); - ret = QosPolicySet(&g_defaultQosPolicy); - EXPECT_EQ(ret, 0); -#endif } /** diff --git a/test/unittest/phone/qos_policy_test.cpp b/test/unittest/phone/qos_policy_test.cpp index 50a3c64b81ea7a81e749cd4fe33dd2f3879a676c..12c8d805fbdc9d7b77bbbca763bb8da23bd70c12 100644 --- a/test/unittest/phone/qos_policy_test.cpp +++ b/test/unittest/phone/qos_policy_test.cpp @@ -58,6 +58,8 @@ HWTEST_F(QosPolicyTest, TaskHandleTest, TestSize.Level1) { QosPolicy qosPolicy; qosPolicy.Init(); + struct QosPolicyDatas test = {}; + EXPECT_EQ(qosPolicy.SetQosPolicy(&test), -1); } } } \ No newline at end of file