Github user tillrohrmann commented on a diff in the pull request:
https://github.com/apache/flink/pull/6147#discussion_r195050299
--- Diff:
flink-runtime/src/test/java/org/apache/flink/runtime/jobgraph/JobGraphTest.java
---
@@ -18,16 +18,30 @@
package org.apache.flink.runtime.jobgraph;
-import static org.junit.Assert.*;
-
-import java.util.List;
-
import org.apache.flink.api.common.InvalidProgramException;
+import org.apache.flink.api.common.cache.DistributedCache;
import org.apache.flink.core.testutils.CommonTestUtils;
import org.apache.flink.runtime.io.network.partition.ResultPartitionType;
+
+import org.junit.ClassRule;
import org.junit.Test;
+import org.junit.rules.TemporaryFolder;
+
+import java.io.IOException;
+import java.nio.file.Files;
+import java.nio.file.Path;
+import java.util.List;
+import java.util.Map;
+
+import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
+import static org.junit.Assert.fail;
public class JobGraphTest {
--- End diff --
Could extend `TestLogger`
---