-
Notifications
You must be signed in to change notification settings - Fork 2.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
openPMD-api: CTest & Install Tests #25300
Conversation
Run CTest at build time with: ``` spack install --test=root openpmd-api@<version> ``` and run smoke-tests after install and loading of the package via ``` spack load -r /<spec> spack test run /<spec> ```
@tldahlgren can you please backport this to e4s branch we need this test |
@ax3l What was the problem you were having that cause you to need to load the module first? |
@tldahlgren |
for exe in exes: | ||
spec_vers_str = '{0}'.format(self.spec.version) | ||
reason = 'test version of {0} is {1}'.format(exe, spec_vers_str) | ||
self.run_test(exe, ['--version'], [spec_vers_str], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tldahlgren this line essentially does not work if not loaded
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There's something in the process that I'm missing that I need to look into. Thanks.
@shahzebsiddiqui can you please submit a PR backporting this to the E4S branch? |
Run CTest at build time with:
Ref.: https://spack.readthedocs.io/en/latest/packaging_guide.html#stand-alone-or-smoke-tests
Run smoke-tests after install and loading of the package via
x-ref: NERSC/buildtest-nersc#56