Replace test_minimum_service_factor_invalid_input with mocking

Author: alexdewar

tests/test_minimum_service.py

@@ -1,7 +1,8 @@
-from itertools import chain, permutations
+from itertools import permutations
+from unittest.mock import patch
 
 import numpy as np
-from pytest import mark, raises
+from pytest import mark
 
 
 def modify_minimum_service_factors(
@@ -25,7 +26,8 @@ def modify_minimum_service_factors(
     "minimum_service_factors",
     permutations((np.linspace(0, 1, 6), [0] * 6)),
 )
-def test_minimum_service_factor(tmpdir, minimum_service_factors):
+@patch("muse.readers.csv.check_utilization_and_minimum_service_factors")
+def test_minimum_service_factor(check_mock, tmpdir, minimum_service_factors):
     import pandas as pd
     from muse import examples
     from muse.mca import MCA
@@ -51,6 +53,7 @@ def test_minimum_service_factor(tmpdir, minimum_service_factors):
 
     with tmpdir.as_cwd():
         MCA.factory(model_path / "settings.toml").run()
+    check_mock.assert_called_once()
 
     supply_timeslice = pd.read_csv(tmpdir / "Results/MCAMetric_Supply.csv")
 
@@ -64,35 +67,3 @@ def test_minimum_service_factor(tmpdir, minimum_service_factors):
                 ].supply
                 >= factor
             ).all()
-
-
-@mark.parametrize(
-    "minimum_service_factors",
-    chain.from_iterable(map(permutations, ((-1, 0), (2, 0), (float("nan"), 0)))),
-)
-def test_minimum_service_factor_invalid_input(tmpdir, minimum_service_factors):
-    from muse import examples
-    from muse.mca import MCA
-
-    sector = "power"
-    processes = ("gasCCGT", "windturbine")
-
-    # Copy the model inputs to tmpdir
-    model_path = examples.copy_model(
-        name="default_timeslice", path=tmpdir, overwrite=True
-    )
-
-    technodata_timeslices = modify_minimum_service_factors(
-        model_path=model_path,
-        sector=sector,
-        processes=processes,
-        minimum_service_factors=minimum_service_factors,
-    )
-
-    technodata_timeslices.to_csv(
-        model_path / "technodata" / sector / "TechnodataTimeslices.csv", index=False
-    )
-
-    with raises(ValueError):
-        with tmpdir.as_cwd():
-            MCA.factory(model_path / "settings.toml").run()