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

[Shafwansafi06]

Metadata:

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

Details:

• Resolved Environment Issue: Fixed a ValueError related to hydra-core and Python 3.13 incompatibility by upgrading hydra-core to version 1.3.2.
• Mocked test_get_dataset: Replaced live server call with a mock.
• Mocked test_download_task: Correctly mocked get_task in the base class to ensure the mock is picked up during super() calls.
• Mocked test_upload_task: Mocked create_task and the publish method, removing network dependencies.

Test Run Results:

tests/test_tasks/test_clustering_task.py::OpenMLClusteringTaskTest::test_download_task PASSED
tests/test_tasks/test_clustering_task.py::OpenMLClusteringTaskTest::test_get_dataset PASSED
tests/test_tasks/test_clustering_task.py::OpenMLClusteringTaskTest::test_upload_task PASSED

3 Passed in 0.09s

[Copilot]

Pull request overview

This PR updates the clustering task unit tests to avoid live OpenML server interactions by replacing network-dependent calls with mocks, aligning with the goal of keeping these as true unit tests (Fixes #1649).

Changes:

• Mock openml.tasks.get_task usage in test_get_dataset to avoid production server dependency.
• Mock get_task at the base-test import site to ensure super() calls pick up the mock in test_download_task.
• Mock openml.tasks.create_task (and related behavior) in test_upload_task to remove network/upload dependencies.

Comments suppressed due to low confidence (1)

tests/test_tasks/test_clustering_task.py:12

• These imports appear unused after switching the tests to mocks (e.g., pytest, OpenMLServerException, TestBase). Please remove unused imports to keep the test module minimal and avoid confusion about required fixtures/markers.

import pytest

import openml
from openml.exceptions import OpenMLServerException
from openml.tasks import TaskType
from openml.testing import TestBase

---

💡 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 3 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.

[Copilot]

There are two identical @mock.patch("tests.test_tasks.test_task.get_task") decorators applied to test_download_task, but the test function only accepts a single injected mock. This will raise a TypeError at runtime (extra argument). Remove the duplicate decorator (and the extra blank line) so only one patch is applied.

[Shafwansafi06]

@amueller can you review this PR and check for the issues my Tests has passed in local machine (Ubuntu 24 LTS)

[Shafwansafi06]

@copilot open a new pull request to apply changes based on the comments in this thread