Mercurial > hg > nginx-tests
comparison lib/Test/Nginx.pm @ 163:450206ed1fd2
Tests: move logging out of alarm.
Logging large amounts of data may be terminated by alarm() which isn't
really good (and likely to corrupt TAP output). Problem seen on limit_req.t
test with logging enabled.
| author | Maxim Dounin <mdounin@mdounin.ru> |
|---|---|
| date | Mon, 01 Aug 2011 21:23:49 +0400 |
| parents | 33f6ec774efa |
| children | f602276b9bb9 |
comparison
equal
deleted
inserted
replaced
| 162:465a711ffa19 | 163:450206ed1fd2 |
|---|---|
| 364 $s->print($request); | 364 $s->print($request); |
| 365 local $/; | 365 local $/; |
| 366 select undef, undef, undef, $extra{sleep} if $extra{sleep}; | 366 select undef, undef, undef, $extra{sleep} if $extra{sleep}; |
| 367 return '' if $extra{aborted}; | 367 return '' if $extra{aborted}; |
| 368 $reply = $s->getline(); | 368 $reply = $s->getline(); |
| 369 log_in($reply); | |
| 370 alarm(0); | 369 alarm(0); |
| 371 }; | 370 }; |
| 372 alarm(0); | 371 alarm(0); |
| 373 if ($@) { | 372 if ($@) { |
| 374 log_in("died: $@"); | 373 log_in("died: $@"); |
| 375 return undef; | 374 return undef; |
| 375 } else { | |
| 376 log_in($reply); | |
| 376 } | 377 } |
| 377 return $reply; | 378 return $reply; |
| 378 } | 379 } |
| 379 | 380 |
| 380 ############################################################################### | 381 ############################################################################### |