Github user vrozov commented on a diff in the pull request:

    https://github.com/apache/drill/pull/1225#discussion_r182902184
  
    --- Diff: 
exec/java-exec/src/test/java/org/apache/drill/exec/expr/fn/FunctionInitializerTest.java
 ---
    @@ -53,17 +56,27 @@
     @Category(SqlFunctionTest.class)
     public class FunctionInitializerTest {
     
    -  private static final String CLASS_NAME = 
"com.drill.udf.CustomLowerFunction";
    +  private static final String CLASS_NAME = 
"org.apache.drill.udf.dynamic.CustomLowerFunction";
       private static URLClassLoader classLoader;
     
    +  @ClassRule
    +  public static TemporaryFolder temporaryFolder = new TemporaryFolder();
    +
       @BeforeClass
       public static void init() throws Exception {
    -    Path jars = TestTools.WORKING_PATH
    -      .resolve(TestTools.TEST_RESOURCES)
    -      .resolve("jars");
    -    String binaryName = "DrillUDF-1.0.jar";
    -    String sourceName = JarUtil.getSourceName(binaryName);
    -    URL[] urls = {jars.resolve(binaryName).toUri().toURL(), 
jars.resolve(sourceName).toUri().toURL()};
    +    String binaryName = "DrillUDF-1.0";
    +    URL template = 
ClassLoader.getSystemClassLoader().getResource("udf/dynamic/CustomLowerFunctionTemplate");
    +    assert template != null;
    --- End diff --
    
    use junit assert in unit tests.


---

Reply via email to