Adjust tests to pass in order_types instead of rate
This commit is contained in:
		| @@ -28,11 +28,12 @@ def test_stoploss_limit_order(default_conf, mocker): | |||||||
|     exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') |     exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') | ||||||
|  |  | ||||||
|     with pytest.raises(OperationalException): |     with pytest.raises(OperationalException): | ||||||
|         order = exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=190, rate=200) |         order = exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=190, | ||||||
|  |                                         order_types={'stoploss_on_exchange_limit_ratio': 1.05}) | ||||||
|  |  | ||||||
|     api_mock.create_order.reset_mock() |     api_mock.create_order.reset_mock() | ||||||
|  |  | ||||||
|     order = exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, rate=200) |     order = exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, order_types={}) | ||||||
|  |  | ||||||
|     assert 'id' in order |     assert 'id' in order | ||||||
|     assert 'info' in order |     assert 'info' in order | ||||||
| @@ -41,30 +42,29 @@ def test_stoploss_limit_order(default_conf, mocker): | |||||||
|     assert api_mock.create_order.call_args[0][1] == order_type |     assert api_mock.create_order.call_args[0][1] == order_type | ||||||
|     assert api_mock.create_order.call_args[0][2] == 'sell' |     assert api_mock.create_order.call_args[0][2] == 'sell' | ||||||
|     assert api_mock.create_order.call_args[0][3] == 1 |     assert api_mock.create_order.call_args[0][3] == 1 | ||||||
|     assert api_mock.create_order.call_args[0][4] == 200 |  | ||||||
|     assert api_mock.create_order.call_args[0][5] == {'stopPrice': 220} |     assert api_mock.create_order.call_args[0][5] == {'stopPrice': 220} | ||||||
|  |  | ||||||
|     # test exception handling |     # test exception handling | ||||||
|     with pytest.raises(DependencyException): |     with pytest.raises(DependencyException): | ||||||
|         api_mock.create_order = MagicMock(side_effect=ccxt.InsufficientFunds("0 balance")) |         api_mock.create_order = MagicMock(side_effect=ccxt.InsufficientFunds("0 balance")) | ||||||
|         exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') |         exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') | ||||||
|         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, rate=200) |         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, order_types={}) | ||||||
|  |  | ||||||
|     with pytest.raises(InvalidOrderException): |     with pytest.raises(InvalidOrderException): | ||||||
|         api_mock.create_order = MagicMock( |         api_mock.create_order = MagicMock( | ||||||
|             side_effect=ccxt.InvalidOrder("binance Order would trigger immediately.")) |             side_effect=ccxt.InvalidOrder("binance Order would trigger immediately.")) | ||||||
|         exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') |         exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') | ||||||
|         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, rate=200) |         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, order_types={}) | ||||||
|  |  | ||||||
|     with pytest.raises(TemporaryError): |     with pytest.raises(TemporaryError): | ||||||
|         api_mock.create_order = MagicMock(side_effect=ccxt.NetworkError("No connection")) |         api_mock.create_order = MagicMock(side_effect=ccxt.NetworkError("No connection")) | ||||||
|         exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') |         exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') | ||||||
|         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, rate=200) |         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, order_types={}) | ||||||
|  |  | ||||||
|     with pytest.raises(OperationalException, match=r".*DeadBeef.*"): |     with pytest.raises(OperationalException, match=r".*DeadBeef.*"): | ||||||
|         api_mock.create_order = MagicMock(side_effect=ccxt.BaseError("DeadBeef")) |         api_mock.create_order = MagicMock(side_effect=ccxt.BaseError("DeadBeef")) | ||||||
|         exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') |         exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') | ||||||
|         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, rate=200) |         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, order_types={}) | ||||||
|  |  | ||||||
|  |  | ||||||
| def test_stoploss_limit_order_dry_run(default_conf, mocker): | def test_stoploss_limit_order_dry_run(default_conf, mocker): | ||||||
| @@ -77,11 +77,12 @@ def test_stoploss_limit_order_dry_run(default_conf, mocker): | |||||||
|     exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') |     exchange = get_patched_exchange(mocker, default_conf, api_mock, 'binance') | ||||||
|  |  | ||||||
|     with pytest.raises(OperationalException): |     with pytest.raises(OperationalException): | ||||||
|         order = exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=190, rate=200) |         order = exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=190, | ||||||
|  |                                         order_types={'stoploss_on_exchange_limit_ratio': 1.05}) | ||||||
|  |  | ||||||
|     api_mock.create_order.reset_mock() |     api_mock.create_order.reset_mock() | ||||||
|  |  | ||||||
|     order = exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, rate=200) |     order = exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, order_types={}) | ||||||
|  |  | ||||||
|     assert 'id' in order |     assert 'id' in order | ||||||
|     assert 'info' in order |     assert 'info' in order | ||||||
|   | |||||||
| @@ -1761,7 +1761,7 @@ def test_get_fee(default_conf, mocker, exchange_name): | |||||||
| def test_stoploss_limit_order_unsupported_exchange(default_conf, mocker): | def test_stoploss_limit_order_unsupported_exchange(default_conf, mocker): | ||||||
|     exchange = get_patched_exchange(mocker, default_conf, 'bittrex') |     exchange = get_patched_exchange(mocker, default_conf, 'bittrex') | ||||||
|     with pytest.raises(OperationalException, match=r"stoploss_limit is not implemented .*"): |     with pytest.raises(OperationalException, match=r"stoploss_limit is not implemented .*"): | ||||||
|         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, rate=200) |         exchange.stoploss_limit(pair='ETH/BTC', amount=1, stop_price=220, order_types={}) | ||||||
|  |  | ||||||
|  |  | ||||||
| def test_merge_ft_has_dict(default_conf, mocker): | def test_merge_ft_has_dict(default_conf, mocker): | ||||||
|   | |||||||
| @@ -1315,7 +1315,7 @@ def test_handle_stoploss_on_exchange_trailing(mocker, default_conf, fee, caplog, | |||||||
|     cancel_order_mock.assert_called_once_with(100, 'ETH/BTC') |     cancel_order_mock.assert_called_once_with(100, 'ETH/BTC') | ||||||
|     stoploss_order_mock.assert_called_once_with(amount=85.25149190110828, |     stoploss_order_mock.assert_called_once_with(amount=85.25149190110828, | ||||||
|                                                 pair='ETH/BTC', |                                                 pair='ETH/BTC', | ||||||
|                                                 rate=0.00002344 * 0.95 * 0.99, |                                                 order_types=freqtrade.strategy.order_types, | ||||||
|                                                 stop_price=0.00002344 * 0.95) |                                                 stop_price=0.00002344 * 0.95) | ||||||
|  |  | ||||||
|  |  | ||||||
| @@ -1492,7 +1492,7 @@ def test_tsl_on_exchange_compatible_with_edge(mocker, edge_conf, fee, caplog, | |||||||
|     cancel_order_mock.assert_called_once_with(100, 'NEO/BTC') |     cancel_order_mock.assert_called_once_with(100, 'NEO/BTC') | ||||||
|     stoploss_order_mock.assert_called_once_with(amount=2131074.168797954, |     stoploss_order_mock.assert_called_once_with(amount=2131074.168797954, | ||||||
|                                                 pair='NEO/BTC', |                                                 pair='NEO/BTC', | ||||||
|                                                 rate=0.00002344 * 0.99 * 0.99, |                                                 order_types=freqtrade.strategy.order_types, | ||||||
|                                                 stop_price=0.00002344 * 0.99) |                                                 stop_price=0.00002344 * 0.99) | ||||||
|  |  | ||||||
|  |  | ||||||
|   | |||||||
		Reference in New Issue
	
	Block a user