chore: fix RUF043 violations

Author: xmatthias

tests/commands/test_commands.py

@@ -658,7 +658,9 @@ def test_start_new_strategy_no_arg():
     args = [
         "new-strategy",
     ]
-    with pytest.raises(OperationalException, match="`new-strategy` requires --strategy to be set."):
+    with pytest.raises(
+        OperationalException, match=r"`new-strategy` requires --strategy to be set\."
+    ):
         start_new_strategy(get_args(args))
 
 
@@ -803,7 +805,7 @@ def test_get_ui_download_url_direct(mocker):
     assert last_version == "0.0.1"
     assert x == "http://download1.zip"
 
-    with pytest.raises(ValueError, match="UI-Version not found."):
+    with pytest.raises(ValueError, match=r"UI-Version not found\."):
         x, last_version = get_ui_download_url("0.0.3", False)
 
 
@@ -1650,15 +1652,15 @@ def fake_iterator(*args, **kwargs):
     pargs = get_args(args)
     pargs["config"] = None
     with pytest.raises(
-        OperationalException, match="The index of the epoch to show should be greater than -4."
+        OperationalException, match=r"The index of the epoch to show should be greater than -4\."
     ):
         start_hyperopt_show(pargs)
 
     args = ["hyperopt-show", "--best", "-n", "4"]
     pargs = get_args(args)
     pargs["config"] = None
     with pytest.raises(
-        OperationalException, match="The index of the epoch to show should be less than 4."
+        OperationalException, match=r"The index of the epoch to show should be less than 4\."
     ):
         start_hyperopt_show(pargs)
 
@@ -2032,5 +2034,7 @@ def test_start_edge():
     ]
 
     pargs = get_args(args)
-    with pytest.raises(OperationalException, match="The Edge module has been deprecated in 2023.9"):
+    with pytest.raises(
+        OperationalException, match=r"The Edge module has been deprecated in 2023\.9"
+    ):
         start_edge(pargs)

tests/data/test_btanalysis.py

@@ -75,7 +75,7 @@ def test_get_latest_hyperopt_file(testdatadir):
     # Test with absolute path
     with pytest.raises(
         OperationalException,
-        match="--hyperopt-filename expects only the filename, not an absolute path.",
+        match=r"--hyperopt-filename expects only the filename, not an absolute path\.",
     ):
         get_latest_hyperopt_file(str(testdatadir.parent), str(testdatadir.parent))
 
@@ -344,7 +344,7 @@ def test_create_cum_profit1(testdatadir):
     assert cum_profits.iloc[0]["cum_profits"] == 0
     assert pytest.approx(cum_profits.iloc[-1]["cum_profits"]) == 9.0225563e-05
 
