From: | Alvaro Herrera <alvherre(at)2ndquadrant(dot)com> |
---|---|
To: | Robins <robins(at)pobox(dot)com> |
Cc: | pgsql-hackers(at)postgresql(dot)org |
Subject: | Re: Increasing code-coverage of 'make check' |
Date: | 2013-03-11 03:46:47 |
Message-ID: | 20130311034647.GB9412@alvh.no-ip.org |
Views: | Raw Message | Whole Thread | Download mbox | Resend email |
Thread: | |
Lists: | pgsql-hackers |
Robins wrote:
> Hi,
>
> I was checking code-coverage of 'make check' and saw that the regression
> tests don't touch files like psql_help.c at all (read 0%).
>
> Attached is a (very small) patch to work on one ABORT help function as a
> sample.
>
> The reason why I post this is, to know if increasing the code-coverage (as
> a task) is considered important at all (to me it is). If so, I could get
> going with creating tests for more untouched lines / functions in 'make
> check'.
I think increasing coverage is a good thing. But psql help? *shrug*
backend code is far more interesting and useful.
Another thing to keep in mind is that there are some corner cases that
are interesting to test that might not necessarily show up in a coverage
chart -- for example how stuff behaves in the face of concurrent
processes, or when various counters wrap around.
Peter Eisentraut has set up a Jenkins instance that publishes coverage
info.
http://pgci.eisentraut.org/jenkins/job/postgresql_master_coverage/Coverage/
(I think he only has it running "make check"; doing the isolation tests
probably raises percentages a bit).
--
Álvaro Herrera http://www.2ndQuadrant.com/
PostgreSQL Development, 24x7 Support, Training & Services
From | Date | Subject | |
---|---|---|---|
Next Message | Daniel Farina | 2013-03-11 04:53:39 | Re: postgres_fdw vs data formatting GUCs (was Re: [v9.3] writable foreign tables) |
Previous Message | Robins | 2013-03-11 01:52:13 | Increasing code-coverage of 'make check' |