[jboss-cvs] JBossAS SVN: r82226 - projects/ejb3/trunk/testsuite/src/test/java/org/jboss/ejb3/test/initial.

[jboss-cvs-commits at lists.jboss.org]

Author: wolfc
Date: 2008-12-11 07:41:30 -0500 (Thu, 11 Dec 2008)
New Revision: 82226

Modified:
   projects/ejb3/trunk/testsuite/src/test/java/org/jboss/ejb3/test/initial/Tester.java
Log:
EJBTHREE-1241: no assumption should be made when PostConstruct on a SLSB is called

Modified: projects/ejb3/trunk/testsuite/src/test/java/org/jboss/ejb3/test/initial/Tester.java
===================================================================
--- projects/ejb3/trunk/testsuite/src/test/java/org/jboss/ejb3/test/initial/Tester.java	2008-12-11 12:17:55 UTC (rev 82225)
+++ projects/ejb3/trunk/testsuite/src/test/java/org/jboss/ejb3/test/initial/Tester.java	2008-12-11 12:41:30 UTC (rev 82226)
@@ -127,7 +127,9 @@
       status.clear();
       InterceptedSLTest test = (InterceptedSLTest) ctx.lookup("initial-ejb3-test/InterceptedSLTestBean/local");
       test.testMethod(5);
-      if (!status.postConstruct()) throw new Exception("PostConstruct should be called for SLSB");
+      // PostConstruct test is invalid, because you can't assume that postConstruct is called during the lookup
+      // the instance might already be available from a previous call or prefill of the pool.
+      //if (!status.postConstruct()) throw new Exception("PostConstruct should be called for SLSB");
       if (status.prePassivate()) throw new Exception("PrePassivate should not be called for SLSB");
       if (status.postActivate()) throw new Exception("PostActivate should not be called for SLSB");