You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by mf...@apache.org on 2011/06/22 21:42:48 UTC
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
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());