[jnigen] Use check:jni for tests and fix warnings (#1077)

dart-lang · Apr 8, 2024 · 26594f2 · 26594f2
1 parent bd96c0a
commit 26594f2
Show file tree

Hide file tree

Showing 16 changed files with 27 additions and 14 deletions.
diff --git a/pkgs/jni/src/dartjni.c b/pkgs/jni/src/dartjni.c
@@ -521,6 +521,8 @@ JniExceptionDetails getExceptionDetails(jthrowable exception) {
   JniExceptionDetails details;
   details.message = (*jniEnv)->CallObjectMethod(
       jniEnv, exception, exceptionMethods.toStringMethod);
+  // No exception is thrown from toString.
+  (*jniEnv)->ExceptionClear(jniEnv);
   jobject buffer =
       (*jniEnv)->NewObject(jniEnv, exceptionMethods.byteArrayOutputStreamClass,
                            exceptionMethods.byteArrayOutputStreamCtor);
@@ -529,8 +531,12 @@ JniExceptionDetails getExceptionDetails(jthrowable exception) {
                            exceptionMethods.printStreamCtor, buffer);
   (*jniEnv)->CallVoidMethod(
       jniEnv, exception, exceptionMethods.printStackTraceMethod, printStream);
+  // No exception is thrown from printStackTrace.
+  (*jniEnv)->ExceptionClear(jniEnv);
   details.stacktrace = (*jniEnv)->CallObjectMethod(
       jniEnv, buffer, exceptionMethods.toStringMethod);
+  // No exception is thrown from toString.
+  (*jniEnv)->ExceptionClear(jniEnv);
   details.message = to_global_ref(details.message);
   details.stacktrace = to_global_ref(details.stacktrace);
   return details;

diff --git a/pkgs/jni/test/boxed_test.dart b/pkgs/jni/test/boxed_test.dart
@@ -13,7 +13,7 @@ void main() {
   // Don't forget to initialize JNI.
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/exception_test.dart b/pkgs/jni/test/exception_test.dart
@@ -20,8 +20,7 @@ void main() {
       Jni.spawn(dylibDir: "wrong_dir");
     } on HelperNotFoundError catch (_) {
       // stderr.write("\n$_\n");
-      Jni.spawnIfNotExists(
-          dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+      spawnJvm();
       caught = true;
     } on JniVmExistsError {
       stderr.writeln('cannot verify: HelperNotFoundError thrown');

diff --git a/pkgs/jni/test/global_env_test.dart b/pkgs/jni/test/global_env_test.dart
@@ -26,7 +26,7 @@ void main() {
 
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/isolate_test.dart b/pkgs/jni/test/isolate_test.dart
@@ -14,7 +14,7 @@ void main() {
   // Don't forget to initialize JNI.
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/jarray_test.dart b/pkgs/jni/test/jarray_test.dart
@@ -13,7 +13,7 @@ void main() {
   // Don't forget to initialize JNI.
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/jbyte_buffer_test.dart b/pkgs/jni/test/jbyte_buffer_test.dart
@@ -14,7 +14,7 @@ void main() {
   // Don't forget to initialize JNI.
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/jlist_test.dart b/pkgs/jni/test/jlist_test.dart
@@ -13,7 +13,7 @@ void main() {
   // Don't forget to initialize JNI.
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/jmap_test.dart b/pkgs/jni/test/jmap_test.dart
@@ -12,7 +12,7 @@ import 'test_util/test_util.dart';
 void main() {
   // Don't forget to initialize JNI.
   if (!Platform.isAndroid) {
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/jobject_test.dart b/pkgs/jni/test/jobject_test.dart
@@ -17,7 +17,7 @@ void main() {
   // Don't forget to initialize JNI.
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/jset_test.dart b/pkgs/jni/test/jset_test.dart
@@ -13,7 +13,7 @@ void main() {
   // Don't forget to initialize JNI.
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/jstring_test.dart b/pkgs/jni/test/jstring_test.dart
@@ -13,7 +13,7 @@ void main() {
   // Don't forget to initialize JNI.
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/load_test.dart b/pkgs/jni/test/load_test.dart
@@ -32,7 +32,7 @@ void doGC() {
 void main() {
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jni/test/test_util/test_util.dart b/pkgs/jni/test/test_util/test_util.dart
@@ -4,6 +4,7 @@
 
 import 'dart:io';
 
+import 'package:jni/jni.dart';
 import 'package:jni/src/build_util/build_util.dart';
 
 typedef TestCaseCallback = void Function();
@@ -34,3 +35,8 @@ void checkDylibIsUpToDate() {
     exit(1);
   }
 }
+
+void spawnJvm() {
+  Jni.spawnIfNotExists(
+      dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m", "-Xcheck:jni"]);
+}
diff --git a/pkgs/jni/test/type_test.dart b/pkgs/jni/test/type_test.dart
@@ -205,7 +205,7 @@ final class $FType extends JObjType<F> {
 void main() {
   if (!Platform.isAndroid) {
     checkDylibIsUpToDate();
-    Jni.spawnIfNotExists(dylibDir: "build/jni_libs", jvmOptions: ["-Xmx128m"]);
+    spawnJvm();
   }
   run(testRunner: test);
 }

diff --git a/pkgs/jnigen/test/test_util/bindings_test_setup.dart b/pkgs/jnigen/test/test_util/bindings_test_setup.dart
@@ -67,6 +67,8 @@ Future<void> bindingsTestSetup() async {
       tempClassDir.path,
       ...jacksonJars,
       kotlinTestJar,
+    ], jvmOptions: [
+      "-Xcheck:jni",
     ]);
   }
 }