pg_restore: TAP test case typo(wrong word) for an error hint in 001_basic.pl

Started by Mahendra Singh Thalor30 days ago5 messageshackers
Jump to latest
#1Mahendra Singh Thalor
mahi6run@gmail.com

Hi,

While doing some changes, I found a typo error(wrong word) in 001_basic.pl
file.

In 31a8b77a9244a0883e1968adc, we added some test cases for pg_dump,
pg_restore and pg_dumpall.
In one error case, we added 'pg_dump' name instead of 'pg_rsetore'.

diff --git a/src/bin/pg_dump/t/001_basic.pl b/src/bin/pg_dump/t/001_basic.pl
index 9f12ca6c51d..ff1d422292e 100644
--- a/src/bin/pg_dump/t/001_basic.pl
+++ b/src/bin/pg_dump/t/001_basic.pl
@@ -144,7 +144,7 @@ command_fails_like(
 command_fails_like(
        [ 'pg_restore', '-f -', '-F', 'garbage' ],
        qr/\Qpg_restore: error: unrecognized archive format "garbage";\E/,
-       'pg_dump: unrecognized archive format');
+       'pg_restore: unrecognized archive format');

Here, I am attaching a patch to fix this.This patch can be applied till
PG14.

I think we should back-patch till PG14. Please review this.

--
Thanks and Regards
Mahendra Singh Thalor
EnterpriseDB: http://www.enterprisedb.com

Attachments:

v01-pg_restore-TAP-test-wrong-word-in-hint.patchtext/x-patch; charset=US-ASCII; name=v01-pg_restore-TAP-test-wrong-word-in-hint.patchDownload+1-2
#2Michael Paquier
michael@paquier.xyz
In reply to: Mahendra Singh Thalor (#1)
Re: pg_restore: TAP test case typo(wrong word) for an error hint in 001_basic.pl

On Sun, Mar 15, 2026 at 10:59:53PM +0530, Mahendra Singh Thalor wrote:

I think we should back-patch till PG14. Please review this.

Thanks, grabbed this one for later. I am not sure if it is worth
bothering about the back-branches. If somebody feels otherwise,
please feel free to overwrite me and I will adapt.
--
Michael

#3Mahendra Singh Thalor
mahi6run@gmail.com
In reply to: Michael Paquier (#2)
Re: pg_restore: TAP test case typo(wrong word) for an error hint in 001_basic.pl

Thanks Michael for the review.

Please can you push this change into master?

Thanks and Regards
Mahendra Singh Thalor
EnterpriseDB: http://www.enterprisedb.com

On Mon, 16 Mar 2026 at 4:59 AM, Michael Paquier <michael@paquier.xyz> wrote:

Show quoted text

On Sun, Mar 15, 2026 at 10:59:53PM +0530, Mahendra Singh Thalor wrote:

I think we should back-patch till PG14. Please review this.

Thanks, grabbed this one for later. I am not sure if it is worth
bothering about the back-branches. If somebody feels otherwise,
please feel free to overwrite me and I will adapt.
--
Michael

#4Nathan Bossart
nathandbossart@gmail.com
In reply to: Mahendra Singh Thalor (#3)
Re: pg_restore: TAP test case typo(wrong word) for an error hint in 001_basic.pl

On Fri, Mar 20, 2026 at 01:33:00AM +0530, Mahendra Singh Thalor wrote:

Please can you push this change into master?

I took care of this in commit ec80215c03.

--
nathan

#5Mahendra Singh Thalor
mahi6run@gmail.com
In reply to: Nathan Bossart (#4)
Re: pg_restore: TAP test case typo(wrong word) for an error hint in 001_basic.pl

On Fri, 20 Mar 2026 at 01:37, Nathan Bossart <nathandbossart@gmail.com> wrote:

On Fri, Mar 20, 2026 at 01:33:00AM +0530, Mahendra Singh Thalor wrote:

Please can you push this change into master?

I took care of this in commit ec80215c03.

--
nathan

Thanks Nathan. I only saw the commit message. Apologize.

--
Thanks and Regards
Mahendra Singh Thalor
EnterpriseDB: http://www.enterprisedb.com