-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Use of INFO #415
Comments
Hi @mintyc. Sorry it's been a long time getting this (I'm just seeing it for the first time during an old tickets review). Ironically what you proposed used to be the case: Anyway I'm going to put this back on the queue as a feature request. |
Sounds good 👍 |
|
IMO INFO currently is limited because it is scoped. It can only be used in the TEST_CASE body itself, not in mocks etc.
I would like to be able to log supplementary information that via the command line I can force to be output irrespective of whether an assertion fails.
Say I have some mock classes that I use in tests, I'd like them to be able to output INFO but they are scoped such that the INFO will never get out.
e.g. I have mocks of element types, Copyable, Movable etc and as well as asserting on numbers of constructions, move assignments etc in the test case itself, I sometimes want individual calls into the various methods to log a copy construction etc.
The existing behaviour of WARN can do this, but tags messages with 'warning' and is always on.
I'd propose existing INFO is renamed SCOPED to convey its restricted (but useful) purpose and INFO is repurposed to become more like WARN with the exception that it is only logged when explicitly requested on the command line.
The text was updated successfully, but these errors were encountered: