[MNT] Replace live server calls with mocks in test_task_methods.py

[Sandipmandal25]

Metadata

• Reference Issue: Fixes [MNT] Replace Live Server Calls with Mocks in Unit Tests #1649 (tests/test_tasks/test_task_methods.py)
• New Tests Added: No
• Documentation Updated: No
• Change Log Entry: "Replace live server calls with mocks in test_task_methods.py"

Details

Both tests in OpenMLTaskMethodsTest previously required a running test server. This PR removes those dependencies:

• test_tagging — patches requests.Session.post to return mock XML responses for task/tag and task/untag endpoints; asserts the HTTP call was made with the correct URL, data, and headers via mock_post.assert_called_once_with(...). No network calls required.

• test_get_train_and_test_split_indices — uses the static cache (task 1882 + dataset 2 already present); no network calls. Also modernises assertRaisesRegex → pytest.raises for consistency with the rest of the test suite.

Test Run

platform darwin -- Python 3.13.3, pytest-9.0.2
tests/test_tasks/test_task_methods.py::OpenMLTaskMethodsTest::test_get_train_and_test_split_indices PASSED
tests/test_tasks/test_task_methods.py::OpenMLTaskMethodsTest::test_tagging PASSED

2 passed in 0.25s

[Copilot]

Pull request overview

This PR removes live test server dependencies from two tests in test_task_methods.py by replacing them with appropriate mocks. The changes align with the broader initiative (issue #1649) to convert integration tests into proper unit tests that don't rely on external server availability.

Changes:

• Removed @pytest.mark.test_server() decorators from both tests
• Replaced live server calls with mocks in test_tagging using stateful simulation
• Migrated assertRaisesRegex to pytest.raises in test_get_train_and_test_split_indices

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.