From e6ae42b1eb44b35a4b864128cfcf91d6c4226b40 Mon Sep 17 00:00:00 2001 From: AcriusWinter <152348900+AcriusWinter@users.noreply.github.com> Date: Tue, 18 Jun 2024 08:40:17 -0700 Subject: vtk-m: Changed test method names and skipping non-applicable tests from old to new approach (#44705) * vtk-m: Changed test method names and skipping non-applicable tests from old to new approach ----- Co-authored-by: Tamara Dahlgren <35777542+tldahlgren@users.noreply.github.com> --- var/spack/repos/builtin/packages/vtk-m/package.py | 13 ++++--------- 1 file changed, 4 insertions(+), 9 deletions(-) diff --git a/var/spack/repos/builtin/packages/vtk-m/package.py b/var/spack/repos/builtin/packages/vtk-m/package.py index 2f5c868374..90eadd0edf 100644 --- a/var/spack/repos/builtin/packages/vtk-m/package.py +++ b/var/spack/repos/builtin/packages/vtk-m/package.py @@ -230,14 +230,12 @@ class VtkM(CMakePackage, CudaPackage, ROCmPackage): return options - # Delegate in the vtk-m built smoke test - def smoke_test(self): + def test_smoke_test(self): + """Build and run ctests""" spec = self.spec if "+examples" not in spec: - raise RuntimeError( - "Examples needed for smoke test missing", "reinstall with `+examples` variant" - ) + raise SkipTest("Package must be installed with +examples") testdir = "smoke_test_build" with working_dir(testdir, create=True): @@ -252,7 +250,4 @@ class VtkM(CMakePackage, CudaPackage, ROCmPackage): @run_after("install") @on_package_attributes(run_tests=True) def build_test(self): - self.smoke_test() - - def test(self): - self.smoke_test() + self.test_smoke_test() -- cgit v1.2.3-70-g09d2