From 50ba20ec9fe1cab05a2b9a4e652d26e7157fa97b Mon Sep 17 00:00:00 2001 From: Matthias Date: Fri, 25 Mar 2022 16:14:18 +0100 Subject: [PATCH] Remove some unused test methods --- tests/conftest.py | 6 +----- tests/test_persistence.py | 8 ++++---- 2 files changed, 5 insertions(+), 9 deletions(-) diff --git a/tests/conftest.py b/tests/conftest.py index 05ff39358..117aeaaed 100644 --- a/tests/conftest.py +++ b/tests/conftest.py @@ -5,7 +5,7 @@ import re from copy import deepcopy from datetime import datetime, timedelta from pathlib import Path -from typing import Optional, Tuple +from typing import Optional from unittest.mock import MagicMock, Mock, PropertyMock import arrow @@ -360,10 +360,6 @@ def create_mock_trades_usdt(fee, use_db: bool = True): Trade.commit() -def get_sides(is_short: bool) -> Tuple[str, str]: - return ("sell", "buy") if is_short else ("buy", "sell") - - @pytest.fixture(autouse=True) def patch_coingekko(mocker) -> None: """ diff --git a/tests/test_persistence.py b/tests/test_persistence.py index 34fc98fef..b83b75cc2 100644 --- a/tests/test_persistence.py +++ b/tests/test_persistence.py @@ -15,8 +15,8 @@ from freqtrade.enums import TradingMode from freqtrade.exceptions import DependencyException, OperationalException from freqtrade.persistence import LocalTrade, Order, Trade, clean_dry_run_db, init_db from freqtrade.persistence.migrations import get_last_sequence_ids, set_sequence_ids -from tests.conftest import (create_mock_trades, create_mock_trades_usdt, - create_mock_trades_with_leverage, get_sides, log_has, log_has_re) +from tests.conftest import (create_mock_trades, + create_mock_trades_with_leverage, log_has, log_has_re) spot, margin, futures = TradingMode.SPOT, TradingMode.MARGIN, TradingMode.FUTURES @@ -77,7 +77,7 @@ def test_init_dryrun_db(default_conf, tmpdir): @pytest.mark.parametrize('is_short', [False, True]) @pytest.mark.usefixtures("init_persistence") def test_enter_exit_side(fee, is_short): - enter_side, exit_side = get_sides(is_short) + enter_side, exit_side = ("sell", "buy") if is_short else ("buy", "sell") trade = Trade( id=2, pair='ADA/USDT', @@ -456,7 +456,7 @@ def test_update_limit_order(fee, caplog, limit_buy_order_usdt, limit_sell_order_ enter_order = limit_sell_order_usdt if is_short else limit_buy_order_usdt exit_order = limit_buy_order_usdt if is_short else limit_sell_order_usdt - enter_side, exit_side = get_sides(is_short) + enter_side, exit_side = ("sell", "buy") if is_short else ("buy", "sell") trade = Trade( id=2,