Skip to content

Add tests #4822

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 6 commits into
base: main
Choose a base branch
from
Draft

Add tests #4822

wants to merge 6 commits into from
+824 −0

Conversation

@Ludy87
Copy link
Collaborator

@Ludy87 Ludy87 commented Nov 3, 2025

Description of Changes

Checklist

General

Documentation

Translations (if applicable)

UI Changes (if applicable)

  • Screenshots or videos demonstrating the UI changes are attached (e.g., as comments or direct attachments in the PR)

Testing (if applicable)

  • I have tested my changes locally. Refer to the Testing Guide for more details.

Ludy87 added 4 commits November 3, 2025 19:51
@Ludy87
Add unit tests
735ea77
@Ludy87
Add unit tests for ConverterWebController
a16850b 
Introduces comprehensive tests for ConverterWebController using standalone MockMvc. Tests cover endpoint view/model mapping, endpoint enable/disable logic, and maxDPI property handling for pdf-to-img and pdf-to-video endpoints.
@Ludy87
Remove LibreOfficeListenerTest unit test class
763db17 
Deleted the LibreOfficeListenerTest.java file from the test suite. This change removes all unit tests related to the LibreOfficeListener class.
@Ludy87
Update ConverterWebControllerTest.java
a23a686
@stirlingbot stirlingbot bot added the Test Testing-related issues or pull requests label Nov 3, 2025
Ludy87 added 2 commits November 3, 2025 22:10
@Ludy87
Add unit tests for GeneralWebController
9582a60 
Introduces comprehensive tests for GeneralWebController, covering simple endpoints, the /sign and /pipeline endpoints, and the getFormatFromExtension method. Tests include model population, error handling, and font resource filtering to ensure correct controller behavior.
@Ludy87
Create HomeWebControllerTest.java
df33f23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@DarioGii DarioGii Awaiting requested review from DarioGii DarioGii will be requested when the pull request is marked ready for review DarioGii is a code owner

@Frooodle Frooodle Awaiting requested review from Frooodle Frooodle will be requested when the pull request is marked ready for review Frooodle is a code owner

@jbrunton96 jbrunton96 Awaiting requested review from jbrunton96 jbrunton96 will be requested when the pull request is marked ready for review jbrunton96 is a code owner

@ConnorYoh ConnorYoh Awaiting requested review from ConnorYoh ConnorYoh will be requested when the pull request is marked ready for review ConnorYoh is a code owner

Copilot code review Copilot Awaiting requested review from Copilot Copilot will automatically review once the pull request is marked ready for review

At least 2 approving reviews are required to merge this pull request.

Assignees

No one assigned

Labels

Test Testing-related issues or pull requests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Ludy87