Dont use start/stop, adjust for new defaults

Author: PGijsbers

.github/workflows/test.yml

@@ -102,7 +102,7 @@ jobs:
         if [ ${{ matrix.code-cov }} ]; then codecov='--cov=openml --long  --cov-report=xml'; fi
         # Most of the time, running only the scikit-learn tests is sufficient
         if [ ${{ matrix.sklearn-only }} = 'true' ]; then marks='sklearn and not production'; else marks='not production'; fi
-        echo pytest -n 4 --durations=20 --dist load -sv $codecov -o log_cli=true $marks
+        echo pytest -n 4 --durations=20 --dist load -sv $codecov -o log_cli=true -m "$marks"
         pytest -n 4 --durations=20 --dist load -sv $codecov -o log_cli=true -m "$marks"
     - name: Run tests on Ubuntu Production
       if: matrix.os == 'ubuntu-latest'

tests/conftest.py

@@ -274,7 +274,8 @@ def with_server(request):
         openml.config.server = "https://www.openml.org/api/v1/xml"
         yield
         return
-    openml.config.start_using_configuration_for_example()
+    openml.config.server = "https://test.openml.org/api/v1/xml"
+    openml.config.apikey = "c0c42819af31e706efe1f4b88c23c6c1"
     yield
 
 

tests/test_datasets/test_dataset_functions.py

@@ -1951,7 +1951,8 @@ def test_get_dataset_parquet(requests_mock, test_files_directory):
     content_file = (
             test_files_directory / "mock_responses" / "datasets" / "data_description_61.xml"
     )
-    requests_mock.get("https://www.openml.org/api/v1/xml/data/61", text=content_file.read_text())
+    # While the mocked example is from production, unit tests by default connect to the test server.
+    requests_mock.get("https://test.openml.org/api/v1/xml/data/61", text=content_file.read_text())
     dataset = openml.datasets.get_dataset(61, download_data=True)
     assert dataset._parquet_url is not None
     assert dataset.parquet_file is not None