[master] f19495802 vtc: Simplify r03241 and prune confusing comment

Dridi Boukelmoune dridi.boukelmoune at gmail.com
Tue Mar 5 06:28:07 UTC 2024


commit f19495802644ea8b181eb70f4bedf3f052b022fc
Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
Date:   Fri Mar 1 18:17:50 2024 +0100

    vtc: Simplify r03241 and prune confusing comment

diff --git a/bin/varnishtest/tests/r03241.vtc b/bin/varnishtest/tests/r03241.vtc
index ac7fab446..3dc19ad01 100644
--- a/bin/varnishtest/tests/r03241.vtc
+++ b/bin/varnishtest/tests/r03241.vtc
@@ -1,24 +1,16 @@
 varnishtest "ESI include out of workspace"
 
-
 server s1 {
 	rxreq
 	expect req.http.esi0 == "foo"
-	txresp -body {
-		<html>
-		Before include
-		<esi:include src="/body" sr="foo"/>
-		After include
-		</html>
-	}
+	txresp -body {before <esi:include src="/body" sr="foo"/> after}
 	rxreq
 	expect req.url == "/body1"
 	expect req.http.esi0 != "foo"
-	txresp -body {
-		Included file
-	}
+	txresp -body "include"
 } -start
 
+varnish v1 -cliok "param.set feature +esi_disable_xml_check"
 varnish v1 -vcl+backend {
 	import vtc;
 
@@ -48,9 +40,8 @@ logexpect l1 -v v1 -g raw {
 client c1 {
 	txreq -hdr "Host: foo"
 	rxresp
-	# XXX this is actually wrong (missed include)
-	expect resp.bodylen == 57
 	expect resp.status == 200
+	expect resp.body == "before  after"
 }  -run
 
 logexpect l1 -wait


More information about the varnish-commit mailing list