autoconf-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [PATCH] Minor testsuite size reduction.


From: Ralf Wildenhues
Subject: Re: [PATCH] Minor testsuite size reduction.
Date: Mon, 2 Aug 2010 22:02:34 +0200
User-agent: Mutt/1.5.20 (2010-04-22)

* Ralf Wildenhues wrote on Thu, Jul 22, 2010 at 08:26:07PM CEST:
> * Ralf Wildenhues wrote on Wed, Jul 21, 2010 at 08:58:06PM CEST:
> > * Eric Blake wrote on Wed, Jul 21, 2010 at 08:26:10PM CEST:
> > > On 07/21/2010 11:54 AM, Ralf Wildenhues wrote:
> > > > * Eric Blake wrote on Wed, Jul 21, 2010 at 03:58:16PM CEST:
> > > >>    # Run serially, avoid forks and other potential surprises.
> > > >> +  at_jobs=1
> 
> > > > Rereading this again, it wasn't right, though, or, put another way, I
> > > > think it causes potentially worse behavior in an unusual setting (not
> > > > verified):
> > > > 
> > > > Say you run './testsuite -jN' on a system without mkfifo, then at_quiet
> > > > gets set to : but above change later sets at_jobs to 1, causing
> > > > $at_desc_line to never be printed.
> 
> > > Is this something we need to revert before 2.67 (seeing as how I've been
> > > unable to upload yet, I could rebase it in)?
> > 
> > Well, if, then you could revert just this one line.  It's not a huge
> > issue, but the change didn't fix a regression, so reverting should be
> > safe.
> 
> I now verified that your reversion of this really did avoid a regression
> in the case I outlined before.  So thanks again.

Here's an exposer.  OK?

Thanks,
Ralf

    Add testsuite exposure for last-minute fix in 2.67.
    
    * tests/autotest.at (parallel args but non-working mkfifo):
    New test, to expose the failure v2.66-23-g991183c avoided.

diff --git a/tests/autotest.at b/tests/autotest.at
index 0ed8226..920b2f6 100644
--- a/tests/autotest.at
+++ b/tests/autotest.at
@@ -1526,6 +1526,28 @@ AT_CHECK([test -s sigpipe-stamp || test ! -f 
micro-suite.dir/7/micro-suite.log],
 AT_CLEANUP
 
 
+# Avoid running into a regression when mkfifo does not work.
+AT_CHECK_AT_TEST([parallel args but non-working mkfifo],
+[AT_CHECK([:])
+AT_CLEANUP
+AT_SETUP([second test])
+AT_CHECK([:])
+],
+[], [], [stdout], [stderr],
+[AT_SKIP_PARALLEL_TESTS
+mkdir bin
+cat >bin/mkfifo <<\EOF
+#! /bin/sh
+exit 1
+EOF
+chmod +x bin/mkfifo
+PATH=`pwd`/bin:$PATH
+export PATH
+],
+[AT_CHECK([grep 'second test' stdout], [], [ignore])
+], [--jobs])
+
+
 # --color
 AT_CHECK_AT_TEST([colored test results],
   [AT_CHECK([:])



reply via email to

[Prev in Thread] Current Thread [Next in Thread]