rn-iso 0.1.0 → 0.2.1

package/test/fixtures/sample-expo-project/app.json

@@ -1,6 +0,0 @@
-{
-  "expo": {
-    "ios": { "bundleIdentifier": "com.example.sample" },
-    "android": { "package": "com.example.sample" }
-  }
-}

package/test/fixtures/sample-expo-project/package.json

@@ -1,4 +0,0 @@
-{
-  "name": "sample-app",
-  "dependencies": { "expo": "~50.0.0" }
-}

package/test/metro.test.js

@@ -1,34 +0,0 @@
-import { test, afterEach } from 'node:test';
-import assert from 'node:assert/strict';
-import { resetExecutor } from '../src/exec.js';
-import { logFileFor, projectHash, buildMetroSpawnArgs } from '../src/metro.js';
-
-afterEach(() => resetExecutor());
-
-test('projectHash is deterministic and short', () => {
-  const a = projectHash('/foo/bar');
-  const b = projectHash('/foo/bar');
-  const c = projectHash('/foo/baz');
-  assert.equal(a, b);
-  assert.notEqual(a, c);
-  assert.equal(a.length, 12);
-});
-
-test('logFileFor uses RN_ISO_HOME and project hash', () => {
-  process.env.RN_ISO_HOME = '/tmp/test-rn-iso';
-  const path = logFileFor('/some/project');
-  assert.match(path, /^\/tmp\/test-rn-iso\/logs\/[0-9a-f]{12}\.log$/);
-  delete process.env.RN_ISO_HOME;
-});
-
-test('buildMetroSpawnArgs returns correct argv for expo', () => {
-  const { cmd, args } = buildMetroSpawnArgs({ isExpo: true, port: 8083 });
-  assert.equal(cmd, 'npx');
-  assert.deepEqual(args, ['expo', 'start', '--port', '8083']);
-});
-
-test('buildMetroSpawnArgs returns correct argv for bare', () => {
-  const { cmd, args } = buildMetroSpawnArgs({ isExpo: false, port: 8083 });
-  assert.equal(cmd, 'npx');
-  assert.deepEqual(args, ['react-native', 'start', '--port', '8083']);
-});

package/test/ports.test.js

@@ -1,76 +0,0 @@
-import { test, beforeEach, afterEach } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, rmSync } from 'fs';
-import { tmpdir } from 'os';
-import { join } from 'path';
-import { resetExecutor } from '../src/exec.js';
-import { upsertProject, setMetro } from '../src/config.js';
-import { computeNextPort, findReclaimablePort, allocatePort } from '../src/ports.js';
-
-let tmpHome;
-
-beforeEach(() => {
-  tmpHome = mkdtempSync(join(tmpdir(), 'rn-iso-test-'));
-  process.env.RN_ISO_HOME = tmpHome;
-});
-
-afterEach(() => {
-  rmSync(tmpHome, { recursive: true, force: true });
-  delete process.env.RN_ISO_HOME;
-  resetExecutor();
-});
-
-test('computeNextPort returns 8082 with no existing ports', () => {
-  assert.equal(computeNextPort(), 8082);
-});
-
-test('computeNextPort returns max + 1', () => {
-  upsertProject('/a', { bundleId: 'a', androidPackage: 'a', isExpo: false });
-  upsertProject('/b', { bundleId: 'b', androidPackage: 'b', isExpo: false });
-  setMetro('/a', 8082, null);
-  setMetro('/b', 8090, null);
-  assert.equal(computeNextPort(), 8091);
-});
-
-test('findReclaimablePort returns null when no projects', async () => {
-  const r = await findReclaimablePort('/excluded');
-  assert.equal(r, null);
-});
-
-test('findReclaimablePort skips the excluded project path', async () => {
-  upsertProject('/a', { bundleId: 'a', androidPackage: 'a', isExpo: false });
-  setMetro('/a', 8082, null);
-  // Mock isMetroRunning to always return false (port is dead)
-  const r = await findReclaimablePort('/a', async () => false);
-  assert.equal(r, null);
-});
-
-test('findReclaimablePort returns first dead port and its owner', async () => {
-  upsertProject('/a', { bundleId: 'a', androidPackage: 'a', isExpo: false });
-  upsertProject('/b', { bundleId: 'b', androidPackage: 'b', isExpo: false });
-  setMetro('/a', 8082, null);
-  setMetro('/b', 8083, null);
-  // 8082 alive, 8083 dead
-  const probe = async (port) => port === 8082;
-  const r = await findReclaimablePort('/c', probe);
-  assert.deepEqual(r, { port: 8083, ownerPath: '/b' });
-});
-
-test('allocatePort reclaims dead ports and removes the dead project', async () => {
-  upsertProject('/a', { bundleId: 'a', androidPackage: 'a', isExpo: false });
-  setMetro('/a', 8082, null);
-  const probe = async () => false;
-  const port = await allocatePort('/new', probe);
-  assert.equal(port, 8082);
-  // Caller should have removed /a -- verify via behavior
-  const { getProject } = await import('../src/config.js');
-  assert.equal(getProject('/a'), null);
-});
-
-test('allocatePort assigns a fresh port when nothing is reclaimable', async () => {
-  upsertProject('/a', { bundleId: 'a', androidPackage: 'a', isExpo: false });
-  setMetro('/a', 8082, null);
-  const probe = async () => true; // everything alive
-  const port = await allocatePort('/new', probe);
-  assert.equal(port, 8083);
-});

