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
495 lines
18 KiB
JavaScript
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 <B> 😊</p>
|
|
<p class="wr_bookList_item_author">O'Neil & 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 � 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 � 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 � 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: '',
|
|
},
|
|
]);
|
|
});
|
|
});
|