Skip to content

Increase converage and add gitlab tests#71

Merged
erseco merged 2 commits intomainfrom
feature/increase-coverage4
Dec 1, 2025
Merged

Increase converage and add gitlab tests#71
erseco merged 2 commits intomainfrom
feature/increase-coverage4

Conversation

@erseco
Copy link
Copy Markdown
Collaborator

@erseco erseco commented Dec 1, 2025

This pull request introduces a comprehensive set of new unit tests for critical components of the Documentate plugin, focusing on export handlers, the admin converter template, and the uninstall process. Additionally, it updates the SonarQube scanner version in CI. These changes significantly improve test coverage, ensuring that key functionality, security checks, and integration points are properly validated.

Testing improvements

  • Added ExportHandlerTest to verify the behavior of Export_Handler and its subclasses (Export_DOCX_Handler, Export_ODT_Handler, Export_PDF_Handler), including format/mime type methods, input sanitization, permission checks, and class structure.
  • Added DocumentateConverterTemplateTest to ensure the admin/documentate-converter-template.php file contains all required HTML, PHP variables, JavaScript integrations (including ZetaJS), AJAX handling, error handling, and CSS styles.
  • Added UninstallTest to check the existence and security of the uninstall.php file, verifying the presence of guard clauses and correct behavior when the uninstall constant is defined.

CI/CD update

  • Updated SonarQube scanner CLI image version from 11.4 to 11.5 in .gitlab-ci.yml to ensure the latest static analysis tooling is used.

@erseco erseco self-assigned this Dec 1, 2025
@chatgpt-codex-connector
Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@codecov
Copy link
Copy Markdown

codecov bot commented Dec 1, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@erseco erseco merged commit b282edc into main Dec 1, 2025
5 checks passed
@erseco erseco deleted the feature/increase-coverage4 branch December 1, 2025 07:59
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.

1 participant