Re: Regression tests failing if not launched on db "regression"

From: Michael Paquier <michael(dot)paquier(at)gmail(dot)com>
To: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
Cc: Robert Haas <robertmhaas(at)gmail(dot)com>, PostgreSQL mailing lists <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Regression tests failing if not launched on db "regression"
Date: 2013-12-05 21:50:18
Message-ID: CBCD561B-1634-4FE0-BD00-36D66FDFB731@gmail.com
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

> On 2013/12/06, at 3:52, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>
> Robert Haas <robertmhaas(at)gmail(dot)com> writes:
>>> On Thu, Dec 5, 2013 at 9:24 AM, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us> wrote:
>>> Michael Paquier <michael(dot)paquier(at)gmail(dot)com> writes:
>>>> It happens that the following regression tests are failing if they are
>>>> run on a database not named "regression":
>
>>> This does not seem like a bug to me, although maybe we'd better update the
>>> documentation to specify that you need to use a DB named regression.
>
>> At the same thing, supporting it might not cost anything.
>
> Well, changing these specific tests today might not be terribly expensive,
> but what is it that prevents more such tests from being committed next
> week? Perhaps by somebody who feels current_database() should be included
> in code coverage, for example?
>
> More generally, we never have and never can promise that the regression
> tests pass regardless of environment. If you turn off enable_seqscan,
> for instance, you'll get a whole lot of not-terribly-exciting diffs.
> I see no particular benefit to promising that the name of the regression
> database isn't significant.
You got a point here. Classifying that as a "don't do that" in the documentation is fine for me as well, as I recall that --dbname has been added to pg_regress only for contrib regression support.

Regards,
--
Michael
(Sent from my mobile phone)

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Robert Treat 2013-12-05 22:48:21 Re: How to detect invisible rows caused by the relfrozenxid bug?
Previous Message bricklen 2013-12-05 21:14:09 Re: Why we are going to have to go DirectIO