From c784970f9c589b346dccfd6ed4835fa5694fe785 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?H=C3=A5vard=20Berland?= Date: Fri, 24 Nov 2023 15:15:11 +0100 Subject: [PATCH] Reactivate now-working tests --- tests/unit_tests/test_libres_facade.py | 5 ----- tests/unit_tests/test_load_forward_model.py | 2 -- tests/unit_tests/test_substitution_list.py | 1 - 3 files changed, 8 deletions(-) diff --git a/tests/unit_tests/test_libres_facade.py b/tests/unit_tests/test_libres_facade.py index 0d53e42b2f7..4f333de4b72 100644 --- a/tests/unit_tests/test_libres_facade.py +++ b/tests/unit_tests/test_libres_facade.py @@ -188,7 +188,6 @@ def test_case_history_data_missing_key(facade): assert isinstance(data, PandasObject) -@pytest.mark.skip(reason="Needs reimplementation") def test_summary_data_verify_indices_and_values( caplog, snake_oil_case_storage, snake_oil_default_storage, snapshot ): @@ -253,7 +252,6 @@ def test_gen_kw_priors(facade): } in priors["SNAKE_OIL_PARAM"] -@pytest.mark.skip(reason="Needs reimplementation") def test_summary_collector( monkeypatch, snake_oil_case_storage, snake_oil_default_storage, snapshot ): @@ -295,7 +293,6 @@ def test_summary_collector( ) -@pytest.mark.skip(reason="Needs reimplementation") def test_misfit_collector(snake_oil_case_storage, snake_oil_default_storage, snapshot): facade = LibresFacade(snake_oil_case_storage) data = facade.load_all_misfit_data(snake_oil_default_storage) @@ -306,7 +303,6 @@ def test_misfit_collector(snake_oil_case_storage, snake_oil_default_storage, sna _ = data.loc[60] -@pytest.mark.skip(reason="Needs reimplementation") def test_gen_kw_collector(snake_oil_case_storage, snake_oil_default_storage, snapshot): facade = LibresFacade(snake_oil_case_storage) data = facade.load_all_gen_kw_data(snake_oil_default_storage) @@ -394,7 +390,6 @@ def test_gen_data_report_steps(): assert obs_key == [] -@pytest.mark.skip(reason="Needs reimplementation") def test_gen_data_collector( snake_oil_case_storage, snapshot, snake_oil_default_storage ): diff --git a/tests/unit_tests/test_load_forward_model.py b/tests/unit_tests/test_load_forward_model.py index c4bb05fdfee..5fa5d779c5d 100644 --- a/tests/unit_tests/test_load_forward_model.py +++ b/tests/unit_tests/test_load_forward_model.py @@ -68,7 +68,6 @@ def run_simulator(time_step_count, start_date) -> Summary: return summary -@pytest.mark.skip(reason="Needs reimplementation") @pytest.mark.usefixtures("copy_snake_oil_case_storage") def test_load_inconsistent_time_map_summary(caplog): """ @@ -111,7 +110,6 @@ def test_load_inconsistent_time_map_summary(caplog): assert loaded == 1 -@pytest.mark.skip(reason="Needs reimplementation") @pytest.mark.usefixtures("copy_snake_oil_case_storage") def test_load_forward_model(snake_oil_default_storage): """ diff --git a/tests/unit_tests/test_substitution_list.py b/tests/unit_tests/test_substitution_list.py index 4c36419e275..ca43a4d6c39 100644 --- a/tests/unit_tests/test_substitution_list.py +++ b/tests/unit_tests/test_substitution_list.py @@ -3,7 +3,6 @@ from ert.substitution_list import SubstitutionList -@pytest.mark.xfail(reason="Needs reimplementation") def test_substitution_list(): subst_list = SubstitutionList()