Author: markt Date: Mon Apr 18 14:14:06 2016 New Revision: 1739762 URL: http://svn.apache.org/viewvc?rev=1739762&view=rev Log: Fix likely cause of occasional unit test failures. Defensive copy was being taken of wrong location.
Modified: tomcat/trunk/test/org/apache/catalina/webresources/AbstractTestFileResourceSet.java tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSet.java tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetReadOnly.java tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetVirtual.java Modified: tomcat/trunk/test/org/apache/catalina/webresources/AbstractTestFileResourceSet.java URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/webresources/AbstractTestFileResourceSet.java?rev=1739762&r1=1739761&r2=1739762&view=diff ============================================================================== --- tomcat/trunk/test/org/apache/catalina/webresources/AbstractTestFileResourceSet.java (original) +++ tomcat/trunk/test/org/apache/catalina/webresources/AbstractTestFileResourceSet.java Mon Apr 18 14:14:06 2016 @@ -31,7 +31,7 @@ public abstract class AbstractTestFileRe this.readOnly = readOnly; } - protected abstract String getDir1(); + protected abstract File getDir2(); @Override public WebResourceRoot getWebResourceRoot() { @@ -41,22 +41,22 @@ public abstract class AbstractTestFileRe root.setMainResources(webResourceSet); WebResourceSet f1 = new FileResourceSet(root, "/f1.txt", - getDir1() + "/f1.txt", "/"); + "test/webresources/dir1/f1.txt", "/"); f1.setReadOnly(readOnly); root.addPreResources(f1); WebResourceSet f2 = new FileResourceSet(root, "/f2.txt", - getDir1() + "/f2.txt", "/"); + "test/webresources/dir1/f2.txt", "/"); f2.setReadOnly(readOnly); root.addPreResources(f2); WebResourceSet d1f1 = new FileResourceSet(root, "/d1/d1-f1.txt", - getDir1() + "/d1/d1-f1.txt", "/"); + "test/webresources/dir1/d1/d1-f1.txt", "/"); d1f1.setReadOnly(readOnly); root.addPreResources(d1f1); WebResourceSet d2f1 = new FileResourceSet(root, "/d2/d2-f1.txt", - getDir1() + "/d2/d2-f1.txt", "/"); + "test/webresources/dir1/d2/d2-f1.txt", "/"); d2f1.setReadOnly(readOnly); root.addPreResources(d2f1); @@ -70,7 +70,7 @@ public abstract class AbstractTestFileRe @Override public File getBaseDir() { - return new File("test/webresources/dir2"); + return getDir2(); } @Override Modified: tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSet.java URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSet.java?rev=1739762&r1=1739761&r2=1739762&view=diff ============================================================================== --- tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSet.java (original) +++ tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSet.java Mon Apr 18 14:14:06 2016 @@ -31,13 +31,13 @@ import org.apache.catalina.startup.Tomca public class TestFileResourceSet extends AbstractTestFileResourceSet { private static Path tempDir; - private static File dir1; + private static File dir2; @BeforeClass public static void before() throws IOException { tempDir = Files.createTempDirectory("test", new FileAttribute[0]); - dir1 = new File(tempDir.toFile(), "dir1"); - TomcatBaseTest.recursiveCopy(new File("test/webresources/dir1").toPath(), dir1.toPath()); + dir2 = new File(tempDir.toFile(), "dir2"); + TomcatBaseTest.recursiveCopy(new File("test/webresources/dir2").toPath(), dir2.toPath()); } @AfterClass @@ -51,8 +51,8 @@ public class TestFileResourceSet extends } @Override - protected String getDir1() { - return dir1.getAbsolutePath(); + protected File getDir2() { + return dir2; } @Override Modified: tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetReadOnly.java URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetReadOnly.java?rev=1739762&r1=1739761&r2=1739762&view=diff ============================================================================== --- tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetReadOnly.java (original) +++ tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetReadOnly.java Mon Apr 18 14:14:06 2016 @@ -31,13 +31,13 @@ import org.apache.catalina.startup.Tomca public class TestFileResourceSetReadOnly extends AbstractTestFileResourceSet { private static Path tempDir; - private static File dir1; + private static File dir2; @BeforeClass public static void before() throws IOException { tempDir = Files.createTempDirectory("test", new FileAttribute[0]); - dir1 = new File(tempDir.toFile(), "dir1"); - TomcatBaseTest.recursiveCopy(new File("test/webresources/dir1").toPath(), dir1.toPath()); + dir2 = new File(tempDir.toFile(), "dir2"); + TomcatBaseTest.recursiveCopy(new File("test/webresources/dir2").toPath(), dir2.toPath()); } @AfterClass @@ -51,8 +51,8 @@ public class TestFileResourceSetReadOnly } @Override - protected String getDir1() { - return dir1.getAbsolutePath(); + protected File getDir2() { + return dir2; } @Override Modified: tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetVirtual.java URL: http://svn.apache.org/viewvc/tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetVirtual.java?rev=1739762&r1=1739761&r2=1739762&view=diff ============================================================================== --- tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetVirtual.java (original) +++ tomcat/trunk/test/org/apache/catalina/webresources/TestFileResourceSetVirtual.java Mon Apr 18 14:14:06 2016 @@ -30,18 +30,18 @@ import org.apache.catalina.startup.Tomca /** * Mounts file resources in sub directories that do not exist in the main - * resoucres. + * resources. */ public class TestFileResourceSetVirtual extends TestFileResourceSet { private static Path tempDir; - private static File dir1; + private static File dir2; @BeforeClass public static void before() throws IOException { tempDir = Files.createTempDirectory("test", new FileAttribute[0]); - dir1 = new File(tempDir.toFile(), "dir1"); - TomcatBaseTest.recursiveCopy(new File("test/webresources/dir1").toPath(), dir1.toPath()); + dir2 = new File(tempDir.toFile(), "dir2"); + TomcatBaseTest.recursiveCopy(new File("test/webresources/dir2").toPath(), dir2.toPath()); } @AfterClass @@ -56,8 +56,8 @@ public class TestFileResourceSetVirtual } @Override - protected String getDir1() { - return dir1.getAbsolutePath(); + protected File getDir2() { + return dir2; } @Override --------------------------------------------------------------------- To unsubscribe, e-mail: dev-unsubscr...@tomcat.apache.org For additional commands, e-mail: dev-h...@tomcat.apache.org