Set proxy admin on first create

OpenZeppelin · Jul 18, 2019 · ba8e3bb · ba8e3bb
1 parent 06cb6c4
commit ba8e3bb
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/packages/cli/src/models/network/NetworkController.ts b/packages/cli/src/models/network/NetworkController.ts
@@ -786,7 +786,7 @@ export default class NetworkController {
         address: instance.address,
         version: semanticVersionToString(packageVersion),
         implementation: implementationAddress,
-        admin: admin || this.networkFile.proxyAdminAddress,
+        admin: admin || this.networkFile.proxyAdminAddress || (await this.project.getAdminAddress()),
         kind,
       });
       return instance;

diff --git a/packages/cli/test/scripts/create.test.js b/packages/cli/test/scripts/create.test.js
@@ -116,6 +116,10 @@ contract('create script', function([_, owner, otherAdmin]) {
       proxyImplementation.should.equalIgnoreCase(minimal);
     }
 
+    if(!minimal) {
+      proxyInfo.admin.should.not.be.undefined;
+    }
+
     return proxyInfo;
   };