Skip to content

Commit

Permalink
Fix tests and update contracts README and (#8)
Browse files Browse the repository at this point in the history
* Update README with latest Foundry installation steps

* Update to latest addresses after solmate upgrade

* Update to new verbosity flag syntax

* chore: upgrade OZ and solmate

* fix alice address in frontend and generator
  • Loading branch information
devanoneth committed Jan 19, 2022
1 parent 9da2383 commit 4c86569
Show file tree
Hide file tree
Showing 6 changed files with 12 additions and 14 deletions.
10 changes: 4 additions & 6 deletions contracts/README.md
Original file line number Diff line number Diff line change
Expand Up @@ -6,11 +6,7 @@ ERC20 token claimable by members of a [Merkle tree](https://en.wikipedia.org/wik

Tests use [Foundry: Forge](https://github.com/gakonst/foundry).

### Install Forge

```bash
cargo install --git https://github.com/gakonst/foundry --bin forge --locked
```
Install Foundry using the installation steps in the README of the linked repo.

### Run tests

Expand All @@ -22,7 +18,9 @@ cd contracts/
forge update

# Run tests
forge test --root . --verbosity 4
forge test --root .
# Run tests with stack traces
forge test --root . -vvvv
```

## Deploy
Expand Down
2 changes: 1 addition & 1 deletion contracts/lib/openzeppelin-contracts
Submodule openzeppelin-contracts updated 58 files
+20 −1 CHANGELOG.md
+3 −3 contracts/access/AccessControl.sol
+2 −2 contracts/access/AccessControlEnumerable.sol
+1 −1 contracts/governance/Governor.sol
+1 −1 contracts/governance/compatibility/GovernorCompatibilityBravo.sol
+8 −6 contracts/interfaces/IERC2981.sol
+1 −1 contracts/interfaces/README.adoc
+20 −0 contracts/interfaces/draft-IERC1822.sol
+11 −0 contracts/mocks/Base64Mock.sol
+7 −0 contracts/mocks/CallReceiverMock.sol
+33 −0 contracts/mocks/ERC721RoyaltyMock.sol
+0 −4 contracts/mocks/MathMock.sol
+23 −0 contracts/mocks/SignedMathMock.sol
+58 −0 contracts/mocks/UUPS/UUPSLegacy.sol
+0 −10 contracts/mocks/UUPS/UUPSUpgradeableMock.sol
+14 −23 contracts/proxy/ERC1967/ERC1967Upgrade.sol
+4 −2 contracts/proxy/README.adoc
+25 −3 contracts/proxy/utils/UUPSUpgradeable.sol
+13 −5 contracts/token/ERC20/ERC20.sol
+2 −1 contracts/token/ERC20/extensions/ERC20FlashMint.sol
+3 −3 contracts/token/ERC20/extensions/ERC20Votes.sol
+2 −2 contracts/token/ERC20/extensions/ERC20VotesComp.sol
+10 −4 contracts/token/ERC20/utils/TokenTimelock.sol
+19 −7 contracts/token/ERC721/README.adoc
+38 −0 contracts/token/ERC721/extensions/ERC721Royalty.sol
+1 −1 contracts/token/ERC721/extensions/IERC721Enumerable.sol
+15 −5 contracts/token/ERC777/ERC777.sol
+112 −0 contracts/token/common/ERC2981.sol
+10 −0 contracts/token/common/README.adoc
+1 −1 contracts/utils/Address.sol
+88 −0 contracts/utils/Base64.sol
+1 −1 contracts/utils/Multicall.sol
+5 −1 contracts/utils/README.adoc
+2 −6 contracts/utils/cryptography/ECDSA.sol
+0 −10 contracts/utils/math/Math.sol
+42 −0 contracts/utils/math/SignedMath.sol
+3 −1 docs/modules/ROOT/pages/erc1155.adoc
+3 −1 docs/modules/ROOT/pages/erc721.adoc
+47 −0 docs/modules/ROOT/pages/utilities.adoc
+7 −7 package-lock.json
+1 −1 package.json
+5 −2 scripts/gen-nav.js
+0 −0 scripts/release/update-comment.js
+60 −12 test/governance/GovernorWorkflow.behavior.js
+67 −116 test/governance/compatibility/GovernorCompatibilityBravo.test.js
+24 −0 test/helpers/erc1967.js
+5 −11 test/proxy/Proxy.behaviour.js
+4 −11 test/proxy/beacon/BeaconProxy.test.js
+7 −14 test/proxy/transparent/TransparentUpgradeableProxy.behaviour.js
+25 −12 test/proxy/utils/UUPSUpgradeable.test.js
+77 −56 test/token/ERC20/ERC20.behavior.js
+28 −34 test/token/ERC20/ERC20.test.js
+40 −0 test/token/ERC721/extensions/ERC721Royalty.test.js
+160 −0 test/token/common/ERC2981.behavior.js
+29 −0 test/utils/Base64.test.js
+5 −2 test/utils/introspection/SupportsInterface.behavior.js
+1 −17 test/utils/math/Math.test.js
+93 −0 test/utils/math/SignedMath.test.js
2 changes: 1 addition & 1 deletion contracts/lib/solmate
8 changes: 4 additions & 4 deletions contracts/src/test/utils/MerkleClaimERC20Test.sol
Original file line number Diff line number Diff line change
Expand Up @@ -30,11 +30,11 @@ contract MerkleClaimERC20Test is DSTest {
"MT",
18,
// Merkle root containing ALICE with 100e18 tokens but no BOB
0x6a0b89fc219e9e72ad683e00d9c152532ec8e5c559600e04160d310936400a00
0xd0aa6a4e5b4e13462921d7518eebdb7b297a7877d6cfe078b0c318827392fb55
);

// Setup airdrop users
ALICE = new MerkleClaimERC20User(TOKEN); // 0x109f93893af4c4b0afc7a9e97b59991260f98313
BOB = new MerkleClaimERC20User(TOKEN); // 0x689856e2a6eb68fc33099eb2ccba0a5a4e8be52f
ALICE = new MerkleClaimERC20User(TOKEN); // 0x185a4dc360ce69bdccee33b3784b0282f7961aea
BOB = new MerkleClaimERC20User(TOKEN); // 0xefc56627233b02ea95bae7e19f648d7dcd5bb132
}
}
}
2 changes: 1 addition & 1 deletion frontend/config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ const config: IConfig = {
decimals: 18,
airdrop: {
"0x016C8780e5ccB32E5CAA342a926794cE64d9C364": 10,
"0x109f93893af4c4b0afc7a9e97b59991260f98313": 100,
"0x185a4dc360ce69bdccee33b3784b0282f7961aea": 100,
},
};

Expand Down
2 changes: 1 addition & 1 deletion generator/config.json
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,6 @@
"decimals": 18,
"airdrop": {
"0x016C8780e5ccB32E5CAA342a926794cE64d9C364": 10,
"0x109f93893af4c4b0afc7a9e97b59991260f98313": 100
"0x185a4dc360ce69bdccee33b3784b0282f7961aea": 100
}
}

0 comments on commit 4c86569

Please sign in to comment.