Test with no initializer

OpenZeppelin · Oct 26, 2023 · 4549110 · 4549110
1 parent b2955c8
commit 4549110
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 1 deletion.
diff --git a/test/Upgrades.t.sol b/test/Upgrades.t.sol
@@ -13,7 +13,7 @@ import {MyToken} from "./contracts/MyToken.sol";
 import {MyTokenV2} from "./contracts/MyTokenV2.sol";
 import {MyTokenProxiable} from "./contracts/MyTokenProxiable.sol";
 import {MyTokenProxiableV2} from "./contracts/MyTokenProxiableV2.sol";
-import {WithConstructor} from "./contracts/WithConstructor.sol";
+import {WithConstructor, NoInitializer} from "./contracts/WithConstructor.sol";
 
 contract UpgradesTest is Test {
     address constant CHEATCODE_ADDRESS = 0x7109709ECfa91a80626fF3989D68f67F5b1DD12D;
@@ -226,6 +226,13 @@ contract UpgradesTest is Test {
         assertEq(WithConstructor(proxy).a(), 123);
         assertEq(WithConstructor(proxy).b(), 456);
     }
+
+    function testNoInitializer() public {
+        Options memory opts;
+        opts.constructorData = abi.encode(123);
+        address proxy = Upgrades.deployTransparentProxy("WithConstructor.sol:NoInitializer", msg.sender, "", opts);
+        assertEq(WithConstructor(proxy).a(), 123);
+    }
 }
 
 contract Validator {

diff --git a/test/contracts/WithConstructor.sol b/test/contracts/WithConstructor.sol
@@ -18,3 +18,13 @@ contract WithConstructor {
         b = _b;
     }
 }
+
+contract NoInitializer {
+    /// @custom:oz-upgrades-unsafe-allow state-variable-immutable
+    uint256 public immutable a;
+
+    /// @custom:oz-upgrades-unsafe-allow constructor
+    constructor(uint256 _a) {
+        a = _a;
+    }
+}