terencemo commented on PR #5913: URL: https://github.com/apache/fineract/pull/5913#issuecomment-4583912965
Hi Adam, thanks for the review! > Please > > 1. create an issue in JIRA and update the PR title > 2. look into failing tests On the failing tests — I looked into both. The failures in `SmsApiResourceIntegrationTest` and `CampaignsTest` are both `403` Forbidden where `200` is expected. My changes are confined to the stretchy reporting/runreports path and have no overlap with the SMS or Campaigns modules. The `403` pattern is typical of test ordering sensitivity in the sharded CI — a prior test leaving the auth state dirty. SmsApiResourceIntegrationTest is also relatively new (FINERACT-2315, July 2025) and likely has pre-existing flakiness here. On the Jira ticket — following up separately with you and @jdailey via email -- 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]
