From 0e01e3d8465cd2f79ad0bb5d6607f82b860021fe Mon Sep 17 00:00:00 2001
From: Anthony Laibe <anthony@laibe.cc>
Date: Fri, 25 Jan 2019 09:38:00 +0000
Subject: [PATCH] PR feedback

---
 src/lib/modules/blockchain_connector/index.js | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)

diff --git a/src/lib/modules/blockchain_connector/index.js b/src/lib/modules/blockchain_connector/index.js
index ef78a54fe..94de8f878 100644
--- a/src/lib/modules/blockchain_connector/index.js
+++ b/src/lib/modules/blockchain_connector/index.js
@@ -109,12 +109,12 @@ class BlockchainConnector {
       const sim = self._getSimulator();
       const options = Object.assign({}, self.config.contractsConfig.deployment, {
         gasPrice: "0x01",
-        gasLimit: "0xfffffffffffff"
+        gasLimit: "0xfffffffffffff",
+        allowUnlimitedContractSize: coverage
       });
+      self.provider = sim.provider(options);
 
       if (coverage) {
-        options.allowUnlimitedContractSize = true;
-        self.provider = sim.provider(options);
         // Here we patch the sendAsync method on the provider. The goal behind this is to force pure/constant/view calls to become
         // transactions, so that we can pull in execution traces and account for those executions in code coverage.
         //
@@ -147,8 +147,6 @@ class BlockchainConnector {
             });
           });
         };
-      } else {
-        self.provider = sim.provider(options);
       }
 
       self.web3.setProvider(self.provider);