Skip to content

tests: fix dovecot: ldap mail delivery works#3252

Merged
georglauterbach merged 2 commits intodocker-mailserver:masterfrom
ap-wtioit:master-fix_tests_github
Apr 11, 2023
Merged

tests: fix dovecot: ldap mail delivery works#3252
georglauterbach merged 2 commits intodocker-mailserver:masterfrom
ap-wtioit:master-fix_tests_github

Conversation

@ap-wtioit
Copy link
Copy Markdown
Contributor

@ap-wtioit ap-wtioit commented Apr 11, 2023

Description

When a new version of docker-mailserver is available the account used in this tests also gets the postmaster notification for the new version. The mailbox then may contain 2 mails but only one with 'This is a test mail.'.

Fixes #3251

Type of change

  • Bug fix (non-breaking change which fixes an issue)

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation (README.md or the documentation under docs/)
  • If necessary I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes

When a new version of docker-mailserver is available the account used in this
tests also gets the postmaster notification for the new version. The mailbox
then may contain 2 mails but only one with 'This is a test mail.'.
@polarathene polarathene added area/tests kind/bug/fix A fix (PR) for a confirmed bug labels Apr 11, 2023
@polarathene polarathene added this to the v12.1.0 milestone Apr 11, 2023
@georglauterbach
Copy link
Copy Markdown
Member

Since this will be refactored anyway, it should be fine 👍🏼 Thank's for the fix ❤️

@georglauterbach georglauterbach merged commit de19c6b into docker-mailserver:master Apr 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/tests kind/bug/fix A fix (PR) for a confirmed bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

dovecot: ldap mail delivery works, breaks when newer version of dms is available

3 participants