Skip to content

Commit f828938

Browse files
committed
[test] Run parser tests at the end of TFormulaTests
The parsing tests need to be run in the end, because they also test what happens when the formula is invalid, resulting in a failed compilation leaves cling in a "bad state". The can cause the vectorization test to fail if run afterwards. See #6439
1 parent 1561f67 commit f828938

File tree

1 file changed

+6
-1
lines changed

1 file changed

+6
-1
lines changed

test/TFormulaTests.cxx

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -571,8 +571,13 @@ int main(int argc, char **argv)
571571
printf("Eval test:%s\n",(test->Eval() ? "PASSED" : "FAILED"));
572572
#endif
573573
printf("Stress test:%s\n",(test->Stress(n) ? "PASSED" : "FAILED"));
574-
printf("Parsing test:%s\n",(test->Parser() ? "PASSED" : "FAILED"));
575574
printf("Vectorization test:%s\n",(test->Vectorize() ? "PASSED" : "FAILED"));
575+
// The parsing tests need to be run in the end, because they also test what
576+
// happens when the formula is invalid, resulting in a failed compilation
577+
// leaves cling in a "bad state". The can cause the vectorization test to
578+
// fail if run afterwards.
579+
// See https://github.com/root-project/root/issues/6439
580+
printf("Parsing test:%s\n",(test->Parser() ? "PASSED" : "FAILED"));
576581

577582
return 0;
578583
}

0 commit comments

Comments
 (0)