add test for non-configured exchange
This commit is contained in:
parent
ae4c4e77bf
commit
162f948729
@ -587,11 +587,10 @@ def test_get_id(default_conf, mocker):
|
|||||||
assert exchange.get_id() == 'binance'
|
assert exchange.get_id() == 'binance'
|
||||||
|
|
||||||
|
|
||||||
def test_get_pair_detail_url(default_conf, mocker):
|
def test_get_pair_detail_url(default_conf, mocker, caplog):
|
||||||
mocker.patch('freqtrade.exchange.Exchange.validate_pairs',
|
mocker.patch('freqtrade.exchange.Exchange.validate_pairs',
|
||||||
side_effect=lambda s: True)
|
side_effect=lambda s: True)
|
||||||
default_conf['exchange']['name'] = 'binance'
|
default_conf['exchange']['name'] = 'binance'
|
||||||
# exchange = get_patched_exchange(mocker, default_conf, api_mock)
|
|
||||||
exchange = Exchange(default_conf)
|
exchange = Exchange(default_conf)
|
||||||
|
|
||||||
url = exchange.get_pair_detail_url('TKN/ETH')
|
url = exchange.get_pair_detail_url('TKN/ETH')
|
||||||
@ -613,6 +612,12 @@ def test_get_pair_detail_url(default_conf, mocker):
|
|||||||
assert 'LOOONG' in url
|
assert 'LOOONG' in url
|
||||||
assert 'BTC' in url
|
assert 'BTC' in url
|
||||||
|
|
||||||
|
default_conf['exchange']['name'] = 'poloniex'
|
||||||
|
exchange = Exchange(default_conf)
|
||||||
|
url = exchange.get_pair_detail_url('LOOONG/BTC')
|
||||||
|
assert '' == url
|
||||||
|
assert log_has('Could not get exchange url for Poloniex', caplog.record_tuples)
|
||||||
|
|
||||||
|
|
||||||
def test_get_trades_for_order(default_conf, mocker):
|
def test_get_trades_for_order(default_conf, mocker):
|
||||||
order_id = 'ABCD-ABCD'
|
order_id = 'ABCD-ABCD'
|
||||||
|
Loading…
Reference in New Issue
Block a user