Logging to all NetSuite operations

Also improved exception handling

Author: flbulgarelli

pom.xml

@@ -95,7 +95,18 @@
 					<appendAssemblyId>false</appendAssemblyId>
 				</configuration>
 			</plugin>
-			<!--
+			<plugin>
+                            <groupId>org.apache.maven.plugins</groupId>
+                            <artifactId>maven-release-plugin</artifactId>
+                            <configuration>
+                               <autoVersionSubmodules>true</autoVersionSubmodules>
+                               <pushChanges>false</pushChanges>
+                           </configuration>
+                           <version>2.1</version>
+                         </plugin>
+
+
+                        <!--
 				<plugin> <groupId>org.apache.cxf</groupId>
 				<artifactId>cxf-codegen-plugin</artifactId>
 				<version>${cxf.version}</version> <executions> <execution>

src/main/java/org/mule/module/netsuite/api/NetSuiteClientAdaptor.java

@@ -10,20 +10,23 @@
 
 package org.mule.module.netsuite.api;
 
+import org.mule.module.netsuite.NetSuiteCloudConnector;
 import org.mule.module.netsuite.api.annotation.NetSuiteOperation;
 
-import com.netsuite.webservices.platform.core_2010_2.Record;
-
 import java.lang.reflect.InvocationHandler;
+import java.lang.reflect.InvocationTargetException;
 import java.lang.reflect.Method;
 import java.lang.reflect.Proxy;
 import java.util.List;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 import org.springframework.core.annotation.AnnotationUtils;
 
 public final class NetSuiteClientAdaptor
 {
     private static final Object LOCK = new Object();
+    private static final Logger log = LoggerFactory.getLogger(NetSuiteCloudConnector.class);
 
     private NetSuiteClientAdaptor()
     {
@@ -37,28 +40,62 @@ public static NetSuiteClient<List<Object>, RuntimeException, Void> adapt(final N
             new InvocationHandler()
             {
                 public Object invoke(Object proxy, Method method, Object[] args) throws Throwable
+                {
+                    if (log.isDebugEnabled())
+                    {
+                        log.debug("Entering into {} with args {}", method.getName(), args);
+                    }
+                    try
+                    {
+                        Object ret = handle(method, args);
+                        if (log.isDebugEnabled())
+                        {
+                            log.debug("Returning from {} with value {}", method.getName(), ret);
+                        }
+                        return ret;
+                    }
+                    catch (Throwable e)
+                    {
+                        log.warn("Throwing {} at {}", e, method.getName());
+                        throw e;
+                    }
+                }
+
+                private Object handle(Method method, Object[] args) throws Throwable
                 {
                     NetSuiteOperation operationMetadata = AnnotationUtils.findAnnotation(method,
                         NetSuiteOperation.class);
                     if (operationMetadata == null)
                     {
-                        return method.invoke(client, args);
+                        return invokeMethod(method, client, args);
                     }
                     try
                     {
                         synchronized (LOCK)
                         {
-                            return adaptReturnType(method.invoke(client, args), operationMetadata);
+                            return adaptReturnType(invokeMethod(method, client, args), operationMetadata);
                         }
                     }
-                    catch (Exception e)
+                    catch (Throwable e)
                     {
                         throw NetSuiteGenericException.soften(e);
                     }
                 }
             });
     }
 
+    private static Object invokeMethod(Method m, Object obj, Object... args) throws Throwable
+    {
+        try
+        {
+            return m.invoke(obj, args);
+        }
+        catch (InvocationTargetException e)
+        {
+            throw e.getCause();
+        }
+    }
+
     private static Object adaptReturnType(Object returnValue, NetSuiteOperation operationMetadata)
         throws Throwable
     {

src/main/java/org/mule/module/netsuite/api/NetSuiteGenericException.java

@@ -24,7 +24,7 @@ public NetSuiteGenericException(String message)
         super(message);
     }
 
-    public static RuntimeException soften(Exception e)
+    public static RuntimeException soften(Throwable e)
     {
         if (!(e instanceof RuntimeException))
         {