Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Refactor tests to use regtest node #263

Merged
merged 61 commits into from
Aug 9, 2022
Merged

Refactor tests to use regtest node #263

merged 61 commits into from
Aug 9, 2022

Conversation

terror
Copy link
Contributor

@terror terror commented Jul 22, 2022 •
edited by casey
Loading

  • get working in parallel
  • refactor and cleanup
  • make faster
  • speed up wallet startup?
  • create wallet in background?
  • speed up block mining?
  • other bottlenecks?
  • get working on CI
  • don't use Result in tests

terror and others added 30 commits July 22, 2022 12:52
@terror
Initial work
3dba2af
@terror
Get first_coinbase_transaction working
79ad708
@terror
Fix two more list tests
646c9ee
@terror
Fix a few find tests
c3c418c
@terror
Get 4th list test working
5e43541
@terror
Start adding bdk wallet in tests
9f498b7
@terror
Cleanup
6cfd1f4
@terror
Merge branch 'master' into regtest
b3f39c5
@terror
Semi-working
6e6b1d0
@terror
Clearer output
1fdbc82
@terror
Get first part of split_ranges_are_tracked_correctly working
be6fb0e
@terror
Fix merge_ranges test
1ad4cc2
@terror
Fix fee_paying_transaction_range
041695d
@terror
Small refactors
2868785
@terror
Fix null_output test
94f84fb
@terror
Use git dependency
01417c3
@terror
Fix two_fee test
c9fc017
@terror
Fix null_input test
380c647
@terror
Get all list tests to pass invidiually
5aa0cde
@terror
Fix find tests
c8cc1df
@terror
Fix info and index tests
f106d2a
@terror
Get server tests to pass
efcfcc2
@terror
Only kill process if healthy
26b67cc
@terror
Get all non-ignored tests to pass
353b78f
@terror
Merge branch 'master' into regtest
8447056
@terror
Make tests slightly faster
0ad28c0
@casey
Add our own bitcoind wrapper
1eca6dc
@terror
Merge branch 'master' into regtest
8219df5
@terror
Remove todo's
10f92cc
@terror
Add logging
ff89123
@terror terror marked this pull request as ready for review August 9, 2022 20:47
@casey casey merged commit bcf78b4 into master Aug 9, 2022
@casey casey deleted the regtest branch August 9, 2022 21:34
@casey casey mentioned this pull request Sep 11, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@terror @casey