-
Notifications
You must be signed in to change notification settings - Fork 11
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
Integration tests #99
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ryanmrichard
pushed a commit
that referenced
this pull request
Nov 6, 2023
* Integration tests (#99) * Upload testing scripts and layout * Update checkout version [skip ci] * Temporarily disable unit tests * Enable integration tests on pull request * Fix cmake_version variable issue * Change cmake version to try * Add extra echo to check making log directory * Debug info * Debug info * Add execution permissions to scripts * Undo some debug info * Fix incorrect token argument * Add more repository tests * Try combined tests * Try combined tests * Fix CMaizePrivateDepend2 combination tests * Debug info * Go to the scripts testing directory before running ctests * Try pwd * Remove double quotes stopping scripts from being 'source'ed * Remove debug info * Remove double quotes for CMaizePublicDepend2 tests and enable them * Enable all integration tests * Exclude integration tests during unit testing * Enable integration tests on PR to 'dev' to test them * Enable unit tests and switch integration tests to only trigger on push to 'dev' * Trigger integration tests only on successful PR merge to 'dev' * Remove planned 'dev' branch for staged testing and run integration tests on PRs to master along with unit tests * Committing license headers * Retrigger tests * Add pre-install of Catch2 for integration tests * install catch2 to local directory * Pin Catch2 v3.0.1 * Debug statements for Catch2 install * Add catch2 install to the integration test toolchain (hopefully) * Change catch2 installation location * Add an ls * Move catch2 install CMAKE_PREFIX_PATH to environment variable --------- Co-authored-by: cmakepp[bot] <cmakepp[bot]@github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Description
This PR adds integration tests using the example repositories at https://github.com/CMaizeExamples. These integration tests are intended to be a secondary check in the commit workflow.
The imagined workflow is for a user to submit a PR to the
dev
branch. Unit tests are run on the PR, and, if the unit tests pass and reviewers approve of the changes, the PR is merged into thedev
branch. From here, the integration tests are run on the newly changeddev
branch. If the integration tests pass, thendev
will be merged intomaster
, causing the changes to go live.