New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Fix tests without secure-node #245
Conversation
e013d6e
to
3e1b848
Compare
Concept ACK. Once #243 has been merged you should rebase, and then I'll ACK. |
This allows running the Python tests without importing secure-node.
- Remove redundant option mainchainrpchost. This option is already provided by bitcoind. - Set a working default for rpcport and rpcuser. Enables use without secure-node.
Previously, the service failed when onion-service was disabled.
'all' includes test 'full', which now succeeds.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Awesome. Looks good to me so far.
scenario=default buildTest "$@" | ||
scenario=netns buildTest "$@" | ||
scenario=full buildTest "$@" | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2 min 30 vs 3 min 15 on my laptop so we can switch to all by default in the future if we miss issues with the full test.
3e1b848
to
bfed10b
Compare
Rebased to master. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK bfed10b
This PR is based on #243 Modularize Tests.
It fixes the tests for scenarios without secure-node.
The following is now working: