Author: julien(a)jboss.com
Date: 2007-09-15 17:23:00 -0400 (Sat, 15 Sep 2007)
New Revision: 8291
Modified:
modules/portlet/trunk/portlet/build.xml
modules/portlet/trunk/portlet/src/main/org/jboss/portal/portlet/impl/state/consumer/AbstractConsumerPersistenceManagerService.java
modules/portlet/trunk/test/build.xml
Log:
- fix minor bug in AbstractConsumerPersistenceManagerService that was making consumer
invoker test failing
- set emptySessionPath=true for the tomcat test in order to make the session test pass
Modified: modules/portlet/trunk/portlet/build.xml
===================================================================
--- modules/portlet/trunk/portlet/build.xml 2007-09-15 20:31:45 UTC (rev 8290)
+++ modules/portlet/trunk/portlet/build.xml 2007-09-15 21:23:00 UTC (rev 8291)
@@ -355,7 +355,7 @@
<x-sysproperty>
<!--
<jvmarg value="-Xdebug"/>
- <jvmarg
value="-Xrunjdwp:transport=dt_socket,address=8787,server=y,suspend=y"/>
+ <jvmarg
value="-Xrunjdwp:transport=dt_socket,address=7878,server=y,suspend=y"/>
-->
</x-sysproperty>
<x-classpath>
Modified:
modules/portlet/trunk/portlet/src/main/org/jboss/portal/portlet/impl/state/consumer/AbstractConsumerPersistenceManagerService.java
===================================================================
---
modules/portlet/trunk/portlet/src/main/org/jboss/portal/portlet/impl/state/consumer/AbstractConsumerPersistenceManagerService.java 2007-09-15
20:31:45 UTC (rev 8290)
+++
modules/portlet/trunk/portlet/src/main/org/jboss/portal/portlet/impl/state/consumer/AbstractConsumerPersistenceManagerService.java 2007-09-15
21:23:00 UTC (rev 8291)
@@ -72,7 +72,7 @@
{
throw new IllegalArgumentException();
}
- if (get(stateId) != null)
+ if (get(stateId) == null)
{
throw new NoSuchStateException(stateId);
}
Modified: modules/portlet/trunk/test/build.xml
===================================================================
--- modules/portlet/trunk/test/build.xml 2007-09-15 20:31:45 UTC (rev 8290)
+++ modules/portlet/trunk/test/build.xml 2007-09-15 21:23:00 UTC (rev 8291)
@@ -326,6 +326,7 @@
<configuration>
<property name="cargo.servlet.port" value="8080"/>
<property name="cargo.logging" value="high"/>
+ <property name="cargo.tomcat.connector.emptySessionPath"
value="true"/>
<deployable type="war"
file="${codehaus.cargo.lib}/manager.war"/>
<deployable type="war"
file="${build.lib}/tomcat-6.0/portlet-test.war"/>
</configuration>