[J2KT] Enable switchexpression integration test.

PiperOrigin-RevId: 697998734
google · Nov 19, 2024 · acb6e59 · acb6e59
1 parent eb93f54
commit acb6e59
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 16 deletions.
diff --git a/transpiler/javatests/com/google/j2cl/integration/java/switchexpression/BUILD b/transpiler/javatests/com/google/j2cl/integration/java/switchexpression/BUILD
@@ -11,8 +11,8 @@ package(
 integration_test(
     name = "switchexpression",
     srcs = glob(["*.java"]),
-    # TODO(b/375058137): Enable when bug is fixed.
-    enable_kt = False,
+    # TODO(b/379697518): Enable once @DoNotAutobox is propagated by J2KT.
+    enable_kt_web = False,
     # TODO(b/375058020): Enable when bug is fixed.
     enable_wasm = False,
     javacopts = [

diff --git a/transpiler/javatests/com/google/j2cl/integration/java/switchexpression/Main.java b/transpiler/javatests/com/google/j2cl/integration/java/switchexpression/Main.java
@@ -176,20 +176,24 @@ private static void testSwitchWithErasureCast() {
           }
         };
 
-    assertThrowsClassCastException(
-        () -> {
-          Supplier<Numbers> integerSupplier = (Supplier) () -> new Integer(1);
-          int unusedResult =
-              // supplier.get() goes through the `T Supplier<T>.get()` method in the interface, and
-              // thus must have an erasure cast. In this case it should throw CCE and not just
-              // fall into the default case.
-              switch (integerSupplier.get()) {
-                default -> {
-                  fail();
-                  yield 0;
-                }
-              };
-        });
+    if (!TestUtils.isJ2KtNative()) {
+      // Switch statements in Kotlin/Native do not throw erasure casts, instead they would flow to
+      // the default case.
+      assertThrowsClassCastException(
+          () -> {
+            Supplier<Numbers> integerSupplier = (Supplier) () -> new Integer(1);
+            int unusedResult =
+                // supplier.get() goes through the `T Supplier<T>.get()` method in the interface,
+                // and thus must have an erasure cast. In this case it should throw CCE and not just
+                // fall into the default case.
+                switch (integerSupplier.get()) {
+                  default -> {
+                    fail();
+                    yield 0;
+                  }
+                };
+          });
+    }
   }
 
   private static void testStringSwitch() {