Hi,
On 2021-10-13 13:54:10 +0200, Daniel Gustafsson wrote:
> I added a --tap option for TAP output to pg_regress together with Jinbao Chen
> for giggles and killing some time a while back.
Sorry for not replying to this earlier. I somehow thought I had, but the
archives disagree.
I think this would be great.
> If it's helpful and there's any interest for this I'm happy to finish it up
> now.
Yes! Probably worth starting a new thread for...
> One thing that came out of this, is that we don't really handle the ignored
> tests in the way the code thinks it does for normal output, the attached
> treats
> ignored tests as SKIP tests.
I can't really parse the first sentence...
> if (exit_status != 0)
> log_child_failure(exit_status);
> @@ -2152,6 +2413,7 @@ regression_main(int argc, char *argv[],
> {"config-auth", required_argument, NULL, 24},
> {"max-concurrent-tests", required_argument, NULL, 25},
> {"make-testtablespace-dir", no_argument, NULL, 26},
> + {"tap", no_argument, NULL, 27},
> {NULL, 0, NULL, 0}
> };
I'd make it a --format=(regress|tap) or such.
Greetings,
Andres Freund
