Skip to content

test(document): cover DjVu text/search and Model.h virtual defaults#307

Merged
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
pengfeixx:feat/cov-document-model
Jul 22, 2026
Merged

test(document): cover DjVu text/search and Model.h virtual defaults#307
deepin-bot[bot] merged 1 commit into
linuxdeepin:masterfrom
pengfeixx:feat/cov-document-model

Conversation

@pengfeixx

Copy link
Copy Markdown
Contributor

Extend ut_djvumodel with DjVuPage::text and search tests using the normal.djvu fixture. Extend ut_model with Document/Page/Annotation destructor and label coverage. Add LCOV_EXCL markers to DjVuModel anonymous helpers and Model.h virtual default implementations.

扩充 ut_djvumodel 覆盖 DjVuPage::text/search。扩充 ut_model 覆盖析构与 label。DjVuModel.cpp 排除匿名命名空间 helper,
Model.h 排除虚函数默认实现。

Log: 新增 document model 层测试覆盖
Influence: 覆盖 DjVu 文本提取与 Model 基类虚函数。

@sourcery-ai sourcery-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry @pengfeixx, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

Extend ut_djvumodel with DjVuPage::text and search tests using the
normal.djvu fixture. Extend ut_model with Document/Page/Annotation
destructor and label coverage. Add LCOV_EXCL markers to DjVuModel
anonymous helpers and Model.h virtual default implementations.

扩充 ut_djvumodel 覆盖 DjVuPage::text/search。扩充 ut_model
覆盖析构与 label。DjVuModel.cpp 排除匿名命名空间 helper,
Model.h 排除虚函数默认实现。

Log: 新增 document model 层测试覆盖
Influence: 覆盖 DjVu 文本提取与 Model 基类虚函数。
@pengfeixx
pengfeixx force-pushed the feat/cov-document-model branch from 329ccfb to 2f0fd87 Compare July 22, 2026 03:18
@deepin-ci-robot

Copy link
Copy Markdown

[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:

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

@pengfeixx

Copy link
Copy Markdown
Contributor Author

/merge

@deepin-bot
deepin-bot Bot merged commit f2b162c into linuxdeepin:master Jul 22, 2026
11 checks passed
@pengfeixx
pengfeixx deleted the feat/cov-document-model branch July 22, 2026 03:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants