Re: TAP tests are badly named

From: Andrew Dunstan <andrew(at)dunslane(dot)net>
To: PostgreSQL-development <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: TAP tests are badly named
Date: 2015-07-30 23:54:27
Message-ID: 55BAB933.20109@dunslane.net
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


On 07/30/2015 12:40 PM, Andrew Dunstan wrote:
>
> We should describe test sets by what they test, not by how they test.
> TAP is a testing tool/protocol. The current set of tests we have test
> the programs in src/bin, and we should really name the test set by a
> name that reflects that, rather than the fact that we are using TAP
> tools to run the tests. What if we decide to test something else using
> TAP? Would we call that set of tests TAP tests too?
>
> --enable-tap-tests is a reasonable configuration setting, because it's
> about whether or not we have a TAP testing framework available, but I
> think we should stop calling the bin tests "TAP tests" and we should
> change the test name in vcregress.pl to a more appropriate name. In
> the buildfarm I'm calling the step "bin-check":
> <http://www.pgbuildfarm.org/cgi-bin/show_stage_log.pl?nm=crake&dt=2015-07-30%2012%3A25%3A58&stg=bin-check>
>
> Thoughts?
>
>

In fact, looking more closely at the changes that have been made to
vcregress.pl, I don't really like the way this has been done. I'm
putting together some changes to bring things more into line with how
the Makefiles work.

cheers

andrew

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Michael Paquier 2015-07-31 00:35:26 Re: Improving test coverage of extensions with pg_dump
Previous Message Fabrízio de Royes Mello 2015-07-30 23:30:02 Re: Doubt about AccessExclusiveLock in ALTER TABLE .. SET ( .. );