You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@wicket.apache.org by al...@apache.org on 2007/06/19 18:17:52 UTC
svn commit: r548784 -
/incubator/wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/util/string/JavascriptStripperTest.java
Author: almaw
Date: Tue Jun 19 09:17:51 2007
New Revision: 548784
URL: http://svn.apache.org/viewvc?view=rev&rev=548784
Log:
WICKET-501 - JavaScript comment stripping doesn't handle regexps
Modified:
incubator/wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/util/string/JavascriptStripperTest.java
Modified: incubator/wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/util/string/JavascriptStripperTest.java
URL: http://svn.apache.org/viewvc/incubator/wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/util/string/JavascriptStripperTest.java?view=diff&rev=548784&r1=548783&r2=548784
==============================================================================
--- incubator/wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/util/string/JavascriptStripperTest.java (original)
+++ incubator/wicket/trunk/jdk-1.4/wicket/src/test/java/org/apache/wicket/util/string/JavascriptStripperTest.java Tue Jun 19 09:17:51 2007
@@ -53,10 +53,18 @@
assertEquals("\nt = jQuery.trim(t).replace( /^\\/\\//i, \"\" );", s);
}
- public void testRegexpWithString()
+ public void testRegexp2()
{
String s = JavascriptStripper
.stripCommentsAndWhitespace("foo.replace(/\"//*strip me*/, \"\"); // strip me\rdoFoo();");
assertEquals("foo.replace(/\"/, \"\"); doFoo();", s);
+ }
+
+ public void testRegexp3()
+ {
+ String s = JavascriptStripper.stripCommentsAndWhitespace(
+ "parseFloat( elem.filter.match(/alpha\\(opacity=(.*)\\)/)[1] ) / 100 : 1;\r//foo"
+ );
+ assertEquals("parseFloat( elem.filter.match(/alpha\\(opacity=(.*)\\)/)[1] ) / 100 : 1;\r", s);
}
}