Fix openml#1033: skip two unit tests on Windows (openml#1040)

mfeurer · PGijsbers · commit 8328a171125e · 2023-02-23T15:32:03.000+01:00
* skip two unit tests on Windows

* make tests less strict for Windows
diff --git a/tests/test_openml/test_config.py b/tests/test_openml/test_config.py
@@ -12,6 +12,7 @@ class TestConfig(openml.testing.TestBase):
     @unittest.mock.patch("os.path.expanduser")
     @unittest.mock.patch("openml.config.openml_logger.warning")
     @unittest.mock.patch("openml.config._create_log_handlers")
+    @unittest.skipIf(os.name == "nt", "https://github.com/openml/openml-python/issues/1033")
     def test_non_writable_home(self, log_handler_mock, warnings_mock, expanduser_mock):
         with tempfile.TemporaryDirectory(dir=self.workdir) as td:
             expanduser_mock.side_effect = (
diff --git a/tests/test_runs/test_run_functions.py b/tests/test_runs/test_run_functions.py
@@ -1623,7 +1623,9 @@ def test__run_task_get_arffcontent_2(self, parallel_mock):
             0.9655172413793104,
         ]
         scores = [v for k, v in res[2]["predictive_accuracy"][0].items()]
-        self.assertSequenceEqual(scores, expected_scores, seq_type=list)
+        np.testing.assert_array_almost_equal(
+            scores, expected_scores, decimal=2 if os.name == "nt" else 7
+        )
 
     @unittest.skipIf(
         LooseVersion(sklearn.__version__) < "0.21",
diff --git a/tests/test_utils/test_utils.py b/tests/test_utils/test_utils.py
@@ -87,6 +87,7 @@ def test_list_all_for_evaluations(self):
         self.assertEqual(len(evaluations), required_size)
 
     @unittest.mock.patch("openml.config.get_cache_directory")
+    @unittest.skipIf(os.name == "nt", "https://github.com/openml/openml-python/issues/1033")
     def test__create_cache_directory(self, config_mock):
         with tempfile.TemporaryDirectory(dir=self.workdir) as td:
             config_mock.return_value = td
