Skip to content
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

Include negated assertion_result messages: #395

Open
wants to merge 2 commits into
base: develop
Choose a base branch
from

Conversation

krwalker
Copy link

@krwalker krwalker commented Sep 6, 2023

Given a helper function isEqualCaseInsensitive() returning helpful detail in the assertion_result message, negation in a BOOST_CHECK() results in that detail being discarded.

This feature preserves that detail in the negated case.

For example:

BOOST_CHECK(isEqualCaseInsensitive('z', 'Z')) included detail like [z isEqualCaseInsensitiveTo Z]

BOOST_CHECK(!isEqualCaseInsensitive('z', 'Z')) discarded that detail until this revision, which now includes it: NOT([z isEqualCaseInsensitiveTo Z])

K. R. Walker and others added 2 commits September 6, 2023 14:42
Given a helper function `isEqualCaseInsensitive()` returning helpful
detail in the resulting assertion_result message, negation of
that result in a `BOOST_CHECK()` results in that detail being
discarded.

This feature preserves that detail in the negated case.

For example:

`BOOST_CHECK(isEqualCaseInsensitive('z', 'Z'))` included detail like `[z
isEqualCaseInsensitiveTo Z]`

`BOOST_CHECK(!isEqualCaseInsensitive('z', 'Z'))` discarded that detail
until this revision, which now includes it: `NOT([z
isEqualCaseInsensitiveTo Z])`
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant