From b4621e524cdade282d89dbf94d78562d90729425 Mon Sep 17 00:00:00 2001 From: Massimiliano Culpo Date: Mon, 8 Jan 2024 18:57:18 +0100 Subject: Minor tweak to a test docstring (#41985) * Minor tweak to a test docstring Shorten the description of a fixture, extend the description of the test where it's used. * Fix typo --- lib/spack/spack/test/concretize.py | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/lib/spack/spack/test/concretize.py b/lib/spack/spack/test/concretize.py index 5c9f559644..c53dc0dd36 100644 --- a/lib/spack/spack/test/concretize.py +++ b/lib/spack/spack/test/concretize.py @@ -134,12 +134,7 @@ def current_host(request, monkeypatch): @pytest.fixture(scope="function", params=[True, False]) def fuzz_dep_order(request, monkeypatch): - """Metafunction that tweaks the order of iteration over dependencies in the concretizer. - - The original concretizer can be sensitive to this, so we use this to ensure that it - is tested forwards and backwards. - - """ + """Meta-function that tweaks the order of iteration over dependencies in a package.""" def reverser(pkg_name): if request.param: @@ -914,6 +909,12 @@ class TestConcretize: ], ) def test_conditional_dependencies(self, spec_str, expected, unexpected, fuzz_dep_order): + """Tests that conditional dependencies are correctly attached. + + The original concretizer can be sensitive to the iteration order over the dependencies of + a package, so we use a fuzzer function to test concretization with dependencies iterated + forwards and backwards. + """ fuzz_dep_order("py-extension3") # test forwards and backwards s = Spec(spec_str).concretized() -- cgit v1.2.3-70-g09d2