Skip to content

Unify tests for compiler command in the same file#16891

Merged
becker33 merged 2 commits intospack:developfrom
alalazo:qa/unify_tests_for_compiler_command
Jun 2, 2020
Merged

Unify tests for compiler command in the same file#16891
becker33 merged 2 commits intospack:developfrom
alalazo:qa/unify_tests_for_compiler_command

Conversation

@alalazo
Copy link
Copy Markdown
Member

@alalazo alalazo commented Jun 1, 2020

Modifications:

  • Unify tests for spack compiler into a single file (previously scattered among different files)
  • Fixed a bug in the tests (they depend on mutable_config not on config since they modify the configuration)

Tests for the "spack compiler" command were previously
scattered among different files.
@alalazo alalazo added tests General test capability(ies) maintainers labels Jun 1, 2020
@alalazo alalazo requested review from adamjstewart and becker33 June 1, 2020 14:59
@alalazo alalazo added the bugfix Something wasn't working, here's a fix label Jun 1, 2020
@becker33 becker33 merged commit d5ffec1 into spack:develop Jun 2, 2020
@alalazo alalazo deleted the qa/unify_tests_for_compiler_command branch June 3, 2020 06:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bugfix Something wasn't working, here's a fix maintainers tests General test capability(ies)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants