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
The insert_txouts
test in test_tx_graph.rs
needs redoing.
#1331
Comments
Let me try this. |
insert_txouts
test in test_tx_graph.rs
does not make any sense.insert_txouts
test in test_tx_graph.rs
needs redoing.
How does this line even compile: bdk/crates/chain/tests/test_tx_graph.rs Line 97 in 7aca884
If |
We will remove |
We aren't enforcing the |
There is no |
yeh, I misread it. |
Describe the bug
bdk/crates/chain/tests/test_tx_graph.rs
Lines 16 to 17 in 7aca884
This test misunderstands
Anchor
s. There is no such thing as an "unconfirmed anchor", andChainPosition
does not implementAnchor
. We should rewrite this test to be comprehensive.Proposed test scenarios
A
, then insert partial txout(s) of the same txid. Expect: No changes toTxGraph
(full tx should still exist in graph). Returned changeset is empty.B
. Then insert full tx with txidB
. Expect: the full tx should replace the partial tx. Check graph and returned changeset to confirm this.The text was updated successfully, but these errors were encountered: