You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@velocity.apache.org by cb...@apache.org on 2016/10/16 07:30:32 UTC
svn commit: r1765120 - in /velocity/tools/trunk: ./ velocity-tools-assembly/
velocity-tools-examples/velocity-tools-examples-struts/
velocity-tools-generic/ velocity-tools-struts/ velocity-tools-view-jsp/
velocity-tools-view/ velocity-tools-view/src/ma...
Author: cbrisson
Date: Sun Oct 16 07:30:32 2016
New Revision: 1765120
URL: http://svn.apache.org/viewvc?rev=1765120&view=rev
Log:
[tools] more library upgrading and pom cleaning
- upgrade maven plugins versions
- junit, slf4j and surefire versions factorization
- servlet api version factorization
- upgrade to servlet api 3.0.1 (implies a few fixes in ParameterTool, CookieToolTests and LinkToolTests)
Modified:
velocity/tools/trunk/pom.xml
velocity/tools/trunk/velocity-tools-assembly/pom.xml
velocity/tools/trunk/velocity-tools-examples/velocity-tools-examples-struts/pom.xml
velocity/tools/trunk/velocity-tools-generic/pom.xml
velocity/tools/trunk/velocity-tools-struts/pom.xml
velocity/tools/trunk/velocity-tools-view-jsp/pom.xml
velocity/tools/trunk/velocity-tools-view/pom.xml
velocity/tools/trunk/velocity-tools-view/src/main/java/org/apache/velocity/tools/view/ParameterTool.java
velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/CookieToolTests.java
velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/LinkToolTests.java
velocity/tools/trunk/velocity-tools-xml/pom.xml
Modified: velocity/tools/trunk/pom.xml
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/pom.xml?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/pom.xml (original)
+++ velocity/tools/trunk/pom.xml Sun Oct 16 07:30:32 2016
@@ -44,7 +44,11 @@
<inceptionYear>2002</inceptionYear>
<properties>
- <slf4j.version>1.7.21</slf4j.version>
+ <project.build.sourceEncoding>UTF-8</project.build.sourceEncoding>
+ <junit.version>4.12</junit.version>
+ <slf4j.version>1.7.21</slf4j.version>
+ <surefire.plugin.version>2.19.1</surefire.plugin.version>
+ <servlet.api.version>3.0.1</servlet.api.version>
</properties>
<build>
Modified: velocity/tools/trunk/velocity-tools-assembly/pom.xml
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-assembly/pom.xml?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-assembly/pom.xml (original)
+++ velocity/tools/trunk/velocity-tools-assembly/pom.xml Sun Oct 16 07:30:32 2016
@@ -37,7 +37,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-assembly-plugin</artifactId>
- <version>2.2-beta-5</version>
+ <version>2.6</version>
<configuration>
<descriptors>
<descriptor>src/main/assembly/all.xml</descriptor>
@@ -49,7 +49,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-install-plugin</artifactId>
- <version>2.3.1</version>
+ <version>2.5.2</version>
<configuration>
<createChecksum>true</createChecksum>
</configuration>
@@ -115,7 +115,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-assembly-plugin</artifactId>
- <version>2.2-beta-5</version>
+ <version>2.6</version>
<executions>
<execution>
<id>make-assembly</id>
@@ -142,7 +142,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-antrun-plugin</artifactId>
- <version>1.3</version>
+ <version>1.8</version>
<executions>
<execution>
<phase>deploy</phase>
@@ -174,6 +174,7 @@
<plugin>
<groupId>org.codehaus.mojo</groupId>
<artifactId>wagon-maven-plugin</artifactId>
+ <version>1.0</version>
<executions>
<execution>
<phase>deploy</phase>
Modified: velocity/tools/trunk/velocity-tools-examples/velocity-tools-examples-struts/pom.xml
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-examples/velocity-tools-examples-struts/pom.xml?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-examples/velocity-tools-examples-struts/pom.xml (original)
+++ velocity/tools/trunk/velocity-tools-examples/velocity-tools-examples-struts/pom.xml Sun Oct 16 07:30:32 2016
@@ -37,8 +37,8 @@
<dependencies>
<dependency>
<groupId>javax.servlet</groupId>
- <artifactId>servlet-api</artifactId>
- <version>2.5</version>
+ <artifactId>javax.servlet-api</artifactId>
+ <version>${servlet.api.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
Modified: velocity/tools/trunk/velocity-tools-generic/pom.xml
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-generic/pom.xml?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-generic/pom.xml (original)
+++ velocity/tools/trunk/velocity-tools-generic/pom.xml Sun Oct 16 07:30:32 2016
@@ -54,13 +54,14 @@
<dependency>
<groupId>junit</groupId>
<artifactId>junit</artifactId>
- <version>4.12</version>
+ <version>${junit.version}</version>
<scope>test</scope>
</dependency>
<dependency>
<groupId>org.slf4j</groupId>
<artifactId>slf4j-simple</artifactId>
<version>${slf4j.version}</version>
+ <scope>test</scope>
</dependency>
</dependencies>
</project>
Modified: velocity/tools/trunk/velocity-tools-struts/pom.xml
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-struts/pom.xml?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-struts/pom.xml (original)
+++ velocity/tools/trunk/velocity-tools-struts/pom.xml Sun Oct 16 07:30:32 2016
@@ -43,8 +43,8 @@
</dependency>
<dependency>
<groupId>javax.servlet</groupId>
- <artifactId>servlet-api</artifactId>
- <version>2.5</version>
+ <artifactId>javax.servlet-api</artifactId>
+ <version>${servlet.api.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
Modified: velocity/tools/trunk/velocity-tools-view-jsp/pom.xml
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-view-jsp/pom.xml?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-view-jsp/pom.xml (original)
+++ velocity/tools/trunk/velocity-tools-view-jsp/pom.xml Sun Oct 16 07:30:32 2016
@@ -38,8 +38,8 @@
</dependency>
<dependency>
<groupId>javax.servlet</groupId>
- <artifactId>servlet-api</artifactId>
- <version>2.5</version>
+ <artifactId>javax.servlet-api</artifactId>
+ <version>${servlet.api.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
Modified: velocity/tools/trunk/velocity-tools-view/pom.xml
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-view/pom.xml?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-view/pom.xml (original)
+++ velocity/tools/trunk/velocity-tools-view/pom.xml Sun Oct 16 07:30:32 2016
@@ -36,6 +36,7 @@
<plugin>
<groupId>org.apache.maven.plugins</groupId>
<artifactId>maven-surefire-plugin</artifactId>
+ <version>${surefire.plugin.version}</version>
<configuration>
<includes>
<include>**/Test*.java</include>
@@ -55,14 +56,14 @@
</dependency>
<dependency>
<groupId>javax.servlet</groupId>
- <artifactId>servlet-api</artifactId>
- <version>2.5</version>
+ <artifactId>javax.servlet-api</artifactId>
+ <version>${servlet.api.version}</version>
<scope>provided</scope>
</dependency>
<dependency>
<groupId>junit</groupId>
<artifactId>junit</artifactId>
- <version>4.12</version>
+ <version>${junit.version}</version>
<scope>test</scope>
</dependency>
<dependency>
Modified: velocity/tools/trunk/velocity-tools-view/src/main/java/org/apache/velocity/tools/view/ParameterTool.java
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-view/src/main/java/org/apache/velocity/tools/view/ParameterTool.java?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-view/src/main/java/org/apache/velocity/tools/view/ParameterTool.java (original)
+++ velocity/tools/trunk/velocity-tools-view/src/main/java/org/apache/velocity/tools/view/ParameterTool.java Sun Oct 16 07:30:32 2016
@@ -195,7 +195,7 @@ public class ParameterTool extends Value
Array.getLength(value) == 1);
}
- private Map<String,Object> expandSingletonArrays(Map<String,Object> original)
+ private Map<String,Object> expandSingletonArrays(Map<String,String[]> original)
{
Map<String,Object> expanded = new HashMap<String,Object>(original);
for (Map.Entry<String,Object> entry : expanded.entrySet())
Modified: velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/CookieToolTests.java
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/CookieToolTests.java?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/CookieToolTests.java (original)
+++ velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/CookieToolTests.java Sun Oct 16 07:30:32 2016
@@ -40,16 +40,20 @@ import org.junit.Test;
*/
public class CookieToolTests
{
- private CookieTool newCookieTool(InvocationHandler handler)
+ private CookieTool newCookieTool(InvocationHandler requestHandler, InvocationHandler responseHandler)
{
- Object proxy
+ Object requestProxy
= Proxy.newProxyInstance(this.getClass().getClassLoader(),
- new Class[] { HttpServletRequest.class,
- HttpServletResponse.class },
- handler);
+ new Class[] { HttpServletRequest.class },
+ requestHandler);
- HttpServletRequest request = (HttpServletRequest)proxy;
- HttpServletResponse response = (HttpServletResponse)proxy;
+ Object responseProxy
+ = Proxy.newProxyInstance(this.getClass().getClassLoader(),
+ new Class[] { HttpServletResponse.class },
+ responseHandler);
+
+ HttpServletRequest request = (HttpServletRequest)requestProxy;
+ HttpServletResponse response = (HttpServletResponse)responseProxy;
CookieTool cookies = new CookieTool();
cookies.setRequest(request);
@@ -59,7 +63,7 @@ public class CookieToolTests
private CookieTool newCookieTool(Map cookies)
{
- return newCookieTool(new ServletAdaptor(cookies));
+ return newCookieTool(new RequestAdaptor(cookies), new ResponseAdaptor(cookies));
}
private CookieTool newCookieTool(String name, Object value)
@@ -132,11 +136,12 @@ public class CookieToolTests
{
Map jar = new LinkedHashMap();
jar.put("a", "b");
- ServletAdaptor proxy = new ServletAdaptor(jar);
- CookieTool cookies = newCookieTool(proxy);
+ RequestAdaptor requestProxy = new RequestAdaptor(jar);
+ ResponseAdaptor responseProxy = new ResponseAdaptor(jar);
+ CookieTool cookies = newCookieTool(requestProxy, responseProxy);
assertEquals("", cookies.add("a","b"));
- cookies = newCookieTool(proxy);
+ cookies = newCookieTool(requestProxy, responseProxy);
assertNotNull(cookies.get("a"));
assertEquals("b", cookies.get("a").getValue());
}
@@ -145,11 +150,12 @@ public class CookieToolTests
{
Map jar = new LinkedHashMap();
jar.put("a", "b");
- ServletAdaptor proxy = new ServletAdaptor(jar);
- CookieTool cookies = newCookieTool(proxy);
+ RequestAdaptor requestProxy = new RequestAdaptor(jar);
+ ResponseAdaptor responseProxy = new ResponseAdaptor(jar);
+ CookieTool cookies = newCookieTool(requestProxy, responseProxy);
assertEquals("", cookies.add("a","b", 10));
- cookies = newCookieTool(proxy);
+ cookies = newCookieTool(requestProxy, responseProxy);
Cookie c = cookies.get("a");
assertNotNull(c);
assertEquals("b", c.getValue());
@@ -160,12 +166,13 @@ public class CookieToolTests
{
Map jar = new LinkedHashMap();
jar.put("a", "b");
- ServletAdaptor proxy = new ServletAdaptor(jar);
- CookieTool cookies = newCookieTool(proxy);
+ RequestAdaptor requestProxy = new RequestAdaptor(jar);
+ ResponseAdaptor responseProxy = new ResponseAdaptor(jar);
+ CookieTool cookies = newCookieTool(requestProxy, responseProxy);
assertEquals("b", cookies.get("a").toString());
cookies.delete("a");
- cookies = newCookieTool(proxy);
+ cookies = newCookieTool(requestProxy, responseProxy);
assertNull(cookies.get("a"));
}
}
Modified: velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/LinkToolTests.java
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/LinkToolTests.java?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/LinkToolTests.java (original)
+++ velocity/tools/trunk/velocity-tools-view/src/test/java/org/apache/velocity/tools/test/blackbox/LinkToolTests.java Sun Oct 16 07:30:32 2016
@@ -46,16 +46,20 @@ public class LinkToolTests
ConfigValues() { setReadOnly(false); }
}
- private LinkTool newLinkTool(InvocationHandler handler)
+ private LinkTool newLinkTool(InvocationHandler requestHandler, InvocationHandler responseHandler)
{
- Object proxy
+ Object requestProxy
= Proxy.newProxyInstance(this.getClass().getClassLoader(),
- new Class[] { HttpServletRequest.class,
- HttpServletResponse.class },
- handler);
+ new Class[] { HttpServletRequest.class },
+ requestHandler);
- HttpServletRequest request = (HttpServletRequest)proxy;
- HttpServletResponse response = (HttpServletResponse)proxy;
+ Object responseProxy
+ = Proxy.newProxyInstance(this.getClass().getClassLoader(),
+ new Class[] { HttpServletResponse.class },
+ responseHandler);
+
+ HttpServletRequest request = (HttpServletRequest)requestProxy;
+ HttpServletResponse response = (HttpServletResponse)responseProxy;
LinkTool link = new LinkTool();
ValueParser properties = new ConfigValues();
@@ -67,7 +71,7 @@ public class LinkToolTests
private LinkTool newLinkTool(Map params)
{
- return newLinkTool(new ServletAdaptor("/test","/link.vm", params));
+ return newLinkTool(new RequestAdaptor("/test","/link.vm", params), new ResponseAdaptor(params));
}
private LinkTool newLinkTool(String key, Object value)
Modified: velocity/tools/trunk/velocity-tools-xml/pom.xml
URL: http://svn.apache.org/viewvc/velocity/tools/trunk/velocity-tools-xml/pom.xml?rev=1765120&r1=1765119&r2=1765120&view=diff
==============================================================================
--- velocity/tools/trunk/velocity-tools-xml/pom.xml (original)
+++ velocity/tools/trunk/velocity-tools-xml/pom.xml Sun Oct 16 07:30:32 2016
@@ -49,7 +49,7 @@
<dependency>
<groupId>junit</groupId>
<artifactId>junit</artifactId>
- <version>4.12</version>
+ <version>${junit.version}</version>
<scope>test</scope>
</dependency>
</dependencies>