Add test for api-server DELETE trade
This commit is contained in:
parent
215972c68f
commit
26c7341b7d
@ -50,6 +50,12 @@ def client_get(client, url):
|
|||||||
'Origin': 'http://example.com'})
|
'Origin': 'http://example.com'})
|
||||||
|
|
||||||
|
|
||||||
|
def client_delete(client, url):
|
||||||
|
# Add fake Origin to ensure CORS kicks in
|
||||||
|
return client.delete(url, headers={'Authorization': _basic_auth_str(_TEST_USER, _TEST_PASS),
|
||||||
|
'Origin': 'http://example.com'})
|
||||||
|
|
||||||
|
|
||||||
def assert_response(response, expected_code=200, needs_cors=True):
|
def assert_response(response, expected_code=200, needs_cors=True):
|
||||||
assert response.status_code == expected_code
|
assert response.status_code == expected_code
|
||||||
assert response.content_type == "application/json"
|
assert response.content_type == "application/json"
|
||||||
@ -352,7 +358,7 @@ def test_api_daily(botclient, mocker, ticker, fee, markets):
|
|||||||
assert rc.json['data'][0]['date'] == str(datetime.utcnow().date())
|
assert rc.json['data'][0]['date'] == str(datetime.utcnow().date())
|
||||||
|
|
||||||
|
|
||||||
def test_api_trades(botclient, mocker, ticker, fee, markets):
|
def test_api_trades(botclient, mocker, fee, markets):
|
||||||
ftbot, client = botclient
|
ftbot, client = botclient
|
||||||
patch_get_signal(ftbot, (True, False))
|
patch_get_signal(ftbot, (True, False))
|
||||||
mocker.patch.multiple(
|
mocker.patch.multiple(
|
||||||
@ -376,6 +382,36 @@ def test_api_trades(botclient, mocker, ticker, fee, markets):
|
|||||||
assert rc.json['trades_count'] == 1
|
assert rc.json['trades_count'] == 1
|
||||||
|
|
||||||
|
|
||||||
|
def test_api_delete_trade(botclient, mocker, fee, markets):
|
||||||
|
ftbot, client = botclient
|
||||||
|
patch_get_signal(ftbot, (True, False))
|
||||||
|
mocker.patch.multiple(
|
||||||
|
'freqtrade.exchange.Exchange',
|
||||||
|
markets=PropertyMock(return_value=markets)
|
||||||
|
)
|
||||||
|
rc = client_delete(client, f"{BASE_URI}/trades/1")
|
||||||
|
# Error - trade won't exist yet.
|
||||||
|
assert_response(rc, 502)
|
||||||
|
|
||||||
|
create_mock_trades(fee)
|
||||||
|
trades = Trade.query.all()
|
||||||
|
assert len(trades) > 2
|
||||||
|
|
||||||
|
rc = client_delete(client, f"{BASE_URI}/trades/1")
|
||||||
|
assert_response(rc)
|
||||||
|
assert rc.json['result_msg'] == 'Deleted trade 1.'
|
||||||
|
assert len(trades) - 1 == len(Trade.query.all())
|
||||||
|
|
||||||
|
rc = client_delete(client, f"{BASE_URI}/trades/1")
|
||||||
|
# Trade is gone now.
|
||||||
|
assert_response(rc, 502)
|
||||||
|
assert len(trades) - 1 == len(Trade.query.all())
|
||||||
|
rc = client_delete(client, f"{BASE_URI}/trades/2")
|
||||||
|
assert_response(rc)
|
||||||
|
assert rc.json['result_msg'] == 'Deleted trade 2.'
|
||||||
|
assert len(trades) - 2 == len(Trade.query.all())
|
||||||
|
|
||||||
|
|
||||||
def test_api_edge_disabled(botclient, mocker, ticker, fee, markets):
|
def test_api_edge_disabled(botclient, mocker, ticker, fee, markets):
|
||||||
ftbot, client = botclient
|
ftbot, client = botclient
|
||||||
patch_get_signal(ftbot, (True, False))
|
patch_get_signal(ftbot, (True, False))
|
||||||
|
Loading…
Reference in New Issue
Block a user