Skip to content

add monero_wallet integration tests#974

Open
fxrstor wants to merge 1 commit intoeigenwallet:masterfrom
fxrstor:master
Open

add monero_wallet integration tests#974
fxrstor wants to merge 1 commit intoeigenwallet:masterfrom
fxrstor:master

Conversation

@fxrstor
Copy link
Copy Markdown

@fxrstor fxrstor commented Apr 28, 2026

related to issue #800

  • covers the required test scope
  • renames monero-wallet crate to monero_wallet because apparently there's a name conflict which prevents the tests from running with default parameters.
  • introduces an integration_test macro. inline async closures can keep captured wallet resources alive past the test body, which can race with harness teardown and trigger a segfault during c++ wallet destruction. the macro moves test logic into a separate async function passed to setup_test which gives wallet resources a clearer drop boundary before teardown.

test results (i3-4160):
test_results

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.

1 participant