zhangyiss/spack
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

tests: make spack diff test more lenient

Browse Source 
The constraints in the `spack diff` test were very specific and assumed
a lot about the structure of what was being diffed. Relax them a bit to
make them more resilient to changes.
This commit is contained in:
Todd Gamblin 2021-08-09 00:28:53 -07:00
parent 9eb94be6dd
commit cfb60ab9e1
1 changed files with 12 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

16
lib/spack/spack/test/cmd/diff.py
View File

@ -29,10 +29,18 @@ def test_diff_cmd(install_mockery, mock_fetch, mock_archive, mock_packages):
# Calculate the comparison (c)
c = spack.cmd.diff.compare_specs(specA, specB, to_string=True)
assert len(c['a_not_b']) == 1
assert len(c['b_not_a']) == 1
assert c['a_not_b'][0] == ['variant_value', 'mpileaks debug False']
assert c['b_not_a'][0] == ['variant_value', 'mpileaks debug True']
# these particular diffs should have the same length b/c thre aren't
# any node differences -- just value differences.
assert len(c['a_not_b']) == len(c['b_not_a'])
# ensure that variant diffs are in here the result
assert ['variant_value', 'mpileaks debug False'] in c['a_not_b']
assert ['variant_value', 'mpileaks debug True'] in c['b_not_a']
# ensure that hash diffs are in here the result
assert ['hash', 'mpileaks %s' % specA.dag_hash()] in c['a_not_b']
assert ['hash', 'mpileaks %s' % specB.dag_hash()] in c['b_not_a']
def test_load_first(install_mockery, mock_fetch, mock_archive, mock_packages):