Github user paul-rogers commented on a diff in the pull request:
https://github.com/apache/drill/pull/984#discussion_r145294641
--- Diff: exec/java-exec/src/test/java/org/apache/drill/TestBugFixes.java
---
@@ -33,8 +35,11 @@
@Category(UnlikelyTest.class)
public class TestBugFixes extends BaseTestQuery {
private static final org.slf4j.Logger logger =
org.slf4j.LoggerFactory.getLogger(TestBugFixes.class);
- private static final String WORKING_PATH = TestTools.getWorkingPath();
- private static final String TEST_RES_PATH = WORKING_PATH +
"/src/test/resources";
+
+ @BeforeClass
+ public static void setupTestFiles() {
+ dirTestWatcher.copyResourceToRoot("/bugs/DRILL-4192");
--- End diff --
Small nit. Presumably `/bugs/DRILL-4192` is a relative path. Should we
express it in the normal Unix fashion as `bugs/DRILL-4192`? That way, in
time-honored fashion, a leading slash tells us the path is absolute.
---
