From mboxrd@z Thu Jan 1 00:00:00 1970 From: ludo@gnu.org (Ludovic =?utf-8?Q?Court=C3=A8s?=) Subject: Re: [PATCH] build: Add a scheme custom test driver using SRFI-64. Date: Mon, 02 Nov 2015 09:48:18 +0100 Message-ID: <878u6gokh9.fsf@gnu.org> References: <87lhaiv30f.fsf@gnu.org> <87h9l54p8p.fsf@gnu.org> <87lhahcxgt.fsf@gnu.org> <87r3k99udi.fsf@gnu.org> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: quoted-printable Return-path: Received: from eggs.gnu.org ([2001:4830:134:3::10]:49233) by lists.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZtAmu-0005Nl-TY for guix-devel@gnu.org; Mon, 02 Nov 2015 03:48:26 -0500 Received: from Debian-exim by eggs.gnu.org with spam-scanned (Exim 4.71) (envelope-from ) id 1ZtAmr-0005ZT-Na for guix-devel@gnu.org; Mon, 02 Nov 2015 03:48:24 -0500 Received: from fencepost.gnu.org ([2001:4830:134:3::e]:36216) by eggs.gnu.org with esmtp (Exim 4.71) (envelope-from ) id 1ZtAmr-0005ZM-KD for guix-devel@gnu.org; Mon, 02 Nov 2015 03:48:21 -0500 In-Reply-To: <87r3k99udi.fsf@gnu.org> (Mathieu Lirzin's message of "Mon, 02 Nov 2015 00:23:37 +0100") List-Id: "Development of GNU Guix and the GNU System distribution." List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org Sender: guix-devel-bounces+gcggd-guix-devel=m.gmane.org@gnu.org To: Mathieu Lirzin Cc: guix-devel@gnu.org Mathieu Lirzin skribis: > Mathieu Lirzin writes: > >> ludo@gnu.org (Ludovic Court=C3=A8s) writes: >>> Awesome! Are the =E2=80=9Cinner=E2=80=9D tests displayed by default? = Or is there some >>> environment variable to control that? >> >> Yes, each test case is displayed by default. >> >>> I=E2=80=99d prefer the default to display only file names, as is curren= tly the >>> case (it=E2=80=99s more concise.) >> >> IMO This would be reasonable to have an option for making the output more >> concise since there are 563 test cases in Guix. this is not part of the >> test driver API specification, but it does not seem too hard to add an >> additional option to the test driver script like what the TAP driver is >> doing: >> >> https://www.gnu.org/software/automake/manual/automake.html#Use-TAP-wit= h-the-Automake-test-harness >> >> I think =E2=80=98--brief=E2=80=99 could be a good name for this option. = Default >> verbosity would specified by adding =E2=80=9CAM_SCM_LOG_DRIVER=3D --brie= f=3Dyes=E2=80=9D in > ^^^ > AM_SCM_LOG_DRIVER_FLAGS Great. > From 8a1f52e08c8d33a33066271f0f39d6072baf9854 Mon Sep 17 00:00:00 2001 > From: Mathieu Lirzin > Date: Mon, 26 Oct 2015 23:47:24 +0100 > Subject: [PATCH] build: Add a scheme custom test driver using SRFI-64. > > This provides support for multiple scheme test cases in a unique file and > fixes the fragmentation of '.log' files. > > * build-aux/test-driver.scm: New file. > * Makefile.am (SCM_LOG_DRIVER, AM_SCM_LOG_DRIVER_FLAGS): New variables. > (SCM_LOG_COMPILER, AM_SCM_LOG_FLAGS): Delete variables. > (AM_TESTS_ENVIRONMENT): Set GUILE_AUTO_COMPILE to 0. > * doc/guix.texi (Running the Test Suite): Reporting a bug does not > require to provide a specific '.log' file anymore. > * tests/base32.scm, tests/build-utils.scm, tests/builders.scm, > tests/challenge.scm, tests/containers.scm, tests/cpan.scm, > tests/cpio.scm, tests/cran.scm, tests/derivations.scm, tests/elpa.scm, > tests/file-systems.scm, tests/gem.scm, tests/gexp.scm, > tests/graph.scm, tests/gremlin.scm, tests/hackage.scm, tests/hash.scm, > tests/lint.scm, tests/monads.scm, tests/nar.scm, tests/packages.scm, > tests/pk-crypto.scm, tests/pki.scm, tests/profiles.scm, > tests/publish.scm, tests/pypi.scm, tests/records.scm, > tests/scripts.scm, tests/services.scm, tests/sets.scm, tests/size.scm, > tests/snix.scm, tests/store.scm, tests/substitute.scm, > tests/syscalls.scm, tests/ui.scm, tests/union.scm, tests/utils.scm: > Don't exit at the end of each test. Looks nice. If you want you can push it to a wip- branch, and we=E2=80=99ll apply it af= ter the release. Or you can keep it locally for later. Thanks, Ludo=E2=80=99.