You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tuscany.apache.org by rf...@apache.org on 2007/04/16 08:54:47 UTC
svn commit: r529147 - in
/incubator/tuscany/java/sca/modules/interface-java-xml/src:
main/java/org/apache/tuscany/interfacedef/java/introspect/
test/java/org/apache/tuscany/interfacedef/java/introspection/impl/
Author: rfeng
Date: Sun Apr 15 23:54:46 2007
New Revision: 529147
URL: http://svn.apache.org/viewvc?view=rev&rev=529147
Log:
Only add checked exception to fault types
Modified:
incubator/tuscany/java/sca/modules/interface-java-xml/src/main/java/org/apache/tuscany/interfacedef/java/introspect/DefaultJavaInterfaceIntrospector.java
incubator/tuscany/java/sca/modules/interface-java-xml/src/test/java/org/apache/tuscany/interfacedef/java/introspection/impl/JavaInterfaceProcessorRegistryImplTestCase.java
Modified: incubator/tuscany/java/sca/modules/interface-java-xml/src/main/java/org/apache/tuscany/interfacedef/java/introspect/DefaultJavaInterfaceIntrospector.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/interface-java-xml/src/main/java/org/apache/tuscany/interfacedef/java/introspect/DefaultJavaInterfaceIntrospector.java?view=diff&rev=529147&r1=529146&r2=529147
==============================================================================
--- incubator/tuscany/java/sca/modules/interface-java-xml/src/main/java/org/apache/tuscany/interfacedef/java/introspect/DefaultJavaInterfaceIntrospector.java (original)
+++ incubator/tuscany/java/sca/modules/interface-java-xml/src/main/java/org/apache/tuscany/interfacedef/java/introspect/DefaultJavaInterfaceIntrospector.java Sun Apr 15 23:54:46 2007
@@ -129,7 +129,10 @@
}
List<DataType> faultDataTypes = new ArrayList<DataType>(faultTypes.length);
for (Class faultType : faultTypes) {
- faultDataTypes.add(new DataTypeImpl<Class>(UNKNOWN_DATABINDING, faultType, faultType));
+ // Only add checked exceptions
+ if (Exception.class.isAssignableFrom(faultType) && (!RuntimeException.class.isAssignableFrom(faultType))) {
+ faultDataTypes.add(new DataTypeImpl<Class>(UNKNOWN_DATABINDING, faultType, faultType));
+ }
}
DataType<List<DataType>> inputType = new DataTypeImpl<List<DataType>>(IDL_INPUT, Object[].class,
Modified: incubator/tuscany/java/sca/modules/interface-java-xml/src/test/java/org/apache/tuscany/interfacedef/java/introspection/impl/JavaInterfaceProcessorRegistryImplTestCase.java
URL: http://svn.apache.org/viewvc/incubator/tuscany/java/sca/modules/interface-java-xml/src/test/java/org/apache/tuscany/interfacedef/java/introspection/impl/JavaInterfaceProcessorRegistryImplTestCase.java?view=diff&rev=529147&r1=529146&r2=529147
==============================================================================
--- incubator/tuscany/java/sca/modules/interface-java-xml/src/test/java/org/apache/tuscany/interfacedef/java/introspection/impl/JavaInterfaceProcessorRegistryImplTestCase.java (original)
+++ incubator/tuscany/java/sca/modules/interface-java-xml/src/test/java/org/apache/tuscany/interfacedef/java/introspection/impl/JavaInterfaceProcessorRegistryImplTestCase.java Sun Apr 15 23:54:46 2007
@@ -24,6 +24,7 @@
import static org.easymock.EasyMock.replay;
import static org.easymock.EasyMock.verify;
+import java.io.IOException;
import java.lang.reflect.Type;
import java.util.List;
@@ -65,8 +66,8 @@
List<DataType> faultTypes = baseInt.getFaultTypes();
assertEquals(1, faultTypes.size());
DataType<Type> fault0 = faultTypes.get(0);
- assertEquals(IllegalArgumentException.class, fault0.getPhysical());
- assertEquals(IllegalArgumentException.class, fault0.getLogical());
+ assertEquals(IOException.class, fault0.getPhysical());
+ assertEquals(IOException.class, fault0.getLogical());
}
public void testUnregister() throws Exception {
@@ -88,7 +89,7 @@
}
private static interface Base {
- int baseInt(int param) throws IllegalArgumentException;
+ int baseInt(int param) throws IllegalArgumentException, IOException;
}
private static interface Simple extends Base {
---------------------------------------------------------------------
To unsubscribe, e-mail: tuscany-commits-unsubscribe@ws.apache.org
For additional commands, e-mail: tuscany-commits-help@ws.apache.org