test: migrate unit tests from Qt5/qmake to Qt6/CMake build system

[pengfeixx]

Add CMake-based test build infrastructure with Qt5/Qt6 dual-version compatibility. Create ut_compat.h for event constructor API differences. Fix Qt6 API changes: QWheelEvent, QMouseEvent, QHoverEvent, QEnterEvent, QProcess::start/startDetached, DPalette, DApplicationHelper, etc.

将单元测试从 Qt5/qmake 迁移到 Qt6/CMake 构建系统，添加事件构造函数
兼容层，修复 Qt6 接口变更导致的编译问题，812个测试用例编译通过。

Log: 单元测试从Qt5/qmake迁移到Qt6/CMake构建系统
Influence: 单元测试可在Qt6环境下编译运行，812个用例中801个通过。

[github-actions]

Note

[静态代码检查]

• 敏感词检查失败, 检测到1个文件存在敏感词

详情

{
    "tests/document/ut_pdfmodel.cpp": [
        {
            "line": "    g_linkAnnots->setUrl(\"http://www.123.com\");",
            "line_number": 136,
            "rule": "S35",
            "reason": "Url link | 29b32844da"
        },
        {
            "line": "    EXPECT_TRUE(link.urlOrFileName == \"http://www.123.com\");",
            "line_number": 265,
            "rule": "S35",
            "reason": "Url link | 29b32844da"
        },
        {
            "line": "    password = \"123\";",
            "line_number": 674,
            "rule": "S106",
            "reason": "Var naming | 7efdc4fb8a"
        }
    ]
}

[github-actions]

Note

[静态代码检查]

• 敏感词检查失败, 检测到1个文件存在敏感词

详情

{
    "tests/document/ut_pdfmodel.cpp": [
        {
            "line": "    g_linkAnnots->setUrl(\"http://www.123.com\");",
            "line_number": 136,
            "rule": "S35",
            "reason": "Url link | 29b32844da"
        },
        {
            "line": "    EXPECT_TRUE(link.urlOrFileName == \"http://www.123.com\");",
            "line_number": 265,
            "rule": "S35",
            "reason": "Url link | 29b32844da"
        },
        {
            "line": "    password = \"123\";",
            "line_number": 674,
            "rule": "S106",
            "reason": "Var naming | 7efdc4fb8a"
        }
    ]
}

[pengfeixx]

/merge

[deepin-ci-robot]

deepin pr auto review

你好！我是CodeGeeX。我已经仔细审查了你提供的Git Diff，该补丁主要为项目引入了基于CMake的单元测试构建框架，并进行了大量从Qt5向Qt6兼容的迁移工作。

以下是我从语法逻辑、代码质量、代码性能和代码安全四个维度的详细审查意见及改进建议：

一、 语法与逻辑

1. CMake cmake_minimum_required 版本覆盖问题

   • 问题：tests/CMakeLists.txt 中第1行声明了 cmake_minimum_required(VERSION 3.13)。由于该文件是通过主 CMakeLists.txt 的 add_subdirectory(tests) 引入的，CMake 规范要求子项目的 cmake_minimum_required 版本不能高于父项目。如果父项目版本低于 3.13，会导致CMake配置失败。
   • 建议：删除 tests/CMakeLists.txt 中的 cmake_minimum_required 和 project() 声明，直接作为子模块继承主项目的配置；或者确保主项目的版本号大于等于 3.13。

2. DocSheet::setAlive 中的列表操作逻辑不一致

   • 问题：在 reader/uiframe/DocSheet.cpp 的修改中：

     if (index >= 0) {
         if (index < g_sheetList.size())
             g_sheetList.removeAt(index);
         g_uuidList.removeAt(index); // 潜在越界风险
     } else {
         g_sheetList.removeOne(this);
     }

     如果 index >= 0 且 index >= g_sheetList.size()，代码会跳过移除 g_sheetList，但依然执行 g_uuidList.removeAt(index)。这会导致两个列表彻底失去同步，且如果 index >= g_uuidList.size()，会引发程序崩溃。
   • 建议：g_uuidList 和 g_sheetList 应该作为强绑定的数据结构操作。如果确实存在不同步的防御需求，应该统一处理：

     if (index >= 0 && index < g_sheetList.size() && index < g_uuidList.size()) {
         g_sheetList.removeAt(index);
         g_uuidList.removeAt(index);
     } else {
         // 列表不同步时的降级处理
         g_sheetList.removeOne(this);
         g_uuidList.removeOne(m_uuid);
     }

3. Qt6 兼容层 ut_compat.h 中 QEnterEvent 的构造函数逻辑

   • 问题：在 tests/widgets/ut_slideplaywidget.cpp 中直接使用了 new QEnterEvent(QPointF(0, 0), QPointF(0, 0), QPointF(0, 0))。但在 ut_compat.h 中并没有为 QEvent::Enter 提供兼容的包装函数。Qt6 中 Enter 事件的对象类型已从 QEvent 变更为 QEnterEvent，如果需要在 Qt5 下也编译通过，需要补充兼容函数。
   • 建议：在 ut_compat.h 中增加 createEnterEvent：

     #if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0)
     inline QEnterEvent* createEnterEvent(const QPointF &localPos, const QPointF &windowPos, const QPointF &globalPos) {
         return new QEnterEvent(localPos, windowPos, globalPos);
     }
     #else
     inline QEvent* createEnterEvent(const QPointF &, const QPointF &, const QPointF &) {
         return new QEvent(QEvent::Enter);
     }
     #endif

     并将 ut_slideplaywidget.cpp 中的代码替换为 QEvent *event = createEnterEvent(...)。

