Skip to content

[TODO]: Tests should share a more thorough helper method for mail delivery checks #3554

@polarathene

Description

@polarathene

Description

Up for grabs for anyone who'd like to contribute, no rush :)


When reviewing a PR that introduced widespread mail delivery failures by changing UID/GID (as Dovecot no longer had permission to write the mail to /var/mail), I suggested a test-case that would better catch that common failure (technically two, as checking the mail logs for failures would be complimentary at catching another set of errors).


It would be nice to eventually do some cleanup across tests to have a common helper method for consistently checking successful mail delivery.

Failures were all shared here, which should help track down which tests need the refactoring. I also identified some common keywords in the failure logs that could be checked. An earlier comment provided more verbosity in logs (for additional context)

Metadata

Metadata

Assignees

No one assigned

    Labels

    area/testskind/improvementImprove an existing feature, configuration file or the documentationmeta/help wantedThe OP requests help from others - chime in! :Dservice/dovecotservice/postfixstale-bot/ignoreIndicates that this issue / PR shall not be closed by our stale-checking CI

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions