[Java] Update Unit Tests to use org.junit.jupiter.api

[vibhatha]

Describe the enhancement requested

At the moment, test cases are using an older API version of Junit, and this should be upgraded to use the latest API.
It would be best if we can do this module per module by creating an issue per module and list it here.

• [Java] Update Unit Tests for Vector Module #42002
• [Java] Update Unit Tests for Algorithm Module #42025
• [Java] Update Unit Tests for Adapter Module #42030
• c
• [Java] Update Unit Tests for Compression Module #42042
• [Java] Update Unit Tests for Flight Module #42045
• [Java] Update Unit Tests for Dataset Module #42162
• [Java] Update Unit Tests for Gandiva Module #42164
• [Java] Update Unit Tests for Memory Module #42165
• performance
• [Java] Update Unit Tests for Tools Module  #42092
• [Java] Remove JUnit 4 Dependencies after JUnit 5 upgrade #42193

Component(s)

Java

[llama90]

@vibhatha I have verified the c module using only JUnit 5. Please cross-check when you have free time. Thank you.

[vibhatha]

@vibhatha I have verified the c module using only JUnit 5. Please cross-check when you have free time. Thank you.

@llama90 thank you for checking this one. I will verify. I will update tomorrow. Really appreciate your support on this upgrade.

[vibhatha]

@llama90 You are correct, we don't need to work on c module. It is already in JUNIT5. Thanks for verifying this.

[llama90]

@vibhatha I think I will start after merging the spotless changes for each module to avoid conflicts.

[vibhatha]

@llama90 that's good idea.

[llama90]

@vibhatha Additionally, the performance module does not include tests, so there is no need to assign tasks for it.

Please cross-check when you have free time. Thank you.

[vibhatha]

@llama90 Done! Thanks for looking into this.

[llama90]

@vibhatha I attached the remaining issues for updating the contents.

• [Java] Update Unit Tests for Flight Module #42045
• [Java] Update Unit Tests for Dataset Module #42162
• [Java] Update Unit Tests for Gandiva Module #42164
• [Java] Update Unit Tests for Memory Module #42165

[vibhatha]

@llama90 Thank you for updating it here. I will make sure to update the list. I am out today, will be a bit slow on reviews.

[llama90]

No problem! Have a nice day.

[lidavidm]

@vibhatha after this is done, we also need to remove all JUnit 4 dependencies and preferably exclude them so we don't accidentally re-take a dependency on JUnit 4

[vibhatha]

@lidavidm I will work on that. Definitely.

[vibhatha]

@lidavidm issue created: #42193

[llama90]

Finally, we've completed this issue. Thank you for your effort, @lidavidm and @vibhatha.

It was interesting and helped me learn a lot. If there are any other issues I can help solve, please let me know. I'm not yet familiar with the entire codebase in Java, but I'm getting the hang of it.

Thank you!

[lidavidm]

@llama90 thank you for finishing this off!

@vibhatha we can close now right?

[vibhatha]

Yes @lidavidm we can.

Thanks a lot @llama90 for pushing this till the end. 🙏