Fabo/fix gas (#250)

* fix gas estimate

* linted

* fixed test

lib/controller/transaction/index.js

@@ -1,23 +1,26 @@
-const { getMessage } = require('./messageConstructor')
+// const { getMessage } = require('./messageConstructor')
 const { networkMap } = require('../../networks')
 const Sentry = require('@sentry/node')
 const { publishUserTransactionAdded } = require('../../subscriptions')
 const reducers = require('../../reducers/cosmosV0-reducers') // TODO the whole transaction service only works for cosmos rn
 
 global.fetch = require('node-fetch')
 
-async function estimate(tx) {
-  const context = {
-    userAddress: tx.address,
-    networkId: tx.networkId,
-    url: networkMap[tx.networkId].api_url,
-    chainId: networkMap[tx.networkId].chain_id
-  }
+async function estimate() {
+  // const context = {
+  //   userAddress: tx.address,
+  //   networkId: tx.networkId,
+  //   url: networkMap[tx.networkId].api_url,
+  //   chainId: networkMap[tx.networkId].chain_id
+  // }
 
-  const message = getMessage(tx.messageType, tx.txProperties, context)
+  // const message = getMessage(tx.messageType, tx.txProperties, context)
 
   try {
-    const gasEstimate = (await message.simulate({ memo: tx.memo })) * 4 // we quadrupled the gas to be safe. rn there are several txs failing on cosmoshub-3
+    // const gasEstimate = (await message.simulate({ memo: tx.memo })) * 4 // we quadrupled the gas to be safe. rn there are several txs failing on cosmoshub-3
+    // HACK: fixed to this value for now. too high gas can lead to transactions never being included. too low gas can lead to rejected txs and cost the user money.
+    const gasEstimate = 450000
+
     return {
       gasEstimate,
       success: true

tests/routes.test.js

@@ -51,7 +51,8 @@ describe('POST /transaction', function() {
         .send({ payload })
         .set('Accept', 'application/json')
         .expect('Content-Type', /json/)
-        .expect({ gasEstimate: 74072, success: true })
+        // .expect({ gasEstimate: 74072, success: true })
+        .expect({ gasEstimate: 450000, success: true }) // fixed for now
         .expect(200, done)
     })
   })