Files
jackwener__opencli/clis/paperreview/utils.test.ts
jakevin 12176de1a5 refactor: simplify core modules (#784)
* refactor: simplify core modules — remove root shims, consolidate error classification, streamline cascade/interceptor, clean up synthesize

1. Remove root-level shim files (errors.ts, logger.ts, registry.ts, types.ts, utils.ts, launcher.ts) — update all ~840 adapter imports to reference src/ directly
2. Consolidate interceptor: reuse shared DISGUISE_FN in tap interceptor instead of reimplementing
3. Unify error classification: single ClassifiedError type with icon/exitCode/hint lookup table, eliminating duplicated pattern matching between resolveExitCode and renderError
4. Simplify cascade probe: replace repetitive switch cases with PROBE_OPTIONS lookup map
12. Clean up synthesize.ts: remove deprecated snake_case field aliases (recommended_args, recommended_columns, recommendedColumnsLegacy) and unnecessary constant aliases

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* docs: update import paths in contributor docs and skill templates

Update all documentation and skill files to reference src/ directly,
matching the shim removal in the previous commit.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: update new Gemini adapter imports to use src/ paths

Fix imports in newly added deep-research adapter files that were
still referencing the deleted root shim files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: update xiaohongshu tests for /search_result/ URL change

Tests now expect /search_result/<id> for bare note IDs (matching
the note-helpers.ts change from PR #774) and updated empty-shell
hint assertion.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* fix: update LessWrong and hupu adapter imports to use src/ paths

Fix imports in newly merged LessWrong and hupu/mentions adapter
files that were still referencing the deleted root shim files.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>

* test(xueqiu): mock logger via src path

---------

Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
2026-04-05 03:35:12 +08:00

69 lines
2.5 KiB
TypeScript

import * as fs from 'node:fs/promises';
import * as os from 'node:os';
import * as path from 'node:path';
import { beforeEach, describe, expect, it, vi } from 'vitest';
import { CliError } from '../../src/errors.js';
import {
MAX_PDF_BYTES,
buildReviewUrl,
parseYesNo,
readPdfFile,
requestJson,
validateHelpfulness,
} from './utils.js';
describe('paperreview utils', () => {
beforeEach(() => {
vi.restoreAllMocks();
});
it('builds review URLs from the token', () => {
expect(buildReviewUrl('tok 123')).toBe('https://paperreview.ai/review?token=tok%20123');
});
it('parses yes/no flags', () => {
expect(parseYesNo('yes', 'critical-error')).toBe(true);
expect(parseYesNo('NO', 'critical-error')).toBe(false);
});
it('rejects invalid yes/no flags with CliError', () => {
expect(() => parseYesNo('maybe', 'critical-error')).toThrow(CliError);
expect(() => parseYesNo('maybe', 'critical-error')).toThrow('"critical-error" must be either "yes" or "no".');
});
it('validates helpfulness scores', () => {
expect(validateHelpfulness(5)).toBe(5);
expect(() => validateHelpfulness(0)).toThrow(CliError);
});
it('reads a valid PDF file and returns metadata', async () => {
const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), 'paperreview-'));
const pdfPath = path.join(tempDir, 'sample.pdf');
const pdfBytes = Buffer.concat([Buffer.from('%PDF-1.4\n'), Buffer.alloc(256, 1)]);
await fs.writeFile(pdfPath, pdfBytes);
const result = await readPdfFile(pdfPath);
expect(result.fileName).toBe('sample.pdf');
expect(result.resolvedPath).toBe(pdfPath);
expect(result.sizeBytes).toBe(pdfBytes.length);
expect(result.buffer.equals(pdfBytes)).toBe(true);
});
it('rejects PDFs larger than the paperreview.ai size limit', async () => {
const tempDir = await fs.mkdtemp(path.join(os.tmpdir(), 'paperreview-'));
const pdfPath = path.join(tempDir, 'large.pdf');
await fs.writeFile(pdfPath, Buffer.alloc(MAX_PDF_BYTES + 1, 1));
await expect(readPdfFile(pdfPath)).rejects.toThrow(CliError);
await expect(readPdfFile(pdfPath)).rejects.toThrow('The PDF is larger than paperreview.ai\'s 10MB limit.');
});
it('normalizes fetch failures into CliError', async () => {
vi.stubGlobal('fetch', vi.fn().mockRejectedValue(new Error('socket hang up')));
await expect(requestJson('/api/review/token')).rejects.toThrow(CliError);
await expect(requestJson('/api/review/token')).rejects.toThrow('Unable to reach paperreview.ai');
});
});