Build broken on Windows: unit test failure in imagemosaic-jdbc

Christian Mueller added a comment - 02/Mar/09 11:59 PM Hi Ben, I attached a patch file. Since I have no possibility to test on a windows environment this week, can you please apply the patch and test it. If the patch does not work, I will remove the test and dig deeper in the next week thanks

Christian Mueller added a comment - 03/Mar/09 12:00 AM Attaching the patch

Ben Caradoc-Davies added a comment - 03/Mar/09 12:13 AM Christian, this patch will not work because OUTPUTDIR_RESOURCES already contains backslashes, so string comparison is doomed to fail. The first failure occurs at AbstractTest line 412, and your patch does not touch this line or any before it. I thought about converting the URLs to URIs and then to Files to use File.equals(), but the URLs have authority and a relative path so are invalid.

Ben Caradoc-Davies added a comment - 03/Mar/09 12:18 AM And making the right hand side paths in assertTrue(importParamList.get(0).getSourceURL().getPath().contains(OUTPUTDIR_RESOURCES)); into a File and then a URL makes them absolute, while the left hand side are relative.

Christian Mueller added a comment - 03/Mar/09 1:14 AM Ben, what exactly is the problem at line 412. OUPUTDIR_RESOURCES is "target/resources/" on Linux and should be "target\resources\" on Windows The getPath method of the URL class relies on File.seperator (according to java doc) importParamList.get(0).getSourceURL().getPath() is "target/resources/" on linux and should be "target\resources\" on Windows. What Strings do you get for 1) OUPUTDIR_RESOURCES 2) importParamList.get(0).getSourceURL().getPath() on line 412

Ben Caradoc-Davies added a comment - 03/Mar/09 1:26 AM Christian, I have attached a patch that fixes the issue. On Windows, URL.getPath() returns a path with forward slashes, because it is the path component of a URL, not a filesystem path. As a consequence, on Windows the URL path cannot match the file path. The attached patch fixes this problem by creating two File objects and comparing their canonical paths to test if they are the same. It works on Windows. The patch adds a helper method, a unit test for the helper, and enables the test in H2Test.

Ben Caradoc-Davies added a comment - 03/Mar/09 1:42 AM And for ease of reading, I did not wrap the lines. Feel free to reformat the changes bits to conform to the GeoTools convention.

Christian Mueller added a comment - 03/Mar/09 2:16 AM Thanks Ben, applied your patch and it is working on linux too.