[master] 06108a1 Test what logexpect can't

Federico Schwindt fgsch at lodoss.net
Sat Feb 4 00:42:06 CET 2017


Why not simply:

shell -match "^$" {
       varnishlog -n ${v1_name} -d -g raw -I "Should not happen"
}

?

Same with m00027.vtc

On Fri, Feb 3, 2017 at 4:39 PM, Dridi Boukelmoune <
dridi.boukelmoune at gmail.com> wrote:

>
> commit 06108a1930009b7186f776ba279a81d3ad3399e3
> Author: Dridi Boukelmoune <dridi.boukelmoune at gmail.com>
> Date:   Fri Feb 3 15:36:49 2017 +0100
>
>     Test what logexpect can't
>
>     We also don't want the test to pass if varnishlog failed.
>
> diff --git a/bin/varnishtest/tests/m00022.vtc b/bin/varnishtest/tests/
> m00022.vtc
> index a9863d6..11ba4cf 100644
> --- a/bin/varnishtest/tests/m00022.vtc
> +++ b/bin/varnishtest/tests/m00022.vtc
> @@ -34,6 +34,12 @@ varnish v1 -errvcl "Planned failure in vcl_init" {
>
>  logexpect l1 -wait
>
> +shell {
> +       set -e
> +       varnishlog -n ${v1_name} -d -g raw >v1.log
> +       ! grep "Should not happen" v1.log
> +}
> +
>  varnish v1 -cliok "param.set nuke_limit 42"
>
>  varnish v1 -errvcl "nuke_limit is not the answer." {
>
> _______________________________________________
> varnish-commit mailing list
> varnish-commit at varnish-cache.org
> https://www.varnish-cache.org/lists/mailman/listinfo/varnish-commit
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://www.varnish-cache.org/lists/pipermail/varnish-commit/attachments/20170203/5eb40b9e/attachment.html>


More information about the varnish-commit mailing list