Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore(email-tests): 3986 email connector e2e tests are flaky #4051

Merged
merged 5 commits into from
Feb 21, 2025

Conversation

mathias-vandaele
Copy link
Collaborator

Description

All the e2e have been migrated to integration tests

Related issues

closes #3986

Checklist

  • PR has a milestone or the no milestone label.

@mathias-vandaele mathias-vandaele requested a review from a team as a code owner February 19, 2025 14:47
@mathias-vandaele mathias-vandaele linked an issue Feb 19, 2025 that may be closed by this pull request
@mathias-vandaele mathias-vandaele self-assigned this Feb 19, 2025
@mathias-vandaele mathias-vandaele added this to the 8.7.0-alpha5 milestone Feb 19, 2025
@mathias-vandaele mathias-vandaele force-pushed the 3986-email-connector-e2e-tests-are-flaky branch 4 times, most recently from 751155f to ef909d9 Compare February 20, 2025 15:40
@mathias-vandaele mathias-vandaele force-pushed the 3986-email-connector-e2e-tests-are-flaky branch from ef909d9 to 09ba824 Compare February 20, 2025 15:41
Copy link
Collaborator

@johnBgood johnBgood left a comment

Choose a reason for hiding this comment

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

Great but I would rename the package. These are not integration tests as we don't have any "real" mail server there. They look like regular UTs to me.
If you want integration tests, I recommend using docker-mailserver along with TestContainers. That would be great, but it's up to you 🇵🇭

@mathias-vandaele mathias-vandaele added this pull request to the merge queue Feb 21, 2025
Merged via the queue into main with commit 73f25a2 Feb 21, 2025
14 checks passed
@mathias-vandaele mathias-vandaele deleted the 3986-email-connector-e2e-tests-are-flaky branch February 21, 2025 12:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Email connector e2e tests are flaky
2 participants