ruff error fix

Author: TamiTakamiya

tests/integration/test_config.py

@@ -25,7 +25,7 @@ def test_load_proper_config():
 def test_load_non_existent_config():
     """Test how loading of non-existent config is handled."""
     with pytest.raises(
-        FileNotFoundError, match="tests/config/non_existent_config.yaml"
+        FileNotFoundError, match=r"tests/config/non_existent_config.yaml"
     ):
         config.reload_from_yaml_file("tests/config/non_existent_config.yaml")
 

tests/unit/app/endpoints/test_ols.py

@@ -371,7 +371,7 @@ def test_validate_question_too_long_query(llm_validate_question_mock):
     query = "Tell me about Kubernetes?"
     llm_request = LLMRequest(query=query, conversation_id=conversation_id)
     # PromptTooLongError should be caught and HTTPException needs to be raised
-    with pytest.raises(HTTPException, match="413: {'response': 'Prompt is too long'"):
+    with pytest.raises(HTTPException, match=r"413: {'response': 'Prompt is too long'"):
         ols.validate_question(conversation_id, llm_request)
 
 

tests/unit/app/models/test_config.py

@@ -1703,7 +1703,7 @@ def test_tls_config_no_data_provided():
     tls_config = TLSConfig(None)
     with pytest.raises(
         InvalidConfigurationError,
-        match="Can not enable TLS without ols_config.tls_config.tls_certificate_path",
+        match=r"Can not enable TLS without ols_config.tls_config.tls_certificate_path",
     ):
         tls_config.validate_yaml(False)
 
@@ -1718,7 +1718,7 @@ def test_tls_config_no_tls_key_path():
     )
     with pytest.raises(
         InvalidConfigurationError,
-        match="Can not enable TLS without ols_config.tls_config.tls_key_path",
+        match=r"Can not enable TLS without ols_config.tls_config.tls_key_path",
     ):
         tls_config.validate_yaml(False)
 
@@ -3615,7 +3615,7 @@ def test_ols_config_with_non_existing_system_prompt(tmpdir):
     """Test the OLSConfig model with system prompt path specification that does not exist."""
     with pytest.raises(
         FileNotFoundError,
-        match="No such file or directory: 'tests/config/non_existing_file.txt'",
+        match=r"No such file or directory: 'tests/config/non_existing_file.txt'",
     ):
         OLSConfig(
             {
@@ -3921,7 +3921,7 @@ def test_missing_lightspeed_id_attribute():
         }
     ]
 
-    with pytest.raises(KeyError, match="lightspeed id is missing."):
+    with pytest.raises(KeyError, match=r"lightspeed id is missing."):
         config._parse_rhdh_lightspeed_config(data)
 
 
@@ -3936,7 +3936,7 @@ def test_missing_lightspeed_url_attribute():
             "token": "lightspeed-token",
         }
     ]
-    with pytest.raises(KeyError, match="lightspeed url is missing."):
+    with pytest.raises(KeyError, match=r"lightspeed url is missing."):
         config._parse_rhdh_lightspeed_config(data)
 
 
@@ -3951,7 +3951,7 @@ def test_missing_lightspeed_models_attribute():
             "token": "lightspeed-token",
         }
     ]
-    with pytest.raises(KeyError, match="lightspeed models missing."):
+    with pytest.raises(KeyError, match=r"lightspeed models missing."):
         config._parse_rhdh_lightspeed_config(data)
 
 
@@ -3966,7 +3966,7 @@ def test_missing_lightspeed_token_attribute():
             "type": "lightspeed-type",
         }
     ]
-    with pytest.raises(KeyError, match="lightspeed token is missing."):
+    with pytest.raises(KeyError, match=r"lightspeed token is missing."):
         config._parse_rhdh_lightspeed_config(data)
 
 

tests/unit/app/models/test_models.py

@@ -67,14 +67,14 @@ def test_llm_request_provider_and_model():
         # model set and provider not
         with pytest.raises(
             ValidationError,
-            match="LLM provider must be specified when the model is specified.",
+            match=r"LLM provider must be specified when the model is specified.",
         ):
             LLMRequest(query="bla", provider=None, model="davinci")
 
         # provider set and model not
         with pytest.raises(
             ValidationError,
-            match="LLM model must be specified when the provider is specified.",
+            match=r"LLM model must be specified when the provider is specified.",
         ):
             LLMRequest(query="bla", provider="openai", model=None)
 

tests/unit/quota/test_quota_limiter_factory.py

@@ -123,5 +123,5 @@ def test_quota_limiters_unknown_limiter(postgres_mock):
             }
         ]
     )
-    with pytest.raises(ValueError, match="Invalid limiter type: UNKNOWN."):
+    with pytest.raises(ValueError, match=r"Invalid limiter type: UNKNOWN."):
         QuotaLimiterFactory.quota_limiters(config)

tests/unit/utils/test_config.py

@@ -1471,7 +1471,7 @@ def test_loading_incorrect_additional_config_file_rhdh_type():
 def test_additional_config_file_incorrect_type():
     """Check how an additional configuration file is handled with an incorrect type."""
     with pytest.raises(
-        ValueError, match="Configuration file type 'wrongtype' not recognized."
+        ValueError, match=r"Configuration file type 'wrongtype' not recognized."
     ):
         config.reload_additional_config_file(
             "tests/config/valid_rhdh_config.yaml", "wrongtype"