-
Notifications
You must be signed in to change notification settings - Fork 45
Conversation
A bunch of things are resolved, is there an unpushed commit? |
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.
Last to do item:
Tests for Validate()
for both structs
@prestwich Tests are finished, but the |
Now just need to add tests for the error cases :) |
resolved some type issues organized Validate and cleaned up code/errors finished Validate function for BitcoinHeader added ConfirmingHeader.Validate() call added documentation comments for validate functions finished testing functions, think data may be wrong tests: fix invalid proof in test vector mostly finished testing fixed for loops removed todos validate-function Restructure data (#67) * restructured data, all tests passing * deleted commented sections * simplified code * reverted back, didn't work * refactor: remove extra unmarshal step from tests * bug: nbsps in error messages somehow? bizarre. bug: vouts were not always 0x prefixed in proof tests (#68) * bug: vouts were not always 0x prefixed in proof tests * bug: remove debug print added valid proof with invalid header test case
8febbb3
to
1cbe9a7
Compare
No description provided.