Skip to content

Dependency management for JUnit 5 is incomplete #12635

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

Closed
wants to merge 1 commit into from
Closed

Dependency management for JUnit 5 is incomplete #12635

wants to merge 1 commit into from

Conversation

clemstoquart
Copy link
Contributor

Hello,

This PR add two new managed dependencies for Junit 5 :

  • junit-jupiter-params : which is one of the major Junit 5 feature
  • junit-vintage-engine : which allow a painless migration from Junit 4.x to Junit 5

Have a nice day.

@pivotal-issuemaster
Copy link

@clemstoquart Please sign the Contributor License Agreement!

Click here to manually synchronize the status of this Pull Request.

See the FAQ for frequently asked questions.

@pivotal-issuemaster
Copy link

@clemstoquart Thank you for signing the Contributor License Agreement!

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Mar 26, 2018
@wilkinsona wilkinsona added this to the 2.0.x milestone Mar 26, 2018
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Mar 26, 2018
@wilkinsona
Copy link
Member

Thanks for the PR, @clemstoquart.

When merging this, we should check that we haven't missed any of JUnit Jupiter's other modules.

@wilkinsona wilkinsona changed the title Add junit 5 params and vintage to managed dependencies Dependency management for Unit 5 is incomplete Mar 26, 2018
@wilkinsona wilkinsona changed the title Dependency management for Unit 5 is incomplete Dependency management for JUnit 5 is incomplete Mar 26, 2018
@snicoll
Copy link
Member

snicoll commented Mar 26, 2018

For the record, params was not added on purpose, see #10240 (comment)

@wilkinsona
Copy link
Member

I'd forgotten that. I can't recall if I agreed with the reasoning then, but I don't agree with it now. The module being experimental shouldn't cause us to make it harder than it needs to be to use the right version of it.

@philwebb philwebb modified the milestones: 2.0.x, 2.0.1 Apr 5, 2018
@philwebb philwebb self-assigned this Apr 5, 2018
@philwebb philwebb closed this in 276037c Apr 5, 2018
philwebb added a commit that referenced this pull request Apr 5, 2018
* pr/12635:
  Add missing managed JUnit Jupiter dependencies
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
type: bug A general bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants