adelorenzo/stable
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Fixed test_ftx patch

Browse Source
This commit is contained in:
Sam Germain 2021-09-10 02:09:27 -06:00
parent 77fc21a16b
commit 77aa372909
2 changed files with 1 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
freqtrade/exchange/exchange.py
View File

@ -75,8 +75,6 @@ class Exchange:
} }
_ft_has: Dict = {} _ft_has: Dict = {}
_leverage_brackets: Dict = {}
_supported_trading_mode_collateral_pairs: List[Tuple[TradingMode, Collateral]] = [ _supported_trading_mode_collateral_pairs: List[Tuple[TradingMode, Collateral]] = [
# TradingMode.SPOT always supported and not required in this list # TradingMode.SPOT always supported and not required in this list
] ]
@ -90,6 +88,7 @@ class Exchange:
self._api: ccxt.Exchange = None self._api: ccxt.Exchange = None
self._api_async: ccxt_async.Exchange = None self._api_async: ccxt_async.Exchange = None
self._markets: Dict = {} self._markets: Dict = {}
self._leverage_brackets: Dict = {}
self._config.update(config) self._config.update(config)

3
tests/exchange/test_ftx.py
View File

@ -227,8 +227,5 @@ def test_get_max_leverage_ftx(default_conf, mocker, pair, nominal_value, max_lev
def test_fill_leverage_brackets_ftx(default_conf, mocker): def test_fill_leverage_brackets_ftx(default_conf, mocker):
# FTX only has one account wide leverage, so there's no leverage brackets # FTX only has one account wide leverage, so there's no leverage brackets
exchange = get_patched_exchange(mocker, default_conf, id="ftx") exchange = get_patched_exchange(mocker, default_conf, id="ftx")
# TODO: This is a patch, develop a real solution
# TODO: _leverage_brackets retains it's value from the binance tests, but shouldn't
exchange._leverage_brackets = {}
exchange.fill_leverage_brackets() exchange.fill_leverage_brackets()
assert exchange._leverage_brackets == {} assert exchange._leverage_brackets == {}