Re: More business with $Test::Builder::Level in the TAP tests - Mailing list pgsql-hackers

From Daniel Gustafsson
Subject Re: More business with $Test::Builder::Level in the TAP tests
Date
Msg-id 84CDB9AE-9A04-419A-A51F-E61A36AB3272@yesql.se
Whole thread Raw
In response to Re: More business with $Test::Builder::Level in the TAP tests  (Michael Paquier <michael@paquier.xyz>)
List pgsql-hackers
> On 8 Oct 2021, at 09:51, Michael Paquier <michael@paquier.xyz> wrote:
>
> On Fri, Oct 08, 2021 at 09:28:04AM +0200, Daniel Gustafsson wrote:
>> LGTM.  Maybe it should be added that it *must* be called before any Test::More
>> function is called, it's sort of self-explanatory but not everyone writing TAP
>> tests will be deeply familiar with Perl.
>
> I think that "must" is too strong in this context, as in some cases it
> does not really make sense to increment the level, when using for
> example a rather long routine that's labelled with one of the
> routine arguments like for pg_rewind.  So I would stick with
> "should".

Fair enough.

--
Daniel Gustafsson        https://vmware.com/




pgsql-hackers by date:

Previous
From: Daniel Gustafsson
Date:
Subject: Re: pgsql: Adjust configure to insist on Perl version >= 5.8.3.
Next
From: Takashi Menjo
Date:
Subject: Re: Map WAL segment files on PMEM as WAL buffers