Skip to content

test(manual-tests): align native wallet cases with actual UI - #32355

Open
Ondra-Zik wants to merge 1 commit into
developfrom
test/native-manual-wallet-cases
Open

test(manual-tests): align native wallet cases with actual UI#32355
Ondra-Zik wants to merge 1 commit into
developfrom
test/native-manual-wallet-cases

Conversation

@Ondra-Zik

@Ondra-Zik Ondra-Zik commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

Description

Aligns mobile wallet manual tests with what Suite Native actually shows:

  • Drop the EVM input-data step from transaction detail (mobile parameters do not show ethereumSpecific.data)
  • Merge coin enabling from Settings and Receive into one case
  • Stop expecting a fee on the device for a native TRX transfer (fee=None in firmware)
  • Remove the ERC20 contract name / fiat-price case; token name, value, and explorer coverage already live in erc20TransactionDetail / tokenManagement
  • Remove cancel-asset-importing (< / X); it only checks generic back/close navigation on XPUB import
  • Retarget custom backend to Electrum (Bitcoin) and RPC (Ethereum) and assign it to the Network stream

Notes for QA

These are skipped Detox annotations only. No app behavior change. Testhive / manual run lists should pick up the renamed coin-enabling and custom-backend cases and the deletions.

Related Issue

N/A

Screenshots:

N/A

🔍 Currents Test Results

🔍 Suite native android test results: View in Currents

🔍 Suite desktop test results: View in Currents

🔍 Suite web test results: View in Currents

🌐 Preview deployments

🌐 Suite Web preview: https://dev.suite.sldev.cz/suite-web/test/native-manual-wallet-cases/web/

🔒 Quarantined E2E Tests

Trezor Suite (web) — 3 test(s)
Test Type
Quarantine test: "Suite Sync - Quota Manager top-up,Exceeded wallet limit is topped up from the device pool" 🙋 manual
Trading - Swap SPL token to coin via CEX > Swap USDT to SOL via CEX 🤖 auto
Trading - Swap > Swap SOL USDT token to ETH 🤖 auto

Updated: 2026-09-10T16:03:50.283Z • 3 test(s) total

Trezor Suite (desktop) — 3 test(s)
Test Type
Trading - Sell BTC > Sell Bitcoin for best offer 🤖 auto
Quarantine test: "Suite Sync - Quota Manager top-up,Exceeded wallet limit is topped up from the device pool" 🙋 manual
Trading - Swap > Swap SOL USDT token to ETH 🤖 auto

Updated: 2026-09-10T16:03:40.860Z • 3 test(s) total

@Ondra-Zik
Ondra-Zik marked this pull request as ready for review September 10, 2026 15:56
@Ondra-Zik
Ondra-Zik force-pushed the test/native-manual-wallet-cases branch from 1d75497 to 7270432 Compare September 10, 2026 15:57
@trezor-bot

trezor-bot Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

✅ Previously successful run of [Test] suite-native Android E2E workflow has been found.
⏭️ Skipping tests for this run.
💡 If you are unsure about your latest changes, please rerun the workflow manually. (Use the Re-run all jobs option)

Drop steps and cases that don't exist or are already covered on mobile,
merge coin enabling from settings and receive, retarget custom backends
to Electrum and RPC on the Network stream, and stop expecting a TRX fee
on the device.
@Ondra-Zik
Ondra-Zik force-pushed the test/native-manual-wallet-cases branch from 7270432 to f7312e5 Compare September 10, 2026 15:58
@Ondra-Zik
Ondra-Zik removed request for a team, jvaclavik and seibei-iguchi September 10, 2026 15:58

@EuryShadow EuryShadow left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions

github-actions Bot commented Sep 10, 2026

Copy link
Copy Markdown

⚡️ Performance report (alpha)

Note

Alpha release: The limits are still being calibrated, and each measurement is
usually a single run on a shared CI runner, so expect noise between runs. Treat a
number as a hint to look closer, not as a verdict.

desktop / group 1

🟢 Within limits.

Each metric reads measured/limit (% of limit).

Scenario TBT Long tasks Longest task React commits Interaction
account-switch [T3W1] 239/700 ms
(34 %)
3/5
(60 %)
278/700 ms
(40 %)
33/80
(41 %)
734 ms
multi-account-discovery [T3W1] 1190/7000 ms
(17 %)
40/100
(40 %)
249/800 ms
(31 %)
224/400
(56 %)
6791 ms
wallet-discovery [T3W1] 555/2000 ms
(28 %)
19/45
(42 %)
170/400 ms
(43 %)
160/410
(39 %)
5242 ms

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants