Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Oxen test fixes and improvements #17

Merged
merged 17 commits into from
May 24, 2023
Merged

Conversation

jagerman
Copy link

@jagerman jagerman commented May 3, 2023

This PR supercedes #16 and adds other some other small changes and capabilities to go with the high-level Oxen wallet ledger test suite (oxen-io/oxen-core#1640).

darcys22 and others added 12 commits April 22, 2023 10:00
To avoid a compiler warning about the fall-through.
This lets the device be forcibly reset to a specific network type by the
companion software wallet.  This command is *only* permitted when the
wallet is compiled in debug mode.

This also allows putting the wallet into fakechain mode, which is needed
for network tests, and allows doing so without needing (or exposing) a
fakechain setting.
Fixes various Ledger display messages that confusing reference LNS
instead of ONS.  This also changes all the constants along the way (to
make it consistent with the Oxen core code, which renamed them as such a
long time ago).
@fbeutin-ledger
Copy link

Hello @jagerman ,
Thanks for the contribution, this PR looks good to me, you'll need to bump the version however

Also you might be interested in the test framework we developed, it's a pytest based framework that handles the Speculos, automatic screen navigation, screenshot comparison, etc
https://github.com/LedgerHQ/ragger
Seen in use here for example
https://github.com/LedgerHQ/app-boilerplate/tree/master/tests
Let me know if you find it useful or have remarks

0x200 hurts nothing on non-X, and 0x40 should be there on X for
being able to request PIN verification.  (This also matches the flags
used, now, in the Monero app and various other official apps).
This was necessary in older Nano S SDK, but apparently isn't needed
anymore.
@fbeutin-ledger
Copy link

Hi @jagerman can you please add the guidelines_enforcer as done here :
https://github.com/LedgerHQ/app-boilerplate/blob/master/.github/workflows/guidelines_enforcer.yml
In a follow-up PR ?

@fbeutin-ledger fbeutin-ledger merged commit 5d7b1f3 into LedgerHQ:develop May 24, 2023
5 checks passed
@fbeutin-ledger
Copy link

fbeutin-ledger commented May 24, 2023

Hello @jagerman , I've taken the liberty of bumping the version number, otherwise we can't deploy on our provider
#18
Not sure why the CI is starting to fail now.. Can you please have a look ? Thanks

@jagerman
Copy link
Author

cc @darcys22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants