This is an automated email from the ASF dual-hosted git repository. azagrebin pushed a commit to branch flip116 in repository https://gitbox.apache.org/repos/asf/flink.git
commit c4ffce9d4485fc0f12797e15ab4c61583883d55c Author: Yangze Guo <karma...@gmail.com> AuthorDate: Thu Apr 16 17:11:16 2020 +0800 [hotfix] Fix the code style in BashJavaUtilsITCase --- .../src/test/java/org/apache/flink/dist/BashJavaUtilsITCase.java | 9 +++++---- 1 file changed, 5 insertions(+), 4 deletions(-) diff --git a/flink-dist/src/test/java/org/apache/flink/dist/BashJavaUtilsITCase.java b/flink-dist/src/test/java/org/apache/flink/dist/BashJavaUtilsITCase.java index 1281ed2..e0587f7 100644 --- a/flink-dist/src/test/java/org/apache/flink/dist/BashJavaUtilsITCase.java +++ b/flink-dist/src/test/java/org/apache/flink/dist/BashJavaUtilsITCase.java @@ -26,7 +26,8 @@ import org.junit.Test; import java.util.Arrays; import java.util.List; -import static org.junit.Assert.assertEquals; +import static org.hamcrest.core.Is.is; +import static org.junit.Assert.assertThat; /** * Tests for BashJavaUtils. @@ -42,8 +43,8 @@ public class BashJavaUtilsITCase extends JavaBashTestBase { String[] commands = {RUN_BASH_JAVA_UTILS_CMD_SCRIPT, BashJavaUtils.Command.GET_TM_RESOURCE_PARAMS.toString()}; List<String> lines = Arrays.asList(executeScript(commands).split(System.lineSeparator())); - assertEquals(2, lines.size()); - ConfigurationUtils.parseJvmArgString(lines.get(lines.size() - 2)); - ConfigurationUtils.parseTmResourceDynamicConfigs(lines.get(lines.size() - 1)); + assertThat(lines.size(), is(2)); + ConfigurationUtils.parseJvmArgString(lines.get(0)); + ConfigurationUtils.parseTmResourceDynamicConfigs(lines.get(1)); } }