pg.ddx.io  pgsql-committers@postgresql.org mailing list archive  
help / color / mirror / Atom feed
From: Tom Lane <tgl@sss.pgh.pa.us>
To: pgsql-committers@lists.postgresql.org
Subject: pgsql: Add basic TAP tests for psql's tab-completion logic.
Date: Thu, 02 Jan 2020 20:02:29 +0000
Message-ID: <E1in6ft-0004zR-6l@gemulon.postgresql.org> (raw)

Add basic TAP tests for psql's tab-completion logic.

Up to now, psql's tab-complete.c has had exactly no regression test
coverage.  This patch is an experimental attempt to add some.

This needs Perl's IO::Pty module, which isn't installed everywhere,
so the test script just skips all tests if that's not present.
There may be other portability gotchas too, so I await buildfarm
results with interest.

So far this just covers a few very basic keyword-completion and
query-driven-completion scenarios, which should be enough to let us
get a feel for whether this is practical at all from a portability
standpoint.  If it is, there's lots more that can be done.

Discussion: https://postgr.es/m/10967.1577562752@sss.pgh.pa.us

Branch
------
master

Details
-------
https://git.postgresql.org/pg/commitdiff/7c015045b9141cc30272930ea88cfa5df47240b7

Modified Files
--------------
configure                            |   2 +
configure.in                         |   1 +
src/Makefile.global.in               |   1 +
src/bin/psql/.gitignore              |   2 +-
src/bin/psql/Makefile                |  10 +++
src/bin/psql/t/010_tab_completion.pl | 122 +++++++++++++++++++++++++++++++++++
src/test/perl/PostgresNode.pm        |  67 +++++++++++++++++++
7 files changed, 204 insertions(+), 1 deletion(-)



view thread (64+ messages)  latest in thread

Message-ID: <E1in6ft-0004zR-6l@gemulon.postgresql.org>
Permalink:  ../E1in6ft-0004zR-6l@gemulon.postgresql.org/
Also on:    postgresql.org/message-id/E1in6ft-0004zR-6l@gemulon.postgresql.org

 · 

reply

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Reply to all the recipients using the --to and --cc options:
  reply via email

  To: pgsql-committers@postgresql.org
  Cc: tgl@sss.pgh.pa.us, pgsql-committers@lists.postgresql.org
  Subject: Re: pgsql: Add basic TAP tests for psql's tab-completion logic.
  In-Reply-To: <E1in6ft-0004zR-6l@gemulon.postgresql.org>

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

This inbox is served by DDX for PostgreSQL; see mirroring instructions
for how to clone and mirror all data and code used for this inbox