Skip to content

test(subscription): snapshot/restore state consistency tests - #794

Merged
aji70 merged 1 commit into
MyFanss:mainfrom
favourawaku:feat/subscription-snapshot-restore-tests
Apr 23, 2026
Merged

test(subscription): snapshot/restore state consistency tests#794
aji70 merged 1 commit into
MyFanss:mainfrom
favourawaku:feat/subscription-snapshot-restore-tests

Conversation

@favourawaku

Copy link
Copy Markdown
Contributor

Per ISSUES.md item 3:

  • Remove duplicate test_init_rejects_fee_bps_over_10000 that blocked compilation
  • Restore missing #[test] attribute on test_set_fee_bps_non_admin_rejected
  • Enables two snapshot/restore tests that were already written but unreachable:
    • test_subscription_state_after_snapshot_restore: saves state after subscribe via env.to_snapshot(), restores with Env::from_snapshot(), asserts plan, expiry, fan, and plan_count all match
    • test_cancel_after_snapshot_restore: restores snapshot, calls cancel, asserts is_subscriber returns false after cancel
  • All 48 unit tests + 1 integration test pass
    closes Subscription: snapshot/restore tests #603

Per ISSUES.md item 3:
- Remove duplicate test_init_rejects_fee_bps_over_10000 that blocked compilation
- Restore missing #[test] attribute on test_set_fee_bps_non_admin_rejected
- Enables two snapshot/restore tests that were already written but unreachable:
    * test_subscription_state_after_snapshot_restore: saves state after
      subscribe via env.to_snapshot(), restores with Env::from_snapshot(),
      asserts plan, expiry, fan, and plan_count all match
    * test_cancel_after_snapshot_restore: restores snapshot, calls cancel,
      asserts is_subscriber returns false after cancel
- All 48 unit tests + 1 integration test pass
@aji70
aji70 merged commit 051938a into MyFanss:main Apr 23, 2026
2 of 9 checks passed
aji70 added a commit that referenced this pull request Apr 30, 2026
…estore-tests

test(subscription): snapshot/restore state consistency tests
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.

Subscription: snapshot/restore tests

2 participants