1
0
mirror of https://github.com/trezor/trezor-firmware.git synced 2024-12-23 14:58:09 +00:00
Commit Graph

8810 Commits

Author SHA1 Message Date
matejcik
1cc1382153 python/trezorctl: improve get-session 2020-02-12 17:18:50 +01:00
matejcik
b6fca537c9 tests: remove wipe_code_activate test fixture 2020-02-12 15:45:17 +01:00
matejcik
81a03edf61 python/debuglink: add docstrings, rename functions for clearer usage 2020-02-12 15:38:18 +01:00
matejcik
4c8c96272c emu: fix flag options with defaults
Click REALLY INSISTS you provide on/off switches for your options.
You can use is_flag, but then the presence of the option changes based
on the default value.

Which makes sense, really:
@option("-f", "foobar", is_flag=True, default=False)
you would expect `./cli -f` to have `foobar is True`

whereas with
@option("-f", "foobar", is_flag=True, default=True)
you would expect `./cli -f` to have `foobar is False`, otherwise it's a
no-op

this becomes fun with `default=os.environ.get("SOMETHING")`, because
then the effect of the option CHANGES with a value of environment
variable!

there's two ways around this:
a) don't use defaults, update the flag explicitly, like:
   foobar = foobar or os.environ.get("FOOBAR") == "1"
b) forget about is_flag and specify an on/off switch, where the default
   value works as intended

