> On 20 Feb 2025, at 14:06, Andrew Dunstan <[email protected]> wrote:
> Actually, since ok() and friends return true iff the test succeeds, instead of
> + ok(! $self->{timeout}->is_expired, 'psql query_until did not time out');
> + return undef if $self->{timeout}->is_expired;
> you can avoid doing the same test twice and say:
> ok(! $self->{timeout}->is_expired, 'psql query_until did not time out')
> || return undef;
> although for slightly technical reasons perlcritic disapproves of "return
> undef" and prefers that you just write a bare "return" so we should also fix
> that.
> Sorry for taking a second bite at the cherry.
Not at all, I agree that this is an improvement so fixed in the attached along
with a fresh pgperltidy.
--
Daniel Gustafsson
v4-0002-Report-test-failure-rather-than-aborting-in-case-.patch
Description: Binary data
v4-0001-Configure-interactive-instance-to-restart-timer.patch
Description: Binary data
