* card-interactions.test.js: group delete tests used `.dialog-ok` (class selector) but the button has `id="dialog-ok"`. querySelector returned null, the click was skipped, the dialog stayed open and blocked every downstream test. Fixed to getElementById. * runner.js closeAll(): added #dialog, #bucket-super-popover (remove), and #gitea-overlay to the cleanup list. A failed test that leaves any of these open no longer poisons the rest of the suite. * runner.js copy handler: when there are failures or flaky tests, the "copy" button now builds the clipboard text from lastResults — only failed + flaky lines (suite name, test name, timing, error, original error for retried). When everything passed, copies the full output as before. lastResults is set in renderResults(). * bucket-context.test.js: move-left test used `[data-bucket]` to find user bucket tabs, which misses super-bucket tiles (they carry `data-bucket-super`). Changed to `[data-bucket-drag]` which is on both regular and super-bucket tiles but not on fixed filters. * context-menu.test.js: make-ticket pre-fill assertion now polls up to 2s for the title input to be populated (async openCreateForm repo-fetch was racing against the immediate assertion). * card-join.test.js: clickJoin sleep bumped from 400ms → 800ms. joinSelectedCards does 3+ serial API calls (fetch bodies, PATCH winner, N×DELETE losers, _reloadAll) which sometimes exceeded the old 400ms window on slow runs. * Runner cache: RUNNER_V bumped from ?v=2 → ?v=3 in loader.js + all 42 test files so the browser picks up the new runner.js content (copy handler, closeAll, retry progress bar) without a hard reload. super-buckets: remember last selection per group * buckets.js: module-scope lastSuperSelection Map tracks the last active member per super-bucket parent. Updated in renderBar() whenever activeBucket matches a group member. tileFor() uses it as a fallback when the user is on a different bucket — the tile shows the last-picked child's name and color instead of reverting to the parent. The dropdown popover also highlights the last selection when no member is currently active. chore: bump version to 0.9.11 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
116 lines
4.8 KiB
JavaScript
116 lines
4.8 KiB
JavaScript
import { describe, it, expect, csrf } from './runner.js?v=3';
|
|
|
|
describe('API: Edge Cases', () => {
|
|
it('create card with special characters in title', async () => {
|
|
const title = '__test <script>alert(1)</script> & "quotes"__';
|
|
const res = await fetch('api/cards', {
|
|
method: 'POST', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ title, body: '', color: '#38bdf8' }),
|
|
}).then(r => r.json());
|
|
expect(res.id).toBeGreaterThan(0);
|
|
|
|
const cards = await fetch(`api/cards?ids=${res.id}`).then(r => r.json());
|
|
expect(cards[0].title).toBe(title);
|
|
});
|
|
|
|
it('PATCH with empty body clears body', async () => {
|
|
const card = await fetch('api/cards', {
|
|
method: 'POST', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ title: '__test_empty_body__', body: 'has content', color: '#38bdf8' }),
|
|
}).then(r => r.json());
|
|
|
|
await fetch(`api/cards/${card.id}`, {
|
|
method: 'PATCH', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ body: '' }),
|
|
});
|
|
|
|
const updated = (await fetch(`api/cards?ids=${card.id}`).then(r => r.json()))[0];
|
|
expect(updated.body).toBe('');
|
|
expect(updated.tags).toBe('');
|
|
});
|
|
|
|
it('PATCH due_date to null clears it', async () => {
|
|
const card = await fetch('api/cards', {
|
|
method: 'POST', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ title: '__test_clear_due__', body: '', color: '#38bdf8' }),
|
|
}).then(r => r.json());
|
|
|
|
await fetch(`api/cards/${card.id}`, {
|
|
method: 'PATCH', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ due_date: '2099-01-01' }),
|
|
});
|
|
|
|
await fetch(`api/cards/${card.id}`, {
|
|
method: 'PATCH', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ due_date: null }),
|
|
});
|
|
|
|
const updated = (await fetch(`api/cards?ids=${card.id}`).then(r => r.json()))[0];
|
|
expect(updated.due_date).toBe(null);
|
|
});
|
|
|
|
it('DELETE nonexistent card returns ok', async () => {
|
|
const res = await fetch('api/cards/999999', {
|
|
method: 'DELETE', headers: { 'X-CSRF-Token': csrf() },
|
|
}).then(r => r.json());
|
|
expect(res.ok).toBe(true);
|
|
});
|
|
|
|
it('PATCH nonexistent card returns card data (null)', async () => {
|
|
const res = await fetch('api/cards/999999', {
|
|
method: 'PATCH', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ title: 'nope' }),
|
|
}).then(r => r.json());
|
|
// Returns false/null for nonexistent card
|
|
expect(res.title || null).toBe(null);
|
|
});
|
|
|
|
it('create bucket with empty name returns error', async () => {
|
|
const res = await fetch('api/buckets', {
|
|
method: 'POST', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ name: '' }),
|
|
});
|
|
expect(res.status).toBe(400);
|
|
});
|
|
|
|
it('CSRF required for mutations', async () => {
|
|
const res = await fetch('api/cards', {
|
|
method: 'POST', headers: { 'Content-Type': 'application/json' },
|
|
body: JSON.stringify({ title: 'no csrf', body: '' }),
|
|
});
|
|
expect(res.status).toBe(403);
|
|
});
|
|
|
|
it('GET requests dont need CSRF', async () => {
|
|
const res = await fetch('api/cards');
|
|
expect(res.status).toBe(200);
|
|
});
|
|
});
|
|
|
|
describe('API: Card Position', () => {
|
|
it('new card gets position', async () => {
|
|
const card = await fetch('api/cards', {
|
|
method: 'POST', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ title: '__test_position__', body: '', color: '#38bdf8', position: 42 }),
|
|
}).then(r => r.json());
|
|
|
|
const all = await fetch(`api/cards?ids=${card.id}`).then(r => r.json());
|
|
expect(all[0].position).toBe(42);
|
|
});
|
|
|
|
it('position update via PATCH', async () => {
|
|
const card = await fetch('api/cards', {
|
|
method: 'POST', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ title: '__test_pos_patch__', body: '', color: '#38bdf8' }),
|
|
}).then(r => r.json());
|
|
|
|
await fetch(`api/cards/${card.id}`, {
|
|
method: 'PATCH', headers: { 'Content-Type': 'application/json', 'X-CSRF-Token': csrf() },
|
|
body: JSON.stringify({ position: 99 }),
|
|
});
|
|
|
|
const all = await fetch(`api/cards?ids=${card.id}`).then(r => r.json());
|
|
expect(all[0].position).toBe(99);
|
|
});
|
|
});
|