This is an automated email from the ASF dual-hosted git repository.
markt pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/tomcat.git
The following commit(s) were added to refs/heads/master by this push:
new 7e65db4 Remove the debug code. Add explicit mapping for jsp-file
defined servlet
7e65db4 is described below
commit 7e65db453b8a7740de48343c12a8a91db96ad79b
Author: Mark Thomas <[email protected]>
AuthorDate: Mon Aug 17 15:32:10 2020 +0100
Remove the debug code. Add explicit mapping for jsp-file defined servlet
---
.../compiler/TestParserNoStrictWhitespace.java | 31 ++--------------------
test/webapp/WEB-INF/web.xml | 4 +++
2 files changed, 6 insertions(+), 29 deletions(-)
diff --git a/test/org/apache/jasper/compiler/TestParserNoStrictWhitespace.java
b/test/org/apache/jasper/compiler/TestParserNoStrictWhitespace.java
index e8f05c9..3101dcc 100644
--- a/test/org/apache/jasper/compiler/TestParserNoStrictWhitespace.java
+++ b/test/org/apache/jasper/compiler/TestParserNoStrictWhitespace.java
@@ -14,18 +14,11 @@
* See the License for the specific language governing permissions and
* limitations under the License.
*/
-
package org.apache.jasper.compiler;
-import java.io.File;
-
-import jakarta.servlet.ServletContext;
-
import org.junit.Assert;
import org.junit.Test;
-import org.apache.catalina.core.StandardContext;
-import org.apache.catalina.core.StandardWrapper;
import org.apache.catalina.startup.TomcatBaseTest;
import org.apache.tomcat.util.buf.ByteChunk;
@@ -40,34 +33,14 @@ public class TestParserNoStrictWhitespace extends
TomcatBaseTest {
public void testBug49297NoSpaceNotStrict() throws Exception {
getTomcatInstanceTestWebapp(false, true);
- // Github / Travis / s390x debug code
- StandardContext ctxt = (StandardContext)
getTomcatInstance().getHost().findChild("/test");
- StandardWrapper w = (StandardWrapper)
ctxt.findChild("bug49297NoSpace");
- System.out.println("JSP strict whitespace: [" +
w.getInitParameter("strictWhitespace") + "]");
-
- File tmpJsp = new File("test/webapp/bug49nnn/bug49297NoSpace.jsp");
- System.out.println(".jsp [" + tmpJsp.lastModified() + "]");
-
- File tmp = (File)
ctxt.getServletContext().getAttribute(ServletContext.TEMPDIR);
- File tmpJava = new File(tmp,
"org/apache/jsp/bug49nnn/bug49297NoSpace_jsp.java");
- File tmpClass = new File(tmp,
"org/apache/jsp/bug49nnn/bug49297NoSpace_jsp.class");
- System.out.println("before request .java [" + tmpJava.lastModified()
+ "]");
- System.out.println("before request .class [" + tmpClass.lastModified()
+ "]");
- // debug code end
-
ByteChunk res = new ByteChunk();
- int sc = getUrl("http://localhost:" + getPort() +
- "/test/bug49nnn/bug49297NoSpace.jsp", res, null);
-
- // Github / Travis / s390x debug code
- System.out.println("after request .java [" + tmpJava.lastModified() +
"]");
- System.out.println("after request .class [" + tmpClass.lastModified()
+ "]");
- // debug code end
+ int sc = getUrl("http://localhost:" + getPort() +
"/test/bug49nnn/bug49297NoSpace.jsp", res, null);
Assert.assertEquals(200, sc);
assertEcho(res.toString(), "Hello World");
}
+
/** Assertion for text printed by tags:echo */
private static void assertEcho(String result, String expected) {
Assert.assertTrue(result.indexOf("<p>" + expected + "</p>") > 0);
diff --git a/test/webapp/WEB-INF/web.xml b/test/webapp/WEB-INF/web.xml
index d5a3c95..3706586 100644
--- a/test/webapp/WEB-INF/web.xml
+++ b/test/webapp/WEB-INF/web.xml
@@ -74,6 +74,10 @@
<param-value>false</param-value>
</init-param>
</servlet>
+ <servlet-mapping>
+ <servlet-name>bug49297NoSpace</servlet-name>
+ <url-pattern>/bug49nnn/bug49297NoSpace.jsp</url-pattern>
+ </servlet-mapping>
<servlet>
<servlet-name>Bug49922Forward</servlet-name>
<servlet-class>
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]