Re: Convert sepgsql tests to TAP - Mailing list pgsql-hackers

From Tom Lane
Subject Re: Convert sepgsql tests to TAP
Date
Msg-id 1271299.1721853353@sss.pgh.pa.us
Whole thread Raw
In response to Re: Convert sepgsql tests to TAP  (Andreas Karlsson <andreas@proxel.se>)
Responses Re: Convert sepgsql tests to TAP
List pgsql-hackers
Andreas Karlsson <andreas@proxel.se> writes:
> 1) As I said earlier I think we should remove the old code.

I agree that carrying two versions of the test doesn't seem great.
However, a large part of the purpose of test_sepgsql is to help
people debug their sepgsql setup, which is why it goes to great
lengths to print helpful error messages.  I'm worried that making
it into a TAP test will degrade the usefulness of that, simply
because the TAP infrastructure is pretty damn unfriendly when it
comes to figuring out why a test failed.  You have to know where
to even look for the test logfile, and then you have to ignore
a bunch of useless-to-you chatter.  I'm not sure if there is much
we can do to improve that.  (Although if we could, it would
yield benefits across the whole tree.)

OTOH, I suspect there are so few people using sepgsql that this
doesn't matter too much.  Probably most of them will be advanced
hackers who won't blink at digging through a TAP log.  We should
update the docs to explain that though.

            regards, tom lane



pgsql-hackers by date:

Previous
From: Tom Lane
Date:
Subject: Re: DRAFT: Pass sk_attno to consistent function
Next
From: Thomas Munro
Date:
Subject: Re: Sporadic connection-setup-related test failures on Cygwin in v15-