Re: svn commit: r1872919 - /subversion/trunk/subversion/tests/libsvn_subr/mergeinfo-test.c

julianfoad_at_apache.org wrote on Fri, Jan 17, 2020 at 14:54:46 -0000:
> @@ -2089,18 +2130,22 @@ test_rangelist_merge_random_canonical_in
> + if (failure_mode)
> {
> - printf("testcase FAIL: %s / %s\n",
> + printf("first example of a failure mode: %s / %s\n"
> + " %s\n",
> rangelist_to_string(rlx, iterpool),
> + rangelist_to_string(rly, iterpool),
> + failure_mode);

Test programs aren't supposed to print arbitrary stuff to stdout.

Specifically, the interface between run_tests.py and the test programs
specifies¹ that test programs shall only print "PASS:" and "FAIL:" lines by
default. We could of course extend that to support printing of new info, for
example, as "INFO:" lines. So would it make sense to prepend "INFO:" to that
line?

Cheers,

Daniel

¹ Citation: subversion/tests/README:34
Received on 2020-01-17 19:01:14 CET