test polish suggested by Dridi, thank you

bin/varnishtest/tests/d00015.vtc

@@ -22,33 +22,36 @@ varnish v1 -vcl+backend {
 
 		std.log("-- invalid replicas");
 		if (!vd.reconfigure(replicas=0)) {
+			# continue intentionally
 			std.log("reconfigure failed");
 		}
 
 		std.log("-- no changes - no debug output");
 		if (!vd.reconfigure(replicas=25)) {
+			# continue intentionally
 			std.log("reconfigure failed");
 		}
 
 		std.log("-- no backends");
 		if (!vd.clear()) {
-			std.log("clear failed");
+			return(fail("clear failed"));
 		}
 		if (!vd.reconfigure(replicas=25)) {
+			# continue intentionally
 			std.log("reconfigure failed");
 		}
 
 		std.log("-- one backend");
 		if (!vd.add_backend(s1)) {
-			std.log("add s1 failed");
+			return(fail("add s1 failed"));
 		}
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- no change - no output");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- change, clear, no backends");
@@ -60,6 +63,7 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s2);
 		vd.clear();
 		if (!vd.reconfigure()) {
+			# continue intentionally
 			std.log("reconfigure failed");
 		}
 
@@ -69,7 +73,7 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s2);
 		vd.add_backend(s1);
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- duplicate add with idents");
@@ -82,19 +86,19 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s2, ident="s1_1");
 		vd.add_backend(s2, ident="s1_2");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- remove s1_2 specifically");
 		vd.remove_backend(ident="s1_2");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- remove all instances of s1");
 		vd.remove_backend(s1);
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- re-add some - no 2nd director");
@@ -110,19 +114,19 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s3, "8");
 		vd.add_backend(s3, "9");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- remove second-last");
 		vd.remove_backend(ident="8");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- remove last");
 		vd.remove_backend(ident="9");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- END");
@@ -146,7 +150,7 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s1, "0x0e");
 		vd.add_backend(s1, "0x0f");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 		vd.remove_backend(s1, "0x00");
 		vd.remove_backend(s1, "0x01");
@@ -165,7 +169,7 @@ varnish v1 -vcl+backend {
 		vd.remove_backend(s1, "0x0e");
 		vd.remove_backend(s1, "0x0f");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 		vd.set_warmup(0.5);
 
@@ -180,7 +184,7 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s3, "4");
 		vd.add_backend(s3, "5");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 		vd.set_warmup(0);
 	}

bin/varnishtest/tests/d00016.vtc

@@ -22,35 +22,39 @@ varnish v1 -vcl+backend {
 	}
 
 	sub vcl_recv {
+
 		std.log("-- invalid replicas");
 		if (!vd.reconfigure(replicas=0)) {
+			# continue intentionally
 			std.log("reconfigure failed");
 		}
 
 		std.log("-- no changes - no debug output");
 		if (!vd.reconfigure(replicas=25)) {
+			# continue intentionally
 			std.log("reconfigure failed");
 		}
 
 		std.log("-- no backends");
 		if (!vd.clear()) {
-			std.log("clear failed");
+			return(fail("clear failed"));
 		}
 		if (!vd.reconfigure(replicas=25)) {
+			# continue intentionally
 			std.log("reconfigure failed");
 		}
 
 		std.log("-- one backend");
 		if (!vd.add_backend(s1)) {
-			std.log("add s1 failed");
+			return(fail("add s1 failed"));
 		}
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- no change - no output");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- change, clear, no backends");
@@ -62,6 +66,7 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s2);
 		vd.clear();
 		if (!vd.reconfigure()) {
+			# continue intentionally
 			std.log("reconfigure failed");
 		}
 
