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

[PROPOSAL] migrate to JUnit 5 #758

Open
rursprung opened this issue Dec 4, 2023 · 2 comments
Open

[PROPOSAL] migrate to JUnit 5 #758

rursprung opened this issue Dec 4, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@rursprung
Copy link
Contributor

What/Why

What are you proposing?

the project should (eventually) migrate to JUnit 5 so that the newer features & patterns can be used and so that it isn't stuck on an old major release of JUnit.

What users have asked for this feature?

n/a

What problems are you trying to solve?

avoid getting stuck on an old library.

What is the developer experience going to be?

  • no changes for users of opensearch-java
  • more modern experience for developers of opensearch-java

Are there any security considerations?

no

Are there any breaking changes to the API

no (only affects the tests)

What is the user experience going to be?

n/a

Are there breaking changes to the User Experience?

no

Why should it be built? Any reason not to?

long-term stability by being on the current major release of the testing library and improved developer experience

What will it take to execute?

manual effort to migrate it (maybe part of it can be automated?).

Any remaining open questions?

n/a

@reta
Copy link
Collaborator

reta commented Dec 4, 2023

@rursprung we use the OpenSearch -> Apache Lucene -> Carrotsearch (https://labs.carrotsearch.com/randomizedtesting.html) scaffolding, in theory we could use JUnit 4 and JUnit 5 together (would be messy), but without updating the whole chain of dependencies, it would not be possible to use JUnit 5 only.

@rursprung
Copy link
Contributor Author

hm, xkcd at work, huh?

thanks for your insights!

it feels like that library is dead in the water, nearly nothing happening and the issue for JUnit 5 has been open since 2018: randomizedtesting/randomizedtesting#256

i also saw a related ticket in the OpenSearch repo: opensearch-project/sql#1974 (sadly that one is proposing to kick out JUnit 5 tests and revert them back to JUnit 4 😞)

@dblock dblock added enhancement New feature or request and removed untriaged labels Dec 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants