From accc1b509be9543f212d840d0d3108a2153cb374 Mon Sep 17 00:00:00 2001
From: Matthias <xmatthias@outlook.com>
Date: Thu, 29 Jun 2023 12:16:10 +0000
Subject: [PATCH] Simplify class setups without inheritance

---
 freqtrade/mixins/logging_mixin.py            | 2 +-
 freqtrade/optimize/hyperopt_tools.py         | 4 ++--
 freqtrade/persistence/key_value_store.py     | 2 +-
 freqtrade/persistence/pairlock_middleware.py | 2 +-
 freqtrade/persistence/trade_model.py         | 2 +-
 freqtrade/plugins/protectionmanager.py       | 2 +-
 freqtrade/rpc/api_server/webserver_bgwork.py | 2 +-
 freqtrade/strategy/interface.py              | 2 +-
 scripts/rest_client.py                       | 2 +-
 tests/exchange/test_ccxt_compat.py           | 2 +-
 10 files changed, 11 insertions(+), 11 deletions(-)

diff --git a/freqtrade/mixins/logging_mixin.py b/freqtrade/mixins/logging_mixin.py
index 06935d5f6..31b49ba55 100644
--- a/freqtrade/mixins/logging_mixin.py
+++ b/freqtrade/mixins/logging_mixin.py
@@ -3,7 +3,7 @@ from typing import Callable
 from cachetools import TTLCache, cached
 
 
-class LoggingMixin():
+class LoggingMixin:
     """
     Logging Mixin
     Shows similar messages only once every `refresh_period`.
diff --git a/freqtrade/optimize/hyperopt_tools.py b/freqtrade/optimize/hyperopt_tools.py
index 1e7befdf6..bc5b85309 100644
--- a/freqtrade/optimize/hyperopt_tools.py
+++ b/freqtrade/optimize/hyperopt_tools.py
@@ -35,7 +35,7 @@ def hyperopt_serializer(x):
     return str(x)
 
 
-class HyperoptStateContainer():
+class HyperoptStateContainer:
     """ Singleton class to track state of hyperopt"""
     state: HyperoptState = HyperoptState.OPTIMIZE
 
@@ -44,7 +44,7 @@ class HyperoptStateContainer():
         cls.state = value
 
 
-class HyperoptTools():
+class HyperoptTools:
 
     @staticmethod
     def get_strategy_filename(config: Config, strategy_name: str) -> Optional[Path]:
diff --git a/freqtrade/persistence/key_value_store.py b/freqtrade/persistence/key_value_store.py
index 110a23d6c..6da7265d6 100644
--- a/freqtrade/persistence/key_value_store.py
+++ b/freqtrade/persistence/key_value_store.py
@@ -42,7 +42,7 @@ class _KeyValueStoreModel(ModelBase):
     int_value: Mapped[Optional[int]]
 
 
-class KeyValueStore():
+class KeyValueStore:
     """
     Generic bot-wide, persistent key-value store
     Can be used to store generic values, e.g. very first bot startup time.
diff --git a/freqtrade/persistence/pairlock_middleware.py b/freqtrade/persistence/pairlock_middleware.py
index 29169a50d..dd6bacf3a 100644
--- a/freqtrade/persistence/pairlock_middleware.py
+++ b/freqtrade/persistence/pairlock_middleware.py
@@ -11,7 +11,7 @@ from freqtrade.persistence.models import PairLock
 logger = logging.getLogger(__name__)
 
 
-class PairLocks():
+class PairLocks:
     """
     Pairlocks middleware class
     Abstracts the database layer away so it becomes optional - which will be necessary to support
diff --git a/freqtrade/persistence/trade_model.py b/freqtrade/persistence/trade_model.py
index 4d3564df6..35a44e3fc 100644
--- a/freqtrade/persistence/trade_model.py
+++ b/freqtrade/persistence/trade_model.py
@@ -283,7 +283,7 @@ class Order(ModelBase):
         return Order.session.scalars(select(Order).filter(Order.order_id == order_id)).first()
 
 
-class LocalTrade():
+class LocalTrade:
     """
     Trade database model.
     Used in backtesting - must be aligned to Trade model!
diff --git a/freqtrade/plugins/protectionmanager.py b/freqtrade/plugins/protectionmanager.py
index 54432e677..6e55ade11 100644
--- a/freqtrade/plugins/protectionmanager.py
+++ b/freqtrade/plugins/protectionmanager.py
@@ -15,7 +15,7 @@ from freqtrade.resolvers import ProtectionResolver
 logger = logging.getLogger(__name__)
 
 
-class ProtectionManager():
+class ProtectionManager:
 
     def __init__(self, config: Config, protections: List) -> None:
         self._config = config
diff --git a/freqtrade/rpc/api_server/webserver_bgwork.py b/freqtrade/rpc/api_server/webserver_bgwork.py
index 3846fe138..13f45227e 100644
--- a/freqtrade/rpc/api_server/webserver_bgwork.py
+++ b/freqtrade/rpc/api_server/webserver_bgwork.py
@@ -14,7 +14,7 @@ class JobsContainer(TypedDict):
     error: Optional[str]
 
 
-class ApiBG():
+class ApiBG:
     # Backtesting type: Backtesting
     bt: Dict[str, Any] = {
         'bt': None,
diff --git a/freqtrade/strategy/interface.py b/freqtrade/strategy/interface.py
index d0655b504..1e9ebe1ae 100644
--- a/freqtrade/strategy/interface.py
+++ b/freqtrade/strategy/interface.py
@@ -168,7 +168,7 @@ class IStrategy(ABC, HyperStrategyMixin):
                 download_all_data_for_training(self.dp, self.config)
         else:
             # Gracious failures if freqAI is disabled but "start" is called.
-            class DummyClass():
+            class DummyClass:
                 def start(self, *args, **kwargs):
                     raise OperationalException(
                         'freqAI is not enabled. '
diff --git a/scripts/rest_client.py b/scripts/rest_client.py
index f9c9858ed..2b4690287 100755
--- a/scripts/rest_client.py
+++ b/scripts/rest_client.py
@@ -29,7 +29,7 @@ logging.basicConfig(
 logger = logging.getLogger("ft_rest_client")
 
 
-class FtRestClient():
+class FtRestClient:
 
     def __init__(self, serverurl, username=None, password=None):
 
diff --git a/tests/exchange/test_ccxt_compat.py b/tests/exchange/test_ccxt_compat.py
index 404b51d10..b1a6f1c65 100644
--- a/tests/exchange/test_ccxt_compat.py
+++ b/tests/exchange/test_ccxt_compat.py
@@ -342,7 +342,7 @@ def exchange_futures(request, exchange_conf, class_mocker):
 
 
 @pytest.mark.longrun
-class TestCCXTExchange():
+class TestCCXTExchange:
 
     def test_load_markets(self, exchange: EXCHANGE_FIXTURE_TYPE):
         exch, exchangename = exchange