Allow to not specify hops

src/api/dexPriceEstimator/DexPriceEstimatorApi.ts

@@ -1,6 +1,6 @@
 import BigNumber from 'bignumber.js'
 import { assert, TEN_BIG_NUMBER, ONE_BIG_NUMBER } from '@gnosis.pm/dex-js'
-import { ORDER_BOOK_HOPS_DEFAULT, ORDER_BOOK_HOPS_MAX } from 'const'
+import { ORDER_BOOK_HOPS_MAX } from 'const'
 
 export interface DexPriceEstimatorApi {
   getPrice(params: GetPriceParams): Promise<BigNumber | null>
@@ -155,16 +155,20 @@ export class DexPriceEstimatorApiImpl implements DexPriceEstimatorApi {
   }
 
   public getOrderBookUrl(params: OrderBookParams): string {
-    const { networkId, baseTokenId, quoteTokenId, hops = ORDER_BOOK_HOPS_DEFAULT, batchId } = params
-    assert(hops >= 0, 'Hops should be positive')
-    assert(hops <= ORDER_BOOK_HOPS_MAX, 'Hops should be not be greater than ' + ORDER_BOOK_HOPS_MAX)
+    const { networkId, baseTokenId, quoteTokenId, hops, batchId } = params
+    if (hops) {
+      assert(hops <= ORDER_BOOK_HOPS_MAX, 'Hops should be not be greater than ' + ORDER_BOOK_HOPS_MAX)
+    }
 
     const baseUrl = this._getBaseUrl(networkId)
 
-    let url = `${baseUrl}markets/${baseTokenId}-${quoteTokenId}?atoms=true&hops=${hops}`
+    let url = `${baseUrl}markets/${baseTokenId}-${quoteTokenId}?atoms=true`
     if (batchId) {
       url += `&batchId=${batchId}`
     }
+    if (hops) {
+      url += `hops=${hops}`
+    }
     return url
   }
 

src/const.ts

@@ -167,7 +167,7 @@ export const WXDAI_ADDRESS_XDAI = '0xe91D153E0b41518A2Ce8Dd3D7944Fa863463a97d'
 export const WETH_ADDRESS_XDAI = '0x6A023CCd1ff6F2045C3309768eAd9E68F978f6e1'
 
 export const ORDER_BOOK_HOPS_DEFAULT = 2
-export const ORDER_BOOK_HOPS_MAX = 2
+export const ORDER_BOOK_HOPS_MAX = 100
 
 export const REFRESH_WHEN_SECONDS_LEFT = 60 // 1min before batch done
 // for stable reference