LVFM-488: Major rewrite of the Bitcoin app APDU-level tests, using a in-house transaction parser#157

Open

hcleonis wants to merge 12 commits intomasterfrom

Commits on Jul 5, 2020

Add transaction parser utility (BTC, segwit, Zcash for now)
hcleonis
committed
Rationalize helpers module file structure
hcleonis
committed
Helpers: Add recognition of overwinter, pre-sapling Zcash tx to TxParse, Fix parser issues related to GetTrustedInput, reworked sendApdus() method, Update txparser, Helpers reworked to make segwit …
hcleonis
committed
Update GetTrustedInput test to use new BTC tx parser
hcleonis
committed
Test: remove old methods calls
hcleonis
committed
Tests: method names refactoring in tests, BTC test updated to remove offsets, Removed some more indexes + fixed issues in segwit inputs script handling, Fixed Zcash Overwinter tests + adapted other…
hcleonis
committed
Tests: add README
hcleonis
committed

Commits on Jul 6, 2020

Update tests/README.md with additional info of test framework & suggested next steps
hcleonis
committed

Commits on Jul 7, 2020

Applied @onyb & @grydz review remarks
hcleonis
committed
Add forgotten review fix
hcleonis
committed

Commits on Jul 9, 2020

Modifications following review from @Yass
hcleonis
committed
Review fix: guard againt hashing a None transaction
hcleonis
committed