spring-cloud / spring-cloud-dataflow

A microservices-based Streaming and Batch data processing in Cloud Foundry and Kubernetes
https://dataflow.spring.io
Apache License 2.0
1.1k stars 578 forks source link

Refactor junit5 and assertj #4436

Open jvalkeal opened 3 years ago

jvalkeal commented 3 years ago

This issue tracks a progress of moving fully on junit5 and assertj.

Devamparikh commented 2 years ago

Hey there, I have read some issues and discussions, you guys are planning to migrate Junit4 to Junit5, Can outsiders work on this issue? I want to help in migrating Junit4 to Junit5. Can I get some issues to work on? Thanks :)

onobc commented 2 years ago

Hi @Devamparikh ,

Yes, you can work on this. There is a PR to do this but it was done back in July 2021 so I am not sure how up-to-date it is. I would start by scanning that and seeing if we can use it w/ minor modifications (if needed).

Here are some tips/points that may help you:

Ping me if you have any questions.

Thanks

Devamparikh commented 2 years ago

Sure, let me first review where I can fit and I'll let you know. :)

Devamparikh commented 2 years ago

Hi @onobc and all, sorry to bother you, I'm curious to learn more about JUnit4 to JUnit5 migration. I have read and consulted official documents, since you are currently working on them at the moment, I would like you to give me a few tips for eg. "How you intended to migrate junit4 to junit5?" and "What challenges did you face and how did you overcome them?" Thanks in advance :)

onobc commented 2 years ago

Hi @Devamparikh , I have migrated other applications from Junit4->Junit5 and it is pretty straightforward. There are many resources out there that give all the details (eg Baeldung).

For this work, I would first see what the current PR is lacking and update it accordingly. Then be sure to read through my comment above.

Good luck - let me know if you have any specific questions I can answer. Thanks.