AshharAhmadKhan commented on PR #5498: URL: https://github.com/apache/fineract/pull/5498#issuecomment-3997022702
@adamsaghy @IOhacker pushed 66212d0, please restart CI when you get a chance. Fixed in this push: **SpotBugs (fineract-e2e-tests-core)** — EC_UNRELATED_TYPES priority 1 in LoanDelinquencyStepDef.java:93. getStatus().getId() returns Long after OpenAPI regeneration but was being compared against LoanStatus.value (Integer) via Integer.equals(Long), always evaluating false. Fixed with getId().intValue() with null guard. **E2E Shard 2 (LoanDelinquency.feature:1243)** — same root cause as above. The broken type comparison was silently skipping the LoanDelinquencyRangeChangeEvent assertion on every run, causing the event timeout. Resolved by the same fix. **API backward compatibility** — all R014 violations are appUserClientMappings.* fields intentionally removed as the entire point of this PR. Needs a maintainer to regenerate the OpenAPI baseline, not a code fix. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
