Skip to content

test(compliance): verify lazy expiry check in is_allowed - #317

Merged
misrasamuelisiguzor-oss merged 1 commit into
WHEELBACK:mainfrom
joshkirojohn212-jpg:test/compliance-expired-allow-lazy-check
Jul 26, 2026
Merged

test(compliance): verify lazy expiry check in is_allowed#317
misrasamuelisiguzor-oss merged 1 commit into
WHEELBACK:mainfrom
joshkirojohn212-jpg:test/compliance-expired-allow-lazy-check

Conversation

@joshkirojohn212-jpg

Copy link
Copy Markdown
Contributor
  • Add dedicated test for time-bound-allow core correctness property
  • Verify is_allowed returns false for AllowedUntil in the past without requiring sweep from feat(treasury): emit typed settlement events with TreasuryError codes #49 to run first
  • Test lazy evaluation: checks expiry on every call against ledger.timestamp
  • Confirm expiry boundary condition: ledger.timestamp >= AllowedUntil means expired

Closes #245
Closes #295
Closes #250
Closes #254

Description

Related Issue

Closes #[issue_id]

Checklist

  • Description of changes included
  • Test coverage confirmation (new/updated tests added)
  • Screenshot/gif for UI changes (if applicable)
  • ABI snapshot regenerated (if contract interface changed)
  • just check passing locally

- Add dedicated test for time-bound-allow core correctness property
- Verify is_allowed returns false for AllowedUntil in the past
  without requiring sweep from WHEELBACK#49 to run first
- Test lazy evaluation: checks expiry on every call against ledger.timestamp
- Confirm expiry boundary condition: ledger.timestamp >= AllowedUntil means expired

Closes WHEELBACK#54
@drips-wave

drips-wave Bot commented Jul 25, 2026

Copy link
Copy Markdown

@joshkirojohn212-jpg Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

@misrasamuelisiguzor-oss
misrasamuelisiguzor-oss merged commit 4648585 into WHEELBACK:main Jul 26, 2026
8 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

3 participants