Re: [PATCH] Add regression tests of ecpg command notice (error / warning)
Fujii Masao <masao.fujii@oss.nttdata.com>
From: Fujii Masao <masao.fujii@oss.nttdata.com>
To: Ryo Kanbayashi <kanbayashi.dev@gmail.com>
Cc: pgsql-hackers@lists.postgresql.org
Date: 2025-03-04T06:01:56Z
Lists: pgsql-hackers
On 2025/03/03 14:09, Ryo Kanbayashi wrote:
>> On Mon, Mar 3, 2025 at 12:23 PM Fujii Masao <masao.fujii@oss.nttdata.com> wrote:
>> On 2025/03/01 19:45, Ryo Kanbayashi wrote:
>>>> +program_help_ok('ecpg');
>>>> +program_version_ok('ecpg');
>>>> +program_options_handling_ok('ecpg');
>>>> +command_fails(['ecpg'], 'ecpg without arguments fails');
>>>>
>>>> These checks seem unnecessary in 002 since they're already covered in 001.
>>>
>>> I reflected above.
>>
>> Thanks for updating the patch!
>>
>> I've made some minor fixes and cosmetic adjustments.
>> The updated patch is attached.
>>
>> Unless there are any objections, I'll commit it.
>
> Thanks for reviewing and adustments.
> There is no objections :)
I've pushed the patch. Thanks!
Regards,
--
Fujii Masao
Advanced Computing Technology Center
Research and Development Headquarters
NTT DATA CORPORATION
Commits
-
ecpg: Fix compiler warning in ecpg build with Meson.
- 9f25b9f73922 18.0 landed
-
ecpg: Add TAP test for the ecpg command.
- 28f04984f0c2 18.0 landed
-
Avoid updating inactive_since for invalid replication slots.
- 0ec3c295e759 18.0 cited