Trunk-6470: Add JUnit tests for OpenmrsUtil.getFileAsBytes() #5566
+89
−0
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR adds unit tests for the method OpenmrsUtil.getFileAsBytes(File file).
The test suite covers the following scenarios:
Valid file → method returns correct byte array
Empty file → method returns zero-length array
Null input → method returns null (without throwing exception)
Non-existent file → method handles exception internally and returns null
The tests help ensure correct behavior of file reading operations and improve code coverage for OpenmrsUtil.
Issue Link
https://issues.openmrs.org/browse/TRUNK-6470
What was changed
Added new JUnit test methods inside OpenmrsUtilTest.java
Covered both positive and negative test scenarios
Verified behavior using FileUtils and StandardCharsets
Testing Instructions
To run only these tests:
mvn -q -pl :openmrs-api -Dtest=OpenmrsUtilTest test
Checklist
My IDE is using Java 21
mvn clean install completes successfully
Tests pass on local environment
PR title follows OpenMRS naming conventions
Additional Notes
This PR does not modify production code.
It only improves test coverage for core utilities.