You are viewing a plain text version of this content. The canonical link for it is here.

Posted to issues@flink.apache.org by GitBox <gi...@apache.org> on 2022/06/03 13:44:38 UTC

[GitHub] [flink] alpreu commented on pull request #19807: [FLINK-27185][connectors] Convert connector-jdbc module to assertj

alpreu commented on PR #19807:
URL: https://github.com/apache/flink/pull/19807#issuecomment-1145981118

   > @alpreu Sorry for replying that late, but what about also migrating from junit4 to junit5 as part of that effort? Since we decided to go for assertj/junit5 in general? What's the motivation of moving to assertj only?
   
   The migration to assertJ was mainly done using the script written by @slinkydeveloper (https://github.com/assertj/assertj-migrator). We did have to manually touch some files though because not all assertions worked out of the box and some had better alternatives. I believe there is another umbrella ticket tracking the Junit5 migration


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: issues-unsubscribe@flink.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org