try to catch exception when invoking a method in ObjCClass

README.md

@@ -96,6 +96,8 @@ implementation of Objective-C interfaces in Java.
 * separate same parts of jna-platform (like jna-platform-extended)
 * deprecate rococoa-contrib
 * ~~selector and java method binding for notification~~
+* exception in callback method cannot be shown as reason, shown as "Exception calling method for selector foo:"
+  * `OCInvocationCallbacks.java:170`
 
 ----
 

rococoa-core/src/main/java/org/rococoa/cocoa/foundation/NSArray.java

@@ -69,7 +69,7 @@ static NSMutableArray toArray(NSObject...objects) {
     @SuppressWarnings("unchecked")
     public <T extends NSObject> List<T> toList() {
         List<NSObject> result = new ArrayList<>();
-        for (int i = 0; i <this.count(); i++) {
+        for (int i = 0; i < this.count(); i++) {
             result.add(this.objectAtIndex(i));
         }
         return (List<T>) result;

rococoa-core/src/main/java/org/rococoa/internal/ObjCObjectInvocationHandler.java

@@ -28,6 +28,7 @@
 import java.util.ConcurrentModificationException;
 import java.util.List;
 import java.util.concurrent.Callable;
+import java.util.logging.Level;
 import java.util.logging.Logger;
 
 import com.sun.jna.Pointer;
@@ -170,7 +171,12 @@ public Object intercept(@This Object proxy, @Origin Method method, @AllArguments
         if (!Modifier.isAbstract(method.getModifiers())) {
             // method is not abstract, so a Java override has been provided, which we call
 logging.finest(String.format("superMethod.invoke [%s %s].%s(%s)", javaClassName, ocInstance, method.getName(), new VarArgsUnpacker(args)));
-            return superMethod.invoke(proxy, args);
+            try {
+                return superMethod.invoke(proxy, args);
+            } catch (Throwable t) {
+logging.log(Level.WARNING, String.format("superMethod.invoke [%s %s].%s(%s) failure", javaClassName, ocInstance, method.getName(), new VarArgsUnpacker(args)), t);
+                throw t;
+            }
         }
         // normal case
         return invokeCocoa(method, args);

rococoa-core/src/test/java/Test1.java

@@ -10,7 +10,6 @@
 import java.util.concurrent.Executors;
 import java.util.concurrent.atomic.AtomicReference;
 
-import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.condition.DisabledIfEnvironmentVariable;
 import org.rococoa.cocoa.foundation.NSString;
@@ -20,29 +19,13 @@
 
 
 /**
- * Test1.
+ * Integration Test.
  *
  * @author <a href="mailto:[email protected]">Naohide Sano</a> (nsano)
  * @version 0.00 2023-06-30 nsano initial version <br>
  */
 public class Test1 {
 
-    @Test
-    @Disabled
-    void test1() throws Exception {
-//        MethodHandle methodHandleFieldDirect = lookup.unreflectGetter(fieldName);
-//        CallSite callSiteField = new ConstantCallSite(methodHandleFieldDirect);
-//        methodHandleFieldDirect = callSiteField.dynamicInvoker();
-//        name = (String) methodHandleFieldDirect.invokeExact(new Employee());
-//
-//
-////Lookup invoke dynamic
-//        methodType = MethodType.methodType(String.class);
-//        methodHandle = lookup.findVirtual(Employee.class, "getName", methodType);
-//        CallSite callSiteMethod = new ConstantCallSite(methodHandleFieldDirect);
-//        methodHandle = callSiteMethod.dynamicInvoker();
-    }
-
     @Test
     @DisabledIfEnvironmentVariable(named = "GITHUB_WORKFLOW", matches = ".*")
     void test2() throws Exception {

rococoa-core/src/test/java/TestCase.java

@@ -0,0 +1,51 @@
+/*
+ * Copyright (c) 2024 by Naohide Sano, All rights reserved.
+ *
+ * Programmed by Naohide Sano
+ */
+
+import java.util.List;
+
+import org.junit.jupiter.api.Disabled;
+import org.junit.jupiter.api.Test;
+import org.rococoa.cocoa.foundation.NSArray;
+
+import static org.junit.jupiter.api.Assertions.assertThrows;
+
+
+/**
+ * TestCase.
+ *
+ * @author <a href="mailto:[email protected]">Naohide Sano</a> (nsano)
+ * @version 0.00 2024-03-26 nsano initial version <br>
+ */
+public class TestCase {
+
+    @Test
+    @Disabled
+    void test1() throws Exception {
+//        MethodHandle methodHandleFieldDirect = lookup.unreflectGetter(fieldName);
+//        CallSite callSiteField = new ConstantCallSite(methodHandleFieldDirect);
+//        methodHandleFieldDirect = callSiteField.dynamicInvoker();
+//        name = (String) methodHandleFieldDirect.invokeExact(new Employee());
+//
+//
+////Lookup invoke dynamic
+//        methodType = MethodType.methodType(String.class);
+//        methodHandle = lookup.findVirtual(Employee.class, "getName", methodType);
+//        CallSite callSiteMethod = new ConstantCallSite(methodHandleFieldDirect);
+//        methodHandle = callSiteMethod.dynamicInvoker();
+    }
+
+    NSArray array() {
+        return null;
+    }
+
+    @Test
+    void test2() throws Exception {
+        NSArray array = array();
+        assertThrows(NullPointerException.class, () -> {
+            List<?> list = array.toList();
+        });
+    }
+}