r3826 - in trunk/varnish-cache/bin/varnishtest: . tests
    tfheen at projects.linpro.no 
    tfheen at projects.linpro.no
       
    Wed Feb 25 11:50:49 CET 2009
    
    
  
Author: tfheen
Date: 2009-02-25 11:50:49 +0100 (Wed, 25 Feb 2009)
New Revision: 3826
Modified:
   trunk/varnish-cache/bin/varnishtest/tests/a00009.vtc
   trunk/varnish-cache/bin/varnishtest/vtc.c
Log:
Make sure system "..." returns successfully
We didn't check the return status of system, making failures in a00009
go unnoticed.  Fix this and fix a00009 to not fail when run
uninstalled.
Modified: trunk/varnish-cache/bin/varnishtest/tests/a00009.vtc
===================================================================
--- trunk/varnish-cache/bin/varnishtest/tests/a00009.vtc	2009-02-25 10:25:43 UTC (rev 3825)
+++ trunk/varnish-cache/bin/varnishtest/tests/a00009.vtc	2009-02-25 10:50:49 UTC (rev 3826)
@@ -2,4 +2,4 @@
 
 test "See that the VCL compiler works"
 
-shell "cd ../varnishd && ./varnishd -b 127.0.0.1:80 -C"
+shell "cd ../varnishd && ./varnishd -b 127.0.0.1:80 -C -n /tmp/__v1"
Modified: trunk/varnish-cache/bin/varnishtest/vtc.c
===================================================================
--- trunk/varnish-cache/bin/varnishtest/vtc.c	2009-02-25 10:25:43 UTC (rev 3825)
+++ trunk/varnish-cache/bin/varnishtest/vtc.c	2009-02-25 10:50:49 UTC (rev 3826)
@@ -246,7 +246,7 @@
 	AN(av[1]);
 	AZ(av[2]);
 	vtc_dump(vl, 4, "shell", av[1]);
-	(void)system(av[1]);	/* XXX: assert ? */
+	assert(WEXITSTATUS(system(av[1])) == 0);
 }
 
 /**********************************************************************
    
    
More information about the varnish-commit
mailing list