qa: Skipped items recheck — wallet connection + UI verification #19

New issue

Closed

opened 2026-03-14 03:17:26 +00:00 by Brouie · 4 comments

Brouie commented

2026-03-14 03:17:26 +00:00

(Migrated from gitlab.com)

Skipped Items Recheck — 2026-03-14

Wallet Connection on LocalTerra

All non-Keplr wallets fail on localterra with clear error messages (not bugs — localterra not supported):

Wallet	Result	Error
Station	⏭️ SKIP	crypto.randomUUID not available (HTTP, needs HTTPS)
Leap	⏭️ SKIP	Invalid chain id (localterra not recognized)
Cosmostation	⏭️ SKIP	No chain info for localterra
LuncDash (WC)	⏭️ SKIP	localterra not supported
Galaxy Station (WC)	⏭️ SKIP	QR shows but localterra not supported
Keplr	✅ PASS	Works via experimentalSuggestChain
Simulated Dev	✅ PASS	Works

Note: All wallets now display proper error messages instead of failing silently. These will need retesting on testnet/mainnet.

UI Items Previously Skipped — Now Verified

Item	Result	Notes
Sound - button press	✅ PASS	Click sounds play
Sound - hover	✅ PASS	Hover sounds play
Sound - success	✅ PASS	Success sound on swap
Sound - error	✅ PASS	Error sound plays
Custom cursor - default	✅ PASS	Custom cursor visible
Custom cursor - pointer	✅ PASS	Changes on hover

Remaining Skipped Items

Cross-browser (Firefox, Safari, Edge, Brave) — not tested
Wallet tx signing for Station/Leap/Cosmostation — blocked by localterra
WalletConnect flows — blocked by localterra
Memory/performance extended testing — not done

## Skipped Items Recheck — 2026-03-14 ### Wallet Connection on LocalTerra All non-Keplr wallets fail on localterra with clear error messages (not bugs — localterra not supported): | Wallet | Result | Error | |---|---|---| | Station | ⏭️ SKIP | crypto.randomUUID not available (HTTP, needs HTTPS) | | Leap | ⏭️ SKIP | Invalid chain id (localterra not recognized) | | Cosmostation | ⏭️ SKIP | No chain info for localterra | | LuncDash (WC) | ⏭️ SKIP | localterra not supported | | Galaxy Station (WC) | ⏭️ SKIP | QR shows but localterra not supported | | Keplr | ✅ PASS | Works via experimentalSuggestChain | | Simulated Dev | ✅ PASS | Works | Note: All wallets now display proper error messages instead of failing silently. These will need retesting on testnet/mainnet. ### UI Items Previously Skipped — Now Verified | Item | Result | Notes | |---|---|---| | Sound - button press | ✅ PASS | Click sounds play | | Sound - hover | ✅ PASS | Hover sounds play | | Sound - success | ✅ PASS | Success sound on swap | | Sound - error | ✅ PASS | Error sound plays | | Custom cursor - default | ✅ PASS | Custom cursor visible | | Custom cursor - pointer | ✅ PASS | Changes on hover | ### Remaining Skipped Items - Cross-browser (Firefox, Safari, Edge, Brave) — not tested - Wallet tx signing for Station/Leap/Cosmostation — blocked by localterra - WalletConnect flows — blocked by localterra - Memory/performance extended testing — not done

Brouie commented

2026-03-14 03:59:53 +00:00

(Migrated from gitlab.com)

Section 21 Test Results — 2026-03-14

20 PASS / 0 FAIL / 1 SKIP / 1 PASS WITH ISSUES

All 22 proposed test cases executed. Key findings:

Decimals scaling confirmed working (100 EMBER = 100,000,000 micro-units deducted)
Balance refreshes after swap without page reload
MAX button fills correct scaled amount
3 rapid sequential swaps all succeed
Provide + withdraw liquidity amounts match on-chain
Simulated wallet correctly does not persist on refresh
LocalTerra chain freeze at block 100800 filed as #21
NETWORKS.local still hardcodes localhost — MR submitted to fix

## Section 21 Test Results — 2026-03-14 20 PASS / 0 FAIL / 1 SKIP / 1 PASS WITH ISSUES All 22 proposed test cases executed. Key findings: - Decimals scaling confirmed working (100 EMBER = 100,000,000 micro-units deducted) - Balance refreshes after swap without page reload - MAX button fills correct scaled amount - 3 rapid sequential swaps all succeed - Provide + withdraw liquidity amounts match on-chain - Simulated wallet correctly does not persist on refresh - LocalTerra chain freeze at block 100800 filed as #21 - NETWORKS.local still hardcodes localhost — MR submitted to fix

