chore: add tests project

[Johnson-zs]

add tests project

Log:

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. CMakeLists.txt:

   • 在添加测试目录时，建议添加对测试目录的描述或注释，以便其他开发者理解为什么需要添加测试。
   • 可以考虑在add_subdirectory(tests)之前添加一个条件检查，确保测试目录存在，以避免在构建时出现错误。

2. docparser.cpp:

   • isTextSuffix函数中使用了std::string_view，这是一个好的做法，因为它避免了不必要的字符串复制。
   • 在doConvertFile函数中，将后缀转换为小写时，使用了lambda表达式，这是一个很好的实践，因为它使代码更加简洁。
   • extensionMap和similarExtensionMap的使用提高了代码的可读性和可维护性，但是需要注意，如果文件扩展名和创建函数的映射关系发生变化，需要手动更新这两个映射。
   • 在doConvertFile函数中，使用std::move来避免不必要的字符串复制，这是一个好的做法。
   • 在main函数中，对于文件不存在的情况，应该抛出一个更具体的异常，而不是直接返回1。

3. tests/CMakeLists.txt:

   • 在链接库时，建议添加对库的描述或注释，以便其他开发者理解为什么需要链接这些库。
   • 可以考虑在target_link_libraries中添加对库版本的检查，以确保链接的是正确的库版本。

4. tests/main.cpp:

   • 在解析文件编码时，使用了Dtk::Core::DTextEncoding::detectFileEncoding，这是一个好的做法，因为它可以自动检测文件的编码。
   • 在转换编码时，使用了Dtk::Core::DTextEncoding::convertTextEncodingEx，这是一个好的做法，因为它可以处理不同编码之间的转换。
   • 在打印解析结果时，使用了QTextStream，这是一个好的做法，因为它可以处理不同编码的字符串。
   • 在处理异常时，应该捕获更具体的异常类型，而不是使用catch (...)，以避免吞掉其他类型的异常。

总体来说，代码的改进主要集中在提高代码的可读性、可维护性和性能上。建议在未来的开发中继续关注这些方面，并确保代码的质量和安全性。

[github-actions]

TAG Bot

TAG: 1.0.17
EXISTED: no
DISTRIBUTION: unstable

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: GongHeng2017, Johnson-zs, liyigang1

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

[Johnson-zs]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)