@react-native-harness/platform-android 1.4.0 → 1.4.1

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@react-native-harness/platform-android",
   "description": "Android platform for React Native Harness",
-  "version": "1.4.0",
+  "version": "1.4.1",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",
@@ -19,9 +19,9 @@
     "vite": "^7.2.2",
     "zod": "^3.25.67",
     "tslib": "^2.3.0",
-    "@react-native-harness/config": "1.4.0",
-    "@react-native-harness/tools": "1.4.0",
-    "@react-native-harness/platforms": "1.4.0"
+    "@react-native-harness/config": "1.4.1",
+    "@react-native-harness/platforms": "1.4.1",
+    "@react-native-harness/tools": "1.4.1"
   },
   "license": "MIT"
 }

package/dist/__tests__/adb-id.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=adb-id.test.d.ts.map

package/dist/__tests__/adb-id.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"adb-id.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/adb-id.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/adb-id.test.js

@@ -1,41 +0,0 @@
-import { beforeEach, describe, expect, it, vi } from 'vitest';
-import { getAdbId, isAdbIdEmulator } from '../adb-id.js';
-import * as adb from '../adb.js';
-describe('adb id resolution', () => {
-    beforeEach(() => {
-        vi.restoreAllMocks();
-    });
-    it('identifies emulator adb ids', () => {
-        expect(isAdbIdEmulator('emulator-5554')).toBe(true);
-        expect(isAdbIdEmulator('device-serial-001')).toBe(false);
-    });
-    it('skips non-emulator ids when resolving an emulator device', async () => {
-        vi.spyOn(adb, 'getDeviceIds').mockResolvedValue([
-            'device-serial-001',
-            'emulator-5554',
-        ]);
-        vi.spyOn(adb, 'getEmulatorName').mockResolvedValue('Test_AVD_API_35');
-        await expect(getAdbId({
-            type: 'emulator',
-            name: 'Test_AVD_API_35',
-        })).resolves.toBe('emulator-5554');
-    });
-    it('resolves matching physical device by manufacturer and model', async () => {
-        vi.spyOn(adb, 'getDeviceIds').mockResolvedValue([
-            'device-serial-001',
-            'emulator-5554',
-        ]);
-        vi.spyOn(adb, 'getDeviceInfo').mockImplementation(async (adbId) => {
-            const results = {
-                'device-serial-001': { manufacturer: 'Acme', model: 'Model A1' },
-                'emulator-5554': { manufacturer: 'Emulator', model: 'Emulator Model' },
-            };
-            return results[adbId] || null;
-        });
-        await expect(getAdbId({
-            type: 'physical',
-            manufacturer: 'acme',
-            model: 'model a1',
-        })).resolves.toBe('device-serial-001');
-    });
-});

package/dist/__tests__/adb-wait.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=adb-wait.test.d.ts.map

package/dist/__tests__/adb-wait.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"adb-wait.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/adb-wait.test.ts"],"names":[],"mappings":""}

package/dist/__tests__/adb-wait.test.js

@@ -1,69 +0,0 @@
-import { afterEach, describe, expect, it, vi } from 'vitest';
-const mocks = vi.hoisted(() => ({
-    spawn: vi.fn(),
-}));
-// The global test/setup.ts mocks '../adb.js' wholesale (including
-// waitForEmulator itself), which is right for consumers of this module but
-// defeats the purpose here: this suite exercises the *real* polling loop
-// (and its private waitWithSignal helper) to catch timer/listener leaks.
-// Re-declaring the mock for this file restores the actual implementation,
-// while `spawn` and `getAdbBinaryPath` (genuine cross-module imports) are
-// swapped out to keep `getDeviceIds()` from touching a real adb binary.
-vi.mock('../adb.js', async () => {
-    const actual = await vi.importActual('../adb.js');
-    return actual;
-});
-vi.mock('../environment.js', async () => {
-    const actual = await vi.importActual('../environment.js');
-    return { ...actual, getAdbBinaryPath: () => '/fake/sdk/platform-tools/adb' };
-});
-vi.mock('@react-native-harness/tools', async () => {
-    const actual = await vi.importActual('@react-native-harness/tools');
-    return { ...actual, spawn: mocks.spawn };
-});
-const { waitForEmulator } = await import('../adb.js');
-afterEach(() => {
-    vi.useRealTimers();
-    vi.restoreAllMocks();
-});
-describe('waitForEmulator polling', () => {
-    it('does not leak an abort listener per poll iteration when the delay wins', async () => {
-        vi.useFakeTimers();
-        // No matching device on any poll: `adb devices` reports an empty list.
-        mocks.spawn.mockResolvedValue({ stdout: 'List of devices attached\n' });
-        const controller = new AbortController();
-        const addSpy = vi.spyOn(controller.signal, 'addEventListener');
-        const removeSpy = vi.spyOn(controller.signal, 'removeEventListener');
-        const resultPromise = waitForEmulator('Test_AVD', controller.signal);
-        resultPromise.catch(() => undefined);
-        for (let i = 0; i < 3; i += 1) {
-            await vi.advanceTimersByTimeAsync(1_000);
-        }
-        // Regression: each 1s poll iteration used to add an 'abort' listener
-        // to the local waitForAbort helper that was never removed when the
-        // delay (not the abort) won the race, leaking one listener per second
-        // of polling against this session-lifetime signal.
-        expect(addSpy.mock.calls.length).toBeGreaterThanOrEqual(3);
-        const addCallsBeforeAbort = addSpy.mock.calls.length;
-        controller.abort(new Error('teardown'));
-        await expect(resultPromise).rejects.toThrow('teardown');
-        // Every listener added while losing the race — including the one still
-        // pending when the abort itself won — must be removed again.
-        expect(removeSpy).toHaveBeenCalledTimes(addCallsBeforeAbort);
-    });
-    it('clears the pending 1s poll timer when the signal aborts mid-wait', async () => {
-        vi.useFakeTimers();
-        mocks.spawn.mockResolvedValue({ stdout: 'List of devices attached\n' });
-        const controller = new AbortController();
-        const resultPromise = waitForEmulator('Test_AVD', controller.signal);
-        resultPromise.catch(() => undefined);
-        // Let the loop enter its 1s wait, then abort before the timer fires.
-        await vi.advanceTimersByTimeAsync(0);
-        controller.abort(new Error('teardown'));
-        await expect(resultPromise).rejects.toThrow('teardown');
-        // Regression: the bare setTimeout-backed `wait()` used to have no
-        // handle to clear, so it stayed alive in the event loop for the rest
-        // of the 1s interval even after the abort branch won.
-        expect(vi.getTimerCount()).toBe(0);
-    });
-});

package/dist/__tests__/adb.test.d.ts

@@ -1,2 +0,0 @@
-export {};
-//# sourceMappingURL=adb.test.d.ts.map

package/dist/__tests__/adb.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"adb.test.d.ts","sourceRoot":"","sources":["../../src/__tests__/adb.test.ts"],"names":[],"mappings":""}