Re: Fix a test case in 035_standby_logical_decoding.pl

Hi,

On 4/27/23 10:11 AM, Yu Shi (Fujitsu) wrote:
> Hi hackers,
>
> In 035_standby_logical_decoding.pl, I think that the check in the following test
> case should be performed on the standby node, instead of the primary node, as
> the slot is created on the standby node.

Oh right, the current test is not done on the right node, thanks!

> The test currently passes because it
> only checks the return value of psql. It might be more appropriate to check the
> error message.

Agree, and it's consistent with what is being done in 006_logical_decoding.pl.

> Please see the attached patch.
>

+
+($result, $stdout, $stderr) = $node_standby->psql(
'otherdb',
"SELECT lsn FROM pg_logical_slot_peek_changes('behaves_ok_activeslot', NULL, NULL) ORDER BY lsn DESC LIMIT 1;"
- ),
- 3,
- 'replaying logical slot from another database fails');
+ );
+ok( $stderr =~
+ m/replication slot "behaves_ok_activeslot" was not created in this database/,
+ "replaying logical slot from another database fails");

That does look good to me.

Nit: I wonder if while at it (as it was already there) we could not remove the " ORDER BY lsn DESC LIMIT 1" part of it.
It does not change anything regarding the test but looks more appropriate to me.

Regards,

--
Bertrand Drouvot
PostgreSQL Contributors Team
RDS Open Source Databases
Amazon Web Services: https://aws.amazon.com