Re: Documenting PROVE_TESTS in section of TAP tests

From: Andres Freund <andres(at)anarazel(dot)de>
To: Michael Paquier <michael(at)paquier(dot)xyz>
Cc: Postgres hackers <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: Documenting PROVE_TESTS in section of TAP tests
Date: 2018-03-01 09:52:09
Message-ID: 20180301095209.apele7fcimzao3tm@alap3.anarazel.de
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers

Hi,

On 2018-02-17 23:03:05 +0900, Michael Paquier wrote:
> The section of the documentation dedicated to TAP tests mentions
> PROVE_FLAGS:
> https://www.postgresql.org/docs/devel/static/regress-tap.html
>
> I think that it would be a good idea to mention PROVE_TESTS as well. I
> personally use and abuse of it, and documenting it instead of keeping it
> hidden in Makefile.global.in is good for new developers and testers.
>
> Proposal of patch attached.

Good plan.

> diff --git a/doc/src/sgml/regress.sgml b/doc/src/sgml/regress.sgml
> index 53716a029f..d14318053b 100644
> --- a/doc/src/sgml/regress.sgml
> +++ b/doc/src/sgml/regress.sgml
> @@ -675,6 +675,17 @@ make -C src/bin check PROVE_FLAGS='--timer'
> See the manual page of <command>prove</command> for more information.
> </para>
>
> + <para>
> + The <command>make</command> variable <varname>PROVE_TESTS</varname>
> + can be used to define a whitespace-separated list of paths relative
> + to the <filename>Makefile</filename> invoking <command>prove</command>
> + to run a subset of tests instead of the default
> + <filename>t/*.pl</filename>. For example:
> +<programlisting>
> +make check PROVE_TESTS='t/001_test1.pl t/003_test3.pl'
> +</programlisting>
> + </para>
> +

Pushed, after replacing "a subset" with "the specified subset".

Greetings,

Andres Freund

In response to

Responses

Browse pgsql-hackers by date

  From Date Subject
Next Message Magnus Hagander 2018-03-01 09:52:12 Re: 2018-03 Commitfest starts tomorrow
Previous Message Andres Freund 2018-03-01 09:42:00 Re: Minor fix for pgbench documentation