remove skip markers

Author: PGijsbers

tests/test_flows/test_flow.py

@@ -179,11 +179,6 @@ def test_to_xml_from_xml(self):
         # Would raise exception if they are not legal
         openml.flows.functions.assert_flows_equal(new_flow, flow)
         assert new_flow is not flow
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_publish_flow(self):
@@ -225,11 +220,6 @@ def test_publish_existing_flow(self, flow_exists_mock):
         TestBase.logger.info(
             f"collected from {__file__.split('/')[-1]}: {flow.flow_id}",
         )
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_publish_flow_with_similar_components(self):
@@ -280,11 +270,6 @@ def test_publish_flow_with_similar_components(self):
         flow3.publish()
         TestBase._mark_entity_for_removal("flow", flow3.flow_id, flow3.name)
         TestBase.logger.info(f"collected from {__file__.split('/')[-1]}: {flow3.flow_id}")
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_semi_legal_flow(self):
@@ -394,11 +379,6 @@ def get_sentinel():
 
         flow_id = openml.flows.flow_exists(name, version)
         assert not flow_id
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_existing_flow_exists(self):
@@ -439,11 +419,6 @@ def test_existing_flow_exists(self):
                 flow.external_version,
             )
             assert downloaded_flow_id == flow.flow_id
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_sklearn_to_upload_to_flow(self):

tests/test_flows/test_flow_functions.py

@@ -309,11 +309,6 @@ def test_get_flow1(self):
         self.use_production_server()
         flow = openml.flows.get_flow(1)
         assert flow.external_version is None
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_get_flow_reinstantiate_model(self):
@@ -396,11 +391,6 @@ def test_get_flow_reinstantiate_flow_not_strict_pre_023(self):
         flow = openml.flows.get_flow(flow_id=8175, reinstantiate=True, strict_version=False)
         assert flow.flow_id is None
         assert "sklearn==0.19.1" not in flow.dependencies
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_get_flow_id(self):

tests/test_openml/test_api_calls.py

@@ -20,11 +20,6 @@ class TestConfig(openml.testing.TestBase):
     def test_too_long_uri(self):
         with pytest.raises(openml.exceptions.OpenMLServerError, match="URI too long!"):
             openml.datasets.list_datasets(data_id=list(range(10000)))
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @unittest.mock.patch("time.sleep")
     @unittest.mock.patch("requests.Session")
     @pytest.mark.test_server()

tests/test_runs/test_run.py

@@ -117,11 +117,6 @@ def _check_array(array, type_):
             np.testing.assert_array_equal(string_part, string_part_prime)
         else:
             assert run_prime_trace_content is None
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_to_from_filesystem_vanilla(self):
@@ -156,11 +151,6 @@ def test_to_from_filesystem_vanilla(self):
         TestBase.logger.info(
             f"collected from {__file__.split('/')[-1]}: {run_prime.run_id}",
         )
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.flaky()
     @pytest.mark.test_server()
@@ -196,11 +186,6 @@ def test_to_from_filesystem_search(self):
         TestBase.logger.info(
             f"collected from {__file__.split('/')[-1]}: {run_prime.run_id}",
         )
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_to_from_filesystem_no_model(self):
@@ -306,11 +291,6 @@ def assert_run_prediction_data(task, run, model):
             # Assert correctness
             assert_method(y_pred, saved_y_pred)
             assert_method(y_test, saved_y_test)
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_publish_with_local_loaded_flow(self):
@@ -354,11 +334,6 @@ def test_publish_with_local_loaded_flow(self):
             # make sure the flow is published as part of publishing the run.
             assert openml.flows.flow_exists(flow.name, flow.external_version)
             openml.runs.get_run(loaded_run.run_id)
-
-    @pytest.mark.skipif(
-        os.getenv("OPENML_USE_LOCAL_SERVICES") == "true",
-        reason="Pending resolution of #1657",
-    )
     @pytest.mark.sklearn()
     @pytest.mark.test_server()
     def test_offline_and_online_run_identical(self):