diff options
author | Ludovic Courtès <ludo@gnu.org> | 2020-03-16 12:18:59 +0100 |
---|---|---|
committer | Ludovic Courtès <ludo@gnu.org> | 2020-03-16 14:05:52 +0100 |
commit | c215d9ec1ce108844b14c0c2952199a0da5f4176 (patch) | |
tree | ae5d749c150f1cc0a395621eb7e2a4347febbc6b /gnu/tests/mail.scm | |
parent | 0f13dd2b7f7675310e4a9bbfd37ee946b9f0dea3 (diff) | |
download | guix-c215d9ec1ce108844b14c0c2952199a0da5f4176.tar guix-c215d9ec1ce108844b14c0c2952199a0da5f4176.tar.gz |
tests: opensmtpd: Gracefully handle test failure.
Previously the 'wait' loop would run for ~1024 seconds, at which point
we'd reach the file descriptor limit due to the leak in 'queue-empty?'.
* gnu/tests/mail.scm (run-opensmtpd-test)[test]("mail arrived"): In
'queue-empty?', close PIPE to avoid file descriptor leak. In 'wait'
loop, arrange to run at most 20 times.
Diffstat (limited to 'gnu/tests/mail.scm')
-rw-r--r-- | gnu/tests/mail.scm | 25 |
1 files changed, 15 insertions, 10 deletions
diff --git a/gnu/tests/mail.scm b/gnu/tests/mail.scm index 298918b3a7..58172cd1d6 100644 --- a/gnu/tests/mail.scm +++ b/gnu/tests/mail.scm @@ -140,16 +140,21 @@ match from any for local action inbound (ice-9 rdelim)) (define (queue-empty?) - (eof-object? - (read-line - (open-input-pipe - (string-append #$(file-append opensmtpd "/sbin/smtpctl") - " show queue"))))) - - (let wait () - (if (queue-empty?) - (file-exists? "/var/mail/root") - (begin (sleep 1) (wait))))) + (let* ((pipe (open-pipe* OPEN_READ + #$(file-append opensmtpd + "/sbin/smtpctl") + "show" "queue")) + (line (read-line pipe))) + (close-pipe pipe) + (eof-object? line))) + + (let wait ((n 20)) + (cond ((queue-empty?) + (file-exists? "/var/mail/root")) + ((zero? n) + (error "root mailbox didn't show up")) + (else + (sleep 1) (wait (- n 1)))))) marionette)) (test-end) |