[master] 24da967 Forgot to remove -err

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Fri Feb 3 17:54:05 CET 2017


commit 24da967b48d074fa7b9c3f067319ba4d1b2be37d
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Fri Feb 3 17:50:28 2017 +0100

    Forgot to remove -err
    
    The original test was:
    
        shell -err {
                varnishlog -n ${v1_name} -d -g raw |
                grep "Should not happen"
        }
    
    But it wouldn't catch errors, like for example varnishlog failing to
    connect to the shmlog.

diff --git a/bin/varnishtest/tests/m00027.vtc b/bin/varnishtest/tests/m00027.vtc
index 92cd417..daadeab 100644
--- a/bin/varnishtest/tests/m00027.vtc
+++ b/bin/varnishtest/tests/m00027.vtc
@@ -18,7 +18,7 @@ varnish v1 -errvcl "Missing dynamic backend address or port" {
 	}
 }
 
-shell -err {
+shell {
 	set -e
 	varnishlog -n ${v1_name} -d -g raw >v1.log
 	! grep "Should not happen" v1.log



More information about the varnish-commit mailing list