Skip to content

tests: fix side effects of default_config fixture#41361

Merged
alalazo merged 3 commits intospack:developfrom
haampie:tests/default-config-fixes
Nov 30, 2023
Merged

tests: fix side effects of default_config fixture#41361
alalazo merged 3 commits intospack:developfrom
haampie:tests/default-config-fixes

Conversation

@haampie
Copy link
Copy Markdown
Member

@haampie haampie commented Nov 30, 2023

scope="session" makes no sense cause it influences all further tests

@spackbot-app spackbot-app bot added core PR affects Spack core functionality tests General test capability(ies) labels Nov 30, 2023
@alalazo alalazo merged commit 40e705d into spack:develop Nov 30, 2023
@haampie haampie deleted the tests/default-config-fixes branch November 30, 2023 17:19
@alalazo alalazo mentioned this pull request Jan 2, 2024
36 tasks
alalazo pushed a commit that referenced this pull request Jan 2, 2024
* tests: default_config drop scope

* use default_config elsewhere

* use parse_install_tree for missing defaults in default config
alalazo pushed a commit that referenced this pull request Jan 10, 2024
* tests: default_config drop scope

* use default_config elsewhere

* use parse_install_tree for missing defaults in default config
haampie added a commit that referenced this pull request Jan 11, 2024
* tests: default_config drop scope

* use default_config elsewhere

* use parse_install_tree for missing defaults in default config
RikkiButler20 pushed a commit to RikkiButler20/spack that referenced this pull request Jan 31, 2024
* tests: default_config drop scope

* use default_config elsewhere

* use parse_install_tree for missing defaults in default config
vjranagit pushed a commit to vjranagit/spack that referenced this pull request Jan 18, 2026
* tests: default_config drop scope

* use default_config elsewhere

* use parse_install_tree for missing defaults in default config
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

core PR affects Spack core functionality tests General test capability(ies)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants