From 04f6881b76791d8b5f825052a26f2e8e6459a724 Mon Sep 17 00:00:00 2001 From: Massimiliano Culpo Date: Thu, 17 Apr 2025 07:55:32 +0200 Subject: [PATCH] Environment: remove leftover code to attach test deps (#50097) This should not be needed anymore after #49405 Signed-off-by: Massimiliano Culpo --- lib/spack/spack/environment/environment.py | 17 ----------------- 1 file changed, 17 deletions(-) diff --git a/lib/spack/spack/environment/environment.py b/lib/spack/spack/environment/environment.py index 93dca484d5a..96d023026f9 100644 --- a/lib/spack/spack/environment/environment.py +++ b/lib/spack/spack/environment/environment.py @@ -1605,23 +1605,6 @@ def _concretize_separately(self, tests=False): # Unify the specs objects, so we get correct references to all parents self._read_lockfile_dict(self._to_lockfile_dict()) - - # Re-attach information on test dependencies - if tests: - # This is slow, but the information on test dependency is lost - # after unification or when reading from a lockfile. - for h in self.specs_by_hash: - current_spec, computed_spec = self.specs_by_hash[h], by_hash[h] - for node in computed_spec.traverse(): - test_edges = node.edges_to_dependencies(depflag=dt.TEST) - for current_edge in test_edges: - test_dependency = current_edge.spec - if test_dependency in current_spec[node.name]: - continue - current_spec[node.name].add_dependency_edge( - test_dependency.copy(), depflag=dt.TEST, virtuals=current_edge.virtuals - ) - return concretized_specs @property