Re: TAP test breakage on MacOS X

From: Andres Freund <andres(at)2ndquadrant(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Jim Nasby <Jim(dot)Nasby(at)BlueTreble(dot)com>, Andrew Dunstan <andrew(at)dunslane(dot)net>, Noah Misch <noah(at)leadboat(dot)com>, Peter Eisentraut <peter_e(at)gmx(dot)net>, Heikki Linnakangas <hlinnakangas(at)vmware(dot)com>, Robert Haas <robertmhaas(at)gmail(dot)com>, "pgsql-hackers(at)postgresql(dot)org" <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: TAP test breakage on MacOS X
Date: 2014-10-30 23:47:00
Message-ID: 20141030234700.GN17724@awork2.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

On 2014-10-30 19:30:04 -0400, Tom Lane wrote:
> Jim Nasby <Jim(dot)Nasby(at)BlueTreble(dot)com> writes:
> > If our policy is that tests are there primarily for developers then I agree with you.
>
> > If not, then would we be OK with make check being a no-op unless you'd configured with --enable-make-check?
>
> > Making this something you have to enable will seriously limit the number of people running the TAP tests, simply because few will know to enable them.
>
> Well, TBH I have no problem with that at the moment, because as Robert has
> pointed out the current TAP tests are of close to zero value. The odds
> that they'll find anything in the hands of Joe Random User are far lower
> than the odds that they'll break Joe's build.

I already have a couple more ready once this has stabilized...

I personally don't agree that they have no value at this point. At least
the pg_basebackup tests test paths that are not executed *at all*
otherwise and which are not trivial. To my knowledge it's the only thing
in our tests that exercises walsender and wal_level != minimal.

> At some point down the road that value judgement might (hopefully will)
> reverse, and then we could deal with it by making --enable-tap-tests the
> default. But even then there would be a place for
> --disable-tap-tests.

Which would be what exactly?

> The current situation, where the only way to disable the TAP tests is to
> not do make check-world, is utterly unacceptable given their low present
> usefulness and lack of proven portability.

Agreed.

> I opined before that we should rip those tests out of the 9.4 branch
> altogether. I'm willing to leave them there if we have an --enable
> switch controlling them, though.

Hm. I'm not convinced that that's the best way, but I'm not going fight
hard.

Greetings,

Andres Freund

--
Andres Freund http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Tom Lane 2014-10-30 23:53:54 Re: TAP test breakage on MacOS X
Previous Message Andres Freund 2014-10-30 23:34:52 Re: proposal: CREATE DATABASE vs. (partial) CHECKPOINT