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

feat(junit5): adds support for Suites #291

Merged
merged 6 commits into from
Dec 1, 2020

Conversation

ingwarsw
Copy link
Contributor

Short description of what this resolves:

This PR adds support for Suites in junit5 (in same way at it was working in junit4)
So it fixes Arquillian Suite Extension for junit5.

Changes proposed in this pull request:

Add suite finalizer based on auto closable objects on jupiter contexts.

Fixes ingwarsw/arquillian-suite-extension#54

@lprimak
Copy link
Contributor

lprimak commented Nov 30, 2020

@bartoszmajsak @Thihup Please merge this. It works perfectly and solves my issue!

@ingwarsw
Copy link
Contributor Author

ingwarsw commented Dec 1, 2020

@bartoszmajsak @Thihup Actually half of code in junit5 plugin can be removed (it was copy/pasted from junit4 and never used)..
I can try cleaning it in other PR..

@bartoszmajsak
Copy link
Member

@bartoszmajsak @Thihup Actually half of code in junit5 plugin can be removed (it was copy/pasted from junit4 and never used)..
I can try cleaning it in other PR..

Awesome, many thanks. Also for all your great work over the years! (been using suite extension heavily in my previous projects).

@bartoszmajsak bartoszmajsak changed the title Add support for Suites in junit5 feat(junit5): adds support for Suites Dec 1, 2020
@bartoszmajsak bartoszmajsak merged commit 5fdf890 into arquillian:master Dec 1, 2020
@bartoszmajsak
Copy link
Member

@ingwarsw I've just released 1.7.0.Alpha6 with this patch. Waiting for more :)

@ingwarsw
Copy link
Contributor Author

ingwarsw commented Dec 7, 2020

@bartoszmajsak Thanks..

I will probably find some spare time in next week.. that one is busy..

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.

Using surefire or failsafe causes multiple deployments
3 participants