Skip to content

Conversation

@ARR4N
Copy link
Collaborator

@ARR4N ARR4N commented Apr 11, 2025

Why this should be merged

Importing some avalanchego test fixtures automatically imports coreth packages that register extras, which affects testing of other VMs that depend on libevm. All other registered extras can be cleared with the pattern in this PR.

How this works

Uses the register.AtMostOnce type already used in params and types. I forgot to update the vm package when implementing the type.

How this was tested

Existing unit tests.

@ARR4N ARR4N enabled auto-merge (squash) April 11, 2025 14:22
@ARR4N ARR4N disabled auto-merge April 11, 2025 14:26
@ARR4N ARR4N changed the title feat(core/vm): allow clearing of registered hooks in tests refactor(core/vm): expose clearing of registered hooks in tests Apr 11, 2025
@ARR4N ARR4N merged commit c2e6df1 into main Apr 16, 2025
11 checks passed
@ARR4N ARR4N deleted the arr4n/test-only-clear-vm-hooks branch April 16, 2025 13:05
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.

2 participants