struts-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Rene Gielen <gie...@it-neering.net>
Subject Re: Portlet 2.0 Plugin was moved to trunk
Date Fri, 24 Jun 2011 07:44:08 GMT
Great work, Johannes!

The only thing that concerns me a bit is that we seem to have lost the
commit history of the new plugin, since you have applied the sandbox
sources as patch to the current plugin. The history how p1 mutated to p2
is imo important ...

How do others feel about this? It's not too late to revert this to a move.

- René

On 23.06.11 23:24, Johannes Geppert wrote:
> Hi Guys,
> 
> I have copied the latest version of the Portlet 1.0 Plugin to the archive
> and replaced the version in trunk with the sandbox version. Also i have made
> same small changes on the Portlet Sample App which is running fine in
> jetspeed for me with the new portlet plugin version.
> 
> I have only one Problem in Test PortletRequestMapTest in Method
> testEntrySet()
> 
>     public void testEntrySet() {
>     	MockPortletRequest request = new MockPortletRequest();
>     	request.setAttribute("testAttribute1", "testValue1");
>     	request.setAttribute("testAttribute2", "testValue2");
> 
>         PortletRequestMap map = new PortletRequestMap(request);
>         Set entries = map.entrySet();
> 
>         //TODO Why is Entry Size 3?
>         assertEquals(2, entries.size());
>         Iterator it = entries.iterator();
>         Map.Entry entry = (Map.Entry)it.next();
>         checkEntry(entry);
>         entry = (Map.Entry)it.next();
>         checkEntry(entry);
> 
>     }
>  
> The Test runs into Failure because the entries.size() == 3 ?!?
> This Test runs in the Sandbox version. Has anyone an Idea or Suggestion?
> 
> Johannes
> 
> -----
> web: http://www.jgeppert.com
> twitter: http://twitter.com/jogep
> --
> View this message in context: http://struts.1045723.n5.nabble.com/Portlet-2-0-Plugin-was-moved-to-trunk-tp4519115p4519115.html
> Sent from the Struts - Dev mailing list archive at Nabble.com.
> 
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
> For additional commands, e-mail: dev-help@struts.apache.org
> 

-- 
René Gielen
IT-Neering.net
Saarstrasse 100, 52062 Aachen, Germany
Tel: +49-(0)241-4010770
Fax: +49-(0)241-4010771
Cel: +49-(0)163-2844164
http://twitter.com/rgielen

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org


Mime
View raw message