Skip to content

test(reporting): lock down query authorization coverage across endpoints#606

Merged
Baskarayelu merged 1 commit into
Remitwise-Org:mainfrom
spartan124:test/reporting-auth-coverage
Apr 29, 2026
Merged

test(reporting): lock down query authorization coverage across endpoints#606
Baskarayelu merged 1 commit into
Remitwise-Org:mainfrom
spartan124:test/reporting-auth-coverage

Conversation

@spartan124
Copy link
Copy Markdown
Contributor

Title: test(reporting): lock down query authorization coverage across endpoints

Description

This PR introduces comprehensive tests to guarantee that all user-facing reporting queries enforce strict authorization checks. It verifies that user.require_auth() is correctly applied across all endpoints, effectively rejecting unauthorized callers. Additionally, it includes test coverage for ACL delegation behavior, ensuring only explicitly granted viewers can access designated reports when ACL is enabled.

Changes Implemented

Added authorization and access control test coverage for the following endpoints:

  • get_remittance_summary
  • get_savings_report
  • get_bill_compliance_report
  • get_insurance_report
  • get_financial_health_report
  • get_stored_report
  • Verified ACL grant and revoke delegation behaviors.

Checklist & Guidelines Met

  • Minimum 95% test coverage achieved for the reporting module.
  • All tests are passing locally (cargo test -p reporting).
  • Clear documentation added to test cases explaining authorization boundaries.
  • Completed within the required 96-hour timeframe.

Related Issues

@Baskarayelu Baskarayelu merged commit 33a8505 into Remitwise-Org:main Apr 29, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

SC-003 Reporting: Add tests ensuring all user-facing queries require auth (and ACL only when enabled)

2 participants