Skip to content

tests: Use mail.example.test as common container hostname#2975

Merged
georglauterbach merged 3 commits intodocker-mailserver:masterfrom
polarathene:tests/use-test-tld-hostname
Jan 4, 2023
Merged

tests: Use mail.example.test as common container hostname#2975
georglauterbach merged 3 commits intodocker-mailserver:masterfrom
polarathene:tests/use-test-tld-hostname

Conversation

@polarathene
Copy link
Copy Markdown
Member

Description

  • Minor change to the common container hostname (presently only affects parallel set tests, serial has been left alone). This is preferred going forward as it's a private TLD that won't make external DNS lookups.
  • Had an unrelated recurring test failure on a weak internet connection, bumped the sleep to match what is used in previous test cases.

All tests converted to the new helper here should be adapted to be compatible with `mail.example.test` hostname.
On slower internet connections for local testing, 10 seconds of sleep is too short to install the packages and connect to the SMTP port.

Increased to 100 seconds to match container sleep time used in previous test cases.
@polarathene polarathene added area/tests kind/improvement Improve an existing feature, configuration file or the documentation labels Jan 4, 2023
@polarathene polarathene added this to the v12.0.0 milestone Jan 4, 2023
@polarathene polarathene self-assigned this Jan 4, 2023
@georglauterbach georglauterbach merged commit 304747f into docker-mailserver:master Jan 4, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/tests kind/improvement Improve an existing feature, configuration file or the documentation priority/low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants