ut

Author: sunnylqm

.github/workflows/test.yml

@@ -0,0 +1,48 @@
+name: test
+
+on:
+  pull_request:
+    branches:
+      - master
+  push:
+    branches:
+      - master
+
+concurrency:
+  group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref }}
+  cancel-in-progress: true
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        node-version: [20.x]
+
+    steps:
+      - uses: actions/checkout@v4
+      - uses: oven-sh/setup-bun@v2
+      - uses: actions/setup-node@v4
+        with:
+          node-version: ${{ matrix.node-version }}
+          registry-url: 'https://registry.npmjs.org'
+
+      - name: Install Dependency
+        env:
+          NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          NODE_OPTIONS: '--max_old_space_size=4096'
+        run: bun install --frozen-lockfile
+
+      - name: Run unit tests with coverage
+        env:
+          NODE_AUTH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+          NODE_OPTIONS: '--max_old_space_size=4096'
+        run: bun run test:coverage
+
+      - name: Upload coverage artifact
+        uses: actions/upload-artifact@v4
+        with:
+          name: coverage
+          path: coverage
+          if-no-files-found: error

package.json

@@ -24,7 +24,9 @@
   "scripts": {
     "build": "rm -rf lib && swc src -d lib --strip-leading-paths && tsc -p tsconfig.build.json",
     "prepublishOnly": "bun scripts/prepublish.ts && bun run build && chmod +x lib/index.js",
-    "lint": "tsc --noEmit && biome check --write ."
+    "lint": "tsc --noEmit && biome check --write .",
+    "test": "bun test",
+    "test:coverage": "bun test --coverage --coverage-reporter=text --coverage-reporter=lcov"
   },
   "repository": {
     "type": "git",

tests/add-gitignore.test.ts

@@ -0,0 +1,67 @@
+import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
+import fs from 'fs';
+import os from 'os';
+import path from 'path';
+import { addGitIgnore } from '../src/utils/add-gitignore';
+import { credentialFile, tempDir } from '../src/utils/constants';
+
+const originalCwd = process.cwd();
+
+function mkTempDir(prefix: string): string {
+  return fs.mkdtempSync(path.join(os.tmpdir(), prefix));
+}
+
+function readGitIgnore(cwd: string): string {
+  return fs.readFileSync(path.join(cwd, '.gitignore'), 'utf-8');
+}
+
+describe('utils/add-gitignore', () => {
+  let tempRoot = '';
+
+  beforeEach(() => {
+    tempRoot = mkTempDir('rn-update-gitignore-');
+    process.chdir(tempRoot);
+  });
+
+  afterEach(() => {
+    process.chdir(originalCwd);
+    if (tempRoot && fs.existsSync(tempRoot)) {
+      fs.rmSync(tempRoot, { recursive: true, force: true });
+    }
+  });
+
+  test('does nothing when .gitignore does not exist', () => {
+    addGitIgnore();
+    expect(fs.existsSync(path.join(tempRoot, '.gitignore'))).toBe(false);
+  });
+
+  test('appends required ignore entries when missing', () => {
+    fs.writeFileSync(path.join(tempRoot, '.gitignore'), 'node_modules\n');
+
+    addGitIgnore();
+
+    const content = readGitIgnore(tempRoot);
+    expect(content).toContain('# react-native-update');
+    expect(content).toContain(credentialFile);
+    expect(content).toContain(tempDir);
+  });
+
+  test('does not duplicate entries that already exist', () => {
+    fs.writeFileSync(
+      path.join(tempRoot, '.gitignore'),
+      ['node_modules', credentialFile].join('\n'),
+    );
+
+    addGitIgnore();
+    const first = readGitIgnore(tempRoot);
+    addGitIgnore();
+    const second = readGitIgnore(tempRoot);
+
+    expect(second).toEqual(first);
+
+    const lines = second.split('\n');
+    expect(lines.filter((line) => line.trim() === credentialFile)).toHaveLength(
+      1,
+    );
+  });
+});

tests/bundle-pack.test.ts

@@ -0,0 +1,79 @@
+import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
+import fs from 'fs';
+import os from 'os';
+import path from 'path';
+import { open as openZipFile } from 'yauzl';
+import { packBundle } from '../src/bundle-pack';
+
+function mkTempDir(prefix: string): string {
+  return fs.mkdtempSync(path.join(os.tmpdir(), prefix));
+}
+
+async function listZipEntries(zipPath: string): Promise<string[]> {
+  const entries: string[] = [];
+  await new Promise<void>((resolve, reject) => {
+    openZipFile(zipPath, { lazyEntries: true }, (error, zipfile) => {
+      if (error || !zipfile) {
+        reject(error ?? new Error('Failed to open zip file'));
+        return;
+      }
+
+      zipfile.on('entry', (entry) => {
+        entries.push(entry.fileName);
+        zipfile.readEntry();
+      });
+      zipfile.once('end', () => resolve());
+      zipfile.once('error', reject);
+
+      zipfile.readEntry();
+    });
+  });
+  return entries;
+}
+
+describe('bundle-pack', () => {
+  let tempRoot = '';
+
+  beforeEach(() => {
+    tempRoot = mkTempDir('rn-update-pack-bundle-');
+  });
+
+  afterEach(() => {
+    if (tempRoot && fs.existsSync(tempRoot)) {
+      fs.rmSync(tempRoot, { recursive: true, force: true });
+    }
+  });
+
+  test('packs bundle and excludes ignored files', async () => {
+    const sourceDir = path.join(tempRoot, 'bundle');
+    const nestedDir = path.join(sourceDir, 'assets');
+    fs.mkdirSync(nestedDir, { recursive: true });
+
+    fs.writeFileSync(path.join(sourceDir, 'index.bundlejs'), 'bundle');
+    fs.writeFileSync(path.join(sourceDir, 'index.bundlejs.map'), 'ignored-map');
+    fs.writeFileSync(
+      path.join(sourceDir, 'bundle.harmony.js.map'),
+      'ignored-map',
+    );
+    fs.writeFileSync(path.join(sourceDir, 'keep.txt'), 'keep');
+    fs.writeFileSync(path.join(sourceDir, 'ignored.txt.map'), 'ignored-map');
+    fs.writeFileSync(path.join(sourceDir, '.DS_Store'), 'ignored');
+    fs.writeFileSync(path.join(nestedDir, 'image.png'), 'image-content');
+
+    const outputZip = path.join(tempRoot, 'dist', 'output.ppk');
+    await packBundle(sourceDir, outputZip);
+
+    expect(fs.existsSync(outputZip)).toBe(true);
+
+    const entries = await listZipEntries(outputZip);
+    expect(entries).toContain('index.bundlejs');
+    expect(entries).toContain('keep.txt');
+    expect(entries).toContain('assets/');
+    expect(entries).toContain('assets/image.png');
+
+    expect(entries).not.toContain('index.bundlejs.map');
+    expect(entries).not.toContain('bundle.harmony.js.map');
+    expect(entries).not.toContain('ignored.txt.map');
+    expect(entries).not.toContain('.DS_Store');
+  });
+});

tests/check-lockfile.test.ts

@@ -0,0 +1,77 @@
+import { afterEach, beforeEach, describe, expect, test } from 'bun:test';
+import fs from 'fs';
+import os from 'os';
+import path from 'path';
+import { checkLockFiles } from '../src/utils/check-lockfile';
+
+const originalCwd = process.cwd();
+
+function mkTempDir(prefix: string): string {
+  return fs.mkdtempSync(path.join(os.tmpdir(), prefix));
+}
+
+function writeJson(filePath: string, value: unknown): void {
+  fs.writeFileSync(filePath, JSON.stringify(value, null, 2));
+}
+
+describe('utils/check-lockfile', () => {
+  let tempRoot = '';
+
+  beforeEach(() => {
+    tempRoot = mkTempDir('rn-update-lockfile-');
+  });
+
+  afterEach(() => {
+    process.chdir(originalCwd);
+    if (tempRoot && fs.existsSync(tempRoot)) {
+      fs.rmSync(tempRoot, { recursive: true, force: true });
+    }
+  });
+
+  test('passes when exactly one lock file exists in current directory', () => {
+    process.chdir(tempRoot);
+    fs.writeFileSync(path.join(tempRoot, 'bun.lock'), '');
+
+    expect(() => checkLockFiles()).not.toThrow();
+  });
+
+  test('throws when multiple lock files exist in current directory', () => {
+    process.chdir(tempRoot);
+    fs.writeFileSync(path.join(tempRoot, 'bun.lock'), '');
+    fs.writeFileSync(path.join(tempRoot, 'yarn.lock'), '');
+
+    expect(() => checkLockFiles()).toThrow();
+  });
+
+  test('detects lock file from monorepo root when cwd has none', () => {
+    const monorepoRoot = path.join(tempRoot, 'repo');
+    const packageDir = path.join(monorepoRoot, 'packages', 'cli');
+    fs.mkdirSync(packageDir, { recursive: true });
+    writeJson(path.join(monorepoRoot, 'package.json'), {
+      private: true,
+      workspaces: ['packages/*'],
+    });
+    fs.writeFileSync(path.join(monorepoRoot, 'pnpm-lock.yaml'), '');
+
+    process.chdir(packageDir);
+    expect(() => checkLockFiles()).not.toThrow();
+  });
+
+  test('warns but does not throw when no lock file is found', () => {
+    process.chdir(tempRoot);
+    writeJson(path.join(tempRoot, 'package.json'), { name: 'no-lock' });
+
+    const warnings: string[] = [];
+    const originalWarn = console.warn;
+    console.warn = (...args: unknown[]) => {
+      warnings.push(args.map((item) => String(item)).join(' '));
+    };
+
+    try {
+      expect(() => checkLockFiles()).not.toThrow();
+      expect(warnings.length).toBeGreaterThan(0);
+    } finally {
+      console.warn = originalWarn;
+    }
+  });
+});

tests/command-result.test.ts

@@ -0,0 +1,56 @@
+import { describe, expect, test } from 'bun:test';
+import {
+  runAsCommandResult,
+  toErrorMessage,
+} from '../src/utils/command-result';
+
+describe('utils/command-result', () => {
+  test('toErrorMessage uses Error.message when possible', () => {
+    expect(toErrorMessage(new Error('boom'), 'fallback')).toBe('boom');
+  });
+
+  test('toErrorMessage falls back for non-Error values', () => {
+    expect(toErrorMessage('boom', 'fallback')).toBe('fallback');
+    expect(toErrorMessage({ message: 'boom' }, 'fallback')).toBe('fallback');
+  });
+
+  test('runAsCommandResult returns success with raw data', async () => {
+    const result = await runAsCommandResult(async () => 123, 'failed');
+    expect(result).toEqual({
+      success: true,
+      data: 123,
+    });
+  });
+
+  test('runAsCommandResult maps successful data when mapper provided', async () => {
+    const result = await runAsCommandResult(
+      async () => ({ id: 1, name: 'pkg' }),
+      'failed',
+      (value) => ({ label: `${value.id}-${value.name}` }),
+    );
+    expect(result).toEqual({
+      success: true,
+      data: { label: '1-pkg' },
+    });
+  });
+
+  test('runAsCommandResult returns thrown Error message', async () => {
+    const result = await runAsCommandResult(async () => {
+      throw new Error('api failed');
+    }, 'fallback');
+    expect(result).toEqual({
+      success: false,
+      error: 'api failed',
+    });
+  });
+
+  test('runAsCommandResult returns fallback for non-Error throw', async () => {
+    const result = await runAsCommandResult(async () => {
+      throw 'failed';
+    }, 'fallback');
+    expect(result).toEqual({
+      success: false,
+      error: 'fallback',
+    });
+  });
+});