!17929 merge ut into master

add ut Created-by: wendel Commit-by: wendel Merged-by: openharmony_ci Description: **IssueNo**: **Description**: **稳定性自检：** | 自检项 | 自检结果 | | ------------------------------------------------------------ | -------- | | 涉及跨进程调用的相关操作需要抛至主线程或加锁防止并发 | pass | | 成员变量进行赋值或创建需要排查并发 | pass | | 谨慎在lambda表达式中使用引用捕获 | pass | | 谨慎在未经拷贝的情况下使用外部传入的string、C字符串 | pass | | map\vector\list\set等stl模板类使用时需要排查并发 | pass | | 谨慎考虑加锁范围 | pass | | 在IPC通信中谨慎使用同步通信方式 | pass | | 禁止传递this指针至其他模块或线程（特别是eventhandler任务） | pass | | 禁止将外部传入的裸指针在内部直接构造智能指针 | pass | | 禁止多个独立创建的智能指针管理同一地址 | pass | | 禁止在析构函数中抛异步任务 | pass | | 禁止js对象在非js线程（例如在IPC线程）创建、使用或销毁 | pass | | 禁止在对外接口中未经判空直接使用外部传入的指针 | pass | | 禁止接口返回局部变量引用 | pass | | 禁止在信号函数中加锁 | pass | | 禁止在关键流程（SA启动、应用启动等主流程）执行耗时的操作 |pass | | 禁止将同一个cpp编译在不同的so中 | pass | **安全编码自检：** | 自检项 | 自检结果 | | -------------------------------------------------------------- | -------- | | 裸指针避免通过隐式转换构造为sptr | pass | | json对象在取值之前必须先判断类型，避免类型不匹配 | pass | | 序列化时必须对传入的数组大小进行校验，避免出现超大数组 | pass | | 避免使用未明确位宽的整型，选择使用int8_t、uint8_t等类型 | pass | | 外部传入的路径要做规范化校验，对路径中的.、..、../等特殊字符严格校验 | pass | | 指针变量、表示资源描述符的变量、bool变量必须赋初值 | pass | | readParcelable获取的对象使用前需要判空 | pass | | 分配和释放内存的函数需要成对出现 | pass | | 申请内存后异常退出前需要及时进行内存释放 | pass | | 内存申请前必须对内存大小进行合法性校验 | pass | | 内存分配后必须判断是否成功 | pass | | 禁止使用realloc、alloca函数 | pass | | 禁止打印文件路径、口令等敏感信息，如有需要，使用private修饰 | pass | | 禁止打印内存地址 | pass | | 整数之间运算时必须严格检查，确保不会出现溢出、反转、除0 | pass | | 禁止对有符号整数进行位操作符运算 | pass | | 禁止对指针进行逻辑或位运算 | pass | | 循环次数如果收外部数据控制，需要检验其合法性 | pass | | 禁止使用内存操作类危险函数，需要使用安全函数 | pass | | 谨慎使用不可重入函数 | pass | | 必须检查安全函数的返回值，并进行正确处理 | pass | | 禁止仅通过TokenType类型判断绕过权限校验 | pass | **TDD Result**: **XTS Result**:不涉及 ### 是否已执行L0用例 - [ ] 已验证 - [x] 不涉及。新增ut用例 See merge request: openharmony/ability_ability_runtime!17929
2026-08-24 12:43:16 -04:00 · 2026-01-05 16:21:44 +08:00
parent fa42a956fc 8af6aef74a
commit 9939db868e
3 changed files with 127 additions and 0 deletions
@@ -479,6 +479,7 @@ group("unittest") {
      "start_options_test:unittest",
      "start_options_utils_test:unittest",
      "start_other_app_interceptor_test:unittest",
+      "start_params_by_SCB_test:unittest",
      "startup_util_test:unittest",
      "state_utils_test:unittest",
      "stop_user_callback_proxy_test:unittest",
@@ -0,0 +1,40 @@
+# Copyright (c) 2026 Huawei Device Co., Ltd.
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#     http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
+import("//build/test.gni")
+import("//foundation/ability/ability_runtime/ability_runtime.gni")
+
+module_output_path = "ability_runtime/ability_runtime/abilitymgr"
+
+ohos_unittest("start_params_by_SCB_test") {
+  module_out_path = module_output_path
+
+  sources = [ "start_params_by_SCB_test.cpp" ]
+
+  cflags = []
+  if (target_cpu == "arm") {
+    cflags += [ "-DBINDER_IPC_32BIT" ]
+  }
+  deps = [ "${ability_runtime_innerkits_path}/ability_manager:ability_manager" ]
+
+  external_deps = [
+    "c_utils:utils",
+    "googletest:gtest_main",
+  ]
+}
+
+group("unittest") {
+  testonly = true
+
+  deps = [ ":start_params_by_SCB_test" ]
+}
@@ -0,0 +1,86 @@
+/*
+ * Copyright (c) 2026 Huawei Device Co., Ltd.
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+#include <gtest/gtest.h>
+
+#include "parcel.h"
+#define private public
+#include "start_params_by_SCB.h"
+#undef private
+
+using namespace testing::ext;
+namespace OHOS {
+namespace AbilityRuntime {
+class StartParamsBySCBTest : public testing::Test {
+public:
+    static void SetUpTestCase();
+    static void TearDownTestCase();
+    void SetUp();
+    void TearDown();
+};
+
+void StartParamsBySCBTest::SetUpTestCase()
+{}
+
+void StartParamsBySCBTest::TearDownTestCase()
+{}
+
+void StartParamsBySCBTest::SetUp()
+{}
+
+void StartParamsBySCBTest::TearDown()
+{}
+
+/**
+ * @tc.name: start_params_by_SCB_test_001
+ * @tc.desc: test class StartParamsBySCB marshalling function
+ * @tc.type: FUNC
+ */
+HWTEST_F(StartParamsBySCBTest, start_params_by_SCB_test_001, TestSize.Level1)
+{
+    StartParamsBySCB params;
+    Parcel parcel;
+    bool marshallResult = params.Marshalling(parcel);
+    EXPECT_EQ(marshallResult, true);
+}
+
+/**
+ * @tc.name: start_params_by_SCB_test_002
+ * @tc.desc: test class StartOptions Unmarshalling function
+ * @tc.type: FUNC
+ */
+HWTEST_F(StartParamsBySCBTest, start_params_by_SCB_test_002, TestSize.Level1)
+{
+    StartParamsBySCB params;
+    Parcel parcel;
+    auto unmarshallResult = params.Unmarshalling(parcel);
+    EXPECT_NE(unmarshallResult, nullptr);
+    ASSERT_FALSE(unmarshallResult->isRestart);
+}
+
+/**
+ * @tc.name: start_params_by_SCB_test_003
+ * @tc.desc: test class StartOptions Unmarshalling function
+ * @tc.type: FUNC
+ */
+HWTEST_F(StartParamsBySCBTest, start_params_by_SCB_test_003, TestSize.Level1)
+{
+    StartParamsBySCB params;
+    Parcel parcel;
+    bool result = params.ReadFromParcel(parcel);
+    ASSERT_TRUE(result);
+}
+}  // namespace AbilityRuntime
+}  // namespace OHOS