diff options
author | Matt Caswell <matt@openssl.org> | 2016-10-14 15:55:49 +0100 |
---|---|---|
committer | Matt Caswell <matt@openssl.org> | 2016-11-04 10:38:54 +0000 |
commit | 9e663b13714f36a2069cfddf493a5d27985e26a0 (patch) | |
tree | 751d79a76196b85a728755b2c46665e291fd1f41 /test | |
parent | d5df08afb43d66c26a512093c22a70d3a079dc3f (diff) |
Add a comment to 90-test_external.t to explain why we need filter_run
Also rename executable to cmd...otherwise it breaks!
Reviewed-by: Richard Levitte <levitte@openssl.org>
Diffstat (limited to 'test')
-rw-r--r-- | test/recipes/90-test_external.t | 15 |
1 files changed, 9 insertions, 6 deletions
diff --git a/test/recipes/90-test_external.t b/test/recipes/90-test_external.t index d08d66faad..49e2bd2005 100644 --- a/test/recipes/90-test_external.t +++ b/test/recipes/90-test_external.t @@ -20,14 +20,17 @@ if (!$ENV{BORING_RUNNER_DIR}) { plan tests => 1; indir $ENV{BORING_RUNNER_DIR} => sub { - ok(filter_run(executable(["go", "test", "-shim-path", - srctop_file("test", "ossl_shim", "ossl_shim"), - "-shim-config", - srctop_file("test", "ossl_shim", - "ossl_config.json"), - "-pipe"])), + ok(filter_run(cmd(["go", "test", "-shim-path", + srctop_file("test", "ossl_shim", "ossl_shim"), + "-shim-config", + srctop_file("test", "ossl_shim", "ossl_config.json"), + "-pipe"])), + "running external tests"); }, create => 0, cleanup => 0; +# Filter the output so that the "ok" printed by go test doesn't confuse +# Test::More. Without that it thinks there has been one more test run than was +# planned sub filter_run { my $cmd = cmdstr(shift); open(PIPE, "-|", $cmd); |