Skip to content

CMS: Increase test coverage to 87.75%#1476

Merged
MrDirkelz merged 4 commits intomainfrom
1463-cms-implement-full-test-coverage
Apr 9, 2026
Merged

CMS: Increase test coverage to 87.75%#1476
MrDirkelz merged 4 commits intomainfrom
1463-cms-implement-full-test-coverage

Conversation

@MrDirkelz
Copy link
Copy Markdown
Collaborator

The remaining low statement coverage on LPaginator (24%), ImageEditor (29%), MediaEditor (31%), UserOverview (28%), and EditContentBasic (41%) is primarily the v8 SFC <script setup> coverage artifact where the entire setup block counts as one uncovered function. Their branch and function coverage has improved significantly.

@MrDirkelz MrDirkelz linked an issue Apr 7, 2026 that may be closed by this pull request
@MrDirkelz MrDirkelz self-assigned this Apr 7, 2026
Copy link
Copy Markdown
Collaborator

@johan-bell johan-bell left a comment

Choose a reason for hiding this comment

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

Tests failed locally,

Copy link
Copy Markdown
Contributor

@ivanslabbert ivanslabbert left a comment

Choose a reason for hiding this comment

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

Approved, but check comment from Johan - might be a flakey test?

The remaining low statement coverage on LPaginator (24%), ImageEditor (29%), MediaEditor (31%), UserOverview (28%), and EditContentBasic (41%) is primarily the v8 SFC <script setup> coverage artifact where the entire setup block counts as one uncovered function. Their branch and function coverage has improved significantly.
@MrDirkelz MrDirkelz force-pushed the 1463-cms-implement-full-test-coverage branch from 6d5ddb9 to 0876e9e Compare April 9, 2026 13:10
@MrDirkelz MrDirkelz merged commit 5c56e81 into main Apr 9, 2026
4 checks passed
@MrDirkelz MrDirkelz deleted the 1463-cms-implement-full-test-coverage branch April 9, 2026 13:16
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.

CMS: Implement full test coverage

3 participants