Re: Create shorthand for including all extra tests - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: Create shorthand for including all extra tests
Date
Msg-id E8A25395-0606-4DE2-9A17-BECA21A5BA3E@yesql.se
Whole thread Raw
In response to Re: Create shorthand for including all extra tests  (Tom Lane <tgl@sss.pgh.pa.us>)
Responses Re: Create shorthand for including all extra tests
List pgsql-hackers
> On 4 Sep 2023, at 17:01, Tom Lane <tgl@sss.pgh.pa.us> wrote:
>
> Nazir Bilal Yavuz <byavuz81@gmail.com> writes:
>> I created an 'all' option for PG_TEST_EXTRA to enable all test suites
>> defined under PG_TEST_EXTRA.
>
> I think this is a seriously bad idea.  The entire point of not including
> certain tests in check-world by default is that the omitted tests are
> security hazards, so a developer or buildfarm owner should review each
> one before deciding whether to activate it on their machine.

I dunno, I've certainly managed to not run the tests I hoped to multiple times.
I think it could be useful for sandboxed testrunners which are destroyed after
each run. There is for sure a foot-gun angle to it, no question about that.

--
Daniel Gustafsson




pgsql-hackers by date:

Previous
From: Matthias van de Meent
Date:
Subject: Re: Commitfest 2023-09 starts soon
Next
From: Noah Misch
Date:
Subject: Re: Create shorthand for including all extra tests