svn commit: r918931 - /tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java

[slaws]

Author: slaws
Date: Thu Mar  4 11:01:36 2010
New Revision: 918931

URL: http://svn.apache.org/viewvc?rev=918931&view=rev
Log:
improve the operation not found error message

Modified:
    
tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java

Modified: 
tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java
URL: 
http://svn.apache.org/viewvc/tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java?rev=918931&r1=918930&r2=918931&view=diff
==============================================================================
--- 
tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java
 (original)
+++ 
tuscany/sca-java-2.x/trunk/modules/assembly/src/main/java/org/apache/tuscany/sca/interfacedef/impl/InterfaceContractMapperImpl.java
 Thu Mar  4 11:01:36 2010
@@ -259,7 +259,11 @@
             Operation targetOperation = map(target.getInterface(), operation);
             if (targetOperation == null) {
                 if (!silent) {
-                    throw new 
IncompatibleInterfaceContractException("Operation not found on target", source, 
target);
+                    throw new 
IncompatibleInterfaceContractException("Operation " + 
+                                                                     
operation.getName() +
+                                                                     " not 
found on target", 
+                                                                     source, 
+                                                                     target);
                 } else {
                     return false;
                 }
@@ -268,8 +272,11 @@
                 // FIXME: for remotable operation, only compare name for now
                 if (!isCompatible(operation, targetOperation, 
Compatibility.SUBSET)) {
                     if (!silent) {
-                        throw new 
IncompatibleInterfaceContractException("Target operations are not compatible",
-                                                                         
source, target);
+                        throw new 
IncompatibleInterfaceContractException("Target operations called " +
+                                                                         
operation.getName() +
+                                                                         " are 
not compatible",
+                                                                         
source, 
+                                                                         
target);
                     } else {
                         return false;
                     }