Files
jackwener--opencli/clis/weread/search-regression.test.js
wehub-resource-sync 9b395f5cc3
E2E Headed Chrome / e2e-headed (macos-15) (push) Has been cancelled
E2E Headed Chrome / e2e-headed (ubuntu-latest) (push) Has been cancelled
E2E Headed Chrome / e2e-headed (windows-latest) (push) Has been cancelled
CI / build (macos-latest) (push) Has been cancelled
CI / build (ubuntu-latest) (push) Has been cancelled
CI / build (windows-latest) (push) Has been cancelled
CI / unit-test (push) Has been cancelled
CI / bun-test (push) Has been cancelled
CI / adapter-test (push) Has been cancelled
CI / smoke-test (macos-latest) (push) Has been cancelled
CI / smoke-test (ubuntu-latest) (push) Has been cancelled
Security Audit / audit (push) Has been cancelled
Build Chrome Extension / build (push) Has been cancelled
Trigger Website Rebuild (Docs Updated) / dispatch (push) Has been cancelled
chore: import upstream snapshot with attribution
2026-07-13 12:39:48 +08:00

495 lines
18 KiB
JavaScript

import { beforeEach, describe, expect, it, vi } from 'vitest';
import { getRegistry } from '@jackwener/opencli/registry';
import { CommandExecutionError, EmptyResultError } from '@jackwener/opencli/errors';
import './search.js';
describe('weread/search regression', () => {
beforeEach(() => {
vi.restoreAllMocks();
});
it('uses the query argument for the search API and returns reader urls from search html', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: 'Deep Work',
author: 'Cal Newport',
bookId: 'abc123',
},
},
],
}),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve(`
<ul class="search_bookDetail_list">
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/reader123"></a>
<p class="wr_bookList_item_title">Deep Work</p>
</li>
</ul>
`),
});
vi.stubGlobal('fetch', fetchMock);
const result = await command.func({ query: 'deep work', limit: 5 });
expect(fetchMock).toHaveBeenCalledTimes(2);
expect(String(fetchMock.mock.calls[0][0])).toContain('keyword=deep+work');
expect(String(fetchMock.mock.calls[1][0])).toContain('/web/search/books?keyword=deep+work');
expect(result).toEqual([
{
rank: 1,
title: 'Deep Work',
author: 'Cal Newport',
bookId: 'abc123',
url: 'https://weread.qq.com/web/reader/reader123',
},
]);
});
it('does not emit stale bookDetail urls when the reader url is unavailable', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: 'Deep Work',
author: 'Cal Newport',
bookId: 'abc123',
},
},
],
}),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve('<html><body><p>no search cards</p></body></html>'),
});
vi.stubGlobal('fetch', fetchMock);
const result = await command.func({ query: 'deep work', limit: 5 });
expect(result).toEqual([
{
rank: 1,
title: 'Deep Work',
author: 'Cal Newport',
bookId: 'abc123',
url: '',
},
]);
});
it('matches reader urls by title queue instead of assuming identical result order', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: 'Deep Work',
author: 'Cal Newport',
bookId: 'abc123',
},
},
{
bookInfo: {
title: 'Digital Minimalism',
author: 'Cal Newport',
bookId: 'xyz789',
},
},
],
}),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve(`
<ul class="search_bookDetail_list">
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/unrelated111"></a>
<p class="wr_bookList_item_title">Atomic Habits</p>
</li>
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/digital222"></a>
<p class="wr_bookList_item_title">Digital Minimalism</p>
</li>
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/deep333"></a>
<p class="wr_bookList_item_title">Deep Work</p>
</li>
</ul>
`),
});
vi.stubGlobal('fetch', fetchMock);
const result = await command.func({ query: 'cal newport', limit: 5 });
expect(result).toEqual([
{
rank: 1,
title: 'Deep Work',
author: 'Cal Newport',
bookId: 'abc123',
url: 'https://weread.qq.com/web/reader/deep333',
},
{
rank: 2,
title: 'Digital Minimalism',
author: 'Cal Newport',
bookId: 'xyz789',
url: 'https://weread.qq.com/web/reader/digital222',
},
]);
});
it('surfaces search html request failures instead of emitting empty urls', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: 'Deep Work',
author: 'Cal Newport',
bookId: 'abc123',
},
},
],
}),
})
.mockRejectedValueOnce(new Error('network timeout'));
vi.stubGlobal('fetch', fetchMock);
await expect(command.func({ query: 'deep work', limit: 5 })).rejects.toBeInstanceOf(CommandExecutionError);
});
it('throws EmptyResultError when the public search API returns no books', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({ books: [] }),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve('<html></html>'),
});
vi.stubGlobal('fetch', fetchMock);
await expect(command.func({ query: 'definitely-missing-book', limit: 5 })).rejects.toBeInstanceOf(EmptyResultError);
});
it('binds reader urls with title and author instead of title alone', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: '文明',
author: '作者甲',
bookId: 'book-a',
},
},
{
bookInfo: {
title: '文明',
author: '作者乙',
bookId: 'book-b',
},
},
],
}),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve(`
<ul class="search_bookDetail_list">
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/book-b-reader"></a>
<p class="wr_bookList_item_title">文明</p>
<p class="wr_bookList_item_author"><a href="/web/search/books?author=%E4%BD%9C%E8%80%85%E4%B9%99">作者乙</a></p>
</li>
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/book-a-reader"></a>
<p class="wr_bookList_item_title">文明</p>
<p class="wr_bookList_item_author"><a href="/web/search/books?author=%E4%BD%9C%E8%80%85%E7%94%B2">作者甲</a></p>
</li>
</ul>
`),
});
vi.stubGlobal('fetch', fetchMock);
const result = await command.func({ query: '文明', limit: 5 });
expect(result).toEqual([
{
rank: 1,
title: '文明',
author: '作者甲',
bookId: 'book-a',
url: 'https://weread.qq.com/web/reader/book-a-reader',
},
{
rank: 2,
title: '文明',
author: '作者乙',
bookId: 'book-b',
url: 'https://weread.qq.com/web/reader/book-b-reader',
},
]);
});
it('decodes named and astral HTML entities before matching search cards', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: 'A <B> 😊',
author: "O'Neil & Co",
bookId: 'entity-book',
},
},
],
}),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve(`
<ul class="search_bookDetail_list">
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/entity-reader"></a>
<p class="wr_bookList_item_title">A &lt;B&gt; &#x1F60A;</p>
<p class="wr_bookList_item_author">O&apos;Neil &amp; Co</p>
</li>
</ul>
`),
});
vi.stubGlobal('fetch', fetchMock);
const result = await command.func({ query: 'entities', limit: 5 });
expect(result).toEqual([
{
rank: 1,
title: 'A <B> 😊',
author: "O'Neil & Co",
bookId: 'entity-book',
url: 'https://weread.qq.com/web/reader/entity-reader',
},
]);
});
it('leaves invalid numeric HTML entities literal instead of throwing raw RangeError', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: 'Bad &#xFFFFFFFF; Entity',
author: 'Tester',
bookId: 'bad-entity-book',
},
},
],
}),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve(`
<ul class="search_bookDetail_list">
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/bad-entity-reader"></a>
<p class="wr_bookList_item_title">Bad &#xFFFFFFFF; Entity</p>
<p class="wr_bookList_item_author">Tester</p>
</li>
</ul>
`),
});
vi.stubGlobal('fetch', fetchMock);
const result = await command.func({ query: 'bad entity', limit: 5 });
expect(result[0]).toMatchObject({
title: 'Bad &#xFFFFFFFF; Entity',
bookId: 'bad-entity-book',
url: 'https://weread.qq.com/web/reader/bad-entity-reader',
});
});
it('leaves urls empty when same-title results are ambiguous and html cards have no author', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: '文明',
author: '作者甲',
bookId: 'book-a',
},
},
{
bookInfo: {
title: '文明',
author: '作者乙',
bookId: 'book-b',
},
},
],
}),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve(`
<ul class="search_bookDetail_list">
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/book-b-reader"></a>
<p class="wr_bookList_item_title">文明</p>
</li>
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/book-a-reader"></a>
<p class="wr_bookList_item_title">文明</p>
</li>
</ul>
`),
});
vi.stubGlobal('fetch', fetchMock);
const result = await command.func({ query: '文明', limit: 5 });
expect(result).toEqual([
{
rank: 1,
title: '文明',
author: '作者甲',
bookId: 'book-a',
url: '',
},
{
rank: 2,
title: '文明',
author: '作者乙',
bookId: 'book-b',
url: '',
},
]);
});
it('leaves urls empty when exact author matching fails and multiple html cards share the same title', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: '文明',
author: '作者甲',
bookId: 'book-a',
},
},
],
}),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve(`
<ul class="search_bookDetail_list">
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/book-a-reader"></a>
<p class="wr_bookList_item_title">文明</p>
<p class="wr_bookList_item_author"><a href="/web/search/books?author=%E4%BD%9C%E8%80%85%E4%B9%99">作者乙</a></p>
</li>
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/book-a-reader-2"></a>
<p class="wr_bookList_item_title">文明</p>
</li>
</ul>
`),
});
vi.stubGlobal('fetch', fetchMock);
const result = await command.func({ query: '文明', limit: 5 });
expect(result).toEqual([
{
rank: 1,
title: '文明',
author: '作者甲',
bookId: 'book-a',
url: '',
},
]);
});
it('leaves urls empty when multiple results share the same title and author identity', async () => {
const command = getRegistry().get('weread/search');
expect(command?.func).toBeTypeOf('function');
const fetchMock = vi.fn()
.mockResolvedValueOnce({
ok: true,
json: () => Promise.resolve({
books: [
{
bookInfo: {
title: '文明',
author: '作者甲',
bookId: 'book-a',
},
},
{
bookInfo: {
title: '文明',
author: '作者甲',
bookId: 'book-b',
},
},
],
}),
})
.mockResolvedValueOnce({
ok: true,
text: () => Promise.resolve(`
<ul class="search_bookDetail_list">
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/book-b-reader"></a>
<p class="wr_bookList_item_title">文明</p>
<p class="wr_bookList_item_author"><a href="/web/search/books?author=%E4%BD%9C%E8%80%85%E7%94%B2">作者甲</a></p>
</li>
<li class="wr_bookList_item">
<a class="wr_bookList_item_link" href="/web/reader/book-a-reader"></a>
<p class="wr_bookList_item_title">文明</p>
<p class="wr_bookList_item_author"><a href="/web/search/books?author=%E4%BD%9C%E8%80%85%E7%94%B2">作者甲</a></p>
</li>
</ul>
`),
});
vi.stubGlobal('fetch', fetchMock);
const result = await command.func({ query: '文明', limit: 5 });
expect(result).toEqual([
{
rank: 1,
title: '文明',
author: '作者甲',
bookId: 'book-a',
url: '',
},
{
rank: 2,
title: '文明',
author: '作者甲',
bookId: 'book-b',
url: '',
},
]);
});
});