summaryrefslogtreecommitdiff
path: root/var
diff options
context:
space:
mode:
authorTamara Dahlgren <35777542+tldahlgren@users.noreply.github.com>2021-08-20 05:27:12 -0700
committerGitHub <noreply@github.com>2021-08-20 14:27:12 +0200
commit201f5bdfe8813e0b3995d72e07f00d8d7257586c (patch)
tree2001162447ee1c481987160a5a9e1da5fb7b72aa /var
parentef32ff0e4ca5b0e9d8ed6d734f7e221f90a480ac (diff)
downloadspack-201f5bdfe8813e0b3995d72e07f00d8d7257586c.tar.gz
spack-201f5bdfe8813e0b3995d72e07f00d8d7257586c.tar.bz2
spack-201f5bdfe8813e0b3995d72e07f00d8d7257586c.tar.xz
spack-201f5bdfe8813e0b3995d72e07f00d8d7257586c.zip
WarpX: Actually skip tests when cannot be run (#25494)
Diffstat (limited to 'var')
-rw-r--r--var/spack/repos/builtin/packages/warpx/package.py4
1 files changed, 2 insertions, 2 deletions
diff --git a/var/spack/repos/builtin/packages/warpx/package.py b/var/spack/repos/builtin/packages/warpx/package.py
index 03d0e3884d..b5cb9ef7ef 100644
--- a/var/spack/repos/builtin/packages/warpx/package.py
+++ b/var/spack/repos/builtin/packages/warpx/package.py
@@ -158,7 +158,7 @@ class Warpx(CMakePackage):
"""Checks after the build phase"""
if '+app' not in self.spec:
print("WarpX check skipped: requires variant +app")
- pass
+ return
with working_dir("spack-check", create=True):
cli_args = self._get_input_options(False)
@@ -175,7 +175,7 @@ class Warpx(CMakePackage):
"""Perform smoke tests on the installed package."""
if '+app' not in self.spec:
print("WarpX smoke tests skipped: requires variant +app")
- pass
+ return
# our executable names are a variant-dependent and naming evolves
exe = find(self.prefix.bin, 'warpx.*', recursive=False)[0]