FINERACT-2494: Add unit tests for ApiParameterHelper in fineract-core#5510
FINERACT-2494: Add unit tests for ApiParameterHelper in fineract-core#5510Ambika-Sony wants to merge 1 commit intoapache:developfrom
Conversation
d3a8f13 to
03c1a73
Compare
|
@Ambika-Sony Testing 1 method does not provide too much value. Would you consider to test the whole class? |
bbe7a08 to
eaf3dee
Compare
I've Implemented pagination (limit, offset), sorting (orderBy, sortOrder), and association helpers in ApiParameterHelper. Added 10 unit tests in ApiParameterHelperTest, covering standard scenarios, null/empty inputs, and defensive checks for whitespace handling. Verified the build locally with ./gradlew :fineract-core:test --tests ApiParameterHelperTest (all 10 passed). Looking forward to your feedback. |
c298bf2 to
5e18810
Compare
|
@adamsaghy I noticed the Cucumber E2E tests are failing in the CI, but since my changes are strictly limited to the ApiParameterHelper utility and its unit tests (2 files), these failures seem unrelated to my code. I saw your recent email about the E2E framework transition—perhaps these are known issues? |
0ef52be to
c335179
Compare
c335179 to
1672c0e
Compare
Description
This PR adds unit test coverage for the ApiParameterHelper utility class in the fineract-core module.
Changes:
Added ApiParameterHelperTest.java to test core methods like extractFieldsForResponseIfProvided.
Verified code formatting using ./gradlew spotlessApply.
Testing:
Successfully ran locally using: ./gradlew :fineract-core:test --tests "org.apache.fineract.infrastructure.core.api.ApiParameterHelperTest"
All tests passed (100% success rate).
Checklist
Please make sure these boxes are checked before submitting your pull request - thanks!
Your assigned reviewer(s) will follow our guidelines for code reviews.