You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@struts.apache.org by cr...@apache.org on 2006/01/25 21:07:23 UTC

svn commit: r372302 - in /struts/shale/trunk/test-framework/src/java/org/apache/shale/test: base/AbstractJsfTestCase.java mock/MockApplication.java

Author: craigmcc
Date: Wed Jan 25 12:07:20 2006
New Revision: 372302

URL: http://svn.apache.org/viewcvs?rev=372302&view=rev
Log:
Ensure that the instance returned by RenderKitFactory.getRenderKit() is the
same as the instance in the renderKit variable.

Improve the exception messages on component lookups that fail.

Modified:
    struts/shale/trunk/test-framework/src/java/org/apache/shale/test/base/AbstractJsfTestCase.java
    struts/shale/trunk/test-framework/src/java/org/apache/shale/test/mock/MockApplication.java

Modified: struts/shale/trunk/test-framework/src/java/org/apache/shale/test/base/AbstractJsfTestCase.java
URL: http://svn.apache.org/viewcvs/struts/shale/trunk/test-framework/src/java/org/apache/shale/test/base/AbstractJsfTestCase.java?rev=372302&r1=372301&r2=372302&view=diff
==============================================================================
--- struts/shale/trunk/test-framework/src/java/org/apache/shale/test/base/AbstractJsfTestCase.java (original)
+++ struts/shale/trunk/test-framework/src/java/org/apache/shale/test/base/AbstractJsfTestCase.java Wed Jan 25 12:07:20 2006
@@ -128,13 +128,8 @@
         facesContext.setApplication(application);
         RenderKitFactory renderKitFactory = (RenderKitFactory)
             FactoryFinder.getFactory(FactoryFinder.RENDER_KIT_FACTORY);
-        renderKit = new MockRenderKit();
-        try {
-            renderKitFactory.addRenderKit(RenderKitFactory.HTML_BASIC_RENDER_KIT,
-                                          renderKit);
-        } catch (IllegalArgumentException e) {
-            ;
-        }
+        renderKit = (MockRenderKit)
+          renderKitFactory.getRenderKit(null, RenderKitFactory.HTML_BASIC_RENDER_KIT);
 
     }
 

Modified: struts/shale/trunk/test-framework/src/java/org/apache/shale/test/mock/MockApplication.java
URL: http://svn.apache.org/viewcvs/struts/shale/trunk/test-framework/src/java/org/apache/shale/test/mock/MockApplication.java?rev=372302&r1=372301&r2=372302&view=diff
==============================================================================
--- struts/shale/trunk/test-framework/src/java/org/apache/shale/test/mock/MockApplication.java (original)
+++ struts/shale/trunk/test-framework/src/java/org/apache/shale/test/mock/MockApplication.java Wed Jan 25 12:07:20 2006
@@ -272,7 +272,14 @@
 
     public UIComponent createComponent(String componentType) {
 
+        if (componentType == null) {
+            throw new NullPointerException("Requested component type is null");
+        }
         String componentClass = (String) components.get(componentType);
+        if (componentClass == null) {
+            throw new FacesException("No component class registered for component type '" +
+                    componentType + "'");
+        }
         try {
             Class clazz = Class.forName(componentClass);
             return ((UIComponent) clazz.newInstance());



---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@struts.apache.org
For additional commands, e-mail: dev-help@struts.apache.org