package/test/project.test.js

@@ -1,109 +0,0 @@
-import { test, beforeEach, afterEach } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, rmSync } from 'fs';
-import { tmpdir } from 'os';
-import { resolve, join } from 'path';
-import { findProjectRoot, detectIsExpo, detectBundleId, detectAndroidPackage, resolveRegisteredProject } from '../src/project.js';
-import { upsertProject } from '../src/config.js';
-
-const FIXTURES = resolve(import.meta.dirname, 'fixtures');
-const EXPO_PROJ = join(FIXTURES, 'sample-expo-project');
-const BARE_PROJ = join(FIXTURES, 'sample-bare-project');
-
-test('findProjectRoot walks up from cwd to find package.json', () => {
-  const nested = join(EXPO_PROJ, 'src');
-  assert.equal(findProjectRoot(nested), EXPO_PROJ);
-});
-
-test('findProjectRoot returns null when no package.json found', () => {
-  assert.equal(findProjectRoot('/'), null);
-});
-
-test('detectIsExpo true when expo deps + app.json has expo block', () => {
-  assert.equal(detectIsExpo(EXPO_PROJ), true);
-});
-
-test('detectIsExpo false when expo is not in dependencies', () => {
-  assert.equal(detectIsExpo(BARE_PROJ), false);
-});
-
-test('detectIsExpo trusts the ios script: react-native script wins even with expo dep', async () => {
-  // Mimics rainbow: `expo` in deps for prebuild/modules, but the ios script
-  // invokes react-native run-ios. Should NOT be flagged as Expo.
-  const { mkdtempSync, mkdirSync, writeFileSync, rmSync } = await import('fs');
-  const { tmpdir } = await import('os');
-  const tmp = mkdtempSync(join((await import('os')).tmpdir(), 'rn-iso-detect-'));
-  try {
-    writeFileSync(join(tmp, 'package.json'), JSON.stringify({
-      dependencies: { expo: '54.0.33' },
-      scripts: { ios: "react-native run-ios --simulator='iPhone 16 Pro'" },
-    }));
-    assert.equal(detectIsExpo(tmp), false);
-  } finally {
-    rmSync(tmp, { recursive: true, force: true });
-  }
-});
-
-test('detectIsExpo trusts the ios script: expo run:ios wins', async () => {
-  const { mkdtempSync, writeFileSync, rmSync } = await import('fs');
-  const tmp = mkdtempSync(join((await import('os')).tmpdir(), 'rn-iso-detect-'));
-  try {
-    writeFileSync(join(tmp, 'package.json'), JSON.stringify({
-      dependencies: { 'react-native': '0.74.0' },
-      scripts: { ios: 'expo run:ios' },
-    }));
-    assert.equal(detectIsExpo(tmp), true);
-  } finally {
-    rmSync(tmp, { recursive: true, force: true });
-  }
-});
-
-test('detectBundleId reads ios.bundleIdentifier from app.json', () => {
-  assert.equal(detectBundleId(EXPO_PROJ), 'com.example.sample');
-});
-
-test('detectBundleId falls back to pbxproj when app config has no bundle id', () => {
-  // BARE_PROJ has no app.json; the fixture pbxproj has main app id "me.sample"
-  // alongside an extension target with a longer suffix. Picks the most-common
-  // concrete value, tie-breaking by shortest length.
-  assert.equal(detectBundleId(BARE_PROJ), 'me.sample');
-});
-
-test('detectAndroidPackage reads android.package from app.json', () => {
-  assert.equal(detectAndroidPackage(EXPO_PROJ), 'com.example.sample');
-});
-
-test('detectAndroidPackage falls back to android/app/build.gradle (namespace)', () => {
-  assert.equal(detectAndroidPackage(BARE_PROJ), 'me.sample');
-});
-
-// resolveRegisteredProject -- needs an isolated config home.
-let tmpHome;
-beforeEach(() => {
-  tmpHome = mkdtempSync(join(tmpdir(), 'rn-iso-resolve-'));
-  process.env.RN_ISO_HOME = tmpHome;
-});
-afterEach(() => {
-  rmSync(tmpHome, { recursive: true, force: true });
-  delete process.env.RN_ISO_HOME;
-});
-
-test('resolveRegisteredProject finds a project by absolute path', () => {
-  upsertProject('/Users/x/Developer/agent-1', { bundleId: 'a', androidPackage: 'a', isExpo: false });
-  const r = resolveRegisteredProject('/Users/x/Developer/agent-1');
-  assert.equal(r.found, '/Users/x/Developer/agent-1');
-});
-
-test('resolveRegisteredProject does NOT do basename matching', () => {
-  upsertProject('/Users/x/Developer/agent-1', { bundleId: 'a', androidPackage: 'a', isExpo: false });
-  const r = resolveRegisteredProject('agent-1');
-  assert.equal(r.found, null);
-  assert.match(r.error, /Pass an absolute path/);
-});
-
-test('resolveRegisteredProject errors when path does not match anything', () => {
-  upsertProject('/Users/x/Developer/agent-1', { bundleId: 'a', androidPackage: 'a', isExpo: false });
-  const r = resolveRegisteredProject('/no/such/path');
-  assert.equal(r.found, null);
-  assert.match(r.error, /No registered project/);
-});