-    with pytest.raises(ValueError, match="Trade dataframe empty."):
+    with pytest.raises(ValueError, match=r"Trade dataframe empty\."):
         create_cum_profit(
             df.set_index("date"),
             bt_data[bt_data["pair"] == "NOTAPAIR"],
@@ -369,10 +369,10 @@ def test_calculate_max_drawdown(testdatadir):
     underwater = calculate_underwater(bt_data)
     assert isinstance(underwater, DataFrame)
 
-    with pytest.raises(ValueError, match="Trade dataframe empty."):
+    with pytest.raises(ValueError, match=r"Trade dataframe empty\."):
         calculate_max_drawdown(DataFrame())
 
-    with pytest.raises(ValueError, match="Trade dataframe empty."):
+    with pytest.raises(ValueError, match=r"Trade dataframe empty\."):
         calculate_underwater(DataFrame())
 
 
@@ -391,7 +391,7 @@ def test_calculate_csum(testdatadir):
     assert csum_min1 == csum_min + 5
     assert csum_max1 == csum_max + 5
 
-    with pytest.raises(ValueError, match="Trade dataframe empty."):
+    with pytest.raises(ValueError, match=r"Trade dataframe empty\."):
         csum_min, csum_max = calculate_csum(DataFrame())
 
 

tests/data/test_converter.py

@@ -49,7 +49,7 @@ def test_ohlcv_to_dataframe(ohlcv_history_list, caplog):
 
 def test_trades_to_ohlcv(trades_history_df, caplog):
     caplog.set_level(logging.DEBUG)
-    with pytest.raises(ValueError, match="Trade-list empty."):
+    with pytest.raises(ValueError, match=r"Trade-list empty\."):
         trades_to_ohlcv(pd.DataFrame(columns=trades_history_df.columns), "1m")
 
     df = trades_to_ohlcv(trades_history_df, "1m")

tests/exchange/test_exchange.py

@@ -859,7 +859,7 @@ def test_validate_pricing(default_conf, mocker):
     default_conf["exchange"]["name"] = "binance"
     ExchangeResolver.load_exchange(default_conf)
     has.update({"fetchTicker": False})
-    with pytest.raises(OperationalException, match="Ticker pricing not available for .*"):
+    with pytest.raises(OperationalException, match=r"Ticker pricing not available for .*"):
         ExchangeResolver.load_exchange(default_conf)
 
     has.update({"fetchTicker": True})
@@ -868,7 +868,7 @@ def test_validate_pricing(default_conf, mocker):
     ExchangeResolver.load_exchange(default_conf)
     has.update({"fetchL2OrderBook": False})
 
-    with pytest.raises(OperationalException, match="Orderbook not available for .*"):
+    with pytest.raises(OperationalException, match=r"Orderbook not available for .*"):
         ExchangeResolver.load_exchange(default_conf)
 
     has.update({"fetchL2OrderBook": True})
@@ -877,7 +877,7 @@ def test_validate_pricing(default_conf, mocker):
     default_conf["trading_mode"] = TradingMode.FUTURES
     default_conf["margin_mode"] = MarginMode.ISOLATED
 
-    with pytest.raises(OperationalException, match="Ticker pricing not available for .*"):
+    with pytest.raises(OperationalException, match=r"Ticker pricing not available for .*"):
         ExchangeResolver.load_exchange(default_conf)
 
 
@@ -3556,7 +3556,7 @@ def test_get_historic_trades_notsupported(
     pair = "ETH/BTC"
 
     with pytest.raises(
-        OperationalException, match="This exchange does not support downloading Trades."
+        OperationalException, match=r"This exchange does not support downloading Trades\."
     ):
         exchange.get_historic_trades(pair, since=trades_history[0][0], until=trades_history[-1][0])
 
@@ -4442,7 +4442,7 @@ def test_get_markets(
 def test_get_markets_error(default_conf, mocker):
     ex = get_patched_exchange(mocker, default_conf)
     mocker.patch(f"{EXMS}.markets", PropertyMock(return_value=None))
-    with pytest.raises(OperationalException, match="Markets were not loaded."):
+    with pytest.raises(OperationalException, match=r"Markets were not loaded\."):
         ex.get_markets("LTC", "USDT", True, False)
 
 
@@ -5243,7 +5243,7 @@ def test__fetch_and_calculate_funding_fees(
     # Return empty "refresh_latest"
     mocker.patch(f"{EXMS}.refresh_latest_ohlcv", return_value={})
     ex = get_patched_exchange(mocker, default_conf, api_mock, exchange=exchange)
-    with pytest.raises(ExchangeError, match="Could not find funding rates."):
+    with pytest.raises(ExchangeError, match=r"Could not find funding rates\."):
         ex._fetch_and_calculate_funding_fees(
             pair="ADA/USDT:USDT", amount=amount, is_short=False, open_date=d1, close_date=d2
         )

tests/freqtradebot/test_freqtradebot.py

@@ -986,7 +986,7 @@ def test_execute_entry(
     # Fail to get price...
     mocker.patch(f"{EXMS}.get_rate", MagicMock(return_value=0.0))
 
-    with pytest.raises(PricingError, match="Could not determine entry price."):
+    with pytest.raises(PricingError, match=r"Could not determine entry price\."):
         freqtrade.execute_entry(pair, stake_amount, is_short=is_short)
 
     # In case of custom entry price

tests/optimize/test_backtesting.py

@@ -444,7 +444,7 @@ def get_timerange(input1):
 
     backtesting = Backtesting(default_conf)
     backtesting._set_strategy(backtesting.strategylist[0])
-    with pytest.raises(OperationalException, match="No data found. Terminating."):
+    with pytest.raises(OperationalException, match=r"No data found. Terminating\."):
         backtesting.start()
 
 
@@ -465,7 +465,7 @@ def test_backtesting_no_pair_left(default_conf, mocker) -> None:
     default_conf["export"] = "none"
     default_conf["timerange"] = "20180101-20180102"
 
-    with pytest.raises(OperationalException, match="No pair in whitelist."):
+    with pytest.raises(OperationalException, match=r"No pair in whitelist\."):
         Backtesting(default_conf)
 
     default_conf.update(
@@ -476,7 +476,7 @@ def test_backtesting_no_pair_left(default_conf, mocker) -> None:
     )
 
     with pytest.raises(
-        OperationalException, match="Detail timeframe must be smaller than strategy timeframe."
+        OperationalException, match=r"Detail timeframe must be smaller than strategy timeframe\."
     ):
         Backtesting(default_conf)
 
@@ -517,7 +517,7 @@ def test_backtesting_pairlist_list(default_conf, mocker, tickers) -> None:
     default_conf["strategy_list"] = [CURRENT_TEST_STRATEGY, "StrategyTestV2"]
     with pytest.raises(
         OperationalException,
-        match="PrecisionFilter not allowed for backtesting multiple strategies.",
+        match=r"PrecisionFilter not allowed for backtesting multiple strategies\.",
     ):
         Backtesting(default_conf)
 

tests/optimize/test_hyperopt.py

@@ -280,7 +280,7 @@ def test_start_no_data(mocker, hyperopt_conf, tmp_path) -> None:
         "5",
     ]
     pargs = get_args(args)
-    with pytest.raises(OperationalException, match="No data found. Terminating."):
+    with pytest.raises(OperationalException, match=r"No data found. Terminating\."):
         start_hyperopt(pargs)
 
     # Cleanup since that failed hyperopt start leaves a lockfile.
@@ -1127,7 +1127,7 @@ def test_in_strategy_auto_hyperopt(mocker, hyperopt_conf, tmp_path, fee) -> None
     assert opt.backtesting.strategy.max_open_trades != 1
 
     opt.custom_hyperopt.generate_estimator = lambda *args, **kwargs: "ET1"
-    with pytest.raises(OperationalException, match="Optuna Sampler ET1 not supported."):
+    with pytest.raises(OperationalException, match=r"Optuna Sampler ET1 not supported\."):
         opt.get_optimizer(42)
 
 

tests/optimize/test_hyperoptloss.py

@@ -11,8 +11,8 @@
 def test_hyperoptlossresolver_noname(default_conf):
     with pytest.raises(
         OperationalException,
-        match="No Hyperopt loss set. Please use `--hyperopt-loss` to specify "
-        "the Hyperopt-Loss class to use.",
+        match=r"No Hyperopt loss set. Please use `--hyperopt-loss` to specify "
+        r"the Hyperopt-Loss class to use\.",
     ):
         HyperOptLossResolver.load_hyperoptloss(default_conf)
 

tests/plugins/test_pairlist.py

@@ -1669,7 +1669,7 @@ def test_rangestabilityfilter_checks(mocker, default_conf, markets, tickers):
 
     with pytest.raises(
         OperationalException,
-        match="RangeStabilityFilter requires sort_direction to be either None.*",
+        match=r"RangeStabilityFilter requires sort_direction to be either None\.*",
     ):
         get_patched_freqtradebot(mocker, default_conf)
 
@@ -2526,7 +2526,7 @@ def test_MarketCapPairList_exceptions(mocker, default_conf_usdt, caplog):
         }
     ]
     with pytest.raises(
-        OperationalException, match="Category layer250 not in coingecko category list."
+        OperationalException, match=r"Category layer250 not in coingecko category list\."
     ):
         PairListManager(exchange, default_conf_usdt)
 

tests/plugins/test_remotepairlist.py

@@ -82,7 +82,7 @@ def test_fetch_pairlist_mock_response_html(mocker, rpl_config):
         exchange, pairlistmanager, rpl_config, rpl_config["pairlists"][0], 0
     )
 
-    with pytest.raises(OperationalException, match="RemotePairList is not of type JSON."):
+    with pytest.raises(OperationalException, match=r"RemotePairList is not of type JSON\."):
         remote_pairlist.fetch_pairlist()