@@ -71,7 +76,7 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s2);
 		vd.add_backend(s1);
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- duplicate add with idents");
@@ -84,19 +89,19 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s2, ident="s1_1");
 		vd.add_backend(s2, ident="s1_2");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- remove s1_2 specifically");
 		vd.remove_backend(ident="s1_2");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- remove all instances of s1");
 		vd.remove_backend(s1);
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- re-add some - no 2nd director");
@@ -112,23 +117,22 @@ varnish v1 -vcl+backend {
 		vd.add_backend(s3, "8");
 		vd.add_backend(s3, "9");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- remove second-last");
 		vd.remove_backend(ident="8");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- remove last");
 		vd.remove_backend(ident="9");
 		if (!vd.reconfigure(replicas=1)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 
 		std.log("-- END");
-
 		set req.backend_hint = vd.backend();
 		return(pass);
 	}

bin/varnishtest/tests/d00017.vtc

@@ -31,16 +31,16 @@ varnish v1 -vcl+backend {
 		new vd = directors.shard();
 		vd.debug(3);
 		if (!vd.add_backend(s1)) {
-			std.log("add s1 failed");
+			return(fail("add s1 failed"));
 		}
 		if (!vd.add_backend(s2)) {
-			std.log("add s2 failed");
+			return(fail("add s2 failed"));
 		}
 		if (!vd.add_backend(s3)) {
-			std.log("add s3 failed");
+			return(fail("add s3 failed"));
 		}
 		if (!vd.reconfigure(replicas=25)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 	}
 

bin/varnishtest/tests/d00018.vtc

@@ -22,16 +22,16 @@ varnish v1 -vcl+backend {
 	sub vcl_init {
 		new vd = directors.shard();
 		if (!vd.add_backend(s1)) {
-			std.log("add s1 failed");
+			return(fail("add s1 failed"));
 		}
 		if (!vd.add_backend(s2)) {
-			std.log("add s2 failed");
+			return(fail("add s2 failed"));
 		}
 		if (!vd.add_backend(s3)) {
-			std.log("add s3 failed");
+			return(fail("add s3 failed"));
 		}
 		if (!vd.reconfigure(replicas=25)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 	}
 

bin/varnishtest/tests/d00021.vtc

@@ -22,32 +22,32 @@ varnish v1 -vcl+backend {
 	sub vcl_init {
 	    new vd = directors.shard();
 	    if (!vd.add_backend(s1)) {
-		std.log("add s1 failed");
+		return(fail("add s1 failed"));
 	    }
 	    if (!vd.add_backend(s2)) {
-		std.log("add s2 failed");
+		return(fail("add s2 failed"));
 	    }
 	    if (!vd.add_backend(s3)) {
-		std.log("add s3 failed");
+		return(fail("add s3 failed"));
 	    }
 	    if (!vd.reconfigure(replicas=25)) {
-		std.log("reconfigure failed");
+		return(fail("reconfigure failed"));
 	    }
 
 	    new l = directors.shard();
 	    new lp = directors.shard_param();
 	    l.associate(lp.use());
 	    if (!l.add_backend(s1)) {
-		std.log("add s1 failed");
+		return(fail("add s1 failed"));
 	    }
 	    if (!l.add_backend(s2)) {
-		std.log("add s2 failed");
+		return(fail("add s2 failed"));
 	    }
 	    if (!l.add_backend(s3)) {
-		std.log("add s3 failed");
+		return(fail("add s3 failed"));
 	    }
 	    if (!l.reconfigure(replicas=25)) {
-		std.log("reconfigure failed");
+		return(fail("reconfigure failed"));
 	    }
 	    new ll = directors.round_robin();
 	    ll.add_backend(l.backend());

bin/varnishtest/tests/d00029.vtc

@@ -22,16 +22,16 @@ varnish v1 -vcl+backend {
 	sub vcl_init {
 	    new vd = directors.shard();
 	    if (!vd.add_backend(s1)) {
-		std.log("add s1 failed");
+		return(fail("add s1 failed"));
 	    }
 	    if (!vd.add_backend(s2)) {
-		std.log("add s2 failed");
+		return(fail("add s2 failed"));
 	    }
 	    if (!vd.add_backend(s3)) {
-		std.log("add s3 failed");
+		return(fail("add s3 failed"));
 	    }
 	    if (!vd.reconfigure(replicas=25)) {
-		std.log("reconfigure failed");
+		return(fail("reconfigure failed"));
 	    }
 
 	    vd.debug(1);

bin/varnishtest/tests/d00041.vtc

@@ -24,16 +24,16 @@ varnish v1 -vcl+backend {
 		new vd = directors.shard();
 		vd.debug(3);
 		if (!vd.add_backend(s1)) {
-			std.log("add s1 failed");
+			return(fail("add s1 failed"));
 		}
 		if (!vd.add_backend(s2, weight=2)) {
-			std.log("add s2 failed");
+			return(fail("add s2 failed"));
 		}
 		if (!vd.add_backend(s3, weight=3)) {
-			std.log("add s3 failed");
+			return(fail("add s3 failed"));
 		}
 		if (!vd.reconfigure(replicas=25)) {
-			std.log("reconfigure failed");
+			return(fail("reconfigure failed"));
 		}
 	}