Closed
Description
openedon Feb 9, 2023
Summary
Perform manual tests with a real Ledger device.
Problem
SIGN_MODE_TEXTUAL's implementation is almost finished, so we are entering the QA phase.
An external audit is happening for Textual (see #11970). After discussing internally, we decided to not wait for the external audit final report to submit a PR to the LedgerHQ repo.
This issue tracks the final manual testing to be done before we submit the app to Ledger.
Manual Tests
Testing on the main branch won't work, as there are some switches that are turned off for Textual. See this diff on how to turn on Textual.
Note: maybe interesting to compare number of screens of amino vs textual on the same tx.
Name | date | tester & device | simd command | comments | #screens (non-expert) | #screens amino | video |
---|---|---|---|---|---|---|---|
MsgDelegate, MsgSend, MsgCreateValidtor | 14.03.2023 | @AmauryM Nano S | Tested the new struct sign doc, tested dynamicpb | ||||
MsgDelegate | 22.02.2023 | @AmauryM Nano S | simd tx staking delegate --from nanos <oper_addr> 10stake |
Fixed the MsgDelegate bug from 14.02 | 14 | 11 | |
With fees and (unicode) memo | 14.02.2023 | @AmauryM Nano S | simd tx bank send --fees 1stake --memo "💪" |
Both memo and fees are shown | 14 | 12 | |
MsgSubmitProposal too big payload | 14.02.2023 | @AmauryM Nano S | 200kb proposal json | Error: hidapi: unknown failure in the CLI |
|||
MsgSubmitProposal with special chars | 14.02.2023 | @AmauryM Nano S | Proposal: {"title": "A 🚗 ","summary": "B\n @"} (trailing whitespaces, trailing @, unicodes) |
All good. | 13 | 12 | video |
MsgDelegate | 14.02.2023 | @AmauryM Nano S | simd tx staking delegate --from nanos <oper_addr> 10stake |
There's one screen with Message (2/2) [1/2]Message (1/1) [1/2] (with last 3 chars truncated), which is confusing |
14 | 11 | video |
MsgSend | 14.02.2023 | @AmauryM Nano S | simd tx bank send nanos <addr> 10stake |
Verified that the tx can be broadcasted and included in a block (passes antehandlers) on a local node. | 12 | 11 | |
MsgMultiSend | 14.02.2023 | @AmauryM Nano S | simd tx bank multi-send nanos <addr1> <addr2> 10stake |
Confusion between Outputs (2/2) and Address [2/2] ? |
22 | doesn't work | |
Expert Mode | 31.01.2023 | @AmauryM Nano S | msg send | Verified all screens are shown (address, gas, hash, pubkey) | video | ||
Nested Authz MsgExec | 31.01.2023 | @AmauryM Nano S | simd tx authz exec tx.json --sign-mode textual with a tx.json containing a MsgSend |
Verified that 1. > for indentation are correctly shown and 2. Long field names are shown in the content part, not the title part |
video |
TODO:
- Basic MsgSend
- Basic MsgDelegate
- Nested Authz MsgExec
- Unicode characters
- Long field names
- TX payload that is too large
- Multi-signer tx
- All screens are shown in expert mode
- Amino still works
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Metadata
Assignees
Labels
No labels