Skip to content

Feature/issue 590 591 592 593 event tests#624

Open
Coredevjay wants to merge 4 commits intoSynapse-bridgez:mainfrom
Coredevjay:feature/issue-590-591-592-593-event-tests
Open

Feature/issue 590 591 592 593 event tests#624
Coredevjay wants to merge 4 commits intoSynapse-bridgez:mainfrom
Coredevjay:feature/issue-590-591-592-593-event-tests

Conversation

@Coredevjay
Copy link
Copy Markdown

@Coredevjay Coredevjay commented Mar 30, 2026

test: event emission tests for relayer, admin, and pause operations

Closes #590
closes #591
closes #592
closes #593

What

Adds typed event assertion tests for four contract operations that previously had no event coverage, and fixes the missing RelayerGranted emission.

Changes

lib.rs
— emit Event::RelayerGranted(relayer) in grant_relayer (resolves the TODO(#3) dependency required by #590)
contract_test.rs
:
Replace weak !events.is_empty() in grant_relayer_emits_relayer_granted_event with a typed Event::RelayerGranted(relayer) assertion (#590)
Add test_revoke_relayer_emits_event — asserts Event::RelayerRevoked(relayer) is the last event (#591)
Add test_transfer_admin_emits_event — asserts Event::AdminTransferred(old, new) and that get_admin() returns the new admin (#592)
Add test_pause_emits_event — asserts Event::ContractPaused(admin) is emitted on pause (#593)
Testing

All new tests follow the existing typed event pattern (event_data helper + (Event::Variant, ledger) comparison).

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 30, 2026

@Coredevjay 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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant