From 4e3d3354397374116577555200319975fc2ff86e Mon Sep 17 00:00:00 2001
From: Ysopwnable <161784452+N0zoM1z0@users.noreply.github.com>
Date: Sat, 10 Oct 2026 14:37:49 +0800
Subject: [PATCH] test(android): retain incomplete cleanup failure reasons
 (#1443)

Include the full stop disposition in the fixture assertion message, so incomplete owned teardown retains its typed reason in Vitest failure reporters. Preserve the existing cleanup requirement without retries or relaxed ownership checks.

Seven process-boundary tests, full serial typecheck/lint/duplication/format checks, and 19 final-head CI checks pass after integrating the combined-main fixture fix. A controlled failure-rendering probe verifies that the explicit message preserves the reason. The original CI cleanup failure remains unreproduced; this is diagnostic work with a synthetic JADX producer, not a root-cause or real-engine fix.

Refs #1441, #1446, #1448, #1065.
---
 tests/fixtures/android/jadx.ts | 5 ++++-
 1 file changed, 4 insertions(+), 1 deletion(-)

diff --git a/tests/fixtures/android/jadx.ts b/tests/fixtures/android/jadx.ts
index 3488647a0..c82469039 100644
--- a/tests/fixtures/android/jadx.ts
+++ b/tests/fixtures/android/jadx.ts
@@ -74,7 +74,10 @@ export const createJadxProtocolFixture = async (
             ownsProcessLifetime: true,
             cleanup: () => cleanupOwnedProcessGroup(spawned.ownership),
           });
-          expect((await supervisor.stop()).status).not.toBe("incomplete");
+          const stopped = await supervisor.stop();
+          expect(stopped, JSON.stringify(stopped)).not.toMatchObject({
+            status: "incomplete",
+          });
           if (options.cwd !== undefined)
             await rm(options.cwd, { recursive: true, force: true });
         });