log warning if bytecode is empty or invalid

0xProject · Sep 25, 2019 · ab1ee63 · ab1ee63
1 parent 23bc086
commit ab1ee63
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 7 deletions.
diff --git a/packages/abi-gen-wrappers/src/generated-wrappers/i_asset_proxy.ts b/packages/abi-gen-wrappers/src/generated-wrappers/i_asset_proxy.ts
diff --git a/packages/abi-gen-wrappers/src/generated-wrappers/i_wallet.ts b/packages/abi-gen-wrappers/src/generated-wrappers/i_wallet.ts
diff --git a/packages/abi-gen/src/index.ts b/packages/abi-gen/src/index.ts
@@ -362,8 +362,21 @@ for (const abiFileName of abiFileNames) {
     let deployedBytecode;
     try {
         deployedBytecode = parsedContent.compilerOutput.evm.deployedBytecode.object;
-    } catch (e) {
-        logUtils.log(`Couldn't find deployedBytecode for ${chalk.bold(namedContent.name)}, using undefined`);
+        if (
+            deployedBytecode === '' ||
+            deployedBytecode === undefined ||
+            deployedBytecode === '0x' ||
+            deployedBytecode === '0x00'
+        ) {
+            throw new Error();
+        }
+    } catch (err) {
+        logUtils.log(
+            `Couldn't find deployedBytecode for ${chalk.bold(
+                namedContent.name,
+            )}, using undefined. Found [${deployedBytecode}]`,
+        );
+        deployedBytecode = undefined;
     }
     let ctor = ABI.find((abi: AbiDefinition) => abi.type === ABI_TYPE_CONSTRUCTOR) as ConstructorAbi;
     if (ctor === undefined) {

diff --git a/packages/base-contract/src/index.ts b/packages/base-contract/src/index.ts
@@ -268,8 +268,9 @@ export class BaseContract {
     ) {
         assert.isString('contractName', contractName);
         assert.isETHAddressHex('address', address);
-        if (deployedBytecode !== undefined) {
+        if (deployedBytecode !== undefined && deployedBytecode !== '') {
             assert.isHexString('deployedBytecode', deployedBytecode);
+            this._deployedBytecodeIfExists = Buffer.from(deployedBytecode.substr(2), 'hex');
         }
         const provider = providerUtils.standardizeOrThrow(supportedProvider);
         if (callAndTxnDefaults !== undefined) {
@@ -287,8 +288,6 @@ export class BaseContract {
             (abiDefinition: AbiDefinition) => abiDefinition.type === AbiType.Function,
         ) as MethodAbi[];
         this._abiEncoderByFunctionSignature = {};
-        this._deployedBytecodeIfExists =
-            deployedBytecode === undefined ? deployedBytecode : Buffer.from(deployedBytecode.substr(2), 'hex');
         _.each(methodAbis, methodAbi => {
             const abiEncoder = new AbiEncoder.Method(methodAbi);
             const functionSignature = abiEncoder.getSignature();