test: add CTest to cmake to enable testing#2753
Open
realstealthninja wants to merge 34 commits intoTheAlgorithms:masterfrom
Open
test: add CTest to cmake to enable testing#2753realstealthninja wants to merge 34 commits intoTheAlgorithms:masterfrom
realstealthninja wants to merge 34 commits intoTheAlgorithms:masterfrom
Conversation
Contributor
|
This pull request has been automatically marked as abandoned because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
4 tasks
vil02
added a commit
to vil02/C-Plus-Plus
that referenced
this pull request
Sep 26, 2025
8 tasks
realstealthninja
added a commit
that referenced
this pull request
Sep 27, 2025
Helps with #2753 Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
Collaborator
Author
|
@vil02 Do you think we should wait until all test cases pass, i.e. fix all the files? |
Member
In short: yes. But the optimistic thing is that maybe one can work directory by directory, i.e. select one, let's say Doing all at once is way too much. |
vil02
added a commit
to vil02/C-Plus-Plus
that referenced
this pull request
Sep 29, 2025
8 tasks
realstealthninja
added a commit
to realstealthninja/C-Plus-Plus
that referenced
this pull request
Oct 1, 2025
Helps with TheAlgorithms#2753 Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
realstealthninja
added a commit
that referenced
this pull request
Oct 11, 2025
* style: remove interaction with the user Helps with #2753 * clang-format and clang-tidy fixes for ff441e0 * tests: add test with result being a zero vector Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com> --------- Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: realstealthninja <68815218+realstealthninja@users.noreply.github.com>
This file contains hidden or 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
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 of Change
Testing
This pr enables testing of the algorithms in the repository
before we get to the stage of running tests on a CI. We will need to remove all
other code from every main function so that the only remaining code is a call to the test function.
for now this draft pull request shows how easy it is to enable testing.
adding it to the awesome CI will be as easy as adding
make test to a test step.
Checklist
Notes: