-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
Not extend Junit Assert classes in Tests. #1316 #1331
Conversation
@@ -110,7 +115,10 @@ public void multiBulkReply() { | |||
expected.add(SafeEncoder.encode("Hello")); | |||
expected.add(SafeEncoder.encode("World")); | |||
|
|||
assertEquals(expected, response); | |||
assertEquals(expected.size(), response.size()); |
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.
We can apply assertByteArrayListEquals this case, right?
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.
Yes. Thats right. But I want to use it without extending. Let me look into it.
Thanks for the work. LGTM. Extending Assert was old fashion and this pull request corrects it. @marcosnils Please review and merge. Thanks! |
👍 LGTM!. thx for the contribution @smadasu |
* Not extend Junit Assert classes in Tests. #1316 Conflicts: src/test/java/redis/clients/jedis/tests/ModuleTest.java src/test/java/redis/clients/jedis/tests/PipeliningTest.java src/test/java/redis/clients/jedis/tests/commands/AllKindOfValuesCommandsTest.java src/test/java/redis/clients/jedis/tests/commands/BinaryValuesCommandsTest.java src/test/java/redis/clients/jedis/tests/commands/HashesCommandsTest.java src/test/java/redis/clients/jedis/tests/commands/ScriptingCommandsTest.java src/test/java/redis/clients/jedis/tests/commands/SetCommandsTest.java
* Not extend Junit Assert classes in Tests. #1316 Conflicts: src/test/java/redis/clients/jedis/tests/ModuleTest.java src/test/java/redis/clients/jedis/tests/PipeliningTest.java src/test/java/redis/clients/jedis/tests/commands/AllKindOfValuesCommandsTest.java src/test/java/redis/clients/jedis/tests/commands/BinaryValuesCommandsTest.java src/test/java/redis/clients/jedis/tests/commands/HashesCommandsTest.java src/test/java/redis/clients/jedis/tests/commands/ScriptingCommandsTest.java src/test/java/redis/clients/jedis/tests/commands/SetCommandsTest.java
Applied to 2.8 and 2.9 respectively. Thx again @smadasu |
Fantastic! Glad I could help :) |
I have cleaned up all the Junit tests extending from Assert class. Please review