diff --git a/freqtrade/exchange/exchange.py b/freqtrade/exchange/exchange.py index 94673fdc3..475baad67 100644 --- a/freqtrade/exchange/exchange.py +++ b/freqtrade/exchange/exchange.py @@ -222,8 +222,11 @@ class Exchange(object): > arrow.utcnow().timestamp): return None logger.debug("Performing scheduled market reload..") - self._api.load_markets(reload=True) - self._last_markets_refresh = arrow.utcnow().timestamp + try: + self._api.load_markets(reload=True) + self._last_markets_refresh = arrow.utcnow().timestamp + except ccxt.NetworkError: + logger.exception("Could not reload markets.") def validate_pairs(self, pairs: List[str]) -> None: """ diff --git a/freqtrade/tests/exchange/test_exchange.py b/freqtrade/tests/exchange/test_exchange.py index 9e471d551..924ed538f 100644 --- a/freqtrade/tests/exchange/test_exchange.py +++ b/freqtrade/tests/exchange/test_exchange.py @@ -301,6 +301,20 @@ def test__reload_markets(default_conf, mocker, caplog): assert log_has('Performing scheduled market reload..', caplog.record_tuples) +def test__reload_markets_exception(default_conf, mocker, caplog): + caplog.set_level(logging.DEBUG) + + api_mock = MagicMock() + api_mock.load_markets = MagicMock(side_effect=ccxt.NetworkError) + default_conf['exchange']['markets_refresh_interval'] = 10 + exchange = get_patched_exchange(mocker, default_conf, api_mock, id="binance") + + # less than 10 minutes have passed, no reload + exchange._reload_markets() + assert exchange._last_markets_refresh == 0 + assert log_has_re(r"Could not reload markets.*", caplog.record_tuples) + + def test_validate_pairs(default_conf, mocker): # test exchange.validate_pairs directly api_mock = MagicMock() type(api_mock).markets = PropertyMock(return_value={