trezor / python-trezor

:snake: Don't use this repo, use the new monorepo instead:
https://github.com/trezor/trezor-firmware
GNU Lesser General Public License v3.0
201 stars 194 forks source link

Add a test case for segwit inputs/outputs with very high amounts #337

Closed strmci closed 5 years ago

strmci commented 5 years ago

Add a test case for segwit inputs/outputs with very high amount, solves issue #332

matejcik commented 5 years ago

you will need to run make style to fix the style warnings in Travis

given that this is a SegWit test, you should move it to test_msg_signtx_segwit.py

Maybe it's worth it to have a non-segwit test for the same thing too, using the transactions you found earlier? I'm not sure if Trezor will let you sign that though.

matejcik commented 5 years ago

looking good, thanks!