search

exercism / java

Exercism exercises in Java.
https://exercism.org/tracks/java
MIT License
688 stars 672 forks source link

Remove nested test classes from practice exercise `rest-api` #2539

Closed sanderploegsma closed 10 months ago

sanderploegsma commented 10 months ago

Due to a bug in the Java test runner the current tests for the rest-api exercise fail to run on Exercism. Since this bug is not easily fixed, the best workaround is to refactor the exercise tests into a single class containing all of the test methods, instead of using nested classes.

Contribution notes

As always, make sure to follow the Exercism contributing guidelines and the Java track contributing guidelines.

Don't hesitate to ask here in case you can't find answers to your questions!

Yordaniss commented 10 months ago

@sanderploegsma hey 🙂

Are the tests for rest-api not already in one single class RestApiTest? Or do you mean not to use static classes like UserManagementTest and move the tests under main class?

sanderploegsma commented 10 months ago

Or do you mean not to use static classes like UserManagementTest and move the tests under main class?

Yes, that is what I mean. Apologies if this wasn’t clear from the issue description. I updated it a bit, perhaps now it’s more clear?

Yordaniss commented 10 months ago

Or do you mean not to use static classes like UserManagementTest and move the tests under main class?

Yes, that is what I mean. Apologies if this wasn’t clear from the issue description. I updated it a bit, perhaps now it’s more clear?

Thanks for an answer. I would like to work than on this issue 🙂

sanderploegsma commented 10 months ago

Great, I assigned it to you!