]> source.dussan.org Git - jgit.git/commitdiff
Fix FileUtils.testRelativize_mixedCase which failed on Mac OS X 37/40937/2
authorMatthias Sohn <matthias.sohn@sap.com>
Mon, 2 Feb 2015 20:21:09 +0000 (21:21 +0100)
committerMatthias Sohn <matthias.sohn@sap.com>
Mon, 2 Feb 2015 20:23:31 +0000 (21:23 +0100)
HFS is case insensitive hence expecting it to return the result for case
sensitive filesystem doesn't work.

Change-Id: I292eab78e50711529a0412f9a54e174a3ac16109
Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
org.eclipse.jgit.test/tst/org/eclipse/jgit/util/FileUtilTest.java

index d4be25c0cdef24d191c4fdcc608ff13600cdeddf..0d7d31b3ad6dc2741f159418f0434c2701f5b16f 100644 (file)
@@ -450,31 +450,20 @@ public class FileUtilTest {
        @Test
        public void testRelativize_mixedCase() {
                SystemReader systemReader = SystemReader.getInstance();
-               String oldOSName = null;
                String base = toOSPathString("C:\\git\\jgit");
                String other = toOSPathString("C:\\Git\\test\\d\\f.txt");
-               String expectedWindows = toOSPathString("..\\test\\d\\f.txt");
-               String expectedUnix = toOSPathString("..\\..\\Git\\test\\d\\f.txt");
+               String expectedCaseInsensitive = toOSPathString("..\\test\\d\\f.txt");
+               String expectedCaseSensitive = toOSPathString("..\\..\\Git\\test\\d\\f.txt");
 
-               if (!systemReader.isWindows()) {
+               if (systemReader.isWindows()) {
                        String actual = FileUtils.relativize(base, other);
-                       assertEquals(expectedUnix, actual);
-
-                       // FS_POSIX#isCaseSensitive will return "false" for mac OS X.
-                       // Use this to test both behaviors.
-                       oldOSName = System.getProperty("os.name");
-                       try {
-                               System.setProperty("os.name", "Mac OS X");
-
-                               actual = FileUtils.relativize(base, other);
-                               assertEquals(expectedWindows, actual);
-                       } finally {
-                               if (oldOSName != null)
-                                       System.setProperty("os.name", oldOSName);
-                       }
+                       assertEquals(expectedCaseInsensitive, actual);
+               } else if (systemReader.isMacOS()) {
+                       String actual = FileUtils.relativize(base, other);
+                       assertEquals(expectedCaseInsensitive, actual);
                } else {
                        String actual = FileUtils.relativize(base, other);
-                       assertEquals(expectedWindows, actual);
+                       assertEquals(expectedCaseSensitive, actual);
                }
        }