update gas limit padding to 50% (#794)

floating · Apr 15, 2022 · d3009b1 · d3009b1
1 parent 51637e3
commit d3009b1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/main/provider/index.ts b/main/provider/index.ts
@@ -465,7 +465,7 @@ export class Provider extends EventEmitter {
         }
 
         const estimatedLimit = parseInt(response.result, 16)
-        const paddedLimit = Math.ceil(estimatedLimit * 1.15)
+        const paddedLimit = Math.ceil(estimatedLimit * 1.5)
 
         log.verbose(`gas estimate for tx to ${txParams.to}: ${estimatedLimit}, using ${paddedLimit} as gas limit`)
         return resolve(addHexPrefix(paddedLimit.toString(16)))

diff --git a/test/main/provider/index.test.js b/test/main/provider/index.test.js
@@ -659,7 +659,7 @@ describe('#send', () => {
       })
     })
 
-    it('pads the gas estimate from the network by 15 percent', done => {
+    it('pads the gas estimate from the network by 50 percent', done => {
       connection.send.mockImplementationOnce((payload, cb) => {
         expect(payload.method).toBe('eth_estimateGas')
         cb({ result: addHexPrefix((150000).toString(16)) })
@@ -670,7 +670,7 @@ describe('#send', () => {
       sendTransaction(() => {
         try {
           const initialRequest = accountRequests[0]
-          expect(initialRequest.data.gasLimit).toBe(addHexPrefix((172500).toString(16)))
+          expect(initialRequest.data.gasLimit).toBe(addHexPrefix((225000).toString(16)))
           done()
         } catch (e) { done(e) }
       })