myfaces-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From mfreed...@apache.org
Subject svn commit: r1138596 - /myfaces/portlet-bridge/tck/trunk_3.0.x/src/main/java/org/apache/myfaces/portlet/faces/testsuite/tests/chapter_6/section_6_1_1/FacesContextFactoryServiceProviderTest.java
Date Wed, 22 Jun 2011 19:42:48 GMT
Author: mfreedman
Date: Wed Jun 22 19:42:48 2011
New Revision: 1138596

URL: http://svn.apache.org/viewvc?rev=1138596&view=rev
Log:
FacesContextfactoryTest wasn't returning the result string in the fail case. 

Modified:
    myfaces/portlet-bridge/tck/trunk_3.0.x/src/main/java/org/apache/myfaces/portlet/faces/testsuite/tests/chapter_6/section_6_1_1/FacesContextFactoryServiceProviderTest.java

Modified: myfaces/portlet-bridge/tck/trunk_3.0.x/src/main/java/org/apache/myfaces/portlet/faces/testsuite/tests/chapter_6/section_6_1_1/FacesContextFactoryServiceProviderTest.java
URL: http://svn.apache.org/viewvc/myfaces/portlet-bridge/tck/trunk_3.0.x/src/main/java/org/apache/myfaces/portlet/faces/testsuite/tests/chapter_6/section_6_1_1/FacesContextFactoryServiceProviderTest.java?rev=1138596&r1=1138595&r2=1138596&view=diff
==============================================================================
--- myfaces/portlet-bridge/tck/trunk_3.0.x/src/main/java/org/apache/myfaces/portlet/faces/testsuite/tests/chapter_6/section_6_1_1/FacesContextFactoryServiceProviderTest.java
(original)
+++ myfaces/portlet-bridge/tck/trunk_3.0.x/src/main/java/org/apache/myfaces/portlet/faces/testsuite/tests/chapter_6/section_6_1_1/FacesContextFactoryServiceProviderTest.java
Wed Jun 22 19:42:48 2011
@@ -61,21 +61,22 @@ public class FacesContextFactoryServiceP
       {
         resultWriter.setStatus(BridgeTCKResultWriter.FAIL);
         resultWriter.setDetail("META-INF/services/javax.faces.context.FacesContextFactory
not found.");
-        return;
       }
-      
-      // See if we can load the class
-      try
-      {
-        ClassLoader loader = Thread.currentThread().getContextClassLoader();
-        Class c = (Class<? extends FacesContextFactory>) loader.loadClass(className);
-        resultWriter.setStatus(BridgeTCKResultWriter.PASS);
-        resultWriter.setDetail("Located and loaded the Bridge's FacesContextFactory class:
" + className);
-      }
-      catch (ClassNotFoundException cnfe)
+      else
       {
-        resultWriter.setStatus(BridgeTCKResultWriter.FAIL);
-        resultWriter.setDetail("Located but unable to load the Bridge's FacesContextFactory
class: " + className);
+        // See if we can load the class
+        try
+        {
+          ClassLoader loader = Thread.currentThread().getContextClassLoader();
+          Class c = (Class<? extends FacesContextFactory>) loader.loadClass(className);
+          resultWriter.setStatus(BridgeTCKResultWriter.PASS);
+          resultWriter.setDetail("Located and loaded the Bridge's FacesContextFactory class:
" + className);
+        }
+        catch (ClassNotFoundException cnfe)
+        {
+          resultWriter.setStatus(BridgeTCKResultWriter.FAIL);
+          resultWriter.setDetail("Located but unable to load the Bridge's FacesContextFactory
class: " + className);
+        }
       }
 
     out.println(resultWriter.toString());



Mime
View raw message