On 9/23/21 5:20 PM, Peter Eisentraut wrote: > On 23.09.21 12:34, Michael Paquier wrote: >> On Wed, Sep 22, 2021 at 03:18:43PM +0000, Jacob Champion wrote: >>> On Wed, 2021-09-22 at 10:20 +0200, Peter Eisentraut wrote: >>>> This should be added to each level of a function call that >>>> represents a >>>> test. This ensures that when a test fails, the line number points to >>>> the top-level location of the test_role() call. Otherwise it would >>>> point to the connect_ok() call inside test_role(). >>> >>> Patch LGTM, sorry about that. Thanks! >> >> For the places of the patch, that seems fine then. Thanks! > > committed > >> Do we need to care about that in other places? We have tests in >> src/bin/ using subroutines that call things from PostgresNode.pm or >> TestLib.pm, like pg_checksums, pg_ctl or pg_verifybackup, just to name >> three. > > Yeah, at first glance, there is probably more that could be done. > Here, I was just looking at a place where it was already and was > accidentally removed.
It probably wouldn't be a bad thing to have something somewhere (src/test/perl/README ?) that explains when and why we need to bump $Test::Builder::Level. cheers andrew -- Andrew Dunstan EDB: https://www.enterprisedb.com