diff options
author | Peter Scheibel <scheibel1@llnl.gov> | 2020-05-01 10:47:46 -0700 |
---|---|---|
committer | GitHub <noreply@github.com> | 2020-05-01 10:47:46 -0700 |
commit | 31d12d380fdbd5c71a02645515d062cfbcc12017 (patch) | |
tree | 4b0a9e112b03952e290d5034375640b905e6dfa1 /COPYRIGHT | |
parent | d0dfa1ea4d844d87efa68a69053b813c4377c557 (diff) | |
download | spack-31d12d380fdbd5c71a02645515d062cfbcc12017.tar.gz spack-31d12d380fdbd5c71a02645515d062cfbcc12017.tar.bz2 spack-31d12d380fdbd5c71a02645515d062cfbcc12017.tar.xz spack-31d12d380fdbd5c71a02645515d062cfbcc12017.zip |
Testing: fix unintended interactions between tests (#16003)
* For tests that use the real Spack package repository, the config
needs to avoid using MPI providers that are not intended to be
installed by Spack. Without this, it is possible that Spack tests
which concretize the MPI virtual will end up trying to use an
implementation that it shouldn't (e.g. one that is always
provided externally). See #15666 for an example.
* The mutable_config test fixture was not initializing the scope
roots to the right directories (so the resulting config was empty).
* The current_host fixture in the concretize.py tests was using the
config fixture rather than mutable_config, and was polluting the
config cache for other tests.
* One test in concretize.py was clearing a nonexistent cache
(PackagePrefs._packages_config_cache). This reference has been
removed.
* The test 'test_preferred_compilers' was was depending on cross
test config pollution to succeed. The initial spec before
concretization has been updated to updated to be explicit about
the desired result.
Diffstat (limited to 'COPYRIGHT')
0 files changed, 0 insertions, 0 deletions