You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by ma...@apache.org on 2016/04/18 16:14:06 UTC
svn commit: r1739762 - in
/tomcat/trunk/test/org/apache/catalina/webresources:
AbstractTestFileResourceSet.java TestFileResourceSet.java
TestFileResourceSetReadOnly.java TestFileResourceSetVirtual.java
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-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org