73238135d6
Based on original contribution by Max Kupriianov <xlab@hey.com> Implemented EIP-712 typed data signatures in Ethereum app. Add eth_abi into pyproject deps device test for EIP 712 fixed hex decoding for address fixup! fixed hex decoding for address code quality, more pythonic code, removing unused imports running black and isort on changed files trezorctl file input for EIP 712 data signing fixup! code quality, more pythonic code, removing unused imports fixup! fixup! code quality, more pythonic code, removing unused imports necessary changes after rebase to master unit tests for sign_typed_data.py new protobuf messages, working for nonarray types simplified and verified solution for our simple data support for simple arrays, without their confirmation reverting protobuf value messages to bytes, appropriate changes showing arrays in Trezor, code quality improvements data validation on Trezor, minor improvements using custom types for storing type data instead of dicts, addressing feedback from review moving helper functions to its own file, tests for decode_data additional overall tests support for arrays of structs adding support for metamask_v4_compat variable using HashWriter object to collect the final hash continously minor improvements in code quality validate_field_type function streaming values from client without saving them, missing UI prototype of streamed UI using confirm_properties accounting for bytes in data, more data types in integration tests rebase on master, using f-strings minor fixes and improvements from code review StructHasher class for the whole hashing process mypy and style changes asking users whether to show structs and arrays protobuf descriptions to fix make defs_check unifying comments, mypy fix unit tests for StructHasher class UI fixtures, skipping device tests for T1 addressing majority of code review comments about code quality and structure changing file structure - layouts, helpers, sign_typed_data decode_data renaming and docstring, renaming unit test file using tuples instead of lists in elifs layout improvements excluding core/src/apps/common/confirm.py file from the PR True/False returning layout with Show more button code review layout improvements forgotten br_type argument to should_show_more |
||
---|---|---|
.. | ||
cardano | ||
ethereum | ||
__init__.py | ||
.gitignore | ||
data_webauthn.py | ||
README.md | ||
signtx.py | ||
test_autolock.py | ||
test_basic.py | ||
test_bip32_speed.py | ||
test_cancel.py | ||
test_cosi.py | ||
test_debuglink.py | ||
test_descriptors.py | ||
test_msg_applysettings.py | ||
test_msg_authorize_coinjoin.py | ||
test_msg_backup_device.py | ||
test_msg_binance_get_address.py | ||
test_msg_binance_get_public_key.py | ||
test_msg_binance_sign_tx.py | ||
test_msg_change_wipe_code_t1.py | ||
test_msg_change_wipe_code_t2.py | ||
test_msg_changepin_t2.py | ||
test_msg_changepin.py | ||
test_msg_cipherkeyvalue.py | ||
test_msg_eos_get_public_key.py | ||
test_msg_eos_signtx.py | ||
test_msg_getaddress_segwit_native.py | ||
test_msg_getaddress_segwit.py | ||
test_msg_getaddress_show.py | ||
test_msg_getaddress.py | ||
test_msg_getecdhsessionkey.py | ||
test_msg_getentropy.py | ||
test_msg_getownershipproof.py | ||
test_msg_getpublickey_curve.py | ||
test_msg_getpublickey.py | ||
test_msg_loaddevice.py | ||
test_msg_monero_getaddress.py | ||
test_msg_monero_getwatchkey.py | ||
test_msg_nem_getaddress.py | ||
test_msg_nem_signtx_mosaics_t2.py | ||
test_msg_nem_signtx_mosaics.py | ||
test_msg_nem_signtx_multisig.py | ||
test_msg_nem_signtx_others.py | ||
test_msg_nem_signtx_transfers.py | ||
test_msg_ping.py | ||
test_msg_recoverydevice_bip39_dryrun.py | ||
test_msg_recoverydevice_bip39_t2.py | ||
test_msg_recoverydevice_bip39.py | ||
test_msg_recoverydevice_slip39_advanced_dryrun.py | ||
test_msg_recoverydevice_slip39_advanced.py | ||
test_msg_recoverydevice_slip39_basic_dryrun.py | ||
test_msg_recoverydevice_slip39_basic.py | ||
test_msg_resetdevice_bip39_skipbackup.py | ||
test_msg_resetdevice_bip39_t2.py | ||
test_msg_resetdevice_bip39.py | ||
test_msg_resetdevice_slip39_advanced.py | ||
test_msg_resetdevice_slip39_basic.py | ||
test_msg_ripple_get_address.py | ||
test_msg_ripple_sign_tx.py | ||
test_msg_sd_protect.py | ||
test_msg_signidentity.py | ||
test_msg_signmessage.py | ||
test_msg_signtx_amount_unit.py | ||
test_msg_signtx_bcash.py | ||
test_msg_signtx_bgold.py | ||
test_msg_signtx_dash.py | ||
test_msg_signtx_decred.py | ||
test_msg_signtx_external.py | ||
test_msg_signtx_firo.py | ||
test_msg_signtx_grs.py | ||
test_msg_signtx_invalid_path.py | ||
test_msg_signtx_komodo.py | ||
test_msg_signtx_mixed_inputs.py | ||
test_msg_signtx_peercoin.py | ||
test_msg_signtx_prevhash.py | ||
test_msg_signtx_replacement.py | ||
test_msg_signtx_segwit_native.py | ||
test_msg_signtx_segwit.py | ||
test_msg_signtx_zcash.py | ||
test_msg_signtx.py | ||
test_msg_tezos_getaddress.py | ||
test_msg_tezos_getpublickey.py | ||
test_msg_tezos_sign_tx.py | ||
test_msg_verifymessage_segwit_native.py | ||
test_msg_verifymessage_segwit.py | ||
test_msg_verifymessage.py | ||
test_msg_webauthn.py | ||
test_msg_wipedevice.py | ||
test_multisig_change.py | ||
test_multisig.py | ||
test_nonstandard_paths.py | ||
test_op_return.py | ||
test_passphrase_slip39_advanced.py | ||
test_passphrase_slip39_basic.py | ||
test_pin.py | ||
test_protection_levels.py | ||
test_reset_backup.py | ||
test_reset_recovery_bip39.py | ||
test_reset_recovery_slip39_advanced.py | ||
test_reset_recovery_slip39_basic.py | ||
test_sdcard.py | ||
test_session_id_and_passphrase.py | ||
test_session.py | ||
test_stellar.py | ||
test_u2f_counter.py |
Running device tests
1. Running the full test suite
Note: You need Poetry, as mentioned in the core's documentation section.
In the trezor-firmware
checkout, in the root of the monorepo, install the environment:
poetry install
And run the automated tests:
poetry run make -C core test_emu
2. Running tests manually
Install the poetry environment as outlined above. Then switch to a shell inside the environment:
poetry shell
If you want to test against the emulator, run it in a separate terminal:
./core/emu.py
Now you can run the test suite with pytest
from the root directory:
pytest tests/device_tests
Useful Tips
The tests are randomized using the pytest-random-order plugin. The random seed is printed in the header of the tests output, in case you need to run the tests in the same order.
If you only want to run a particular test, pick it with -k <keyword>
or -m <marker>
:
pytest -k nem # only runs tests that have "nem" in the name
pytest -m stellar # only runs tests marked with @pytest.mark.stellar
If you want to see debugging information and protocol dumps, run with -v
.
If you would like to interact with the device (i.e. press the buttons yourself), just prefix pytest with INTERACT=1
:
INTERACT=1 pytest tests/device_tests
3. Using markers
When you're developing a new currency, you should mark all tests that belong to that currency. For example, if your currency is called NewCoin, your device tests should have the following marker:
@pytest.mark.newcoin
This marker must be registered in REGISTERED_MARKERS file.
If you wish to run a test only on TT, mark it with @pytest.mark.skip_t1
.
If the test should only run on T1, mark it with @pytest.mark.skip_t2
.
You must not use both on the same test.
Extended testing and debugging
Building for debugging (Emulator only)
Build the debuggable unix binary so you can attach the gdb or lldb. This removes optimizations and reduces address space randomizaiton.
make build_unix_debug
The final executable is significantly slower due to ASAN(Address Sanitizer) integration. If you wan't to catch some memory errors use this.
time ASAN_OPTIONS=verbosity=1:detect_invalid_pointer_pairs=1:strict_init_order=true:strict_string_checks=true TREZOR_PROFILE="" poetry run make test_emu
Coverage (Emulator only)
Get the Python code coverage report.
If you want to get HTML/console summary output you need to install the coverage.py tool.
pip3 install coverage
Run the tests with coverage output.
make build_unix && make coverage