Fix overly-lax regex pattern in TAP test of READ_REPLICATION_SLOT
authorMichael Paquier <[email protected]>
Tue, 26 Oct 2021 02:16:06 +0000 (11:16 +0900)
committerMichael Paquier <[email protected]>
Tue, 26 Oct 2021 02:16:06 +0000 (11:16 +0900)
The case checking for a NULL output when a slot does not exist was
too lax, as it was passing for any output generated by the query.  This
fixes the matching pattern to be what it should be, matching only on
"||".

Oversight in b4ada4e.

src/test/recovery/t/001_stream_rep.pl

index b3dc68d54bc827ee61d4b1f185e60d0ecf09bd74..c70c08e27bf4840ae98e0778fd2c90975686c734 100644 (file)
@@ -263,7 +263,7 @@ my $slotname = 'test_read_replication_slot_physical';
        'READ_REPLICATION_SLOT non_existent_slot;',
        extra_params => [ '-d', $connstr_rep ]);
 ok($ret == 0, "READ_REPLICATION_SLOT exit code 0 on success");
-like($stdout, qr/^||$/,
+like($stdout, qr/^\|\|$/,
        "READ_REPLICATION_SLOT returns NULL values if slot does not exist");
 
 $node_primary->psql(