test: add tests for version info JSON parsing in core.ts (#522)

- Added `src/__tests__/core.test.ts` to test `src/core.ts` initialization.
- Mocks `NativeModules.Pushy` to simulate invalid and valid JSON for `currentVersionInfo`.
- Uses dynamic import with query parameters to bypass Bun's module cache for testing top-level side effects.
- Updated `src/__tests__/setup.ts` to provide better default mocks for the `Pushy` native module.

Co-authored-by: google-labs-jules[bot] <161369871+google-labs-jules[bot]@users.noreply.github.com>
Co-authored-by: sunnylqm <615282+sunnylqm@users.noreply.github.com>

Author: sunnylqm

src/__tests__/core.test.ts

@@ -0,0 +1,102 @@
+import { describe, expect, test, mock } from 'bun:test';
+
+// In Bun, top-level imports are cached.
+// We can use mock.module to change the implementation of a module,
+// but if a module has already been executed (like core.ts),
+// re-importing it might not re-run the top-level code unless we use some tricks
+// or run tests in isolation.
+// Actually, bun test runs each file in its own environment usually,
+// BUT if we run multiple test files in one process, they might share the cache.
+
+describe('core info parsing', () => {
+  test('should call error when currentVersionInfo is invalid JSON', async () => {
+    const mockError = mock(() => {});
+
+    mock.module('react-native', () => ({
+      Platform: {
+        OS: 'ios',
+        Version: 13,
+      },
+      NativeModules: {
+        Pushy: {
+          currentVersionInfo: '{invalid}',
+          downloadRootDir: '/tmp',
+          packageVersion: '1.0.0',
+          currentVersion: 'hash1',
+          isFirstTime: false,
+          rolledBackVersion: '',
+          buildTime: '2023-01-01',
+          uuid: 'existing-uuid',
+          setLocalHashInfo: mock(() => {}),
+          getLocalHashInfo: mock(() => Promise.resolve('{}')),
+          setUuid: mock(() => {}),
+        },
+      },
+      NativeEventEmitter: class {
+        addListener = mock(() => ({ remove: mock(() => {}) }));
+      },
+    }));
+
+    mock.module('react-native/Libraries/Core/ReactNativeVersion', () => ({
+      version: { major: 0, minor: 73, patch: 0 },
+    }));
+
+    mock.module('nanoid/non-secure', () => ({
+      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 import('../core?error');
+
+    expect(mockError).toHaveBeenCalledWith(
+      expect.stringContaining('error_parse_version_info')
+    );
+  });
+
+  test('should not call error when currentVersionInfo is valid JSON', async () => {
+    const mockError = mock(() => {});
+    const mockSetLocalHashInfo = mock(() => {});
+
+    mock.module('react-native', () => ({
+      Platform: {
+        OS: 'ios',
+        Version: 13,
+      },
+      NativeModules: {
+        Pushy: {
+          currentVersionInfo: JSON.stringify({ name: 'v1', debugChannel: true }),
+          downloadRootDir: '/tmp',
+          packageVersion: '1.0.0',
+          currentVersion: 'hash1',
+          isFirstTime: false,
+          rolledBackVersion: '',
+          buildTime: '2023-01-01',
+          uuid: 'existing-uuid',
+          setLocalHashInfo: mockSetLocalHashInfo,
+          getLocalHashInfo: mock(() => Promise.resolve('{}')),
+          setUuid: mock(() => {}),
+        },
+      },
+      NativeEventEmitter: class {
+        addListener = mock(() => ({ remove: mock(() => {}) }));
+      },
+    }));
+
+    mock.module('../utils', () => ({
+      error: mockError,
+      log: mock(() => {}),
+      emptyModule: {},
+    }));
+
+    await import('../core?success');
+
+    expect(mockError).not.toHaveBeenCalled();
+  });
+});

src/__tests__/setup.ts

@@ -4,14 +4,34 @@ mock.module('react-native', () => {
   return {
     Platform: {
       OS: 'ios',
+      Version: 13,
+    },
+    NativeModules: {
+      Pushy: {
+        currentVersionInfo: '{}',
+        downloadRootDir: '/tmp',
+        packageVersion: '1.0.0',
+        currentVersion: 'hash',
+        isFirstTime: false,
+        rolledBackVersion: '',
+        buildTime: '2023-01-01',
+        uuid: 'uuid',
+        setLocalHashInfo: () => {},
+        getLocalHashInfo: () => Promise.resolve('{}'),
+        setUuid: () => {},
+      },
+    },
+    NativeEventEmitter: class {
+      addListener = () => ({ remove: () => {} });
+      removeAllListeners = () => {};
     },
   };
 });
 
 mock.module('../i18n', () => {
   return {
     default: {
-      t: (key: string) => key,
+      t: (key: string, params?: any) => `${key}${params ? JSON.stringify(params) : ''}`,
     },
   };
 });