You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@ant.apache.org by jg...@apache.org on 2012/03/05 20:26:27 UTC

svn commit: r1297188 - /ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/SymlinkUtilsTest.java

Author: jglick
Date: Mon Mar  5 19:26:26 2012
New Revision: 1297188

URL: http://svn.apache.org/viewvc?rev=1297188&view=rev
Log:
More idiomatic check with Os.isFamily.

Modified:
    ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/SymlinkUtilsTest.java

Modified: ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/SymlinkUtilsTest.java
URL: http://svn.apache.org/viewvc/ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/SymlinkUtilsTest.java?rev=1297188&r1=1297187&r2=1297188&view=diff
==============================================================================
--- ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/SymlinkUtilsTest.java (original)
+++ ant/core/trunk/src/tests/junit/org/apache/tools/ant/util/SymlinkUtilsTest.java Mon Mar  5 19:26:26 2012
@@ -18,9 +18,9 @@
 
 package org.apache.tools.ant.util;
 
-import java.io.File;
 import java.io.IOException;
 import junit.framework.TestCase;
+import org.apache.tools.ant.taskdefs.condition.Os;
 
 public class SymlinkUtilsTest extends TestCase {
 
@@ -28,7 +28,7 @@ public class SymlinkUtilsTest extends Te
         SymbolicLinkUtils.getSymbolicLinkUtils();
 
     public void testRootIsNoSymlink() throws IOException {
-        if (File.pathSeparatorChar == ';') {
+        if (Os.isFamily("windows")) {
             return; // test is meaningless on Windows
         }
         assertFalse(SYMLINK_UTILS.isSymbolicLink("/"));