diff --git a/src/ooxml/testcases/org/apache/poi/openxml4j/opc/TestFileHelper.java b/src/ooxml/testcases/org/apache/poi/openxml4j/opc/TestFileHelper.java index 280d6d992..8829ca292 100755 --- a/src/ooxml/testcases/org/apache/poi/openxml4j/opc/TestFileHelper.java +++ b/src/ooxml/testcases/org/apache/poi/openxml4j/opc/TestFileHelper.java @@ -33,13 +33,23 @@ public class TestFileHelper extends TestCase { public void testGetDirectory() { TreeMap expectedValue = new TreeMap(); - expectedValue.put("c:\\test\\test.doc", "c:\\test"); - expectedValue.put("d:\\test\\test2\\test.doc.xml", "d:\\test\\test2"); + expectedValue.put("/dir1/test.doc", "/dir1"); + expectedValue.put("/dir1/dir2/test.doc.xml", "/dir1/dir2"); for (String filename : expectedValue.keySet()) { - assertTrue(expectedValue.get(filename).equalsIgnoreCase( - FileHelper.getDirectory(new File(filename)) - .getAbsolutePath())); + File f1 = new File(expectedValue.get(filename)); + File f2 = FileHelper.getDirectory(new File(filename)); + + /* + * YK: The original version asserted expected values against File#getAbsolutePath(): + * assertTrue(expectedValue.get(filename).equalsIgnoreCase( + * FileHelper.getDirectory(new File(filename)) + * .getAbsolutePath())); + * + * This comparison is platform dependent and resulted in build errors in Gump since 21/02/2009. + * Assertion via File#equals(File otherFile) is a better approach. + */ + assertTrue(f1.equals(f2)); } } }