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

Make Testcontainers extension annotation public to allow using in ordered ExtendWith #2262

Closed

Conversation

bedla
Copy link
Contributor

@bedla bedla commented Jan 19, 2020

Hi,
I found fix of #2045 useful and created PR. My usecase could be found in TestcontainersSharedContainerWithGeneratedResourcesTests. Problem is that I want to have static containers that use some generated resources and without @ExtendWith I am unable to tell order in which extensions are invoked.
What do you think?
Thx
Ivos

@bedla bedla force-pushed the issue-2045-public-extension branch from f3d2fbb to 220bf16 Compare February 23, 2020 11:01
@bedla bedla force-pushed the issue-2045-public-extension branch 2 times, most recently from daba628 to a001926 Compare April 20, 2020 18:06
@bedla bedla force-pushed the issue-2045-public-extension branch from a001926 to 5c3ef67 Compare April 21, 2020 06:41
@stale
Copy link

stale bot commented Jul 20, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. If you believe this is a mistake, please reply to this comment to keep it open. If there isn't one already, a PR to fix or at least reproduce the problem in a test case will always help us get back on track to tackle this.

@stale stale bot added the stale label Jul 20, 2020
@stale
Copy link

stale bot commented Aug 3, 2020

This issue has been automatically closed due to inactivity. We apologise if this is still an active problem for you, and would ask you to re-open the issue if this is the case.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant