3ea9e630ed
Prep PR for the admin auth module (docs/plan-admin-auth.md step 1).
better-auth 1.7 ships ESM only, so the API moves off CommonJS:
- "type": "module", module nodenext, target ES2024, .js suffixes on all
relative imports, require('mariadb'|'cors') replaced by imports, and
export= packages (winston, app-root-path, bcrypt) consumed via default
imports. The logger now uses appRoot.path explicitly.
- TypeScript 5.9, @types/node 26, tslint removed. Node 26 pinned via
engines and .nvmrc (Plesk runs 26).
- Jest 28 + ts-jest replaced by vitest 5. Eight test files depend on
hoisted module mocks with static imports and resetModules + require,
which Jest's ESM mode does not support; vitest keeps them nearly
verbatim. Coverage via @vitest/coverage-v8 (lcov), Sonar generic report
via vitest-sonar-reporter, so sonar-project.properties is unchanged.
vitest.config.ts sets FEEDBACK_IP_SALT so the suite passes without a
local .env.
- dotenv 8 -> 16 and axios 0.24 -> 1.x: their old typings are not
resolvable under nodenext.
- autoCommit: false dropped from the pool configs; it is not a mariadb
connector option and was silently ignored.
tsc clean, 96/96 tests green, compiled app boots and serves /, /docs and
CORS under Node ESM.
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
133 lines
4.7 KiB
TypeScript
133 lines
4.7 KiB
TypeScript
// tickets.confirmation-email builds and sends the redemption confirmation
|
|
// email, shared by the public redeem path and the admin resend action.
|
|
|
|
import {vi, describe, it, expect, beforeEach, type Mock} from 'vitest';
|
|
vi.mock('../../src/models/calendar/events/events.service.js', () => ({
|
|
getEventById: vi.fn()
|
|
}));
|
|
vi.mock('../../src/models/calendar/events/icalgenerator.service.js', () => ({
|
|
convertToIcal: vi.fn()
|
|
}));
|
|
vi.mock('../../src/common/common.mail.js', () => ({
|
|
MailService: {sendMail: vi.fn()}
|
|
}));
|
|
vi.mock('../../src/models/tickets/Tickets.db.js', () => ({
|
|
NachklangTicketsDB: {getConnection: vi.fn()}
|
|
}));
|
|
vi.mock('../../src/middleware/logger.js', () => ({
|
|
__esModule: true,
|
|
default: {info: vi.fn(), warn: vi.fn(), error: vi.fn()}
|
|
}));
|
|
|
|
import * as EventsService from '../../src/models/calendar/events/events.service.js';
|
|
import * as IcalService from '../../src/models/calendar/events/icalgenerator.service.js';
|
|
import {MailService} from '../../src/common/common.mail.js';
|
|
import logger from '../../src/middleware/logger.js';
|
|
import {NachklangTicketsDB} from '../../src/models/tickets/Tickets.db.js';
|
|
import {recordConfirmationEmailResult, sendRedemptionConfirmation} from '../../src/models/tickets/tickets.confirmation-email.js';
|
|
|
|
const mockGetEvent = EventsService.getEventById as Mock;
|
|
const mockToIcal = IcalService.convertToIcal as Mock;
|
|
const mockSendMail = MailService.sendMail as Mock;
|
|
const mockLogger = logger as unknown as {info: Mock; warn: Mock; error: Mock};
|
|
const mockGetConnection = NachklangTicketsDB.getConnection as Mock;
|
|
|
|
const EVENT = {
|
|
eventId: 42,
|
|
name: 'Sommerkonzert 2026',
|
|
startDateTime: new Date('2026-07-01T19:00:00Z'),
|
|
location: 'Christuskirche',
|
|
status: 'PUBLISHED'
|
|
};
|
|
|
|
const RECIPIENT = {
|
|
eventId: 42,
|
|
contactName: 'Erika Mustermann',
|
|
contactEmail: 'erika@example.com',
|
|
guestNames: ['Erika Mustermann', 'Hans Mustermann']
|
|
};
|
|
|
|
beforeEach(() => {
|
|
vi.clearAllMocks();
|
|
mockGetEvent.mockResolvedValue(EVENT);
|
|
mockToIcal.mockResolvedValue('BEGIN:VCALENDAR\nEND:VCALENDAR');
|
|
mockSendMail.mockResolvedValue(true);
|
|
});
|
|
|
|
describe('sendRedemptionConfirmation', () => {
|
|
it('sends the German confirmation with the event details, guest list and .ics attachment', async () => {
|
|
const result = await sendRedemptionConfirmation(RECIPIENT);
|
|
|
|
expect(result).toBe(true);
|
|
expect(mockSendMail).toHaveBeenCalledTimes(1);
|
|
const [to, subject, body, options] = mockSendMail.mock.calls[0];
|
|
expect(to).toBe('erika@example.com');
|
|
expect(subject).toBe('Bestätigung: Sommerkonzert 2026');
|
|
expect(body).toContain('Hallo Erika Mustermann,');
|
|
expect(body).toContain('"Sommerkonzert 2026"');
|
|
expect(body).toContain('- Hans Mustermann');
|
|
expect(options.attachments).toEqual([
|
|
{filename: 'konzert.ics', content: 'BEGIN:VCALENDAR\nEND:VCALENDAR', contentType: 'text/calendar'}
|
|
]);
|
|
});
|
|
|
|
it('still sends (without an attachment) and warns when the .ics build fails', async () => {
|
|
mockToIcal.mockRejectedValue(new Error('ical boom'));
|
|
|
|
await sendRedemptionConfirmation(RECIPIENT);
|
|
|
|
const options = mockSendMail.mock.calls[0][3];
|
|
expect(options.attachments).toBeUndefined();
|
|
expect(mockLogger.warn).toHaveBeenCalledWith(expect.stringContaining('without .ics attachment'));
|
|
});
|
|
|
|
it('returns false and does not send when the event no longer exists', async () => {
|
|
mockGetEvent.mockResolvedValue(null);
|
|
|
|
const result = await sendRedemptionConfirmation(RECIPIENT);
|
|
|
|
expect(result).toBe(false);
|
|
expect(mockSendMail).not.toHaveBeenCalled();
|
|
});
|
|
|
|
it('propagates the relay result', async () => {
|
|
mockSendMail.mockResolvedValue(false);
|
|
|
|
expect(await sendRedemptionConfirmation(RECIPIENT)).toBe(false);
|
|
});
|
|
});
|
|
|
|
describe('recordConfirmationEmailResult', () => {
|
|
const makeConn = () => ({query: vi.fn().mockResolvedValue(undefined), end: vi.fn().mockResolvedValue(undefined)});
|
|
|
|
it('writes SENT when the mail was accepted', async () => {
|
|
const conn = makeConn();
|
|
mockGetConnection.mockResolvedValue(conn);
|
|
|
|
await recordConfirmationEmailResult(7, true);
|
|
|
|
expect(conn.query).toHaveBeenCalledWith(
|
|
'UPDATE redemptions SET confirmation_email_status = ? WHERE redemption_id = ?',
|
|
['SENT', 7]
|
|
);
|
|
expect(conn.end).toHaveBeenCalled();
|
|
});
|
|
|
|
it('writes FAILED when the mail was not accepted', async () => {
|
|
const conn = makeConn();
|
|
mockGetConnection.mockResolvedValue(conn);
|
|
|
|
await recordConfirmationEmailResult(7, false);
|
|
|
|
expect(conn.query).toHaveBeenCalledWith(expect.any(String), ['FAILED', 7]);
|
|
});
|
|
|
|
it('swallows a DB error rather than throwing', async () => {
|
|
const conn = {query: vi.fn().mockRejectedValue(new Error('db down')), end: vi.fn().mockResolvedValue(undefined)};
|
|
mockGetConnection.mockResolvedValue(conn);
|
|
|
|
await expect(recordConfirmationEmailResult(7, true)).resolves.toBeUndefined();
|
|
expect(conn.end).toHaveBeenCalled();
|
|
});
|
|
});
|