@react-native-harness/jest 1.1.0-rc.1 → 1.1.0-rc.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@react-native-harness/jest",
-  "version": "1.1.0-rc.1",
+  "version": "1.1.0-rc.3",
   "type": "module",
   "main": "./dist/index.js",
   "module": "./dist/index.js",
@@ -40,11 +40,12 @@
     "p-limit": "^7.1.1",
     "tslib": "^2.3.0",
     "yargs": "^17.7.2",
-    "@react-native-harness/bridge": "1.1.0-rc.1",
-    "@react-native-harness/bundler-metro": "1.1.0-rc.1",
-    "@react-native-harness/config": "1.1.0-rc.1",
-    "@react-native-harness/platforms": "1.1.0-rc.1",
-    "@react-native-harness/tools": "1.1.0-rc.1"
+    "@react-native-harness/bridge": "1.1.0-rc.3",
+    "@react-native-harness/config": "1.1.0-rc.3",
+    "@react-native-harness/bundler-metro": "1.1.0-rc.3",
+    "@react-native-harness/platforms": "1.1.0-rc.3",
+    "@react-native-harness/tools": "1.1.0-rc.3",
+    "@react-native-harness/plugins": "1.1.0-rc.3"
   },
   "devDependencies": {
     "@types/yargs": "^17.0.32",

package/src/__tests__/errors.test.ts

@@ -1,5 +1,13 @@
 import { describe, expect, it } from 'vitest';
-import { NativeCrashError } from '../errors.js';
+import { NativeCrashError, PlatformReadyTimeoutError } from '../errors.js';
+
+describe('PlatformReadyTimeoutError', () => {
+  it('includes the configured timeout and config hint', () => {
+    expect(new PlatformReadyTimeoutError(300000).message).toBe(
+      'The platform did not become ready within 300000ms. Increase "platformReadyTimeout" if your device, simulator, or emulator needs more time to start.'
+    );
+  });
+});
 
 describe('NativeCrashError', () => {
   it('formats the extracted stack trace in the error message', () => {
@@ -35,7 +43,9 @@ describe('NativeCrashError', () => {
       exceptionType: 'EXC_CRASH',
     });
 
-    expect(error.message).not.toContain('notify_get_state check indicated test daemon not ready');
+    expect(error.message).not.toContain(
+      'notify_get_state check indicated test daemon not ready'
+    );
     expect(error.message).toContain('Signal: SIGABRT');
     expect(error.message).toContain('Exception: EXC_CRASH');
     expect(error.message).toContain('Process: HarnessPlayground (pid 18007)');
@@ -54,9 +64,9 @@ describe('NativeCrashError', () => {
       stackTrace: [frame],
     });
 
-    expect(error.message.match(/MainActivity\.onCreate\(MainActivity\.kt:38\)/g)).toHaveLength(
-      1
-    );
+    expect(
+      error.message.match(/MainActivity\.onCreate\(MainActivity\.kt:38\)/g)
+    ).toHaveLength(1);
   });
 
   it('collapses the native crash stack header so jest does not reprint multiline messages', () => {
@@ -65,6 +75,8 @@ describe('NativeCrashError', () => {
       summary: ['line one', 'line two'].join('\n'),
     });
 
-    expect(error.stack).toBe('NativeCrashError: The native app crashed while preparing to run this test file.');
+    expect(error.stack).toBe(
+      'NativeCrashError: The native app crashed while preparing to run this test file.'
+    );
   });
 });

package/src/__tests__/harness-cache.test.ts

@@ -0,0 +1,74 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import type { Config as HarnessConfig } from '@react-native-harness/config';
+import type { HarnessPlatform } from '@react-native-harness/platforms';
+
+const mocks = vi.hoisted(() => ({
+  isMetroCacheReusable: vi.fn(),
+  logMetroCacheReused: vi.fn(),
+  logMetroPrewarmCompleted: vi.fn(),
+}));
+
+vi.mock('@react-native-harness/bundler-metro', () => ({
+  isMetroCacheReusable: mocks.isMetroCacheReusable,
+}));
+
+vi.mock('../logs.js', () => ({
+  logMetroCacheReused: mocks.logMetroCacheReused,
+  logMetroPrewarmCompleted: mocks.logMetroPrewarmCompleted,
+}));
+
+import { maybeLogMetroCacheReuse } from '../harness.js';
+
+const platform: HarnessPlatform = {
+  name: 'ios',
+  platformId: 'ios',
+  runner: '/virtual/platform-runner.js',
+  config: {},
+  getResourceLockKey: () => 'ios:simulator:iPhone 17 Pro:26.2',
+};
+
+const createHarnessConfig = (
+  overrides: Partial<HarnessConfig> = {}
+): HarnessConfig =>
+  ({
+    appRegistryComponentName: 'App',
+    disableViewFlattening: false,
+    bridgeTimeout: 5000,
+    entryPoint: 'index.js',
+    unstable__enableMetroCache: true,
+    forwardClientLogs: false,
+    ...overrides,
+  } as HarnessConfig);
+
+describe('maybeLogMetroCacheReuse', () => {
+  beforeEach(() => {
+    vi.clearAllMocks();
+    mocks.isMetroCacheReusable.mockReturnValue(false);
+  });
+
+  it('logs when an existing metro cache will be reused', () => {
+    mocks.isMetroCacheReusable.mockReturnValue(true);
+
+    maybeLogMetroCacheReuse(createHarnessConfig(), platform, '/tmp/project');
+
+    expect(mocks.isMetroCacheReusable).toHaveBeenCalledWith('/tmp/project');
+    expect(mocks.logMetroCacheReused).toHaveBeenCalledWith(platform);
+  });
+
+  it('does not log when metro cache reuse is disabled', () => {
+    maybeLogMetroCacheReuse(
+      createHarnessConfig({ unstable__enableMetroCache: false }),
+      platform,
+      '/tmp/project'
+    );
+
+    expect(mocks.isMetroCacheReusable).not.toHaveBeenCalled();
+    expect(mocks.logMetroCacheReused).not.toHaveBeenCalled();
+  });
+
+  it('does not log when the metro cache is absent', () => {
+    maybeLogMetroCacheReuse(createHarnessConfig(), platform, '/tmp/project');
+
+    expect(mocks.logMetroCacheReused).not.toHaveBeenCalled();
+  });
+});