fix e2e ci module wiring

Author: sunnylqm

.github/workflows/e2e_android.yml

@@ -20,6 +20,8 @@ jobs:
     steps:
       - name: Checkout react-native-update
         uses: actions/checkout@v6
+        with:
+          submodules: recursive
 
       - name: Checkout react-native-update-cli
         uses: actions/checkout@v6
@@ -52,6 +54,18 @@ jobs:
       - name: Install e2etest dependencies
         run: cd Example/e2etest && bun install --frozen-lockfile
 
+      - name: Link local react-native-update
+        run: |
+          set -euo pipefail
+          export LOCAL_RNU_VERSION="$(node -p "require('./Example/e2etest/node_modules/react-native-update/package.json').version")"
+          LOCAL_RNU_DIR="$PWD/Example/e2etest/.e2e-artifacts/local-react-native-update"
+          rm -rf "$LOCAL_RNU_DIR" Example/e2etest/node_modules/react-native-update
+          mkdir -p "$LOCAL_RNU_DIR"
+          cp package.json react-native-update.podspec react-native.config.js expo-module.config.json "$LOCAL_RNU_DIR"/
+          rsync -a --exclude='.git' src ios android cpp scripts "$LOCAL_RNU_DIR"/
+          node -e "const fs = require('fs'); const file = 'Example/e2etest/.e2e-artifacts/local-react-native-update/package.json'; const pkg = JSON.parse(fs.readFileSync(file, 'utf8')); pkg.version = process.env.LOCAL_RNU_VERSION; fs.writeFileSync(file, JSON.stringify(pkg, null, 2) + '\n');"
+          ln -s ../.e2e-artifacts/local-react-native-update Example/e2etest/node_modules/react-native-update
+
       - name: Install react-native-update-cli dependencies
         run: cd react-native-update-cli && bun install --frozen-lockfile
 

.github/workflows/e2e_ios.yml

@@ -19,6 +19,8 @@ jobs:
     steps:
       - name: Checkout react-native-update
         uses: actions/checkout@v6
+        with:
+          submodules: recursive
 
       - name: Checkout react-native-update-cli
         uses: actions/checkout@v6
@@ -53,6 +55,18 @@ jobs:
       - name: Install e2etest dependencies
         run: cd Example/e2etest && bun install --frozen-lockfile
 
+      - name: Link local react-native-update
+        run: |
+          set -euo pipefail
+          export LOCAL_RNU_VERSION="$(node -p "require('./Example/e2etest/node_modules/react-native-update/package.json').version")"
+          LOCAL_RNU_DIR="$PWD/Example/e2etest/.e2e-artifacts/local-react-native-update"
+          rm -rf "$LOCAL_RNU_DIR" Example/e2etest/node_modules/react-native-update
+          mkdir -p "$LOCAL_RNU_DIR"
+          cp package.json react-native-update.podspec react-native.config.js expo-module.config.json "$LOCAL_RNU_DIR"/
+          rsync -a --exclude='.git' src ios android cpp scripts "$LOCAL_RNU_DIR"/
+          node -e "const fs = require('fs'); const file = 'Example/e2etest/.e2e-artifacts/local-react-native-update/package.json'; const pkg = JSON.parse(fs.readFileSync(file, 'utf8')); pkg.version = process.env.LOCAL_RNU_VERSION; fs.writeFileSync(file, JSON.stringify(pkg, null, 2) + '\n');"
+          ln -s ../.e2e-artifacts/local-react-native-update Example/e2etest/node_modules/react-native-update
+
       - name: Install react-native-update-cli dependencies
         run: cd react-native-update-cli && bun install --frozen-lockfile
 

src/__tests__/core.test.ts

@@ -1,4 +1,4 @@
-import { describe, expect, test, mock } from 'bun:test';
+import { describe, expect, test, mock, spyOn } from 'bun:test';
 
 // In Bun, top-level imports are cached.
 // We can use mock.module to change the implementation of a module,
@@ -11,13 +11,16 @@ const importFreshCore = (cacheKey: string) => import(`../core?${cacheKey}`);
 
 describe('core info parsing', () => {
   test('should call error when currentVersionInfo is invalid JSON', async () => {
-    const mockError = mock(() => {});
+    const consoleError = spyOn(console, 'error').mockImplementation(() => {});
 
     mock.module('react-native', () => ({
       Platform: {
         OS: 'ios',
         Version: 13,
       },
+      DeviceEventEmitter: {
+        addListener: mock(() => ({ remove: mock(() => {}) })),
+      },
       NativeModules: {
         Pushy: {
           currentVersionInfo: '{invalid}',
@@ -46,30 +49,30 @@ describe('core info parsing', () => {
       nanoid: () => 'mock-uuid',
     }));
 
-    mock.module('../utils', () => ({
-      error: mockError,
-      log: mock(() => {}),
-      emptyModule: {},
-    }));
-
     // Use a unique query parameter to bypass cache if supported, or just rely on fresh environment per file.
     // In Bun, you can sometimes use a cache buster if it's dynamic import.
     await importFreshCore('error');
 
-    expect(mockError).toHaveBeenCalledWith(
+    expect(consoleError).toHaveBeenCalledWith(
+      expect.any(String),
       expect.stringContaining('error_parse_version_info')
     );
+
+    consoleError.mockRestore();
   });
 
   test('should not call error when currentVersionInfo is valid JSON', async () => {
-    const mockError = mock(() => {});
+    const consoleError = spyOn(console, 'error').mockImplementation(() => {});
     const mockSetLocalHashInfo = mock(() => {});
 
     mock.module('react-native', () => ({
       Platform: {
         OS: 'ios',
         Version: 13,
       },
+      DeviceEventEmitter: {
+        addListener: mock(() => ({ remove: mock(() => {}) })),
+      },
       NativeModules: {
         Pushy: {
           currentVersionInfo: JSON.stringify({ name: 'v1', debugChannel: true }),
@@ -90,14 +93,10 @@ describe('core info parsing', () => {
       },
     }));
 
-    mock.module('../utils', () => ({
-      error: mockError,
-      log: mock(() => {}),
-      emptyModule: {},
-    }));
-
     await importFreshCore('success');
 
-    expect(mockError).not.toHaveBeenCalled();
+    expect(consoleError).not.toHaveBeenCalled();
+
+    consoleError.mockRestore();
   });
 });