[dna-commits] DNA SVN: r728 - trunk/extensions/dna-connector-filesystem/src/test/java/org/jboss/dna/connector/filesystem.

dna-commits at lists.jboss.org dna-commits at lists.jboss.org
Mon Feb 16 11:34:26 EST 2009


Author: rhauch
Date: 2009-02-16 11:34:26 -0500 (Mon, 16 Feb 2009)
New Revision: 728

Modified:
   trunk/extensions/dna-connector-filesystem/src/test/java/org/jboss/dna/connector/filesystem/FileSystemConnectorCreateWorkspacesTest.java
Log:
DNA-284 FileSystemConnectorCreateWorkspacesTest Fails on Windows XP

Applied a patch that fixes the issue on XP.

Modified: trunk/extensions/dna-connector-filesystem/src/test/java/org/jboss/dna/connector/filesystem/FileSystemConnectorCreateWorkspacesTest.java
===================================================================
--- trunk/extensions/dna-connector-filesystem/src/test/java/org/jboss/dna/connector/filesystem/FileSystemConnectorCreateWorkspacesTest.java	2009-02-13 21:55:20 UTC (rev 727)
+++ trunk/extensions/dna-connector-filesystem/src/test/java/org/jboss/dna/connector/filesystem/FileSystemConnectorCreateWorkspacesTest.java	2009-02-16 16:34:26 UTC (rev 728)
@@ -101,7 +101,11 @@
             workspaceNames.add(workspace.getName());
         }
         // The actual names should be the absolute paths to the directories representing the root ...
-        String absolutePathToRepositories = new File(".").getCanonicalPath() + "/src/test/resources/repositories/";
+        String absolutePathToRepositories = new File("./src/test/resources/repositories/").getCanonicalPath();
+        
+        // getCanonicalPath strips the trailing separator character, so we need to re-added it for each case
+        absolutePathToRepositories += File.separatorChar;	
+        
         assertThat(workspaceNames.remove(absolutePathToRepositories + "airplanes"), is(true));
         assertThat(workspaceNames.remove(absolutePathToRepositories + "cars"), is(true));
         assertThat(workspaceNames.isEmpty(), is(true));




More information about the dna-commits mailing list