gnosis / dex-services / 4697

31%

push

travis-pro

Unit parameter with default (#1300)

Recreated PR from #1287 as it was merged into the wrong branch.

Fixes #1106

This PR adds defaults the unit to use base units instead of requiring the atoms parameter.

### Test Plan

Run the price estimator and check the new routes:
```
$ cargo run -p price-estimator
...
$ curl -s 'http://localhost:8080/api/v1/markets/7-1/estimated-buy-amount/1' | jq '.buyAmountInBase'
"373.97582492490005"
$ curl -s 'http://localhost:8080/api/v1/markets/7-1/estimated-buy-amount/1?unit=baseunits' | jq '.buyAmountInBase'
"373.97582492490005"
$ curl -s 'http://localhost:8080/api/v1/markets/7-1/estimated-buy-amount/1?atoms=false' | jq '.buyAmountInBase'
"373.97582492490005"
$ curl -s 'http://localhost:8080/api/v1/markets/7-1/estimated-buy-amount/1000000000000000000?unit=atoms' | jq '.buyAmountInBase'
"373975824924900065280"
$ curl -s 'http://localhost:8080/api/v1/markets/7-1/estimated-buy-amount/1000000000000000000?atoms=true' | jq '.buyAmountInBase'
"373975824924900065280"
$ curl -s 'http://localhost:8080/api/v1/markets/7-1/estimated-buy-amount/1?atoms=true&unit=atoms' | jq
{
  "message": "invalid url query"
}
```

12279 of 48698 branches covered (25.21%)

Branch coverage included in aggregate %.

3352 of 5238 relevant lines covered (63.99%)

27590.29 hits per line