Re: Tighten up a few overly lax regexes in pg_dump's tap tests
David Rowley <david.rowley@2ndquadrant.com>
From: David Rowley <david.rowley@2ndquadrant.com>
To: Michael Paquier <michael@paquier.xyz>
Cc: Daniel Gustafsson <daniel@yesql.se>, PostgreSQL Hackers <pgsql-hackers@lists.postgresql.org>
Date: 2019-02-05T00:50:50Z
Lists: pgsql-hackers
On Tue, 5 Feb 2019 at 13:15, Michael Paquier <michael@paquier.xyz> wrote: > Some tests are missing the update, and it seems to me that > tightening things up is a good thing, still we ought to do it > consistently. Some places missing the update: > - ALTER OPERATOR FAMILY > - ALTER OPERATOR CLASS > - ALTER SEQUENCE > - ALTER TABLE (ONLY, partitioned table) > - BLOB load > - COMMENT ON > - COPY > - INSERT INTO > - CREATE COLLATION I think I've done all the ones that use \E. Those \Q ones don't need any escaping. I assume that's the ones you've found. I didn't do an exhaustive check though. > test_pg_dump's 001_base.pl needs also a refresh. Yeah, I thought about looking, but I thought it might be a bottomless pit. -- David Rowley http://www.2ndQuadrant.com/ PostgreSQL Development, 24x7 Support, Training & Services
Commits
-
Align better test output regex with grammar in pg_dump TAP tests
- f339a998ffe6 12.0 landed
-
Tighten some regexes with proper character escaping in pg_dump TAP tests
- d07fb6810e51 12.0 landed