Skip to content

tests: Migrate Dovecot DBox tests (sdbox + mdbox)#3051

Merged
polarathene merged 4 commits intodocker-mailserver:masterfrom
polarathene:tests/migrate-dovecot-dbox
Feb 2, 2023
Merged

tests: Migrate Dovecot DBox tests (sdbox + mdbox)#3051
polarathene merged 4 commits intodocker-mailserver:masterfrom
polarathene:tests/migrate-dovecot-dbox

Conversation

@polarathene
Copy link
Copy Markdown
Member

Description

Two more tests migrated. These two have been failing more regularly, but should work more reliably now 👍

They test the Dovecot DBox mailbox format, which was contributed in Nov 2019 as mostly an ENV to make 1-2 changes.

Type of change

  • Improvement (non-breaking change that does improve existing functionality)

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
  • New and existing unit tests pass locally with my changes

@polarathene polarathene added service/dovecot area/tests kind/improvement Improve an existing feature, configuration file or the documentation labels Feb 1, 2023
@polarathene polarathene added this to the v12.0.0 milestone Feb 1, 2023
@polarathene polarathene self-assigned this Feb 1, 2023
Comment thread test/tests/parallel/set1/dovecot/mailbox_format_dbox.bats Outdated
Copy link
Copy Markdown
Member

@georglauterbach georglauterbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One question, one remark - rest looks good!

Comment thread test/tests/parallel/set1/dovecot/mailbox_format_dbox.bats
Comment thread test/helper/common.bash
@polarathene polarathene enabled auto-merge (squash) February 2, 2023 02:29
@polarathene polarathene merged commit ed6917f into docker-mailserver:master Feb 2, 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 service/dovecot

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants