test gas estimate for account to account transfer (#204)

Add regression test for gas estimate for simple value transfer between accounts. There was previously a same-day hot fix on a release for this issue, but there was no time for the test at that moment... so here is the test!
trufflesuite · Nov 2, 2018 · 9b28b38 · 9b28b38
1 parent b7c312a
commit 9b28b38
Show file tree

Hide file tree

Showing 2 changed files with 37 additions and 7 deletions.
diff --git a/npm-shrinkwrap.json b/npm-shrinkwrap.json
diff --git a/test/gas.js b/test/gas.js
@@ -314,6 +314,21 @@ describe("Gas", function() {
 
       return to.hex(Buffer.from(bytes));
     }
+
+    it("matches usage for simple account to account transfer", async function() {
+      let transferAmount = web3.utils.toBN(web3.utils.toWei("5", "finney"));
+      let transactionData = {
+        from: accounts[0],
+        to: accounts[1],
+        value: transferAmount
+      };
+
+      let gasEstimate = await web3.eth.estimateGas(transactionData);
+
+      let receipt = await web3.eth.sendTransaction(transactionData);
+
+      assert.strictEqual(receipt.gasUsed, gasEstimate);
+    });
   });
 
   describe("Expenditure", function() {