style: better function name with overwrite_config_context

eddiebergman · eddiebergman · commit 0bbd67dfc6c0 · 2024-10-16T14:15:24.000+02:00
diff --git a/openml/config.py b/openml/config.py
@@ -499,7 +499,7 @@ def set_root_cache_directory(root_cache_directory: str | Path) -> None:
 
 
 @contextmanager
-def set_context(config: dict[str, Any]) -> Iterator[_Config]:
+def overwrite_config_context(config: dict[str, Any]) -> Iterator[_Config]:
     """A context manager to temporarily override variables in the configuration."""
     existing_config = get_config_as_dict()
     merged_config = {**existing_config, **config}
diff --git a/tests/test_openml/test_api_calls.py b/tests/test_openml/test_api_calls.py
@@ -120,6 +120,6 @@ def test_authentication_endpoints_requiring_api_key_show_relevant_help_link(
     method: str,
 ) -> None:
     # We need to temporarily disable the API key to test the error message
-    with openml.config.set_context({"apikey": None}):
+    with openml.config.overwrite_config_context({"apikey": None}):
         with pytest.raises(openml.exceptions.OpenMLNotAuthorizedError, match=API_TOKEN_HELP_LINK):
             openml._api_calls._perform_api_call(call=endpoint, request_method=method, data=None)
