mirror of
https://github.com/docker-mailserver/docker-mailserver.git
synced 2024-01-19 02:48:50 +00:00
apply feedback from @polarathene
ref: https://github.com/docker-mailserver/docker-mailserver/pull/3403#discussion_r1453332402
This commit is contained in:
parent
e1fd02c1f7
commit
68a67c95e2
|
@ -60,10 +60,6 @@ function teardown_file() { _default_teardown ; }
|
|||
'action=greylist' \
|
||||
'reason=new' \
|
||||
'client_address=127.0.0.1/32, sender=user@external.tld, recipient=user1@localhost.localdomain'
|
||||
|
||||
_repeat_until_success_or_timeout 10 _run_in_container grep \
|
||||
'Recipient address rejected: Delayed by Postgrey' \
|
||||
/var/log/mail/mail.log
|
||||
}
|
||||
|
||||
# NOTE: This test case depends on the previous one
|
||||
|
@ -77,7 +73,7 @@ function teardown_file() { _default_teardown ; }
|
|||
_should_have_log_entry \
|
||||
'action=pass' \
|
||||
'reason=triplet found' \
|
||||
'client_address=127.0.0.1, sender=user@external.tld, recipient=user1@localhost.localdomain'
|
||||
'client_address=127.0.0.1/32, sender=user@external.tld, recipient=user1@localhost.localdomain'
|
||||
}
|
||||
|
||||
# NOTE: These two whitelist tests use `files/nc/` instead of `files/emails`.
|
||||
|
@ -95,7 +91,7 @@ function teardown_file() { _default_teardown ; }
|
|||
_should_have_log_entry \
|
||||
'action=pass' \
|
||||
'reason=client whitelist' \
|
||||
'client_address=127.0.0.1, sender=test@whitelist.tld, recipient=user1@localhost.localdomain'
|
||||
'client_address=127.0.0.1/32, sender=test@whitelist.tld, recipient=user1@localhost.localdomain'
|
||||
}
|
||||
|
||||
@test "should whitelist recipient 'user2@otherdomain.tld'" {
|
||||
|
@ -104,7 +100,7 @@ function teardown_file() { _default_teardown ; }
|
|||
_should_have_log_entry \
|
||||
'action=pass' \
|
||||
'reason=recipient whitelist' \
|
||||
'client_address=127.0.0.1, sender=test@nonwhitelist.tld, recipient=user2@otherdomain.tld'
|
||||
'client_address=127.0.0.1/32, sender=test@nonwhitelist.tld, recipient=user2@otherdomain.tld'
|
||||
}
|
||||
|
||||
function _should_have_log_entry() {
|
||||
|
|
Loading…
Reference in a new issue