dawidwys commented on a change in pull request #11969:
URL: https://github.com/apache/flink/pull/11969#discussion_r419242981
##########
File path: flink-core/src/test/java/org/apache/flink/util/StringUtilsTest.java
##########
@@ -39,21 +40,26 @@ public void testControlCharacters() {
}
@Test
- public void testArrayOfBytesArray() {
- byte[][] expectedArray = new byte[][]{
- {1, -97, 49, 74 },
- {2, -92, 48, 73 }
- };
+ public void testArrayAwareToString() {
Review comment:
nit: I very much prefer to have a single case in a single `Test` method.
It makes it much easier to track back the failing case. Moreover a subsequent
assertions/cases are run even if an earlier one fails.
----------------------------------------------------------------
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.
For queries about this service, please contact Infrastructure at:
[email protected]
