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

[#35] Migrate project to start using testcontainers #45

Merged
merged 4 commits into from
Jan 4, 2021
Merged

Conversation

smcvb
Copy link
Member

@smcvb smcvb commented Dec 29, 2020

This pull request removes the usages of "flap doodle" as the means to run embedded Mongo in favour of using Test Containers.
As such, this PR changes the dependencies and the five tests (Saga, TokenStore and 3 EventStorageEngine tests) to use Mongo Test Containers.

These five tests used a mix of a custom MongoLauncher and Spring to wire everything.
Lastly, the MongoLauncher (which used "flap doodle") and the MongoTestContext were no longer used through these changes and thus have been removed.

This PR resolves #35

Replace flapdoodle dependency in favor of Mongo Test Containers
dependencies

#35
Remove the Spring / FlapDoodle mix in favor of a Mongo Test Container

#35
Remove MongoLauncher and MongoTestContext as both have been replaced in
favor of test containers

#35
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

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

LGTM

Introduce a MongoTemplateFactory to reuse the functionality to construct
 a MongoTemplate based on a given host/port combination during testing.

#35
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

Copy link
Contributor

@lfgcampos lfgcampos left a comment

Choose a reason for hiding this comment

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

Very nice and lots of lines removed!! 👏

@smcvb smcvb merged commit 2326f8f into master Jan 4, 2021
@smcvb smcvb deleted the feature/35 branch January 4, 2021 09:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Migrate project to start using testcontainers
2 participants