package/test/runner.test.js

@@ -1,209 +0,0 @@
-import { test, beforeEach, afterEach } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, mkdirSync, writeFileSync, rmSync } from 'fs';
-import { tmpdir } from 'os';
-import { join } from 'path';
-import { setExecutor, resetExecutor } from '../src/exec.js';
-import {
-  buildIosCommand,
-  buildAndroidCommand,
-  buildMetroCommand,
-  buildScriptCommand,
-  detectPackageManager,
-  detectScriptCli,
-  findLockfile,
-  getProjectScript,
-  resolveSimNameByUdid,
-} from '../src/runner.js';
-
-let tmpProj;
-
-function makeProj(files) {
-  tmpProj = mkdtempSync(join(tmpdir(), 'rn-iso-runner-'));
-  for (const [rel, content] of Object.entries(files)) {
-    const dest = join(tmpProj, rel);
-    mkdirSync(join(dest, '..'), { recursive: true });
-    writeFileSync(dest, content);
-  }
-  return tmpProj;
-}
-
-afterEach(() => {
-  if (tmpProj) rmSync(tmpProj, { recursive: true, force: true });
-  tmpProj = null;
-  resetExecutor();
-});
-
-// ---- Direct CLI fallback (no script) ----
-
-test('buildIosCommand expo fallback uses --device with UDID', () => {
-  const root = makeProj({ 'package.json': '{}' });
-  const cmd = buildIosCommand({ projectRoot: root, packageManager: 'npm', scriptName: 'ios', isExpo: true, udid: 'UDID-1', port: 8083, useScript: false });
-  assert.equal(cmd, 'npx expo run:ios --device UDID-1 --port 8083');
-});
-
-test('buildIosCommand bare fallback uses --udid (not --simulator)', () => {
-  const root = makeProj({ 'package.json': '{}' });
-  const cmd = buildIosCommand({ projectRoot: root, packageManager: 'npm', scriptName: 'ios', isExpo: false, udid: 'UDID-1', port: 8083, useScript: false });
-  assert.equal(cmd, 'npx react-native run-ios --udid UDID-1 --port 8083');
-});
-
-test('buildAndroidCommand expo fallback uses --device serial', () => {
-  const root = makeProj({ 'package.json': '{}' });
-  const cmd = buildAndroidCommand({ projectRoot: root, packageManager: 'npm', scriptName: 'android', isExpo: true, serial: 'emulator-5554', port: 8083, useScript: false });
-  assert.equal(cmd, 'npx expo run:android --device emulator-5554 --port 8083');
-});
-
-test('buildAndroidCommand bare fallback uses --deviceId and RCT_METRO_PORT', () => {
-  const root = makeProj({ 'package.json': '{}' });
-  const cmd = buildAndroidCommand({ projectRoot: root, packageManager: 'npm', scriptName: 'android', isExpo: false, serial: 'emulator-5554', port: 8083, useScript: false });
-  assert.equal(cmd, 'RCT_METRO_PORT=8083 npx react-native run-android --deviceId emulator-5554');
-});
-
-// ---- Script-based path ----
-
-test('buildIosCommand uses npm script with -- separator and --udid for RN script', () => {
-  const root = makeProj({
-    'package.json': JSON.stringify({ scripts: { ios: 'react-native run-ios --simulator="iPhone 16 Pro"' } }),
-  });
-  const cmd = buildIosCommand({ projectRoot: root, packageManager: 'npm', scriptName: 'ios', isExpo: false, udid: 'UDID-1', port: 8083 });
-  assert.equal(cmd, 'npm run ios -- --udid UDID-1 --port 8083');
-});
-
-test('buildIosCommand uses yarn script (no -- separator) and --udid for RN script', () => {
-  const root = makeProj({
-    'package.json': JSON.stringify({ scripts: { ios: 'react-native run-ios' } }),
-  });
-  const cmd = buildIosCommand({ projectRoot: root, packageManager: 'yarn', scriptName: 'ios', isExpo: false, udid: 'UDID-1', port: 8083 });
-  assert.equal(cmd, 'yarn ios --udid UDID-1 --port 8083');
-});
-
-test('buildIosCommand uses --device flag for expo script', () => {
-  const root = makeProj({
-    'package.json': JSON.stringify({ scripts: { ios: 'expo run:ios' } }),
-  });
-  const cmd = buildIosCommand({ projectRoot: root, packageManager: 'pnpm', scriptName: 'ios', isExpo: true, udid: 'UDID-1', port: 8083 });
-  assert.equal(cmd, 'pnpm ios --device UDID-1 --port 8083');
-});
-
-test('buildIosCommand falls back to direct CLI when script does not exist', () => {
-  const root = makeProj({
-    'package.json': JSON.stringify({ scripts: { build: 'echo' } }),
-  });
-  const cmd = buildIosCommand({ projectRoot: root, packageManager: 'npm', scriptName: 'ios', isExpo: false, udid: 'UDID-1', port: 8083 });
-  assert.equal(cmd, 'npx react-native run-ios --udid UDID-1 --port 8083');
-});
-
-test('buildAndroidCommand uses script via bun and --deviceId for RN script', () => {
-  const root = makeProj({
-    'package.json': JSON.stringify({ scripts: { android: 'react-native run-android' } }),
-  });
-  const cmd = buildAndroidCommand({ projectRoot: root, packageManager: 'bun', scriptName: 'android', isExpo: false, serial: 'emulator-5554', port: 8083 });
-  assert.equal(cmd, 'bun run android --deviceId emulator-5554 --port 8083');
-});
-
-// ---- Helpers ----
-
-test('detectPackageManager picks based on lockfile', () => {
-  const yarnRoot = makeProj({ 'yarn.lock': '' });
-  assert.equal(detectPackageManager(yarnRoot), 'yarn');
-  rmSync(yarnRoot, { recursive: true });
-
-  const pnpmRoot = makeProj({ 'pnpm-lock.yaml': '' });
-  assert.equal(detectPackageManager(pnpmRoot), 'pnpm');
-  rmSync(pnpmRoot, { recursive: true });
-
-  const bunRoot = makeProj({ 'bun.lock': '' });
-  assert.equal(detectPackageManager(bunRoot), 'bun');
-  rmSync(bunRoot, { recursive: true });
-
-  const npmRoot = makeProj({ 'package-lock.json': '' });
-  assert.equal(detectPackageManager(npmRoot), 'npm');
-  rmSync(npmRoot, { recursive: true });
-
-  const noLock = makeProj({ 'package.json': '{}' });
-  assert.equal(detectPackageManager(noLock), 'npm'); // default
-});
-
-test('detectPackageManager walks up to find lockfile in monorepo root', () => {
-  // Layout:
-  //   /tmp/.../    <-- yarn.lock here (workspace root)
-  //   /tmp/.../apps/mobile/  <-- our "project" (no lockfile of its own)
-  const root = makeProj({
-    'yarn.lock': '',
-    'package.json': JSON.stringify({ workspaces: ['apps/*'] }),
-    'apps/mobile/package.json': JSON.stringify({ name: 'mobile' }),
-  });
-  const projectRoot = join(root, 'apps/mobile');
-  assert.equal(detectPackageManager(projectRoot), 'yarn');
-});
-
-test('findLockfile returns the lockfile dir and pm', () => {
-  const root = makeProj({
-    'pnpm-lock.yaml': '',
-    'apps/mobile/package.json': '{}',
-  });
-  const found = findLockfile(join(root, 'apps/mobile'));
-  assert.equal(found.pm, 'pnpm');
-  assert.equal(found.dir, root);
-});
-
-test('findLockfile prefers nearest lockfile when nested ones exist', () => {
-  // Some monorepos intentionally have nested lockfiles per package; pick the
-  // closest one, not the workspace root's.
-  const root = makeProj({
-    'yarn.lock': '',
-    'apps/mobile/pnpm-lock.yaml': '',
-    'apps/mobile/package.json': '{}',
-  });
-  const found = findLockfile(join(root, 'apps/mobile'));
-  assert.equal(found.pm, 'pnpm');
-  assert.equal(found.dir, join(root, 'apps/mobile'));
-});
-
-test('detectScriptCli identifies expo, react-native, or unknown', () => {
-  assert.equal(detectScriptCli('expo run:ios'), 'expo');
-  assert.equal(detectScriptCli('npx expo run:ios'), 'expo');
-  assert.equal(detectScriptCli('react-native run-ios --simulator x'), 'react-native');
-  assert.equal(detectScriptCli('npx react-native start'), 'react-native');
-  assert.equal(detectScriptCli('echo hello'), 'unknown');
-  assert.equal(detectScriptCli(''), 'unknown');
-});
-
-test('getProjectScript reads scripts from package.json', () => {
-  const root = makeProj({
-    'package.json': JSON.stringify({ scripts: { ios: 'react-native run-ios' } }),
-  });
-  assert.equal(getProjectScript(root, 'ios'), 'react-native run-ios');
-  assert.equal(getProjectScript(root, 'missing'), null);
-});
-
-test('buildScriptCommand uses the right convention per package manager', () => {
-  assert.equal(buildScriptCommand('npm', 'ios', ['--udid X', '--port 8083']), 'npm run ios -- --udid X --port 8083');
-  assert.equal(buildScriptCommand('yarn', 'ios', ['--udid X']), 'yarn ios --udid X');
-  assert.equal(buildScriptCommand('pnpm', 'ios', ['--udid X']), 'pnpm ios --udid X');
-  assert.equal(buildScriptCommand('bun', 'ios', ['--udid X']), 'bun run ios --udid X');
-  assert.equal(buildScriptCommand('npm', 'ios', []), 'npm run ios');
-});
-
-// ---- Existing helpers retained ----
-
-test('buildMetroCommand picks expo or react-native', () => {
-  assert.equal(buildMetroCommand({ isExpo: true, port: 8083 }), 'npx expo start --port 8083');
-  assert.equal(buildMetroCommand({ isExpo: false, port: 8083 }), 'npx react-native start --port 8083');
-});
-
-test('resolveSimNameByUdid returns name from simctl JSON', () => {
-  setExecutor({
-    run: () => JSON.stringify({
-      devices: {
-        'com.apple.CoreSimulator.SimRuntime.iOS-17-0': [
-          { udid: 'UDID-1', name: 'iPhone 15', state: 'Booted', isAvailable: true },
-        ],
-      },
-    }),
-    runQuiet: () => null,
-    spawn: () => null,
-  });
-  assert.equal(resolveSimNameByUdid('UDID-1'), 'iPhone 15');
-});

