From d82ad1faf0b2774c56a10d8a81b09defacd99c7b Mon Sep 17 00:00:00 2001
From: Ysopwnable <161784452+N0zoM1z0@users.noreply.github.com>
Date: Wed, 14 Oct 2026 08:28:23 +0800
Subject: [PATCH] test(android): preserve malformed DEX prototype rejection
 (#2917)

Preserve malformed DEX parameter/return rejection with tagged producer errors and exact issue paths.

The equivalent portable descriptor fix is already on main via #2910.

Co-authored-by: Kaoru0822 <208281322+kaoru0822-kitauji@users.noreply.github.com>
---
 tests/boundary/android/JadxAnalysis.test.ts | 62 +++++++++++++--------
 1 file changed, 40 insertions(+), 22 deletions(-)

diff --git a/tests/boundary/android/JadxAnalysis.test.ts b/tests/boundary/android/JadxAnalysis.test.ts
index 36b0ab9d9..ad7d0d614 100644
--- a/tests/boundary/android/JadxAnalysis.test.ts
+++ b/tests/boundary/android/JadxAnalysis.test.ts
@@ -279,26 +279,44 @@ it("preserves bounded instruction coverage separately from using-node coverage",
   });
 });
 
-it("rejects malformed method descriptors at the Android adapter boundary", async () => {
-  const invalidSummary = {
-    ...summary,
-    methods: summary.methods.map((method) => ({
-      ...method,
-      dex_descriptor: "not-a-dex-descriptor",
-    })),
-  };
-  await expect(
-    analyzeJadxRequest(
-      {
-        json: () => Promise.resolve(invalidSummary),
-        text: () => Promise.reject(new Error("unexpected text call")),
+it.each([
+  "not-a-dex-descriptor",
+  "(Lbad(name;)V",
+  "(Lbad)name;)V",
+  "()Lbad(name;",
+  "()Lbad)name;",
+])(
+  "rejects malformed DEX prototype %s at the adapter boundary",
+  async (descriptor) => {
+    const invalidSummary = {
+      ...summary,
+      methods: summary.methods.map((method) => ({
+        ...method,
+        dex_descriptor: descriptor,
+      })),
+    };
+    await expect(
+      analyzeJadxRequest(
+        {
+          json: () => Promise.resolve(invalidSummary),
+          text: () => Promise.reject(new Error("unexpected text call")),
+        },
+        {
+          operation: "inspect_android_class",
+          input: { path: "/fixture.apk", class_name: "fixture.Target" },
+        },
+        {} as never,
+        engine,
+      ),
+    ).rejects.toMatchObject({
+      _tag: "AnalysisOutputError",
+      cause: {
+        issues: expect.arrayContaining([
+          expect.objectContaining({
+            path: ["methods", 0, "dex_descriptor"],
+          }),
+        ]),
       },
-      {
-        operation: "inspect_android_class",
-        input: { path: "/fixture.apk", class_name: "fixture.Target" },
-      },
-      {} as never,
-      engine,
-    ),
-  ).rejects.toThrow();
-});
+    });
+  },
+);