Skip to content

fix(e2e): probe /handshake for readiness instead of removed /servers #61

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
roziscoding merged 3 commits into from
Jun 28, 2026
Merged

fix(e2e): probe /handshake for readiness instead of removed /servers #61

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
4b69051
fix(e2e): probe /handshake for readiness instead of removed /servers
roziscoding Jun 28, 2026
66d1010
test(e2e): expect opaque Unauthorized error from the peer-facing torznab
roziscoding Jun 28, 2026
7ba6942
chore: remove .cursor editor rules
roziscoding Jun 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 0 additions & 1 deletion .cursor/rules/use-bun-instead-of-node-vite-npm-pnpm.mdc
View file
Open in desktop

This file was deleted.

4 changes: 2 additions & 2 deletions .mise/tasks/test/e2e
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -51,11 +51,11 @@ echo "🚀 Starting Jack instances..."
docker compose -f "$E2E_DIR/docker-compose.yml" up -d --build jack-alpha jack-beta

echo "⏳ Waiting for Jack Alpha..."
timeout 60 bash -c 'until curl -sf -H "X-Api-Key: '"$JACK_ALPHA_API_KEY"'" http://localhost:13000/servers > /dev/null 2>&1; do sleep 2; done'
timeout 60 bash -c 'until curl -sf -H "X-Api-Key: '"$JACK_ALPHA_API_KEY"'" http://localhost:13000/handshake > /dev/null 2>&1; do sleep 2; done'
echo "✅ Jack Alpha ready"

echo "⏳ Waiting for Jack Beta..."
timeout 60 bash -c 'until curl -sf -H "X-Api-Key: '"$JACK_BETA_API_KEY"'" http://localhost:13001/servers > /dev/null 2>&1; do sleep 2; done'
timeout 60 bash -c 'until curl -sf -H "X-Api-Key: '"$JACK_BETA_API_KEY"'" http://localhost:13001/handshake > /dev/null 2>&1; do sleep 2; done'
echo "✅ Jack Beta ready"

echo ""
Expand Down
5 changes: 4 additions & 1 deletion e2e/tests/torznab-search.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,10 @@ describe('Torznab Search (e2e)', () => {
expect(res.status).toBe(200)
const xml = await res.text()
expect(xml).toContain('<error code="100"')
expect(xml).toContain('Unauthorized: invalid API key')
// The peer-facing app returns opaque errors — a generic reason, never the
// detailed "invalid API key" (which would leak auth detail to clients).
expect(xml).toContain('Unauthorized')
expect(xml).not.toContain('invalid API key')
})

test('Torznab rejects unknown function', async () => {
Expand Down