test(filedialog): replace core tests with comprehensive UI tests

[add-uos]

Remove basic and simple core unit tests and replace with comprehensive UI component tests for FileDialog, FileDialogHandleDBus, and FileDialogStatusBar. The new tests provide better coverage for dialog functionality, including file selection, filter management, custom widgets, and D-Bus interface testing. Update existing FileDialogHandle tests to fix lambda signature issues and improve test reliability.

[sourcery-ai]

Sorry @add-uos, your pull request is larger than the review limit of 150000 diff characters

[deepin-ci-robot]

deepin pr auto review

我来对这个代码变更进行审查。这是一个文件对话框(file dialog)相关的测试代码重构，主要变更包括：

1. 删除了两个旧的测试文件：

• test_core_basic.cpp
• test_core_simple.cpp

2. 新增了四个新的测试文件：

• test_filedialog.cpp
• test_filedialoghandle.cpp
• test_filedialoghandledbus.cpp
• test_filedialogstatusbar.cpp

3. 修改了 test_filedialogmanagerdbus.cpp

从代码质量、性能和安全角度，我提出以下改进建议：

1. 代码组织与结构：

• 优点：新的测试文件结构更清晰，按功能模块划分
• 建议：可以进一步将测试用例按功能分组，使用TEST_F的命名规范体现分组

2. 测试覆盖率：

• 优点：新增的测试用例覆盖了更多边界情况
• 建议：增加对异常情况的测试，比如空指针、无效参数等场景

3. 内存管理：

• 优点：使用了智能指针和RAII
• 建议：在TearDown中统一使用deleteLater()来避免潜在的内存问题

4. 性能优化：

• 建议：对频繁调用的stub函数考虑使用static lambda来减少重复创建
• 建议：对耗时测试添加超时机制

5. 安全性：

• 优点：使用了Mock来避免实际文件系统操作
• 建议：增加对路径遍历、注入攻击等安全场景的测试

6. 代码可维护性：

• 建议：将重复的Mock代码抽取为公共函数
• 建议：为复杂的测试场景添加注释说明测试目的

7. 异常处理：

• 建议：增加对异常抛出场景的测试
• 建议：使用EXPECT_NO_THROW的地方改为具体的异常检查

8. 测试数据管理：

• 建议：将测试数据统一管理，避免硬编码
• 建议：使用参数化测试来减少重复代码

9. 并发测试：

• 建议：增加多线程场景下的测试用例

10. 资源清理：

• 建议：确保所有测试用例都能正确清理资源，避免内存泄漏

这些改进建议将有助于提高代码质量、测试覆盖率和系统的健壮性。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: add-uos, Johnson-zs

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

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

[Johnson-zs]

/forcemerge

[deepin-bot]

This pr force merged! (status: behind)