Mercurial > hg > nginx-tests
changeset 1982:fb25cbe9d4ec
Tests: explicit Valgrind support.
Valgrind logging is done to a separate file, as it is not able to
follow stderr redirection within nginx or append to a file without
corrupting it. Further, Valgrind logging seems to interfere with
error suppression in tests, and catches various startup errors and
warnings, so the log is additionally filtered.
Since startup under Valgrind can be really slow, timeout in waitforfile()
was changed to 10 seconds.
Prodded by Robert Mueller.
author | Maxim Dounin <mdounin@mdounin.ru> |
---|---|
date | Fri, 31 May 2024 06:23:00 +0300 |
parents | 0e2b14c75232 |
children | c7315caf2110 |
files | README lib/Test/Nginx.pm |
diffstat | 2 files changed, 19 insertions(+), 3 deletions(-) [+] |
line wrap: on
line diff
--- a/README Fri May 31 06:22:48 2024 +0300 +++ b/README Fri May 31 06:23:00 2024 +0300 @@ -40,6 +40,10 @@ Run unsafe tests. +TEST_NGINX_VALGRIND + + Run nginx under Valgrind during tests. + TEST_NGINX_GLOBALS Sets additional directives in main context.
--- a/lib/Test/Nginx.pm Fri May 31 06:22:48 2024 +0300 +++ b/lib/Test/Nginx.pm Fri May 31 06:23:00 2024 +0300 @@ -87,6 +87,12 @@ Test::More::is($errors, '', 'no sanitizer errors'); } + if (Test::More->builder->expected_tests && $ENV{TEST_NGINX_VALGRIND}) { + my $errors = $self->read_file('valgrind.log'); + $errors = join "\n", $errors =~ /^==\d+== .+/gm; + Test::More::is($errors, '', 'no valgrind errors'); + } + if ($ENV{TEST_NGINX_CATLOG}) { system("cat $self->{_testdir}/error.log"); } @@ -365,7 +371,10 @@ sub plan($) { my ($self, $plan) = @_; - Test::More::plan(tests => $plan + 2); + $plan += 2; + $plan += 1 if $ENV{TEST_NGINX_VALGRIND}; + + Test::More::plan(tests => $plan); return $self; } @@ -395,7 +404,10 @@ my @globals = $self->{_test_globals} ? () : ('-g', "pid $testdir/nginx.pid; " . "error_log $testdir/error.log debug;"); - exec($NGINX, '-p', "$testdir/", '-c', 'nginx.conf', + my @valgrind = (not $ENV{TEST_NGINX_VALGRIND}) ? + () : ('valgrind', '-q', + "--log-file=$testdir/valgrind.log"); + exec(@valgrind, $NGINX, '-p', "$testdir/", '-c', 'nginx.conf', '-e', 'error.log', @globals) or die "Unable to exec(): $!\n"; } @@ -481,7 +493,7 @@ # wait for file to appear # or specified process to exit - for (1 .. 50) { + for (1 .. 100) { return 1 if -e $file; return 0 if $exited; $exited = waitpid($pid, WNOHANG) != 0 if $pid;