-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
[FLINK-35490][cdc][JUnit5 Migration] For flink-cdc-connect/flink-cdc-source-connectors
module
#3395
base: master
Are you sure you want to change the base?
Conversation
ConnectionPoolId poolId = | ||
new ConnectionPoolId( | ||
HOSTNAME, | ||
PORT, | ||
USER_NAME, | ||
DATABASE, | ||
MockConnectionPoolFactory.class.getName()); | ||
Assert.assertThrows( | ||
assertThrowsExactly( |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wondering if we can use assertThatThrownBy#isExactlyInstanceOf
to replace assertThrowsExactly
, and replace JUnit assertions with AssertJ completely?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @yuxiqian! Good catch!
Indeed I was planning to migrate totally to assertj where possible, I will update this and check the rest of files 👍
https://issues.apache.org/jira/browse/FLINK-35490
Junit4 to Junit5 migration of
flink-cdc-connect/flink-cdc-source-connectors
module. Reference document: Junit5 Migration Guide.