Hermes
993ab82b45
Connecting the owner's DNS3 Radicale calendar (appt.sami) as a DashCaddy
calendar source exposed a real parser bug: extractCalendarsAndEvents split
multistatus on <\/[\w-]+:response> — the namespace prefix was REQUIRED, but
Radicale emits bare <response> in the default DAV namespace. Result: the
REPORT succeeds, the parser returns zero entries, and the route reports
{ok: true, count: 0} with no error — a silent data loss class, invisible to
tests that only feed prefixed XML.
Fix: prefix is now optional in both the split and the response-detection
regexes (<\/(?:[\w-]+:)?response>). Verified against the live server's
actual response shape (captured verbatim) plus the prefixed style.
Tests: NEW __tests__/calendar/caldav-default-ns.test.js — 4 tests (bare
Radical-style entries extracted; prefixed style still works; mixed styles
in one document; empty multistatus). Existing calendar suites: 15 pass
unchanged. eslint 0 errors on touched files.
55 lines
2.3 KiB
JavaScript
55 lines
2.3 KiB
JavaScript
/**
|
|
* DC-167: default-namespace multistatus parsing (Radicale compatibility).
|
|
* Judge-lesson class: the parser required a namespace prefix on <response>,
|
|
* which Radicale (default DAV ns, bare <response>) never emits — result was
|
|
* a silent zero events, ok:true. Real-server XML style is now covered.
|
|
*/
|
|
'use strict';
|
|
|
|
const { extractCalendarsAndEvents } = require('../../src/calendar/caldav-client');
|
|
|
|
const BARE = '<?xml version="1.0"?><multistatus xmlns="DAV:">' +
|
|
'<response><href>/sami7777/calendar/a.ics</href><propstat><prop>' +
|
|
'<getetag>"aa"</getetag>' +
|
|
'<calendar-data xmlns:C="urn:ietf:params:xml:ns:caldav">BEGIN:VCALENDAR END:VCALENDAR</calendar-data>' +
|
|
'</prop></propstat></response>' +
|
|
'<response><href>/sami7777/calendar/b.ics</href><propstat><prop>' +
|
|
'<getetag>"bb"</getetag>' +
|
|
'<calendar-data>BEGIN:VCALENDAR</calendar-data>' +
|
|
'</prop></propstat></response></multistatus>';
|
|
|
|
const PREFIXED = BARE
|
|
.replace(/<response>/g, '<d:response>')
|
|
.replace(/<\/response>/g, '</d:response>')
|
|
.replace('<calendar-data', '<d:calendar-data')
|
|
.replace('</calendar-data>', '</d:calendar-data>');
|
|
|
|
describe('DC-167 default-namespace multistatus (Radicale)', () => {
|
|
test('bare <response> entries are extracted (Radicale style)', () => {
|
|
const out = extractCalendarsAndEvents(BARE);
|
|
expect(out).toHaveLength(2);
|
|
expect(out[0].href).toBe('/sami7777/calendar/a.ics');
|
|
expect(out[0].calendarData).toContain('BEGIN:VCALENDAR');
|
|
expect(out[1].href).toBe('/sami7777/calendar/b.ics');
|
|
});
|
|
|
|
test('prefixed <d:response> entries still extract (back-compat)', () => {
|
|
const out = extractCalendarsAndEvents(PREFIXED);
|
|
expect(out).toHaveLength(2);
|
|
expect(out[0].href).toBe('/sami7777/calendar/a.ics');
|
|
expect(out[0].calendarData).toContain('BEGIN:VCALENDAR');
|
|
});
|
|
|
|
test('mixed styles in one document do not double-count or drop', () => {
|
|
const mixed = BARE.replace('</response>', '</d:response>');
|
|
const out = extractCalendarsAndEvents(mixed);
|
|
expect(out.length).toBeGreaterThanOrEqual(1);
|
|
expect(out.some(e => e.href === '/sami7777/calendar/a.ics')).toBe(true);
|
|
});
|
|
|
|
test('empty multistatus yields zero entries (no throw)', () => {
|
|
const out = extractCalendarsAndEvents("<?xml version='1.0'?><multistatus xmlns='DAV:' />");
|
|
expect(out).toEqual([]);
|
|
});
|
|
});
|