Add boundaries to oracle's gas price

src/services/gasPrice.js

@@ -7,7 +7,7 @@ const logger = require('../services/logger').child({
   module: 'gasPrice'
 })
 const { setIntervalAndRun } = require('../utils/utils')
-const { DEFAULT_UPDATE_INTERVAL } = require('../utils/constants')
+const { DEFAULT_UPDATE_INTERVAL, GAS_PRICE_BOUNDARIES } = require('../utils/constants')
 
 const HomeABI = bridgeConfig.homeBridgeAbi
 const ForeignABI = bridgeConfig.foreignBridgeAbi
@@ -31,12 +31,20 @@ const foreignBridge = new web3Foreign.eth.Contract(ForeignABI, FOREIGN_BRIDGE_AD
 
 let cachedGasPrice = null
 
+function gasPriceWithinLimits(gasPrice) {
+  return GAS_PRICE_BOUNDARIES.MIN <= gasPrice && gasPrice <= GAS_PRICE_BOUNDARIES.MAX
+}
+
 async function fetchGasPriceFromOracle(oracleUrl, speedType) {
   const response = await fetch(oracleUrl)
   const json = await response.json()
   const gasPrice = json[speedType]
   if (!gasPrice) {
     throw new Error(`Response from Oracle didn't include gas price for ${speedType} type.`)
+  } else if (!gasPriceWithinLimits(gasPrice)) {
+    throw new Error(
+      `Response from Oracle included a gas price out of boundaries ${gasPrice} for ${speedType} type.`
+    )
   }
   return Web3Utils.toWei(gasPrice.toString(), 'gwei')
 }
@@ -102,5 +110,6 @@ function getPrice() {
 module.exports = {
   start,
   fetchGasPrice,
-  getPrice
+  getPrice,
+  gasPriceWithinLimits
 }

src/utils/constants.js

@@ -12,5 +12,9 @@ module.exports = {
     GENERAL_ERROR: 1,
     INCOMPATIBILITY: 10,
     MAX_TIME_REACHED: 11
+  },
+  GAS_PRICE_BOUNDARIES: {
+    MIN: 1,
+    MAX: 250
   }
 }

test/gasPrice.test.js

@@ -1,7 +1,7 @@
 const sinon = require('sinon')
 const { expect } = require('chai')
 const proxyquire = require('proxyquire').noPreserveCache()
-const { fetchGasPrice } = require('../src/services/gasPrice')
+const { fetchGasPrice, gasPriceWithinLimits } = require('../src/services/gasPrice')
 const { DEFAULT_UPDATE_INTERVAL } = require('../src/utils/constants')
 
 describe('gasPrice', () => {
@@ -138,4 +138,42 @@ describe('gasPrice', () => {
       expect(utils.setIntervalAndRun.args[0][1]).to.equal(DEFAULT_UPDATE_INTERVAL)
     })
   })
+  describe('gasPriceWithinLimits', () => {
+    it('should return true if gas price is between boundaries', () => {
+      // given
+      const minGasPrice = 1
+      const middleGasPrice = 10
+      const maxGasPrice = 250
+
+      // when
+      const minGasPriceWithinLimits = gasPriceWithinLimits(minGasPrice)
+      const middleGasPriceWithinLimits = gasPriceWithinLimits(middleGasPrice)
+      const maxGasPriceWithinLimits = gasPriceWithinLimits(maxGasPrice)
+
+      // then
+      expect(minGasPriceWithinLimits).to.equal(true)
+      expect(middleGasPriceWithinLimits).to.equal(true)
+      expect(maxGasPriceWithinLimits).to.equal(true)
+    })
+    it('should return false if gas price is below min boundary', () => {
+      // Given
+      const gasPrice = 0.5
+
+      // When
+      const isGasPriceWithinLimits = gasPriceWithinLimits(gasPrice)
+
+      // Then
+      expect(isGasPriceWithinLimits).to.equal(false)
+    })
+    it('should return false if gas price is above max boundary', () => {
+      // Given
+      const gasPrice = 260
+
+      // When
+      const isGasPriceWithinLimits = gasPriceWithinLimits(gasPrice)
+
+      // Then
+      expect(isGasPriceWithinLimits).to.equal(false)
+    })
+  })
 })