Closed cschreib closed 2 months ago
I still need to update the approval test data. But first I need to fix another issue, which was highlighted by these test failures: in the XML or TeamCity reporters, we need to escape some of the strings before reporting them. When the resulting string is too long and truncated, there is a risk of truncating halfway through an escaped sequence, which would render the escaped string invalid. I opened the bug ticket https://github.com/snitch-org/snitch/issues/164 to capture this.
I'm working on a solution.
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 94.05%. Comparing base (
ccd3afa
) to head (65ee518
). Report is 7 commits behind head on main.
Reported in #158. The
*REQUIRE_THAT
macros were assuming thatSNITCH_MAX_MESSAGE_LENGTH
andSNITCH_MAX_EXPR_LENGTH
were the same. If they weren't, this would cause a compilation error. The tests have now been modified so none of the lengths are identical, to catch future instances of this problem.Also fixes https://github.com/snitch-org/snitch/issues/164.