PlasticDigits commented

2026-03-14 05:22:16 +00:00

(Migrated from gitlab.com)

mentioned in commit 434d3c0267

mentioned in commit 434d3c026785ff5562dac4adc196d658fa6d2638

PlasticDigits commented

2026-03-14 05:22:30 +00:00

(Migrated from gitlab.com)

Fix Applied — `434d3c0`

Three of the five failing wallets have been fixed:

What changed

Leap & Cosmostation — These extensions use the exact same Keplr-compatible API (LeapExtension = KeplrExtension, CosmostationExtension = KeplrExtension in cosmes). They all support experimentalSuggestChain() but it was only being called for Keplr. Now suggestChainToExtension() calls it for all three before connecting, so Leap and Cosmostation can learn about the localterra chain.

Station — The Station extension internally calls crypto.randomUUID(), which requires HTTPS (secure context). Added a crypto.randomUUID polyfill in main.tsx that uses crypto.getRandomValues() (available on HTTP) to generate UUIDs before any wallet code loads.

LuncDash & Galaxy Station (WC) — These remain unfixable from our side. LuncDash/Station WalletConnect hardcodes supported chains to columbus-5/phoenix-1/pisco-1 inside the cosmes library, and the mobile apps themselves don't recognize localterra.

Expected results

Wallet	Before	Expected After
Keplr	PASS	PASS
Simulated Dev	PASS	PASS
Leap	SKIP	PASS
Cosmostation	SKIP	PASS
Station	SKIP	PASS
LuncDash (WC)	SKIP	SKIP (mobile app limitation)
Galaxy Station (WC)	SKIP	SKIP (mobile app limitation)

@Brouie — can you re-run the wallet connection QA on LocalTerra to verify Leap, Cosmostation, and Station now connect?

## Fix Applied — 434d3c0 Three of the five failing wallets have been fixed: ### What changed **Leap & Cosmostation** — These extensions use the exact same Keplr-compatible API (`LeapExtension = KeplrExtension`, `CosmostationExtension = KeplrExtension` in cosmes). They all support `experimentalSuggestChain()` but it was only being called for Keplr. Now `suggestChainToExtension()` calls it for all three before connecting, so Leap and Cosmostation can learn about the `localterra` chain. **Station** — The Station extension internally calls `crypto.randomUUID()`, which requires HTTPS (secure context). Added a `crypto.randomUUID` polyfill in `main.tsx` that uses `crypto.getRandomValues()` (available on HTTP) to generate UUIDs before any wallet code loads. **LuncDash & Galaxy Station (WC)** — These remain unfixable from our side. LuncDash/Station WalletConnect hardcodes supported chains to `columbus-5`/`phoenix-1`/`pisco-1` inside the cosmes library, and the mobile apps themselves don't recognize `localterra`. ### Expected results | Wallet | Before | Expected After | |--------|--------|----------------| | Keplr | PASS | PASS | | Simulated Dev | PASS | PASS | | Leap | SKIP | **PASS** | | Cosmostation | SKIP | **PASS** | | Station | SKIP | **PASS** | | LuncDash (WC) | SKIP | SKIP (mobile app limitation) | | Galaxy Station (WC) | SKIP | SKIP (mobile app limitation) | @Brouie — can you re-run the wallet connection QA on LocalTerra to verify Leap, Cosmostation, and Station now connect?

Brouie (Migrated from gitlab.com) closed this issue

2026-03-17 05:04:17 +00:00

Brouie commented

2026-03-17 05:04:26 +00:00

(Migrated from gitlab.com)

Verified fixed — all three wallets now connect on LocalTerra:

Leap: ✅ PASS (experimentalSuggestChain now called)
Cosmostation: ✅ PASS (experimentalSuggestChain now called)
Station: ✅ PASS (crypto.randomUUID polyfill working)
LuncDash (WC): SKIP (mobile app limitation, expected)
Galaxy Station (WC): SKIP (mobile app limitation, expected)

Verified fixed — all three wallets now connect on LocalTerra: - Leap: ✅ PASS (experimentalSuggestChain now called) - Cosmostation: ✅ PASS (experimentalSuggestChain now called) - Station: ✅ PASS (crypto.randomUUID polyfill working) - LuncDash (WC): SKIP (mobile app limitation, expected) - Galaxy Station (WC): SKIP (mobile app limitation, expected)