[test] turn app into navigator #304
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
spalmer25
changed the base branch from
main
to
palmer@functori@sort-and-merge-nano-tests
spalmer25
force-pushed
the
palmer@functori@sort-and-merge-nano-tests
branch
from
8b81fc8 to
474d918
Compare
spalmer25
force-pushed
the
palmer@functori@test-turn-app-into-navigator
branch
from
d28f0b4 to
2982d2a
Compare
spalmer25
force-pushed
the
palmer@functori@test-turn-app-into-navigator
branch
from
2982d2a to
5c00ba0
Compare
spalmer25
force-pushed
the
palmer@functori@sort-and-merge-nano-tests
branch
from
474d918 to
17226a0
Compare
spalmer25
force-pushed
the
palmer@functori@test-turn-app-into-navigator
branch
5 times, most recently
from
08c07a7 to
8f1126b
Compare
spalmer25
force-pushed
the
palmer@functori@sort-and-merge-nano-tests
branch
from
17226a0 to
666cf09
Compare
spalmer25
force-pushed
the
palmer@functori@sort-and-merge-nano-tests
branch
from
666cf09 to
b505dfa
Compare
spalmer25
force-pushed
the
palmer@functori@test-turn-app-into-navigator
branch
from
8f1126b to
b1c4f9b
Compare
spalmer25
force-pushed
the
palmer@functori@sort-and-merge-nano-tests
branch
from
b505dfa to
6a3d0d6
Compare
encode signature as pytezos do
This allow easy public key check
spalmer25
force-pushed
the
palmer@functori@test-turn-app-into-navigator
branch
6 times, most recently
from
5611fab to
568e8ce
Compare
This allow a better error handling for send-and-navigate
Instead of using it directly + Move DEFAULT_ACCOUNT and DEFAULT_SEED into `account.py`
- move functions and value not related to navigator out of `navigator.py`
- remove `test_nanosp_regression_press_right_works_across_apdu_recieves` that does not make sens anymore
spalmer25
force-pushed
the
palmer@functori@test-turn-app-into-navigator
branch
from
568e8ce to
f3d2381
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
depends on #303
In test, the
TezosAppScreenclass has become a class for batching every function needed for tests.This situation has moved the aim of the tests inside this class, and it has become more difficult to add tests or modify them.
This PR change the responsibility of the
TezosAppScreento araggernavigator for the Tezos app.The
TezosAppScreenclass will be the main code changed when theTouchdevice tests will be included.