You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by de...@apache.org on 2006/02/20 05:38:59 UTC

svn commit: r379025 - /myfaces/core/trunk/api/src/test/java/javax/faces/FactoryFinderTest.java

Author: dennisbyrne
Date: Sun Feb 19 20:38:57 2006
New Revision: 379025

URL: http://svn.apache.org/viewcvs?rev=379025&view=rev
Log:
FactoryFinder.releaseFactories() needs to be called in setup()

Modified:
    myfaces/core/trunk/api/src/test/java/javax/faces/FactoryFinderTest.java

Modified: myfaces/core/trunk/api/src/test/java/javax/faces/FactoryFinderTest.java
URL: http://svn.apache.org/viewcvs/myfaces/core/trunk/api/src/test/java/javax/faces/FactoryFinderTest.java?rev=379025&r1=379024&r2=379025&view=diff
==============================================================================
--- myfaces/core/trunk/api/src/test/java/javax/faces/FactoryFinderTest.java (original)
+++ myfaces/core/trunk/api/src/test/java/javax/faces/FactoryFinderTest.java Sun Feb 19 20:38:57 2006
@@ -37,10 +37,16 @@
 
   protected void setUp() throws Exception {
     super.setUp();
+    // this needs to be called *before* the first Test test is run, 
+    // as there may be left over FactoryFinder configurations from
+    // that previous tests that may interfere with the first test here. 
+    FactoryFinder.releaseFactories(); 
   }
 
   protected void tearDown() throws Exception {
     super.tearDown();
+    // call this again so there is no possibility of messing up tests that will
+    // run after this one
     FactoryFinder.releaseFactories();
     releaseRegisteredFactoryNames();
   }