Switch to using module-level fixture for tmp_path workdir

idaes/conftest.py

@@ -270,6 +270,22 @@ def run_class_in_tmp_path(
         os.chdir(prev_workdir)
 
 
+@pytest.fixture(scope="module")
+def run_module_in_tmp_path(
+    request: pytest.FixtureRequest, tmp_path_factory: pytest.TempPathFactory
+):
+    prev_workdir = os.getcwd()
+
+    modname = request.module.__name__
+
+    tmp_path = tmp_path_factory.mktemp(modname, numbered=True)
+    try:
+        os.chdir(tmp_path)
+        yield
+    finally:
+        os.chdir(prev_workdir)
+
+
 def _get_repo_root_dir() -> Union[Path, None]:
     try:
         text = subprocess.check_output(

idaes/core/surrogate/tests/test_pysmo_surrogate.py

@@ -46,7 +46,7 @@
 from idaes.core.surrogate.metrics import compute_fit_metrics
 
 
-dirpath = Path(__file__).parent.resolve()
+pytestmark = pytest.mark.usefixtures("run_module_in_tmp_path")
 
 
 # String representation of json output for testing
@@ -228,7 +228,6 @@
 )
 
 
-@pytest.mark.usefixtures("run_class_in_tmp_path")
 class TestSurrogateTrainingResult:
     @pytest.fixture
     def pysmo_output_pr(self):

idaes/models_extra/power_generation/flowsheets/test/test_subcritical_boiler.py

@@ -35,12 +35,14 @@
 solver = get_solver()
 
 
+pytestmark = pytest.mark.usefixtures("run_module_in_tmp_path")
+
+
 @pytest.fixture(scope="module")
 def model():
     return main()
 
 
-@pytest.mark.usefixtures("run_in_tmp_path")
 @pytest.mark.skipif(not helmholtz_available(), reason="General Helmholtz not available")
 @pytest.mark.component
 def test_basic_build(model):