implement the suggestions

Author: Omswastik-11

openml/_api/resources/run.py

@@ -1,8 +1,7 @@
 from __future__ import annotations
 
 import builtins
-from collections.abc import Mapping
-from typing import TYPE_CHECKING, Any
+from typing import TYPE_CHECKING
 
 import pandas as pd
 import xmltodict
@@ -195,23 +194,6 @@ def _parse_list_xml(self, xml_string: str) -> pd.DataFrame:
         }
         return pd.DataFrame.from_dict(runs, orient="index")
 
-    def publish(self, path: str, files: Mapping[str, Any] | None = None) -> int:
-        """Publish a run on the OpenML server.
-
-        Parameters
-        ----------
-        path : str
-            The endpoint path to publish to (e.g., "run").
-        files : Mapping[str, Any], optional
-            The files to publish.
-
-        Returns
-        -------
-        int
-            The ID of the published run.
-        """
-        return super().publish(path=path, files=files)
-
 
 class RunV2API(ResourceV2API, RunAPI):
     """V2 API resource for runs. Currently read-only until V2 server supports POST."""
@@ -266,26 +248,3 @@ def list(  # type: ignore[valid-type]  # noqa: PLR0913
             V2 server API not yet available for this operation.
         """
         self._not_supported(method="list")
-
-    def publish(self, path: str, files: Mapping[str, Any] | None = None) -> int:  # noqa: ARG002
-        """Publish a run on the V2 server.
-
-        Parameters
-        ----------
-        path : str
-            The endpoint path to publish to.
-        files : Mapping[str, Any], optional
-            The files to publish.
-
-        Returns
-        -------
-        int
-            The ID of the published run.
-
-        Raises
-        ------
-        OpenMLNotSupportedError
-            V2 server does not yet support POST /runs/ endpoint.
-            Expected availability: Q2 2025
-        """
-        self._not_supported(method="publish")

tests/test_runs/test_run_functions.py

@@ -7,7 +7,6 @@
 import time
 import unittest
 import warnings
-from urllib.parse import urljoin
 
 from openml_sklearn import SklearnExtension, cat, cont
 from packaging.version import Version
@@ -41,7 +40,6 @@
     OpenMLNotAuthorizedError,
     OpenMLServerException,
 )
-#from openml.extensions.sklearn import cat, cont
 from openml.runs.functions import (
     _run_task_get_arffcontent,
     delete_run,
@@ -1806,9 +1804,9 @@ def test_initialize_model_from_run_nonstrict(self):
 
 
 @mock.patch.object(requests.Session, "request")
-def test_delete_run_not_owned(mock_delete, test_files_directory, test_server_v1, test_apikey_v1):
+def test_delete_run_not_owned(mock_request, test_files_directory, test_server_v1, test_apikey_v1):
     content_file = test_files_directory / "mock_responses" / "runs" / "run_delete_not_owned.xml"
-    mock_delete.return_value = create_request_response(
+    mock_request.return_value = create_request_response(
         status_code=412,
         content_filepath=content_file,
     )
@@ -1820,15 +1818,15 @@ def test_delete_run_not_owned(mock_delete, test_files_directory, test_server_v1,
         openml.runs.delete_run(40_000)
 
     run_url = test_server_v1 + "run/40000"
-    assert run_url == mock_delete.call_args.kwargs.get("url")
-    assert "DELETE" == mock_delete.call_args.kwargs.get("method")
-    assert test_apikey_v1 == mock_delete.call_args.kwargs.get("params", {}).get("api_key")
+    assert run_url == mock_request.call_args.kwargs.get("url")
+    assert "DELETE" == mock_request.call_args.kwargs.get("method")
+    assert test_apikey_v1 == mock_request.call_args.kwargs.get("params", {}).get("api_key")
 
 
 @mock.patch.object(requests.Session, "request")
-def test_delete_run_success(mock_delete, test_files_directory, test_server_v1, test_apikey_v1):
+def test_delete_run_success(mock_request, test_files_directory, test_server_v1, test_apikey_v1):
     content_file = test_files_directory / "mock_responses" / "runs" / "run_delete_successful.xml"
-    mock_delete.return_value = create_request_response(
+    mock_request.return_value = create_request_response(
         status_code=200,
         content_filepath=content_file,
     )
@@ -1837,15 +1835,15 @@ def test_delete_run_success(mock_delete, test_files_directory, test_server_v1, t
     assert success
 
     run_url = test_server_v1 + "run/10591880"
-    assert run_url == mock_delete.call_args.kwargs.get("url")
-    assert "DELETE" == mock_delete.call_args.kwargs.get("method")
-    assert test_apikey_v1 == mock_delete.call_args.kwargs.get("params", {}).get("api_key")
+    assert run_url == mock_request.call_args.kwargs.get("url")
+    assert "DELETE" == mock_request.call_args.kwargs.get("method")
+    assert test_apikey_v1 == mock_request.call_args.kwargs.get("params", {}).get("api_key")
 
 
 @mock.patch.object(requests.Session, "request")
-def test_delete_unknown_run(mock_delete, test_files_directory, test_server_v1, test_apikey_v1):
+def test_delete_unknown_run(mock_request, test_files_directory, test_server_v1, test_apikey_v1):
     content_file = test_files_directory / "mock_responses" / "runs" / "run_delete_not_exist.xml"
-    mock_delete.return_value = create_request_response(
+    mock_request.return_value = create_request_response(
         status_code=412,
         content_filepath=content_file,
     )
@@ -1857,9 +1855,9 @@ def test_delete_unknown_run(mock_delete, test_files_directory, test_server_v1, t
         openml.runs.delete_run(9_999_999)
 
     run_url = test_server_v1 + "run/9999999"
-    assert run_url == mock_delete.call_args.kwargs.get("url")
-    assert "DELETE" == mock_delete.call_args.kwargs.get("method")
-    assert test_apikey_v1 == mock_delete.call_args.kwargs.get("params", {}).get("api_key")
+    assert run_url == mock_request.call_args.kwargs.get("url")
+    assert "DELETE" == mock_request.call_args.kwargs.get("method")
+    assert test_apikey_v1 == mock_request.call_args.kwargs.get("params", {}).get("api_key")
 
 
 @pytest.mark.sklearn()