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

Move all tests to JUnit 5 #507

Open
gavlyukovskiy opened this issue Jul 26, 2020 · 3 comments
Open

Move all tests to JUnit 5 #507

gavlyukovskiy opened this issue Jul 26, 2020 · 3 comments

Comments

@gavlyukovskiy
Copy link
Member

It would be nice to move all tests to JUnit 5 and maybe we can also use testcontainers to start containers on demand instead of using docker-compose on Travis.

@rjo67
Copy link

rjo67 commented Apr 17, 2021

Hi, I'll take this one, if that's ok. (junit5, not the containers)

@typekpb
Copy link
Member

typekpb commented Apr 17, 2021 via email

@rjo67
Copy link

rjo67 commented Apr 18, 2021

ok, I'll do an initial PR to migrate the majority of the tests.
Is there a restriction on using java 1.7? To migrate P6TestOptionsRepository I'd like to use lambdas (assertThrows).

The parameterised tests are not really going to be able to be migrated in their current form, afaik. It would be certainly rather ugly, at least -- the JUnit5-Parameterized is just too different. The test-setup in the base-class would have to be completely different. Therefore I think it would be just as good to leave those tests on JUnit-4 for the time being.

rjo67 added a commit to rjo67/p6spy that referenced this issue Apr 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants