Removed default registristation of generic exception types

Author: sigiesec

com.btc.serviceidl.tests/src/com/btc/serviceidl/tests/generator/cpp/CppGeneratorTest.xtend

@@ -76,9 +76,7 @@ class CppGeneratorTest extends AbstractGeneratorTest
             #include "modules/Infrastructure/ServiceHost/Demo/API/ServiceAPI/include/IKeyValueStore.h"
             
             #include "modules/Commons/include/BeginCabInclude.h"     // CAB -->
-            #include "Commons/Core/include/InvalidArgumentException.h"
             #include "Commons/Core/include/String.h"
-            #include "Commons/Core/include/UnsupportedOperationException.h"
             #include "Commons/CoreExtras/include/UUID.h"
             #include "ServiceComm/API/include/IServiceFaultHandler.h"
             #include "ServiceComm/Base/include/DefaultServiceFaultHandler.h"
@@ -103,12 +101,6 @@ class CppGeneratorTest extends AbstractGeneratorTest
                
                void RegisterKeyValueStoreServiceFaults(BTC::ServiceComm::API::IServiceFaultHandlerManager& serviceFaultHandlerManager)
                {
-                  
-                  // most commonly used exception types
-                  BTC::ServiceComm::Base::RegisterServiceFault<BTC::Commons::Core::InvalidArgumentException>(
-                     serviceFaultHandlerManager, BTC::Commons::Core::String("BTC.Commons.Core.InvalidArgumentException"));
-                  BTC::ServiceComm::Base::RegisterServiceFault<BTC::Commons::Core::UnsupportedOperationException>(
-                     serviceFaultHandlerManager, BTC::Commons::Core::String("BTC.Commons.Core.UnsupportedOperationException"));
                }
             }}}}}}}
         ''')

com.btc.serviceidl.tests/src/com/btc/serviceidl/tests/generator/java/JavaGeneratorTest.xtend

@@ -316,8 +316,6 @@ public class TypesCodec {
             import com.btc.cab.servicecomm.api.IServiceFaultHandler;
             import com.btc.cab.servicecomm.faulthandling.DefaultServiceFaultHandler;
             import com.btc.cab.servicecomm.faulthandling.ErrorMessage;
-            import java.lang.IllegalArgumentException;
-            import java.lang.UnsupportedOperationException;
             import java.lang.reflect.Constructor;
             import java.util.Optional;
             import org.apache.commons.collections4.BidiMap;
@@ -330,10 +328,6 @@ public class TypesCodec {
 
                static
                {
-                  
-                  // most commonly used exception types
-                  errorMap.put("BTC.Commons.Core.InvalidArgumentException", IllegalArgumentException.class);
-                  errorMap.put("BTC.Commons.Core.UnsupportedOperationException", UnsupportedOperationException.class);
                }
 
                public static final IServiceFaultHandler createServiceFaultHandler()

com.btc.serviceidl/src/com/btc/serviceidl/generator/cpp/CodecGenerator.xtend

@@ -68,12 +68,6 @@ class CodecGenerator extends BasicCppGenerator
                       «val exceptionName = exception.getCommonExceptionName(qualifiedNameProvider)»
                       faultHandlers["«exceptionName»"] = [](«cabString» const& msg) { return «cabCreateUnique»<«exceptionType»>(msg); };
                   «ENDFOR»
-                  
-                  // most commonly used exception types
-                  «val defaultExceptions = typeResolver.defaultExceptionRegistration»
-                  «FOR exception : defaultExceptions.keySet.sort»
-                      faultHandlers["«exception»"] = [](«cabString» const& msg) { return «cabCreateUnique»<«defaultExceptions.get(exception)»>(msg); };
-                  «ENDFOR»
                });
             }
             

com.btc.serviceidl/src/com/btc/serviceidl/generator/cpp/ServiceAPIGenerator.xtend

@@ -231,13 +231,6 @@ class ServiceAPIGenerator extends BasicCppGenerator {
             «registerServiceFault»<«resolveExcName»>(
                serviceFaultHandlerManager, «cabString»("«exception.getCommonExceptionName(qualifiedNameProvider)»"));
          «ENDFOR»
-         
-         // most commonly used exception types
-         «val defaultExceptions = typeResolver.defaultExceptionRegistration»
-         «FOR exception : defaultExceptions.keySet.sort»
-            «registerServiceFault»<«defaultExceptions.get(exception)»>(
-               serviceFaultHandlerManager, «cabString»("«exception»"));
-         «ENDFOR»
       }
       '''
     }

com.btc.serviceidl/src/com/btc/serviceidl/generator/cpp/TypeResolverExtensions.xtend

@@ -2,8 +2,6 @@ package com.btc.serviceidl.generator.cpp
 
 import com.btc.serviceidl.generator.common.GuidMapper
 import com.btc.serviceidl.idl.StructDeclaration
-import com.btc.serviceidl.util.Constants
-import java.util.Map
 
 class TypeResolverExtensions {
     static def String makeEventGUIDImplementations(extension TypeResolver typeResolver,
@@ -27,14 +25,4 @@ class TypeResolverExtensions {
       '''
     }
 
-   static def Map<String, String> getDefaultExceptionRegistration(extension TypeResolver typeResolver)
-   {
-      // TODO these must not be registered, the default must be empty
-      #{
-          Constants.INVALID_ARGUMENT_EXCEPTION_FAULT_HANDLER         -> resolveSymbol("BTC::Commons::Core::InvalidArgumentException")
-         ,Constants.UNSUPPORTED_OPERATION_EXCEPTION_FAULT_HANDLER    -> resolveSymbol("BTC::Commons::Core::UnsupportedOperationException")
-      }
-   }
-      
-    
 }

com.btc.serviceidl/src/com/btc/serviceidl/generator/dotnet/ServiceFaultHandlingGenerator.xtend

@@ -39,10 +39,6 @@ class ServiceFaultHandlingGenerator extends GeneratorBase
          
          static «className»()
          {
-            // most commonly used exception types
-            errorMap["«Constants.INVALID_ARGUMENT_EXCEPTION_FAULT_HANDLER»"] = typeof(«resolve("System.ArgumentException").fullyQualifiedName»);
-            errorMap["«Constants.UNSUPPORTED_OPERATION_EXCEPTION_FAULT_HANDLER»"] = typeof(«resolve("System.NotSupportedException").fullyQualifiedName»);
-            
             «FOR e : raisedExceptions.sortBy[name] SEPARATOR System.lineSeparator»
                errorMap["«e.getCommonExceptionName(qualifiedNameProvider)»"] = typeof(«resolve(e)»);
             «ENDFOR»

com.btc.serviceidl/src/com/btc/serviceidl/generator/dotnet/TypeResolver.xtend

@@ -52,9 +52,8 @@ class TypeResolver
 
     def ResolvedName resolve(String name)
     {
-        val effectiveName = resolveException(name) ?: name
-        val fullyQualifiedName = QualifiedName.create(
-            effectiveName.split(Pattern.quote(Constants.SEPARATOR_PACKAGE)))
+        val effectiveName = name
+        val fullyQualifiedName = QualifiedName.create(effectiveName.split(Pattern.quote(Constants.SEPARATOR_PACKAGE)))
         val namespace = fullyQualifiedName.skipLast(1).toString
 
         if (namespace.startsWith("System"))
@@ -148,19 +147,6 @@ class TypeResolver
         return new ResolvedName(result, TransformType.PACKAGE, FULLY_QUALIFIED)
     }
 
-    private static def String resolveException(String name)
-    {
-        // temporarily some special handling for exceptions, because not all
-        // C++ CAB exceptions are supported by the .NET CAB
-        switch (name)
-        {
-            case "BTC.Commons.Core.InvalidArgumentException":
-                return "System.ArgumentException"
-            default:
-                return null
-        }
-    }
-
     private def boolean isSameProject(QualifiedName referencedPackage)
     {
         GeneratorUtil.getTransformedModuleName(parameterBundle, ArtifactNature.DOTNET, TransformType.PACKAGE) ==

com.btc.serviceidl/src/com/btc/serviceidl/generator/java/BasicJavaSourceGenerator.xtend

@@ -54,8 +54,7 @@ class BasicJavaSourceGenerator
 
     def dispatch String toText(ExceptionDeclaration element)
     {
-        typeResolver.resolveException(qualifiedNameProvider.getFullyQualifiedName(element).toString) ?:
-            '''«typeResolver.resolve(element)»'''
+        '''«typeResolver.resolve(element)»'''
     }
 
     def dispatch String toText(SequenceDeclaration item)

com.btc.serviceidl/src/com/btc/serviceidl/generator/java/ServiceFaultHandlerFactoryGenerator.xtend

@@ -47,8 +47,6 @@ class ServiceFaultHandlerFactoryGenerator
 
         // TODO except for the static initializer, this can be extracted into a reusable class, which can be provided 
         // from com.btc.cab.servicecomm
-        // TODO InvalidArgumentException and UnsupportedOperationException should not be added to the error map, only 
-        // service-specific subtypes 
         '''
         public class «className»
         {
@@ -61,10 +59,6 @@ class ServiceFaultHandlerFactoryGenerator
               «FOR exception : exceptions.sortBy[name]»
                   errorMap.put("«Util.getCommonExceptionName(exception, basicJavaSourceGenerator.qualifiedNameProvider)»", «getClassOrObject(typeResolver.resolve(exception))»);
               «ENDFOR»
-              
-              // most commonly used exception types
-              errorMap.put("«Constants.INVALID_ARGUMENT_EXCEPTION_FAULT_HANDLER»", «getClassOrObject(typeResolver.resolve("java.lang.IllegalArgumentException"))»);
-              errorMap.put("«Constants.UNSUPPORTED_OPERATION_EXCEPTION_FAULT_HANDLER»", «getClassOrObject(typeResolver.resolve("java.lang.UnsupportedOperationException"))»);
            }
            
            public static final «typeResolver.resolve(JavaClassNames.SERVICE_FAULT_HANDLER)» createServiceFaultHandler()

com.btc.serviceidl/src/com/btc/serviceidl/generator/java/TypeResolver.xtend

@@ -105,14 +105,6 @@ class TypeResolver
     {
         var name = qualifiedNameProvider.getFullyQualifiedName(element)
 
-        // try to resolve CAB-related pseudo-exceptions
-        if (element.isException)
-        {
-            val exceptionName = resolveException(name.toString)
-            if (exceptionName !== null)
-                return new ResolvedName(exceptionName, TransformType.PACKAGE, fullyQualified)
-        }
-
         if (name === null)
         {
             if (element instanceof AbstractType)
@@ -157,20 +149,6 @@ class TypeResolver
         return new ResolvedName(fullyQualifiedName, TransformType.PACKAGE, fullyQualified)
     }
 
-    def String resolveException(String name)
-    {
-        // temporarily some special handling for exceptions, because not all
-        // C++ CAB exceptions are supported by the Java CAB
-        switch (name)
-        {
-            case "BTC.Commons.Core.InvalidArgumentException":
-                // TODO shouldn't this use resolve("java.util.IllegalArgumentException")?
-                "IllegalArgumentException"
-            default:
-                null
-        }
-    }
-
     private static def String getObservableName(EventDeclaration event)
     {
         if (event.name === null)

com.btc.serviceidl/src/com/btc/serviceidl/util/Constants.java

@@ -55,11 +55,6 @@ public interface Constants {
     public static final String FILE_NAME_ODB_COMMON    = "common";
     public static final String FILE_NAME_ODB_TRAITS    = "traits";
 
-    // GUID for default exception registration thrown by not implemented methods
-    public static final String UNSUPPORTED_OPERATION_EXCEPTION_FAULT_HANDLER = "BTC.Commons.Core.UnsupportedOperationException";
-
-    public static final String INVALID_ARGUMENT_EXCEPTION_FAULT_HANDLER = "BTC.Commons.Core.InvalidArgumentException";
-
     public static final int DEFAULT_PORT = 5555;
 
     public static final String AUTO_GENERATED_METHOD_STUB_MESSAGE = "Auto-generated method stub is not implemented!";