since the latter is also technically speaking more correct, i'm doing it
2020-02-12 13:32:05 +01:00
matejcik
271da3fa39 python: add detailed logging to emulator runner 2020-02-12 13:31:58 +01:00
matejcik
1bcf856946 tests: simplify change_wipe_code test with PIN queuing 2020-02-12 13:31:31 +01:00
matejcik
ba3d90b994 tests: add wipe code activation test 2020-02-12 13:31:31 +01:00
matejcik
7a253a6c0b python/debuglink: properly clean up at end of "with client" 2020-02-12 13:31:31 +01:00
matejcik
c14429c445 all: shut down emulator on error_shutdown 2020-02-12 13:31:31 +01:00
Tomas Susanka
b4846e484a docs/common: add links and fix some typos 2020-02-12 09:45:33 +00:00
matejcik
d3b88a37be core: do not catch SystemExit in handle_session (#826) 2020-02-12 10:36:42 +01:00
matejcik
94b85efba1 python/debuglink: make pin sequences configurable 2020-02-12 10:36:42 +01:00
Tomas Susanka
f947fe97cc core: fix style 2020-02-11 16:41:43 +01:00
Tomas Susanka
0a13f7a441 core: properly limit passphrase to 50 bytes 2020-02-11 15:39:08 +00:00
Tomas Susanka
2c0504ad1c
Merge pull request #803 from trezor/passphrase
Passphrase Redesign
2020-02-11 16:01:59 +01:00
matejcik
1ebaaa7e77 python/emulator: increase tolerances
This should help in CI where things can sometimes take a long time.
2020-02-11 13:20:35 +01:00
matejcik
a375460afc python/emulator: fix timeout parameter in emulator.wait() 2020-02-11 13:20:35 +01:00
matejcik
3d1fefe38a legacy: make sure nanopb_generator is ready 2020-02-11 13:19:37 +01:00
matejcik
05a1d6f771 core/debug: add "show arbitrary screen" capability, for easier prototyping 2020-02-11 11:53:34 +01:00
matejcik
8c75c94125 ci: ensure system Python 3.7 is used even if other pythons are available 2020-02-11 11:53:34 +01:00
matejcik
c01d04f26e common/protob: drop a default field declaration
which triggers nanopb bug https://github.com/nanopb/nanopb/issues/487
2020-02-11 11:53:34 +01:00
matejcik
37ab510622 ci: remove sleeps for coverage, restore Monero
with the new emulator runner, this should be more stable

This reverts commit 1480f5a25c.
2020-02-11 11:19:56 +01:00
matejcik
741c0c8888 core: improve stability of Monero test runner 2020-02-11 11:16:28 +01:00
Tomas Susanka
8a499b3067 ci: run coverage only for core 2020-02-10 16:12:46 +00:00
Tomas Susanka
f4c38af825 ci: use 'extends' instead of yaml anchors
We might refactor the ci folder a bit more using this but I believe it
is fine for now.

Should solve #835.
2020-02-10 16:12:46 +00:00
matejcik
4b0d8d5f6d python/trezorctl: tweak error message in get-session 2020-02-10 16:22:48 +01:00
matejcik
7f63cf28b0 python: only set session_id from Features on firmwares that send it 2020-02-10 16:15:45 +01:00
matejcik
6dace16033 python: update changelog 2020-02-10 16:14:27 +01:00
matejcik
d4343ad8b7 python/trezorctl: add get-session 2020-02-10 16:14:21 +01:00
matejcik
64838bca2d tests: fix style 2020-02-10 15:44:13 +01:00
matejcik
8d93503c9c python: do not send deprecated state in PassphraseAck on T1 2020-02-10 15:38:26 +01:00
matejcik
ff702c7d7f common: add proper markers for deprecated messages 2020-02-10 15:38:26 +01:00
matejcik
41aa818c4c tests: shut down test suite if no device is available
this seems better than erroring out dozens of tests
2020-02-10 15:38:26 +01:00
Tomas Susanka
82c7b56875
Merge pull request #837 from trezor/tsusanka/docs-build-update
Docs/core: Improve build section
2020-02-10 13:53:46 +01:00
Tomas Susanka
e4077e0634 docs/core: improve build section 2020-02-10 13:05:41 +01:00
matejcik
f7608f55ff python: implement backwards compatibility for pre-2.3.0 firmwares 2020-02-10 13:01:47 +01:00
matejcik
cfe0e34215 tests: ignore trezor.log 2020-02-10 12:36:53 +01:00
matejcik
4c9689d7a7 common: return deprecated fields to protobuf, to allow cross-version compatible code 2020-02-10 12:35:28 +01:00
Pavol Rusnak
e58225fe02
core/boot: don't call mkfs in emulator 2020-02-07 15:30:21 +00:00
Pavol Rusnak
fdfa64ac2a
core/ui: fix HoldToConfirm redraw 2020-02-07 15:21:16 +00:00
Pavol Rusnak
32f8f1cb61
core/embed: update fatfs to 0.14; use upstream version 2020-02-07 15:21:16 +00:00
Tomas Susanka
56dc952495 common: fix PassphraseAck.on_device id 2020-02-07 14:19:01 +00:00
Pavol Rusnak
73ed2f3450
legacy: small nitpicks related to the new passphrase handling 2020-02-07 11:41:03 +00:00
Tomas Susanka
15ed5cd19e
core: do not clear cache on ClearSession 2020-02-07 11:41:03 +00:00
Tomas Susanka
da4743c234
tests/ui: regenerate records
It seems counterintuitive that we need to regenerate ui fixtures because
of some work on passphrase.

The reason is that we have changed where the state (now session id) is
generated. We have moved it to Features, so it is generated already
after the Initialize message is received. It used to be generated later
- during the PassphraseReq/Ack workflow and in some tests we ask for
randomness between those steps.
2020-02-07 11:41:03 +00:00
Tomas Susanka
5717e647ed
tests/device: remove test_expected_responses 2020-02-07 11:41:03 +00:00
matejcik
49727740bc
tests: remove unnecessary passphrase_on_host 2020-02-07 11:41:03 +00:00
matejcik
9c75e27433
tests: unify Ping replacements
for raw calls: just messages.GetAddress()
for library calls: btc.get_address(client, "Testnet", [0])
2020-02-07 11:41:03 +00:00
matejcik
a8cfa8fbed
tests: simplify ClearSession test 2020-02-07 11:41:03 +00:00
Tomas Susanka
fd0dc8ed66
core/passphrase: merge 'get from user' functions 2020-02-07 11:41:03 +00:00