Skip to content

Commit

Permalink
Merge branch 'master' into chore/add-view-to-upgradeable-proxy
Browse files Browse the repository at this point in the history
  • Loading branch information
ernestognw committed Oct 16, 2023
2 parents b5c7e18 + 9c22fb5 commit c899579
Show file tree
Hide file tree
Showing 10 changed files with 861 additions and 742 deletions.
3 changes: 2 additions & 1 deletion contracts/proxy/transparent/TransparentUpgradeableProxy.sol
Original file line number Diff line number Diff line change
Expand Up @@ -50,7 +50,8 @@ interface ITransparentUpgradeableProxy is IERC1967 {
* IMPORTANT: This contract avoids unnecessary storage reads by setting the admin only during construction as an
* immutable variable, preventing any changes thereafter. However, the admin slot defined in ERC-1967 can still be
* overwritten by the implementation logic pointed to by this proxy. In such cases, the contract may end up in an
* undesirable state where the admin slot is different from the actual admin.
* undesirable state where the admin slot is different from the actual admin. Relying in the value of the admin slot
* is generally fine if the implementation is trusted.
*
* WARNING: It is not recommended to extend this contract to add additional external functions. If you do so, the
* compiler will not check that there are no selector conflicts, due to the note above. A selector clash between any new
Expand Down
42 changes: 15 additions & 27 deletions package-lock.json

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions test/access/manager/AccessManaged.test.js
Original file line number Diff line number Diff line change
Expand Up @@ -59,7 +59,7 @@ contract('AccessManaged', function (accounts) {
});

it('reverts if the operation is not scheduled', async function () {
const calldata = await this.managed.contract.methods[method]().encodeABI();
const calldata = this.managed.contract.methods[method]().encodeABI();
const opId = await this.authority.hashOperation(roleMember, this.managed.address, calldata);

await expectRevertCustomError(this.managed.methods[method]({ from: roleMember }), 'AccessManagerNotScheduled', [
Expand All @@ -70,7 +70,7 @@ contract('AccessManaged', function (accounts) {
it('succeeds if the operation is scheduled', async function () {
// Arguments
const delay = time.duration.hours(12);
const calldata = await this.managed.contract.methods[method]().encodeABI();
const calldata = this.managed.contract.methods[method]().encodeABI();

// Schedule
const timestamp = await time.latest();
Expand Down
Loading

0 comments on commit c899579

Please # to comment.