From ddc49643fe558c5f91b1ac10e91cbde3c9ba9ca4 Mon Sep 17 00:00:00 2001 From: 4gray Date: Thu, 1 Oct 2026 19:29:47 +0200 Subject: [PATCH] test(ui): give the dialog service spec the now-required confirm labels ConfirmDialogData.confirmLabel became required, and the spec still built confirmations without one. Jest only transpiles, so the suite stayed green, but the "Typecheck Jest spec programs" CI step rejected it. Co-Authored-By: Claude Opus 5.5 --- .../components/src/lib/confirm-dialog/dialog.service.spec.ts | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/libs/ui/components/src/lib/confirm-dialog/dialog.service.spec.ts b/libs/ui/components/src/lib/confirm-dialog/dialog.service.spec.ts index 348d3ec79..3dda815f8 100644 --- a/libs/ui/components/src/lib/confirm-dialog/dialog.service.spec.ts +++ b/libs/ui/components/src/lib/confirm-dialog/dialog.service.spec.ts @@ -33,6 +33,7 @@ describe('DialogService', () => { service.openConfirmDialog({ title: 'Remove', message: 'Confirm removal?', + confirmLabel: 'Remove playlist', onConfirm: jest.fn(), }); @@ -57,6 +58,7 @@ describe('DialogService', () => { service.openConfirmDialog({ title: 'Refresh Xtream Playlist', message: 'Confirm refresh?', + confirmLabel: 'Refresh playlist', width: '400px', onConfirm: jest.fn(), }); @@ -78,6 +80,7 @@ describe('DialogService', () => { service.openConfirmDialog({ title: 'Remove', message: 'Confirm removal?', + confirmLabel: 'Remove playlist', onConfirm, }); @@ -89,6 +92,7 @@ describe('DialogService', () => { service.openConfirmDialog({ title: 'Recovery', message: 'Path', + confirmLabel: 'Copy path', keepOpenOnConfirm: true, onConfirm, });