Re: Add more regression tests for CREATE OPERATOR - Mailing list pgsql-hackers

From Josh Berkus
Subject Re: Add more regression tests for CREATE OPERATOR
Date
Msg-id 51CB3CAC.1050003@agliodbs.com
Whole thread Raw
In response to Add more regression tests for CREATE OPERATOR  (Robins Tharakan <tharakan@gmail.com>)
Responses Re: Add more regression tests for CREATE OPERATOR  (Szymon Guz <mabewlun@gmail.com>)
List pgsql-hackers
> Is it enough to provide the description in the commitfest app, or is that
> better to send an email and provide link in commitfest?

Better to do it here, on the list.

> This is a patch only with regression tests, is that enough to write
> something like: "This patch applies cleanly on trunk code. All tests pass,
> the test coverage increses as provided."? Or do you expect some more info?

Yes, mainly:

a) does it test what it purports to test?

b) do the tests pass on your machine?


-- 
Josh Berkus
PostgreSQL Experts Inc.
http://pgexperts.com



pgsql-hackers by date:

Previous
From: Fabien COELHO
Date:
Subject: Re: Add more regression tests for dbcommands
Next
From: Josh Berkus
Date:
Subject: Re: Kudos for Reviewers -- straw poll