package/test/sim-android.test.js

@@ -1,140 +0,0 @@
-import { test, beforeEach, afterEach } from 'node:test';
-import assert from 'node:assert/strict';
-import { mkdtempSync, rmSync } from 'fs';
-import { tmpdir } from 'os';
-import { join } from 'path';
-import { setExecutor, resetExecutor } from '../src/exec.js';
-import {
-  parseAvdList,
-  parseAdbDevices,
-  selectAndroidDevice,
-  nextConsolePort,
-} from '../src/sim/android.js';
-
-let tmpHome;
-
-beforeEach(() => {
-  tmpHome = mkdtempSync(join(tmpdir(), 'rn-iso-test-'));
-  process.env.RN_ISO_HOME = tmpHome;
-});
-
-afterEach(() => {
-  rmSync(tmpHome, { recursive: true, force: true });
-  delete process.env.RN_ISO_HOME;
-  resetExecutor();
-});
-
-test('parseAvdList strips header and blanks', () => {
-  const out = `INFO    | Storing AVDs in...\nPixel_6_API_34\nPixel_7_API_33\n`;
-  const avds = parseAvdList(out);
-  assert.deepEqual(avds, ['Pixel_6_API_34', 'Pixel_7_API_33']);
-});
-
-test('parseAdbDevices extracts running emulator console ports', () => {
-  const out = `List of devices attached\nemulator-5554\tdevice\nemulator-5556\tdevice\n0123456789ABCDEF\tdevice\n`;
-  const result = parseAdbDevices(out);
-  assert.deepEqual(result.emulators.sort((a, b) => a.consolePort - b.consolePort), [
-    { serial: 'emulator-5554', consolePort: 5554 },
-    { serial: 'emulator-5556', consolePort: 5556 },
-  ]);
-});
-
-test('parseAdbDevices ignores offline emulators', () => {
-  const out = `List of devices attached\nemulator-5554\toffline\nemulator-5556\tdevice\n`;
-  const result = parseAdbDevices(out);
-  assert.deepEqual(result.emulators, [{ serial: 'emulator-5556', consolePort: 5556 }]);
-});
-
-test('nextConsolePort returns 5554 when none claimed', () => {
-  assert.equal(nextConsolePort([]), 5554);
-});
-
-test('nextConsolePort returns next even port above max claimed', () => {
-  assert.equal(nextConsolePort([5554, 5556]), 5558);
-});
-
-test('selectAndroidDevice prefers existing assignment when AVD still exists', () => {
-  setExecutor({
-    run: (cmd) => {
-      if (cmd.includes('list-avds')) return 'Pixel_6_API_34\n';
-      if (cmd.includes('adb devices')) return 'List of devices attached\n';
-      throw new Error('unexpected: ' + cmd);
-    },
-    runQuiet: () => null,
-    spawn: () => null,
-  });
-  const result = selectAndroidDevice({
-    existingAvd: 'Pixel_6_API_34',
-    existingConsolePort: 5554,
-    claimedAvds: [],
-    claimedConsolePorts: [],
-  });
-  assert.deepEqual(result, {
-    kind: 'reuse',
-    avdName: 'Pixel_6_API_34',
-    consolePort: 5554,
-    isRunning: false,
-  });
-});
-
-test('selectAndroidDevice marks running when serial present in adb devices', () => {
-  setExecutor({
-    run: (cmd) => {
-      if (cmd.includes('list-avds')) return 'Pixel_6_API_34\n';
-      if (cmd.includes('adb devices')) return 'List of devices attached\nemulator-5554\tdevice\n';
-      throw new Error('unexpected');
-    },
-    runQuiet: () => null,
-    spawn: () => null,
-  });
-  const result = selectAndroidDevice({
-    existingAvd: 'Pixel_6_API_34',
-    existingConsolePort: 5554,
-    claimedAvds: [],
-    claimedConsolePorts: [],
-  });
-  assert.equal(result.isRunning, true);
-});
-
-test('selectAndroidDevice allocates first unclaimed AVD with next console port', () => {
-  setExecutor({
-    run: (cmd) => {
-      if (cmd.includes('list-avds')) return 'Pixel_6_API_34\nPixel_7_API_33\n';
-      if (cmd.includes('adb devices')) return 'List of devices attached\n';
-      throw new Error('unexpected');
-    },
-    runQuiet: () => null,
-    spawn: () => null,
-  });
-  const result = selectAndroidDevice({
-    existingAvd: null,
-    existingConsolePort: null,
-    claimedAvds: ['Pixel_6_API_34'],
-    claimedConsolePorts: [5554],
-  });
-  assert.deepEqual(result, {
-    kind: 'allocate',
-    avdName: 'Pixel_7_API_33',
-    consolePort: 5556,
-    isRunning: false,
-  });
-});
-
-test('selectAndroidDevice returns noAvd when no AVDs exist', () => {
-  setExecutor({
-    run: (cmd) => {
-      if (cmd.includes('list-avds')) return '';
-      if (cmd.includes('adb devices')) return 'List of devices attached\n';
-      throw new Error('unexpected');
-    },
-    runQuiet: () => null,
-    spawn: () => null,
-  });
-  const result = selectAndroidDevice({
-    existingAvd: null,
-    existingConsolePort: null,
-    claimedAvds: [],
-    claimedConsolePorts: [],
-  });
-  assert.equal(result.kind, 'noAvd');
-});