Skip to content
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

testing: Convert more tests to gtest_add_tests_helper #2232

Merged
merged 2 commits into from
May 15, 2020

Conversation

bradking
Copy link
Collaborator

Take advantage of the infrastructure from #2200 to simplify test definitions.

@bradking bradking requested a review from chuckatkins May 12, 2020 18:53
@chuckatkins
Copy link
Contributor

See #2234 for the unrelated SST test failures and re-base once it's been addressed

@bradking
Copy link
Collaborator Author

This is blocked in CI by the problem discussed here.

@chuckatkins
Copy link
Contributor

Debian builds disabled. Please rebase.

bradking added 2 commits May 15, 2020 07:42
Convert sst engine tests to `gtest_add_tests_helper` to take advantage
of the infrastructure from commit 0422878 (testing: Teach test helper
functions to add both Serial and MPI tests, 2020-04-30).
Convert the tests to `gtest_add_tests_helper`.
@chuckatkins chuckatkins merged commit f9aa430 into ornladios:master May 15, 2020
@bradking bradking deleted the test-split-mpi branch May 15, 2020 13:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants