-
Notifications
You must be signed in to change notification settings - Fork 504
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
Add CHECK_EQUAL_ZERO macros #1423
Conversation
No test? |
Since they are just aliases for But I guess I could add some tests to tests/CppUTest/TestUTestMacro.cpp similar to the |
Yes, please do put a test there, and perhaps also tests that it behaves properly as a macro. I'd suggest CHECK_EQUAL_ZERO instead to CHECK_ZERO. |
These are useful when working with error code return values, where one often compares to zero which means no error has happened.
c5980e7
to
7fbbec2
Compare
I did the rename and added some tests. Btw. It took me quite some time to figure out how to run the tests (apparently |
Sorry, how to run the tests also does depend on the build system that you are using. We support multiple. The test failed though. |
7fbbec2
to
57b1558
Compare
A copy paste error 🙈 |
These are useful when working with error code return values, where one
often compares to zero which means no error has happened.