Ignore symbolic link test failure on jdk6

ejona86 · ejona86 · commit 6884507af7e8 · 2015-10-26T12:54:51.000-07:00
Not much we can do; the platform simply fails to have useful behavior
for at least openjdk 6u34 and oracle jdk 6u45.
diff --git a/google-http-client/src/test/java/com/google/api/client/util/IOUtilsTest.java b/google-http-client/src/test/java/com/google/api/client/util/IOUtilsTest.java
@@ -57,6 +57,9 @@ public void testIsSymbolicLink_true() throws IOException, InterruptedException {
       // ignore because ln command may not be defined
       return;
     }
-    assertTrue(IOUtils.isSymbolicLink(file2));
+    // multiple versions of jdk6 cannot detect the symbolic link. Consider support best-effort on
+    // jdk6
+    boolean jdk6 = System.getProperty("java.version").startsWith("1.6.0_");
+    assertTrue(IOUtils.isSymbolicLink(file2) || jdk6);
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -57,6 +57,9 @@ public void testIsSymbolicLink_true() throws IOException, InterruptedException {`
`57`	`57`	`// ignore because ln command may not be defined`
`58`	`58`	`return;`
`59`	`59`	`}`
`60`		`- assertTrue(IOUtils.isSymbolicLink(file2));`
	`60`	`+ // multiple versions of jdk6 cannot detect the symbolic link. Consider support best-effort on`
	`61`	`+ // jdk6`
	`62`	`+ boolean jdk6 = System.getProperty("java.version").startsWith("1.6.0_");`
	`63`	`+ assertTrue(IOUtils.isSymbolicLink(file2) \|\| jdk6);`
`61`	`64`	`}`
`62`	`65`	`}`