Skip to content

Commit 014a028

Browse files
Update release test list (#3682)
We're marking some of the tests as optional and some as good candidates for the automation. Once automated, we will be able to remove some of the tests from the list. Latest build: [extension-builds-3682](https://github.com/tahowallet/extension/suites/18522812396/artifacts/1075128205) (as of Mon, 27 Nov 2023 12:38:11 GMT).
2 parents 4fe9acf + eec4828 commit 014a028

File tree

1 file changed

+103
-80
lines changed

1 file changed

+103
-80
lines changed

.github/workflows/test-list/release-test-list.md

+103-80
Original file line numberDiff line numberDiff line change
@@ -20,32 +20,35 @@ This release checklist should be performed before release is published.
2020

2121
## Manual tests
2222

23+
🤖 Items marked with this emoji are good candidates for automation (although
24+
it does not mean that they all would be obsolete once automated).
25+
2326
### 📨 Add account
2427

2528
1. Add read-only account with ENS
26-
- [ ] check assets
27-
- [ ] check balance
28-
- [ ] check NFTs
29-
- [ ] check abilities (abilities should not be displayed)
29+
- [ ] check assets 🤖
30+
- [ ] check balance 🤖
31+
- [ ] check NFTs 🤖
32+
- [ ] check abilities (abilities should not be displayed) 🤖
3033
- [ ] check activities
31-
- [ ] check portfolio page
32-
- [ ] check export options (export recovery phase and export private key should not be available)
34+
- [ ] check portfolio page 🤖
35+
- [ ] check export options (export recovery phrase and export private key should not be available) 🤖
3336
2. Add read-only account with UNS
34-
- [ ] check assets
35-
- [ ] check balance
36-
- [ ] check NFTs
37-
- [ ] check abilities (abilities should not be displayed)
37+
- [ ] check assets 🤖
38+
- [ ] check balance 🤖
39+
- [ ] check NFTs 🤖
40+
- [ ] check abilities (abilities should not be displayed) 🤖
3841
- [ ] check activities
39-
- [ ] check portfolio page
40-
- [ ] check export options (export recovery phase and export private key should not be available)
42+
- [ ] check portfolio page 🤖
43+
- [ ] check export options (export recovery phrase and export private key should not be available) 🤖
4144
3. Add read-only account with 0x address
42-
- [ ] check assets
43-
- [ ] check balance
44-
- [ ] check NFTs
45-
- [ ] check abilities (abilities should not be displayed)
45+
- [ ] check assets 🤖
46+
- [ ] check balance 🤖
47+
- [ ] check NFTs 🤖
48+
- [ ] check abilities (abilities should not be displayed) 🤖
4649
- [ ] check activities
47-
- [ ] check portfolio page
48-
- [ ] check export options (export recovery phase and export private key should not be available)
50+
- [ ] check portfolio page 🤖
51+
- [ ] check export options (export recovery phrse and export private key should not be available) 🤖
4952
4. Import account with a seed phrase
5053
- [ ] check assets
5154
- [ ] check balance
@@ -54,68 +57,74 @@ This release checklist should be performed before release is published.
5457
- [ ] check activities
5558
- [ ] check portfolio page
5659
- [ ] check seed phrase export
57-
- [ ] check private key export for first account
60+
- [ ] check private key export for first account 🤖
5861
5. Add another account from the same seed phrase
5962
- [ ] check assets
6063
- [ ] check balance
6164
- [ ] check NFTs
6265
- [ ] check abilities
6366
- [ ] check activities
6467
- [ ] check portfolio page
65-
- [ ] check private key export for second account
68+
- [ ] check private key export for second account 🤖
6669
6. Add account with a Ledger
6770
- [ ] check assets
6871
- [ ] check balance
6972
- [ ] check NFTs
7073
- [ ] check abilities
7174
- [ ] check activities
7275
- [ ] check portfolio page
73-
- [ ] check export options (export recovery phase and export private key should not be available)
76+
- [ ] check export options (export recovery phrase and export private key should not be available)
7477
7. Create new wallet
75-
- [ ] check assets
76-
- [ ] check balance
77-
- [ ] check NFTs
78-
- [ ] check abilities
79-
- [ ] check activities
80-
- [ ] check portfolio page
81-
- [ ] check private key export
78+
- [ ] check assets 🤖
79+
- [ ] check balance 🤖
80+
- [ ] check NFTs 🤖
81+
- [ ] check abilities 🤖
82+
- [ ] check activities 🤖
83+
- [ ] check portfolio page 🤖
84+
- [ ] check private key export 🤖
8285
8. Add account with private key
8386
- [ ] check assets
8487
- [ ] check balance
8588
- [ ] check NFTs
8689
- [ ] check abilities
8790
- [ ] check activities
8891
- [ ] check portfolio page
89-
- [ ] check private key export
92+
- [ ] check private key export 🤖
9093
9. Add account with JSON keystore
91-
- [ ] check assets
92-
- [ ] check balance
93-
- [ ] check NFTs
94-
- [ ] check abilities
95-
- [ ] check activities
96-
- [ ] check portfolio page
97-
- [ ] check private key export
94+
- [ ] check assets 🤖
95+
- [ ] check balance 🤖
96+
- [ ] check NFTs 🤖
97+
- [ ] check abilities 🤖
98+
- [ ] check activities 🤖
99+
- [ ] check portfolio page 🤖
100+
- [ ] check private key export 🤖
98101

99102
### 🗑️ Remove account
100103

101104
1. Remove account by type
102-
- [ ] remove read-only account
103-
- [ ] remove address from imported account
105+
- [ ] remove read-only account 🤖
106+
- [ ] remove address from imported account 🤖
104107
- [ ] remove Ledger account
105-
- [ ] remove account imported with private key
106-
- [ ] remove account imported with JSON keystore
107-
- [ ] remove selected account
108-
- [ ] remove not selected account
109-
- [ ] remove all accounts
108+
- [ ] remove account imported with private key 🤖
109+
- [ ] remove account imported with JSON keystore 🤖
110+
- [ ] remove selected account 🤖
111+
- [ ] remove not selected account 🤖
112+
- [ ] remove all accounts 🤖
110113
2. Remove account and check functionalities
111-
- [ ] add removed address from imported account
112-
- [ ] remove all addresses from imported account
113-
- [ ] check if NFTs are removed for removed accounts
114-
- [ ] check if abilities are removed for removed accounts
115-
- [ ] check if Portfolio page is updated for removed accounts
114+
- [ ] add removed address from imported account 🤖
115+
- [ ] remove all addresses from imported account 🤖
116+
- [ ] check if NFTs are removed for removed accounts 🤖
117+
- [ ] check if abilities are removed for removed accounts 🤖
118+
- [ ] check if Portfolio page is updated for removed accounts 🤖
116119

117120
### 💌 Send base asset
118121

122+
When there are no changes touching sending txs in the release, only one base
123+
asset send transaction on some mainnet chain is required to be tested. Make sure
124+
it's on different chain than the ERC-20 asset send transaction.
125+
When there are changes in the release that may affect transactions, lets test
126+
everything from the list.
127+
119128
1. Ethereum mainnet
120129
- [ ] check gas settings and gas price
121130
- [ ] sign
@@ -143,6 +152,12 @@ This release checklist should be performed before release is published.
143152

144153
### 🎁 Send ERC20
145154

155+
When there are no changes touching sending txs in the release, only one ERC20
156+
send transaction on some mainnet chain is required to be tested. Make sure it's
157+
on different chain than the base asset send transaction.
158+
When there are changes in the release that may affect transactions, lets test
159+
everything from the list.
160+
146161
1. Ethereum mainnet
147162
- [ ] check gas settings and gas price
148163
- [ ] sign
@@ -170,11 +185,16 @@ This release checklist should be performed before release is published.
170185

171186
### 💸 Internal swap
172187

188+
When there are no changes touching swaps in the release, only one swap on some
189+
mainnet chain is required to be tested.
190+
When there are changes in the release that may affect swaps, lets test
191+
everything from the list.
192+
173193
1. Ethereum mainnet
174194
- [ ] check token list
175195
- [ ] sign token spend approval
176196
- [ ] check gas settings and gas price
177-
- [ ] sign
197+
- [ ] sign (if expensive, can be executed on testnet)
178198
- [ ] check transaction on scan website
179199
2. Polygon
180200
- [ ] check token list
@@ -209,20 +229,23 @@ This release checklist should be performed before release is published.
209229

210230
### 📡 dApps connections
211231

212-
- [ ] connect to [Uniswap](https://app.uniswap.org/)
213-
- [ ] connect to [Opensea](https://opensea.io/)
214-
- [ ] connect to [Galxe](https://galxe.com/)
215-
- [ ] connect to [PancakeSwap](https://pancakeswap.finance/)
216-
- [ ] connect to [TraderJoe](https://traderjoexyz.com/avalanche)
217-
- [ ] connect to [SpookySwap](https://spooky.fi/#/)
218-
- [ ] connect to [Velodrome](https://app.velodrome.finance/swap)
219-
- [ ] connect to [GMX](https://app.gmx.io/#/trade)
220-
- [ ] connect to [MetaMask Test dApp](https://metamask.github.io/test-dapp/) & verify Personal Sign
232+
When there are no changes touching connectivity to dApps, it's enough to do only
233+
a few reandom tests from the list.
234+
235+
- [ ] connect to [Uniswap](https://app.uniswap.org/) 🤖
236+
- [ ] connect to [Opensea](https://opensea.io/) 🤖
237+
- [ ] connect to [Galxe](https://galxe.com/)🤖
238+
- [ ] connect to [PancakeSwap](https://pancakeswap.finance/) 🤖
239+
- [ ] connect to [TraderJoe](https://traderjoexyz.com/avalanche) 🤖
240+
- [ ] connect to [SpookySwap](https://spooky.fi/#/) 🤖
241+
- [ ] connect to [Velodrome](https://app.velodrome.finance/swap) 🤖
242+
- [ ] connect to [GMX](https://app.gmx.io/#/trade) 🤖
243+
- [ ] connect to [MetaMask Test dApp](https://metamask.github.io/test-dapp/) 🤖
221244

222245
### 🎭 Sign in with Ethereum
223246

224-
- [ ] SIWE on [login.xyz](https://login.xyz/)
225-
- [ ] SIWE on [Taho Pledge](https://taho.xyz/web3pledge)
247+
- [ ] SIWE on [login.xyz](https://login.xyz/) 🤖
248+
- [ ] SIWE on [Taho Pledge](https://taho.xyz/web3pledge) 🤖
226249

227250
### 🐶 Default wallet
228251

@@ -235,38 +258,38 @@ This release checklist should be performed before release is published.
235258

236259
### ⚙️ Settings
237260

238-
- [ ] check `hide balance under $2` option
239-
- [ ] check bug reports - export logs
240-
- [ ] check connected dapp - confirm you are able to disconnect from a dapp and connect again on different network
261+
- [ ] check `hide balance under $2` option 🤖
262+
- [ ] check bug reports - export logs 🤖
263+
- [ ] check connected dapp - confirm you are able to disconnect from a dapp and connect again on different network 🤖
241264

242265
### ☀️ Abilities
243266

244-
- [ ] check each filter option
245-
- [ ] delete ability
246-
- [ ] mark ability as completed
267+
- [ ] check each filter option 🤖
268+
- [ ] delete ability 🤖
269+
- [ ] mark ability as completed 🤖
247270

248271
### 🌠 NFTs page
249272

250273
1. NFTs page
251-
- [ ] browse NFTs
252-
- [ ] expand and collapse collection
253-
- [ ] check NFT preview
254-
- [ ] check Badges tab
274+
- [ ] browse NFTs 🤖
275+
- [ ] expand and collapse collection 🤖
276+
- [ ] check NFT preview 🤖
277+
- [ ] check Badges tab 🤖
255278
2. NFTs filters
256-
- [ ] check each sort type
257-
- [ ] disable account
258-
- [ ] disable collection
279+
- [ ] check each sort type 🤖
280+
- [ ] disable account 🤖
281+
- [ ] disable collection 🤖
259282

260283
### 🦾 Flashbots
261284

262285
1. Enable Flashbots in settings:
263-
- [ ] send/swap an asset & confirm trxn on Ethereum shows the Flashbots option with it auto-selected
264-
- [ ] confirm trxn is then sent through Flashbots
286+
- [ ] send/swap an asset & confirm trxn on Ethereum shows the Flashbots option with it auto-selected 🤖
287+
- [ ] confirm trxn is then sent through Flashbots 🤖 (send only)
265288
- [ ] send/swap another asset deselect Flashbots option
266-
- [ ] confirm that trxn is not sent through Flashbots
289+
- [ ] confirm that trxn is not sent through Flashbots 🤖 (send only)
267290
2. Check trxns on other networks
268-
- [ ] confirm Flashbots checkbox isn't shown at all
269-
- [ ] confirm trxn is not sent through Flashbots
291+
- [ ] confirm Flashbots checkbox isn't shown at all 🤖
292+
- [ ] confirm trxn is not sent through Flashbots 🤖
270293
3. Disable Flashbots option in settings
271-
- [ ] send/swap an asset on Ethereum & confirm trxn shows without the Flashbots option
272-
- [ ] confirm trxn is not sent through Flashbots
294+
- [ ] send/swap an asset on Ethereum & confirm trxn shows without the Flashbots option 🤖
295+
- [ ] confirm trxn is not sent through Flashbots 🤖 (send only)

0 commit comments

Comments
 (0)