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>