Add googletest based tests #767
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
Use GTest to run tests. This should help us in quite a few places. Signed-off-by: Johannes Demel <[email protected]>
This is an ongoing effort to introduce some more sophisticated QA tests. We use googletest and write custom test suits. This approach requires more manual work but yields a more accurate result as well. Signed-off-by: Johannes Demel <[email protected]>
Potentially, we want to be more verbose. This is a first effort to improve in this area. Signed-off-by: Johannes Demel <[email protected]>
It's probably a good thing to have googletest as an install dependency. Signed-off-by: Johannes Demel <[email protected]>
This was
linked to
issues
Aug 19, 2024
We want to gradually migrate to a more stable test framework. This requires that our CI runs the new framework. Here we go! Signed-off-by: Johannes Demel <[email protected]>
We can run an individual test for each implementation now. Further, the test names are easier to recognize. Signed-off-by: Johannes Demel <[email protected]>
|
This PR was sitting here for long enough. Merging it. At this point, some changes are kinda urgent to fix the failing flyci test. |
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.
This PR adds googletest as a dependency and starts to implement new tests. This should offer more flexibility and thus, better support for more complex kernels and better coverage of corner cases. Finally, different vector sizes should be easily supported.
For now, this is a second set of tests that still needs to evolve but the foundation is in place.
Finally, the CI system receives an update with newer compiler versions and older versions being removed. Here, the target is to test on Ubuntu versions that still receive general support. At the moment the oldest distro would be Ubuntu 20.04.