From: | Heikki Linnakangas <hlinnaka(at)iki(dot)fi> |
---|---|
To: | Peter Eisentraut <peter(dot)eisentraut(at)2ndquadrant(dot)com>, pgsql-hackers <pgsql-hackers(at)postgresql(dot)org> |
Subject: | Re: location reporting in TAP test failures |
Date: | 2018-06-21 17:30:03 |
Message-ID: | 67e25e42-fb52-2227-82f1-9e5edcadaa18@iki.fi |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
On 05/06/18 18:28, Peter Eisentraut wrote:
> Right now, when a TAP test reports a failure, it looks something like this:
>
> # Failed test 'creating a replication slot'
> # at
> /....../postgresql/src/bin/pg_basebackup/../../../src/test/perl/TestLib.pm
> line 371.
>
> That file location is where we call out to the test function provided by
> Test::More.
>
> What we'd really want is
>
> # Failed test 'creating a replication slot'
> # at t/020_pg_receivewal.pl line 36.
>
> because that's where the code that's doing the testing is.
>
> To achieve that, we need to have our test library functions tell that
> they are support functions and not the actual tests. The attached patch
> does that. The mechanism is (somewhat) explained in the Test::Builder
> man page.
Looks good. I wish there was some way to remind to do this for any
future test helper functions, too, but I don't have any ideas off the
top of my head.
Since this is purely a test change, IMO this could go to master now, no
need to wait for until v12 development starts.
- Heikki
From | Date | Subject | |
---|---|---|---|
Next Message | Andrew Dunstan | 2018-06-21 17:31:48 | Re: Fast default stuff versus pg_upgrade |
Previous Message | Andrew Dunstan | 2018-06-21 17:28:46 | Re: Fast default stuff versus pg_upgrade |