qa: Proposed additional QA checks for template #20

New issue

Closed

opened 2026-03-14 03:18:50 +00:00 by Brouie · 8 comments

Brouie commented

2026-03-14 03:18:50 +00:00

(Migrated from gitlab.com)

Proposed Additional QA Test Cases

Based on bugs found during QA passes, these test cases should be added to QA_TEMPLATE.md:

Swap — Decimals & Amounts (add to Section 2)

#	Test Case	Expected
2.7.1	On-chain amount matches UI input	Swap 100 tokens → on-chain balance decreases by exactly 100 × 10^decimals
2.7.2	Balance refreshes after swap	Balance updates in UI without manual page refresh
2.7.3	MAX button sends correct scaled amount	Click MAX → swap → full balance deducted
2.7.4	Multiple rapid swaps	Execute 3 swaps in quick succession — all succeed, balances correct
2.7.5	Swap on all 20 pairs	Cycle through every pair, execute small swap on each
2.7.6	Gas estimation accuracy	gasUsed should not exceed gasWanted

Pool — Decimals & Amounts (add to Section 3)

#	Test Case	Expected
3.4.1	Provide liquidity amount matches on-chain	Add 100 tokens → pool reserves increase by 100 × 10^decimals
3.4.2	Withdraw returns correct scaled amount	Withdraw LP → returned tokens match expected
3.4.3	Add/remove on multiple pairs	Test provide + withdraw on 3+ different pairs

Wallet — Persistence & Edge Cases (add to Section 1)

#	Test Case	Expected
1.9.1	Simulated wallet note	Note: simulated wallet does not persist on refresh (by design)
1.9.2	Keplr chain suggestion	experimentalSuggestChain fires on first connect, chain appears in Keplr
1.9.3	Wallet reconnect retry	Auto-reconnect retries up to 3x with backoff on transient failures
1.9.4	Clear localStorage	Clear cl8y_wallet_connection → reload → no crash, shows Connect Wallet

Configuration & Deploy (add to Section 18)

#	Test Case	Expected
18.3.1	Deploy script generates .env.local	make deploy-local writes correct contract addresses
18.3.2	Deploy script generates indexer/.env	Indexer env written with correct values
18.3.3	CSP auto-detects host IP	cspDevHosts plugin injects VPS IP into CSP during vite dev
18.3.4	NETWORKS.local uses env vars	Keplr suggestChain gets LCD/RPC from VITE_ env vars, not hardcoded localhost

Error Handling (add to Section 12)

#	Test Case	Expected
12.4.1	Chain stopped producing blocks	Frontend shows timeout/error, does not hang
12.4.2	Account sequence mismatch	Clear error message, not silent failure
12.4.3	Wallet not funded on localterra	Clear error: account not found message

Settings — Token Verification (add to Section 13 equivalent for DEX)

#	Test Case	Expected
NEW.1	LUNC native token in verification	Expected failures — native denom, not CW20 contract

Fee Tiers (add to Section 4)

#	Test Case	Expected
4.4.1	Fee CTA link on swap page	'Hold CL8Y to reduce swap fees' navigates to /tiers
4.4.2	Register with insufficient CL8Y	Clear error with required vs actual balance

Total: 22 new test cases proposed.

@PlasticDigits — let me know if you want me to PR these into QA_TEMPLATE.md directly.

## Proposed Additional QA Test Cases Based on bugs found during QA passes, these test cases should be added to QA_TEMPLATE.md: ### Swap — Decimals & Amounts (add to Section 2) | # | Test Case | Expected | |---|-----------|----------| | 2.7.1 | On-chain amount matches UI input | Swap 100 tokens → on-chain balance decreases by exactly 100 × 10^decimals | | 2.7.2 | Balance refreshes after swap | Balance updates in UI without manual page refresh | | 2.7.3 | MAX button sends correct scaled amount | Click MAX → swap → full balance deducted | | 2.7.4 | Multiple rapid swaps | Execute 3 swaps in quick succession — all succeed, balances correct | | 2.7.5 | Swap on all 20 pairs | Cycle through every pair, execute small swap on each | | 2.7.6 | Gas estimation accuracy | gasUsed should not exceed gasWanted | ### Pool — Decimals & Amounts (add to Section 3) | # | Test Case | Expected | |---|-----------|----------| | 3.4.1 | Provide liquidity amount matches on-chain | Add 100 tokens → pool reserves increase by 100 × 10^decimals | | 3.4.2 | Withdraw returns correct scaled amount | Withdraw LP → returned tokens match expected | | 3.4.3 | Add/remove on multiple pairs | Test provide + withdraw on 3+ different pairs | ### Wallet — Persistence & Edge Cases (add to Section 1) | # | Test Case | Expected | |---|-----------|----------| | 1.9.1 | Simulated wallet note | Note: simulated wallet does not persist on refresh (by design) | | 1.9.2 | Keplr chain suggestion | experimentalSuggestChain fires on first connect, chain appears in Keplr | | 1.9.3 | Wallet reconnect retry | Auto-reconnect retries up to 3x with backoff on transient failures | | 1.9.4 | Clear localStorage | Clear cl8y_wallet_connection → reload → no crash, shows Connect Wallet | ### Configuration & Deploy (add to Section 18) | # | Test Case | Expected | |---|-----------|----------| | 18.3.1 | Deploy script generates .env.local | make deploy-local writes correct contract addresses | | 18.3.2 | Deploy script generates indexer/.env | Indexer env written with correct values | | 18.3.3 | CSP auto-detects host IP | cspDevHosts plugin injects VPS IP into CSP during vite dev | | 18.3.4 | NETWORKS.local uses env vars | Keplr suggestChain gets LCD/RPC from VITE_ env vars, not hardcoded localhost | ### Error Handling (add to Section 12) | # | Test Case | Expected | |---|-----------|----------| | 12.4.1 | Chain stopped producing blocks | Frontend shows timeout/error, does not hang | | 12.4.2 | Account sequence mismatch | Clear error message, not silent failure | | 12.4.3 | Wallet not funded on localterra | Clear error: account not found message | ### Settings — Token Verification (add to Section 13 equivalent for DEX) | # | Test Case | Expected | |---|-----------|----------| | NEW.1 | LUNC native token in verification | Expected failures — native denom, not CW20 contract | ### Fee Tiers (add to Section 4) | # | Test Case | Expected | |---|-----------|----------| | 4.4.1 | Fee CTA link on swap page | 'Hold CL8Y to reduce swap fees' navigates to /tiers | | 4.4.2 | Register with insufficient CL8Y | Clear error with required vs actual balance | Total: 22 new test cases proposed. @PlasticDigits — let me know if you want me to PR these into QA_TEMPLATE.md directly.

Brouie commented

2026-03-14 03:19:28 +00:00

(Migrated from gitlab.com)

22 new test cases proposed based on bugs found during QA. Happy to PR these into QA_TEMPLATE.md if you want. @PlasticDigits