A | B | C | D | E | F | G | H | I | J | K | L | M | N | O | P | Q | R | S | T | U | V | W | X | Y | Z | AA | AB | AC | AD | AE | AF | |
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
1 | Tag | Action | Branch | Min Test Network | Pass Criteria | Unit Test Status | Integration Test Status | Ethereum Goerli Staging Test network | Ethereum Goerli Production Test network | Arbitrum Goerli Staging Test network | Arbitrum Goerli Production Test network | Arbitrum Sepolia Production Test network | Notes | |||||||||||||||||||
2 | Deploy/Initialize | |||||||||||||||||||||||||||||||
3 | 1-1 | Deploy Contracts | all-branches | Anvil Testnet (Integration) | Deployment Successful | N/A | N/A | Pass | Pass | Pass | Pass | Pass | https://docs.google.com/spreadsheets/d/1osioidyxlkyw49CEOAvfmPph_JbTQCIbblLTyi9UFy4/edit#gid=1670927759 | |||||||||||||||||||
4 | 1-2 | Set up mock indexers, subgraphs, curators and allocations | all-branches | Anvil Testnet (Integration) | Able to create a non-reverted transaction creating an allocation in the staking contract | N/A | N/A | Pass | Pass | Pass | Pass | TODO | ||||||||||||||||||||
5 | Deposit/Thaw/Withdraw Funds | |||||||||||||||||||||||||||||||
6 | 2-1 | Deposit funds to single escrow account using valid sender address | all-branches | Forge Unit Testnet | Non-revert transaction resulting in escrow contract tokens and escrow mapping amount increasing by amount deposited | Pass | Pass | Skip | Pass | Skip | Pass | To test | ||||||||||||||||||||
7 | 2-2 | Deposit funds to single escrow account using invalid sender address | 23-add-an-allow-list-for-senders-controlled-through-a-governance | Forge Unit Testnet | Transaction should revert | Skip | Skip | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
8 | 2-3 | Thaw funds and withdraw prior to thaw period | any-branch | Forge Unit Testnet | Transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
9 | 2-4 | Thaw funds, cancel the thaw, and attempt to withdraw | any-branch | Forge Unit Testnet | Transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
10 | 2-5 | Thaw funds and reduce amount thawing, attempt to withdraw prior to thaw period | any-branch | Forge Unit Testnet | Transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
11 | 2-6 | Thaw funds and withdraw after thaw period | any-branch | Forge Unit Testnet | Escrow account should reduce by amount thawing | Pass | Pass | Skip | Pass | Skip | Pass | To test | ||||||||||||||||||||
12 | 2-7 | Thaw funds and reduce amount thawing, withdraw after thaw period | any-branch | Forge Unit Testnet | Escrow account should reduce by amount thawing | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
13 | 2-8 | Test multiple thaw requests being sent within the thaw window and withdraw prior to thaw end | any-branch | Forge Unit Testnet | Transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
14 | Authorize/Thaw/CancelThaw/Revoke Signers | |||||||||||||||||||||||||||||||
15 | 3-1 | Authorize Signer using valid sender address | all-branches | Forge Unit Testnet | Non-revert transaction adding signer to authorize list of sender | Pass | Pass | Pass | Pass | Pass | Pass | To test | ||||||||||||||||||||
16 | 3-2 | Authorize Signer using invalid sender address | 23-add-an-allow-list-for-senders-controlled-through-a-governance | Forge Unit Testnet | Transaction should revert | Skip | Skip | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
17 | 3-3 | Authorize signer using random bits for proof | any-branch | Forge Unit Testnet | Transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
18 | 3-4 | Authorize signer using different signer for proof | any-branch | Forge Unit Testnet | Transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
19 | 3-5 | Thaw signer and revoke prior to thaw period | any-branch | Forge Unit Testnet | Transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
20 | 3-6 | Thaw signer, cancel thaw, and attempt to revoke | any-branch | Forge Unit Testnet | Transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
21 | 3-7 | Thaw signer and revoke after thaw period | any-branch | Forge Unit Testnet | Non-revert transaction removing signer from authorize list of sender, RAV signed by signer should revert | Pass | Pass | Skip | Pass | Skip | Untested | To test | ||||||||||||||||||||
22 | Redeem RAV | |||||||||||||||||||||||||||||||
23 | 4-1 | Redeem Valid RAV with open allocationID | any-branch | Anvil Testnet (Integration) | Non-revert transaction that sends receivers funds to staking contract and removes the funds from escrow contract and escrow account | N/A | Pass | Pass | Pass | Pass | Pass | To test | we're enabling redeem of a valid RAV disregarding the statust of the allocation | |||||||||||||||||||
24 | 4-2 | Redeem Valid RAV with closed allocationID | any-branch | Anvil Testnet (Integration) | Non-revert transaction that sends receivers funds to staking contract and removes the funds from escrow contract and escrow account | N/A | Skip | Skip | Skip | Skip | Skip | Skip | we're enabling redeem of a valid RAV disregarding the statust of the allocation | |||||||||||||||||||
25 | 4-3 | Redeem Valid RAV with insufficient funds in escrow account | any-branch | Forge Unit Testnet | Non-revert transaction that sends all available funds to staking contract and removes the funds from escrow contract and escrow account | Pass | Pass | Skip | Skip | Skip | Skip | Skip | we're passing the value to the other contract - so skipping | |||||||||||||||||||
26 | 4-4 | Redeem RAV with invalid signer | any-branch | Forge Unit Testnet | Transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | when signer is invalid then the execution is not reaching the contract | |||||||||||||||||||
27 | 4-5 | Redeem multiple RAVs with different gateway operators (sender) and same allocation ID | any-branch | Forge Unit Testnet | Non-revert transactions that send receivers funds to staking contract and removes the funds from escrow contract and escrow account | Pass | Pass | Skip | To test | Skip | To test | To test | is this even possible? does the staking contract have any idea on the gateway? | |||||||||||||||||||
28 | 4-6 | Redeem multiple RAVs with same gateway operators (sender) and same allocation ID | any-branch | Forge Unit Testnet | First transaction should not revert, all subsequent transactions are expected to revert | Pass | Pass | Skip | To test | Skip | To test | To test | ||||||||||||||||||||
29 | 4-7 | Redeem RAV with non-registered allocationID | any-branch | Anvil Testnet (Integration) | reverting transaction failing from staking contract | N/A | N/A | Skip | To test | Skip | To test | To test | ||||||||||||||||||||
30 | 4-8 | Redeem RAV with random bits as signature | any-branch | Forge Unit Testnet | transaction should revert | Pass | Pass | Skip | Skip | Skip | Skip | Skip | ||||||||||||||||||||
31 | Escrow Subgraph | |||||||||||||||||||||||||||||||
32 | 5-1 | event Deposit | Event should be caught with no error when full transaction is finished. Also info should be caught by the subgraph and amount should be validated against a call to the subgraph. | N/A | Pass | Skip | Pass | Skip | Pass | To test | Arbitrum escrow transactions: https://goerli.arbiscan.io/address/0xE805CC3166fc0d03C15165303F5e54e1f57A1BCb It is expected for both that the signer authorization fails, but its not an error it is expected since it has already been authorized | |||||||||||||||||||||
33 | 5-2 | event Redeem | Event should be caught with no error when full transaction is finished. Also info should be caught by the subgraph and amount should be validated against a call to the subgraph. | N/A | Pass | Skip | Pass | Skip | Pass | To test | Eth escrow transactions: https://goerli.etherscan.io/address/0xD46c60558F7960407F4D00098145D77Fd061aD90 It is expected for both that the signer authorization fails, but its not an error it is expected since it has already been authorized | |||||||||||||||||||||
34 | event Thaw | Event should be caught with no error when full transaction is finished. Also info should be caught by the subgraph and amount should be validated against a call to the subgraph. | N/A | Pass | Skip | Pass | Skip | Pass | To test | |||||||||||||||||||||||
35 | event CancelThaw | Event should be caught with no error when full transaction is finished. | N/A | Pass | Skip | Pass | Skip | Pass | To test | |||||||||||||||||||||||
36 | event ThawSigner | Event should be caught with no error when full transaction is finished. | N/A | Pass | Skip | Pass | Skip | Pass | To test | |||||||||||||||||||||||
37 | event CancelThawSigner | Event should be caught with no error when full transaction is finished. | N/A | Pass | Skip | Pass | Skip | Pass | To test | |||||||||||||||||||||||
38 | event RevokeAuthorizedSigner | Event should be caught with no error when full transaction is finished. Also info should be caught by the subgraph and amount should be validated against a call to the subgraph. | N/A | Pass | Skip | Skip | Skip | Skip | Skip | There is no easy way to test this right away in a script since the thawing takes more than 2 hours | ||||||||||||||||||||||
39 | event Withdraw | Event should be caught with no error when full transaction is finished. Also info should be caught by the subgraph and amount should be validated against a call to the subgraph. | N/A | Pass | Skip | Skip | Skip | Skip | Skip | |||||||||||||||||||||||
40 | event AuthorizeSigner | Event should be caught with no error when full transaction is finished. Also info should be caught by the subgraph and amount should be validated against a call to the subgraph. | N/A | Pass | Skip | Pass | Skip | Pass | To test | |||||||||||||||||||||||
41 | event AllocationIDUsed | Event should be caught with no error when full transaction is finished. | N/A | Pass | Skip | Pass | Skip | Pass | To test | |||||||||||||||||||||||
42 | TODOs | |||||||||||||||||||||||||||||||
43 | TODO: Carlos Vargas del Rio please update Escrow section - add one event per row | TODO: update our CI/CD in our repo to include Integration Tests | TODO: add links to cells to the etherscan/arbiscan | |||||||||||||||||||||||||||||
44 | TODO: Escrow repo - setup CI/CD | |||||||||||||||||||||||||||||||
45 | Please take a look TAP contract repo / Graph CI/CD | |||||||||||||||||||||||||||||||
46 | ||||||||||||||||||||||||||||||||
47 | ||||||||||||||||||||||||||||||||
48 | ||||||||||||||||||||||||||||||||
49 | ||||||||||||||||||||||||||||||||
50 | ||||||||||||||||||||||||||||||||
51 | ||||||||||||||||||||||||||||||||
52 | ||||||||||||||||||||||||||||||||
53 | ||||||||||||||||||||||||||||||||
54 | ||||||||||||||||||||||||||||||||
55 | ||||||||||||||||||||||||||||||||
56 | ||||||||||||||||||||||||||||||||
57 | ||||||||||||||||||||||||||||||||
58 | ||||||||||||||||||||||||||||||||
59 | ||||||||||||||||||||||||||||||||
60 | ||||||||||||||||||||||||||||||||
61 | ||||||||||||||||||||||||||||||||
62 | ||||||||||||||||||||||||||||||||
63 | ||||||||||||||||||||||||||||||||
64 | ||||||||||||||||||||||||||||||||
65 | ||||||||||||||||||||||||||||||||
66 | ||||||||||||||||||||||||||||||||
67 | ||||||||||||||||||||||||||||||||
68 | ||||||||||||||||||||||||||||||||
69 | ||||||||||||||||||||||||||||||||
70 | ||||||||||||||||||||||||||||||||
71 | ||||||||||||||||||||||||||||||||
72 | ||||||||||||||||||||||||||||||||
73 | ||||||||||||||||||||||||||||||||
74 | ||||||||||||||||||||||||||||||||
75 | ||||||||||||||||||||||||||||||||
76 | ||||||||||||||||||||||||||||||||
77 | ||||||||||||||||||||||||||||||||
78 | ||||||||||||||||||||||||||||||||
79 | ||||||||||||||||||||||||||||||||
80 | ||||||||||||||||||||||||||||||||
81 | ||||||||||||||||||||||||||||||||
82 | ||||||||||||||||||||||||||||||||
83 | ||||||||||||||||||||||||||||||||
84 | ||||||||||||||||||||||||||||||||
85 | ||||||||||||||||||||||||||||||||
86 | ||||||||||||||||||||||||||||||||
87 | ||||||||||||||||||||||||||||||||
88 | ||||||||||||||||||||||||||||||||
89 | ||||||||||||||||||||||||||||||||
90 | ||||||||||||||||||||||||||||||||
91 | ||||||||||||||||||||||||||||||||
92 | ||||||||||||||||||||||||||||||||
93 | ||||||||||||||||||||||||||||||||
94 | ||||||||||||||||||||||||||||||||
95 | ||||||||||||||||||||||||||||||||
96 | ||||||||||||||||||||||||||||||||
97 | ||||||||||||||||||||||||||||||||
98 | ||||||||||||||||||||||||||||||||
99 | ||||||||||||||||||||||||||||||||
100 |