Re: [HACKERS] no test coverage for ALTER FOREIGN DATA WRAPPER nameHANDLER ... - Mailing list pgsql-hackers

From Ashutosh Bapat
Subject Re: [HACKERS] no test coverage for ALTER FOREIGN DATA WRAPPER nameHANDLER ...
Date
Msg-id CAFjFpRe0FPWZ12L3Vb8hXVPhmdFQe0zLzNNzQpEy0RrnHF_rWw@mail.gmail.com
Whole thread Raw
In response to [HACKERS] no test coverage for ALTER FOREIGN DATA WRAPPER name HANDLER ...  (Mark Dilger <hornschnorter@gmail.com>)
List pgsql-hackers
On Fri, May 5, 2017 at 6:08 AM, Mark Dilger <hornschnorter@gmail.com> wrote:
> Hackers,
>
> just FYI, I cannot find any regression test coverage of this part of the grammar, not
> even in the contrib/ directory or TAP tests.  I was going to submit a patch to help out,
> and discovered it is not so easy to do, and perhaps that is why the coverage is missing.
>
> My apologies for the noise if this is already common knowledge.  Also, if I'm wrong,
> I'd appreciate a pointer to the test that I am failing to find.
>

It depends on what do you want to test. You could write a simple test
in postgres_fdw or file_fdw where you specify the same handler as
CREATE FOREIGN DATA WRAPPER command (basically a no-op) to check
whether the command succeeds. If you want to do any more serious
testing, it will require defining at some of the FDW hooks that will
be returned by the handler, and then exercising those hooks.

-- 
Best Wishes,
Ashutosh Bapat
EnterpriseDB Corporation
The Postgres Database Company



pgsql-hackers by date:

Previous
From: Neha Khatri
Date:
Subject: Re: [HACKERS] Should pg_current_wal_location() become pg_current_wal_lsn()
Next
From: Dmitriy Sarafannikov
Date:
Subject: Re: [HACKERS] [PROPOSAL] Use SnapshotAny in get_actual_variable_range