From 40e705d39e8d4ef680f7d4c3295acae216828bc0 Mon Sep 17 00:00:00 2001 From: Harmen Stoppels Date: Thu, 30 Nov 2023 18:19:10 +0100 Subject: tests: fix side effects of default_config fixture (#41361) * tests: default_config drop scope * use default_config elsewhere * use parse_install_tree for missing defaults in default config --- lib/spack/spack/test/config.py | 6 +++--- lib/spack/spack/test/conftest.py | 2 +- 2 files changed, 4 insertions(+), 4 deletions(-) (limited to 'lib') diff --git a/lib/spack/spack/test/config.py b/lib/spack/spack/test/config.py index 5f544a3129..2453172bec 100644 --- a/lib/spack/spack/test/config.py +++ b/lib/spack/spack/test/config.py @@ -1239,11 +1239,11 @@ def test_user_config_path_is_default_when_env_var_is_empty(working_env): assert os.path.expanduser("~%s.spack" % os.sep) == spack.paths._get_user_config_path() -def test_default_install_tree(monkeypatch): +def test_default_install_tree(monkeypatch, default_config): s = spack.spec.Spec("nonexistent@x.y.z %none@a.b.c arch=foo-bar-baz") monkeypatch.setattr(s, "dag_hash", lambda: "abc123") - projection = spack.config.get("config:install_tree:projections:all", scope="defaults") - assert s.format(projection) == "foo-bar-baz/none-a.b.c/nonexistent-x.y.z-abc123" + _, _, projections = spack.store.parse_install_tree(spack.config.get("config")) + assert s.format(projections["all"]) == "foo-bar-baz/none-a.b.c/nonexistent-x.y.z-abc123" def test_local_config_can_be_disabled(working_env): diff --git a/lib/spack/spack/test/conftest.py b/lib/spack/spack/test/conftest.py index 326ee143e7..c832c4d1d5 100644 --- a/lib/spack/spack/test/conftest.py +++ b/lib/spack/spack/test/conftest.py @@ -630,7 +630,7 @@ def platform_config(): spack.config.add_default_platform_scope(spack.platforms.real_host().name) -@pytest.fixture(scope="session") +@pytest.fixture def default_config(): """Isolates the default configuration from the user configs. -- cgit v1.2.3-60-g2f50