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

using testcontaines in dev mode #1101

Merged
merged 4 commits into from
Jun 17, 2023
Merged

Conversation

rajadilipkolli
Copy link
Contributor

No description provided.

@hantsy
Copy link
Owner

hantsy commented Jun 15, 2023

@rajadilipkolli Thanks for sending this PR to demonstrate the new features. I also tried them in my spring6-sandbox, I am still confused in some config, spring-projects/spring-boot#35824

For this PR, can you do a simple refactoring on it, keep the original Repository tests there, and add a new RepositoryTest for the new feature.

@ActiveProfiles("test")
@Import(TestDemoApplication.class)
Copy link
Owner

Choose a reason for hiding this comment

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

We can specify the class attribute of @SpringBootTest instead of @Import here.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

@hantsy hantsy merged commit 1656a30 into hantsy:master Jun 17, 2023
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.

2 participants