二、 代码质量

1. CMake 中使用 GLOB 收集源文件

   • 问题：tests/CMakeLists.txt 大量使用了 file(GLOB_RECURSE ...) 来收集源文件。CMake 官方强烈不建议这样做，因为新增或删除文件时，CMake 不会自动重新配置，必须手动触发，容易导致构建失效。
   • 建议：在大型项目中，建议显式列举出参与编译的源文件。如果一定要用 GLOB，请在此处添加明确的注释警告，提醒开发者增删文件后需重新运行 CMake。

2. 测试脚本 test-prj-running.sh 残留与注释混乱

   • 问题：脚本中把原来的 rm -rf 和 mkdir 注释掉了，却在下面继续 cd ../$builddir，如果目录不存在会导致脚本退出。此外，注释写着 # cmake ... 但实际并未执行 CMake，依然保留了 make，逻辑不闭合。
   • 建议：清理无用注释，如果该脚本已废弃或待重构，应明确标出；若要使用，需补全 CMake 的构建命令。

3. Qt5/Qt6 打桩（Stub）代码的宏隔离

   • 问题：在 ut_model.cpp、ut_centraldocpage.cpp 等文件中，出现了大量 #if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0) 的条件编译来处理 QProcess::startDetached 等API的差异，导致业务测试代码非常臃肿。
   • 建议：可以效仿 ut_compat.h 的做法，为常用的打桩函数（如 QProcess 的操作）封装一层统一的 Stub 兼容层，将条件编译移至头文件中，保持测试用例代码的整洁。

三、 代码性能

1. 单测可执行文件体积与编译耗时

   • 问题：tests/CMakeLists.txt 将 reader 目录下几乎所有的业务代码（APP_SOURCES, BROWSER_SOURCES 等）和所有测试代码打包成了一个单一的可执行文件 test-deepin-reader。
   • 建议：随着项目规模增长，这会导致单测编译极慢，且任何修改都会引发整个单测巨物的重新链接。建议按模块拆分测试目标（如 test_browser, test_sidebar 等），或者使用对象库（OBJECT library）将业务代码编译一次，再链接到不同的单测目标中：

     add_library(reader_lib OBJECT ${ALL_READER_SOURCES})
     add_executable(test_browser ${TEST_BROWSER_SOURCES} $<TARGET_OBJECTS:reader_lib>)

2. g_sheetList.removeOne(this) 的复杂度

   • 问题：QList::removeOne 是线性查找 $O(N)$。虽然这在 setAlive 析构逻辑中可能不是高频操作，但在列表不同步的异常分支中，隐含了性能损耗。
   • 建议：如果 DocSheet 跟踪逻辑复杂，长远来看建议将 g_sheetList 和 g_uuidList 替换为 QMap<QUuid, DocSheet*> 或 QHash，以提升查找和移除的效率至 $O(1)$，同时彻底避免双列表不同步的问题。

四、 代码安全

1. 链接选项 -z lazy 削弱了安全保护

   • 问题：在 tests/CMakeLists.txt 中有：target_link_options(${PROJECT_NAME} PRIVATE -pie -z lazy)。-z lazy（即延迟绑定）会关闭 ELF 的 RELRO（RELocation Read-Only）保护，使得 GOT 表在运行时可写，增加了 GOT 覆盖攻击的风险。现代安全编译规范推荐使用 -z relro 或 -z now。
   • 建议：测试代码虽然不直接暴露在生产环境，但为了规范统一和防止供应链污染，建议将 -z lazy 移除，或显式替换为更安全的 -z relro -z now。

2. 编译选项 -fno-access-control 的潜在风险

   • 问题：target_compile_options 中包含了 -fno-access-control，这允许测试代码直接访问类的 private/protected 成员。
   • 建议：虽然这极大方便了单元测试书写，但也破坏了封装性，使得测试与内部实现高度耦合。如果后续重构私有变量，测试将大面积崩溃。建议仅在必要时局部使用，或者通过友元类/友元测试的方式替代全局编译选项。

3. asan 日志拷贝的容错处理（好的改进）

   • 亮点：test-prj-running.sh 中将 cp -r asan*.log* ... 改为了 cp -r asan*.log* ... 2>/dev/null || true，这是一个很好的安全改进，避免了无 asan 日志时脚本异常退出。

总结

该 Diff 做了大量且必要的 Qt5 到 Qt6 的兼容性适配工作，引入了 ut_compat.h 是一个很好的设计。但在列表同步逻辑、CMake 构建规范以及安全编译选项上存在需要修正的隐患。建议优先修复 DocSheet 的越界风险和 -z lazy 安全选项问题。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, pengfeixx

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment