test: Add initFFmpegVideoThumbnailer unit test.

[rb-union]

完善 initFFmpegVideoThumbnailer() ，添加备用的查找库方式
和提示信息。在 QLibrary 无法查找时尝试使用手动查找方式取得
其它版本的 libffmpegthumbnailer.so 动态库。

Log: 补充适配缩略图查找方式和UT
Bug: https://pms.uniontech.com/bug-view-213565.html
Bug: https://pms.uniontech.com/bug-view-221557.html
Influence: VideoThumbnailer

[deepin-bot]

Deepin ChatOps Bot
Deepin redeliveried failed, need update or recreate pull request!!!

[chenchongbiao]

/check obs

[rb-union]

/check obs

[deepin-bot]

Deepin Obs Bot
Starting find obs webhook event and retrigger!

[deepin-bot]

TAG Bot

New tag: 1.0.40
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #112

[deepin-bot]

TAG Bot

New tag: 1.0.41
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #114

[deepin-bot]

TAG Bot

New tag: 1.0.42
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #127

[deepin-bot]

TAG Bot

New tag: 1.0.43
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #129

[deepin-bot]

TAG Bot

New tag: 1.0.44
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #134

[deepin-bot]

TAG Bot

New tag: 1.0.45
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #140

[deepin-bot]

TAG Bot

New tag: 1.0.46
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #145

[deepin-bot]

TAG Bot

New tag: 1.0.47
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #153

[deepin-bot]

TAG Bot

New tag: 6.5.0
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #156

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 代码注释：

   • 在libPath函数和initFFmpegVideoThumbnailer函数中，注释应该更详细地解释函数的目的、参数、返回值和可能的副作用。例如，libPath函数的注释应该说明它是如何处理动态库查找的，以及为什么需要这个函数。

2. 错误处理：

   • 在initFFmpegVideoThumbnailer函数中，当QLibrary加载失败时，应该有更详细的错误处理逻辑，比如记录错误日志或者抛出异常。当前的实现只是简单地返回false，这可能不足以诊断问题。

3. 代码重复：

   • 在initFFmpegVideoThumbnailer函数中，qWarning和qInfo的日志输出重复了多次，可以考虑将日志输出封装成一个函数，以减少代码重复。

4. 资源管理：

   • QLibrary对象在initFFmpegVideoThumbnailer函数中创建，但没有看到相应的资源释放代码。如果initFFmpegVideoThumbnailer函数可能会被多次调用，应该考虑如何管理QLibrary对象的生命周期。

5. 测试代码：

   • 在ut_ffmpegvideothumbnailer测试类中，SetUp和TearDown函数应该有更明确的注释，说明它们的作用。此外，测试用例initFFmpegVideoThumbnailer和runFFmpegVideoThumbnailer应该有更详细的测试描述，说明它们测试的具体行为。

6. 代码风格：

   • 在initFFmpegVideoThumbnailer函数中，if语句的缩进不一致，应该保持一致的缩进风格以提高代码的可读性。

7. 安全性：

   • 在runFFmpegVideoThumbnailer函数中，直接使用QUrl::fromLocalFile可能会导致安全问题，应该验证文件路径是否合法，避免路径遍历攻击。

8. 性能：

   • 在libPath函数中，dir.entryList调用可能会影响性能，如果目录中包含大量文件，可以考虑使用异步方式或者缓存结果来提高性能。

综上所述，代码需要改进的地方包括：增加详细的注释、优化错误处理、减少代码重复、管理资源生命周期、改进测试描述、保持一致的代码风格、增强安全性以及考虑性能优化。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: pengfeixx, rb-union

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