-
-
Notifications
You must be signed in to change notification settings - Fork 168
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
Unit test tidyup #936
Comments
#948 introduced crases and caused unit tests to fail:
As such this breaks CI so we had to roll it back. Anyone welcome to try again but we must make sure that it doesnt break the tests. The failing tests are skipped when ran as part of a PR (you dont have access to the token) but when it is merged into dev the online tests are ran. This was not tested with a token with the online tests before submitting the PR. |
This is still open if anyone wants to take this! |
Gonna take this on myself. I will be basing it off #948 as we agreed that the format was good. |
I'd put this off for a bit as I faced issues with Mac and whatnot, will look to maybe pick this up over Christmas if no-one else does (as I am busy until then). |
This issue has had no activity and is being marked as stale. If you still wish to continue with this issue please comment to reopen it. |
Reminder, we're looking for people to complete this project! I'll post this in the Discord too! |
Hello @Jaskowicz1, I would like to work on this issue, is it still open? |
sure feel free to grab it, please do not add any external dependencies such as catch2! |
Going to actually take this on myself this time. |
This issue has had no activity and is being marked as stale. If you still wish to continue with this issue please comment to reopen it. |
Bump |
This issue has had no activity and is being marked as stale. If you still wish to continue with this issue please comment to reopen it. |
test.cpp and friends are large and unwieldy. it is now over 2000 lines, this is 🤢 - as a small project this could do with tidying up and breaking down into smaller files. anything under the unittest folder will be linked into one executable.
This is an ideal first issue for an intermediate developer wanting to contribute to D++.
The text was updated successfully, but these errors were encountered: