diff --git a/lib/spack/spack/test/mock_packages/multimethod.py b/lib/spack/spack/test/mock_packages/multimethod.py index fa8497eafc..e5c2f5243e 100644 --- a/lib/spack/spack/test/mock_packages/multimethod.py +++ b/lib/spack/spack/test/mock_packages/multimethod.py @@ -92,10 +92,9 @@ def different_by_dep(self): # # Make sure we can switch on virtual dependencies # + def different_by_virtual_dep(self): + return 1 + @when('^mpi@2:') def different_by_virtual_dep(self): - return 'mpi@2:' - - @when('^mpi@:1') - def different_by_virtual_dep(self): - return 'mpi@:1' + return 2 diff --git a/lib/spack/spack/test/multimethod.py b/lib/spack/spack/test/multimethod.py index 4627e06142..969c987da8 100644 --- a/lib/spack/spack/test/multimethod.py +++ b/lib/spack/spack/test/multimethod.py @@ -76,9 +76,9 @@ def test_ambiguous_dep(self): self.assertRaises(AmbiguousMethodError, pkg.different_by_dep) - def test_one_dep_match(self): - pass + def test_virtual_dep_match(self): + pkg = packages.get('multimethod^mpich2') + self.assertEqual(pkg.different_by_virtual_dep(), 2) - - def test_one_dep_match(self): - pass + pkg = packages.get('multimethod^mpich@1.0') + self.assertEqual(pkg.different_by_virtual_dep(), 1)