fix secure upgrade pattern

OpenZeppelin · Mar 23, 2021 · 1a477c7 · 1a477c7
1 parent e1f8449
commit 1a477c7
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 4 deletions.
diff --git a/contracts/proxy/ERC1967/ERC1967Upgrade.sol b/contracts/proxy/ERC1967/ERC1967Upgrade.sol
@@ -69,9 +69,8 @@ abstract contract ERC1967Upgrade is ERC1967Storage {
             Address.functionDelegateCall(
                 newImplementation,
                 abi.encodeWithSignature(
-                    "upgradeToAndCall(address,bytes)",
-                    oldImplementation,
-                    bytes("")
+                    "upgradeTo(address)",
+                    oldImplementation
                 )
             );
             upgradePending.value = false;

diff --git a/test/proxy/UUPS/Proxiable.test.js b/test/proxy/UUPS/Proxiable.test.js
@@ -5,7 +5,7 @@ const ProxiableMock = artifacts.require('ProxiableMock');
 const ProxiableUnsafeMock = artifacts.require('ProxiableUnsafeMock');
 const ProxiableBrokenMock = artifacts.require('ProxiableBrokenMock');
 
-contract('ERC1967Upgrade', function (accounts) {
+contract('Proxiable', function (accounts) {
   before(async function () {
     this.testimpl0 = await ProxiableMock.new();
     this.testimpl1 = await ProxiableMock.new();