@stamhoofd/models 2.96.2 → 2.97.0

package/src/models/Email.test.ts

@@ -1,10 +1,13 @@
-import { EmailRecipientFilter, EmailRecipientFilterType, EmailRecipientsStatus, EmailRecipient as EmailRecipientStruct, EmailRecipientSubfilter, EmailStatus, LimitedFilteredRequest, OrganizationMetaData, PaginatedResponse } from '@stamhoofd/structures';
+import { BalanceItemType, EmailRecipientFilter, EmailRecipientFilterType, EmailRecipientsStatus, EmailRecipient as EmailRecipientStruct, EmailRecipientSubfilter, EmailStatus, LimitedFilteredRequest, OrganizationMetaData, PaginatedResponse } from '@stamhoofd/structures';
 import { Email } from './Email';
 import { EmailRecipient } from './EmailRecipient';
 import { EmailMocker } from '@stamhoofd/email';
 import { STExpect, TestUtils } from '@stamhoofd/test-utils';
 import { OrganizationFactory } from '../factories/OrganizationFactory';
 import { Platform } from './Platform';
+import { BalanceItemFactory, MemberFactory, UserFactory } from '../factories';
+import { CachedBalance } from './CachedBalance';
+import { Formatter } from '@stamhoofd/utility';
 
 async function buildEmail(data: {
     recipients: EmailRecipientStruct[];
@@ -37,13 +40,14 @@ async function buildEmail(data: {
     });
 
     model.subject = data.subject ?? 'This is a test email';
-    model.html = data.html ?? '<p>This is a test email</p>';
-    model.text = data.text ?? 'This is a test email';
+    model.html = data.html ?? '<p>This is a test email</p> {{unsubscribeUrl}}';
+    model.text = data.text ?? 'This is a test email {{unsubscribeUrl}}';
     model.json = data.json ?? {};
     model.status = data.status ?? EmailStatus.Draft;
     model.attachments = [];
     model.fromAddress = data.fromAddress ?? 'test@stamhoofd.be';
     model.fromName = data.fromName ?? null;
+    model.emailType = data.emailType ?? null;
 
     await model.save();
 
@@ -79,13 +83,14 @@ async function buildFailEmail(data: {
     });
 
     model.subject = data.subject ?? 'This is a test email';
-    model.html = data.html ?? '<p>This is a test email</p>';
-    model.text = data.text ?? 'This is a test email';
+    model.html = data.html ?? '<p>This is a test email</p> {{unsubscribeUrl}}';
+    model.text = data.text ?? 'This is a test email {{unsubscribeUrl}}';
     model.json = data.json ?? {};
     model.status = data.status ?? EmailStatus.Draft;
     model.attachments = [];
     model.fromAddress = data.fromAddress ?? 'test@stamhoofd.be';
     model.fromName = data.fromName ?? null;
+    model.emailType = data.emailType ?? null;
 
     await model.save();
 
@@ -120,8 +125,8 @@ describe('Model.Email', () => {
         expect(model.emailErrors).toBe(null);
         expect(model.recipientsErrors).toBe(null);
         expect(model.status).toBe(EmailStatus.Sent);
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0); // never tried to send any failed emails (whitelist)
+        expect(await EmailMocker.getSucceededCount()).toBe(1);
+        expect(await EmailMocker.getFailedCount()).toBe(0); // never tried to send any failed emails (whitelist)
 
         // Load recipietns
         const recipients = await EmailRecipient.select().where('emailId', model.id).fetch();
@@ -177,8 +182,8 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(2);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(1); // One retry
+        expect(await EmailMocker.getSucceededCount()).toBe(2);
+        expect(await EmailMocker.getFailedCount()).toBe(1); // One retry
 
         // Load recipietns
         const recipients = await EmailRecipient.select().where('emailId', model.id).fetch();
@@ -229,8 +234,8 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(0);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0); // One retry
+        expect(await EmailMocker.getSucceededCount()).toBe(0);
+        expect(await EmailMocker.getFailedCount()).toBe(0); // One retry
     }, 15_000);
 
     it('Marks email recipient as failed if fails three times', async () => {
@@ -268,8 +273,8 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(0);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(6); // Two retries for each recipient
+        expect(await EmailMocker.getSucceededCount()).toBe(0);
+        expect(await EmailMocker.getFailedCount()).toBe(6); // Two retries for each recipient
 
         // Load recipietns
         const recipients = await EmailRecipient.select().where('emailId', model.id).fetch();
@@ -318,8 +323,8 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+        expect(await EmailMocker.getSucceededCount()).toBe(1);
+        expect(await EmailMocker.getFailedCount()).toBe(0);
 
         // Load recipietns
         const recipients = await EmailRecipient.select().where('emailId', model.id).fetch();
@@ -332,7 +337,7 @@ describe('Model.Email', () => {
         ]);
 
         // Check to header
-        expect(EmailMocker.broadcast.getSucceededEmail(0).to).toEqual('example@domain.be');
+        expect(EmailMocker.getSucceededEmail(0).to).toEqual('example@domain.be');
     }, 15_000);
 
     it('Includes recipient names in mail header', async () => {
@@ -358,8 +363,8 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+        expect(await EmailMocker.getSucceededCount()).toBe(1);
+        expect(await EmailMocker.getFailedCount()).toBe(0);
 
         // Load recipietns
         const recipients = await EmailRecipient.select().where('emailId', model.id).fetch();
@@ -374,7 +379,7 @@ describe('Model.Email', () => {
         ]);
 
         // Check to header
-        expect(EmailMocker.broadcast.getSucceededEmail(0).to).toEqual('"John Von Doe" <example@domain.be>');
+        expect(EmailMocker.getSucceededEmail(0).to).toEqual('"John Von Doe" <example@domain.be>');
     }, 15_000);
 
     it('Skips invalid email addresses', async () => {
@@ -400,8 +405,8 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(0);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+        expect(await EmailMocker.getSucceededCount()).toBe(0);
+        expect(await EmailMocker.getFailedCount()).toBe(0);
 
         // Load recipietns
         const recipients = await EmailRecipient.select().where('emailId', model.id).fetch();
@@ -449,8 +454,8 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+        expect(await EmailMocker.getSucceededCount()).toBe(1);
+        expect(await EmailMocker.getFailedCount()).toBe(0);
 
         // Load recipietns
         const recipients = await EmailRecipient.select().where('emailId', model.id).fetch();
@@ -496,11 +501,11 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+        expect(await EmailMocker.getSucceededCount()).toBe(1);
+        expect(await EmailMocker.getFailedCount()).toBe(0);
 
         // Check to header
-        expect(EmailMocker.broadcast.getSucceededEmail(0)).toMatchObject({
+        expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
             to: 'example@domain.be',
             from: '"My Platform" <info@my-platform.com>',
             replyTo: undefined,
@@ -540,11 +545,11 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+        expect(await EmailMocker.getSucceededCount()).toBe(1);
+        expect(await EmailMocker.getFailedCount()).toBe(0);
 
         // Check to header
-        expect(EmailMocker.broadcast.getSucceededEmail(0)).toMatchObject({
+        expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
             to: 'example@domain.be',
             from: '"My Platform" <noreply@broadcast.my-platform.com>', // domain has changed here
             replyTo: '"My Platform" <info@other-platform.com>', // Reply to should be set
@@ -589,11 +594,11 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+        expect(await EmailMocker.getSucceededCount()).toBe(1);
+        expect(await EmailMocker.getFailedCount()).toBe(0);
 
         // Check to header
-        expect(EmailMocker.broadcast.getSucceededEmail(0)).toMatchObject({
+        expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
             to: 'example@domain.be',
             from: '"My Platform" <noreply-uritest@broadcast.my-platform.com>', // domain has changed here
             replyTo: '"My Platform" <info@my-platform.com>', // Reply to should be set
@@ -640,11 +645,11 @@ describe('Model.Email', () => {
         expect(model.softFailedCount).toBe(0);
 
         // Both have succeeded
-        expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-        expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+        expect(await EmailMocker.getSucceededCount()).toBe(1);
+        expect(await EmailMocker.getFailedCount()).toBe(0);
 
         // Check to header
-        expect(EmailMocker.broadcast.getSucceededEmail(0)).toMatchObject({
+        expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
             to: 'example@domain.be',
             from: '"My Platform" <info@my-platform.com>', // domain has changed here
             replyTo: undefined,
@@ -677,6 +682,7 @@ describe('Model.Email', () => {
                 html: '{{fromAddress}}',
                 text: '{{fromAddress}}',
                 subject: '{{fromAddress}}',
+                emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
             });
 
             await model.queueForSending(true);
@@ -688,11 +694,11 @@ describe('Model.Email', () => {
             expect(model.status).toBe(EmailStatus.Sent);
 
             // Both have succeeded
-            expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-            expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+            expect(await EmailMocker.getSucceededCount()).toBe(1);
+            expect(await EmailMocker.getFailedCount()).toBe(0);
 
             // Check to header
-            expect(EmailMocker.broadcast.getSucceededEmail(0)).toMatchObject({
+            expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
                 to: 'example@domain.be',
                 from: '"Custom Name" <noreply-' + organization.uri + '@broadcast.my-platform.com>',
                 replyTo: '"Custom Name" <custom@customdomain.com>', // domain has changed here
@@ -733,6 +739,7 @@ describe('Model.Email', () => {
                 html: '{{primaryColor}};{{primaryColorContrast}};{{organizationName}};{{fromName}}',
                 text: '{{primaryColor}};{{primaryColorContrast}};{{organizationName}};{{fromName}}',
                 subject: '{{primaryColor}};{{primaryColorContrast}};{{organizationName}};{{fromName}}',
+                emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
             });
 
             await model.queueForSending(true);
@@ -744,11 +751,11 @@ describe('Model.Email', () => {
             expect(model.status).toBe(EmailStatus.Sent);
 
             // Both have succeeded
-            expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-            expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+            expect(await EmailMocker.getSucceededCount()).toBe(1);
+            expect(await EmailMocker.getFailedCount()).toBe(0);
 
             // Check to header
-            expect(EmailMocker.broadcast.getSucceededEmail(0)).toMatchObject({
+            expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
                 subject: `${brightYellow};${expectedContrastColor};${organization.name};${organization.name}`,
                 html: `${brightYellow};${expectedContrastColor};${organization.name};${organization.name}`,
                 text: `${brightYellow};${expectedContrastColor};${organization.name};${organization.name}`,
@@ -791,6 +798,7 @@ describe('Model.Email', () => {
                 html: '{{primaryColor}};{{primaryColorContrast}};{{organizationName}};{{fromName}}',
                 text: '{{primaryColor}};{{primaryColorContrast}};{{organizationName}};{{fromName}}',
                 subject: '{{primaryColor}};{{primaryColorContrast}};{{organizationName}};{{fromName}}',
+                emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
             });
 
             await model.queueForSending(true);
@@ -802,11 +810,11 @@ describe('Model.Email', () => {
             expect(model.status).toBe(EmailStatus.Sent);
 
             // Both have succeeded
-            expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-            expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+            expect(await EmailMocker.getSucceededCount()).toBe(1);
+            expect(await EmailMocker.getFailedCount()).toBe(0);
 
             // Check to header
-            expect(EmailMocker.broadcast.getSucceededEmail(0)).toMatchObject({
+            expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
                 subject: `${brightBlue};${expectedContrastColor};${organization.name};Custom Name`,
                 html: `${brightBlue};${expectedContrastColor};${organization.name};Custom Name`,
                 text: `${brightBlue};${expectedContrastColor};${organization.name};Custom Name`,
@@ -841,6 +849,7 @@ describe('Model.Email', () => {
                 html: '{{primaryColor}};{{primaryColorContrast}};{{organizationName}};{{fromName}}',
                 text: '{{primaryColor}};{{primaryColorContrast}};{{organizationName}};{{fromName}}',
                 subject: '{{primaryColor}};{{primaryColorContrast}};{{organizationName}};{{fromName}}',
+                emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
             });
 
             await model.queueForSending(true);
@@ -852,15 +861,360 @@ describe('Model.Email', () => {
             expect(model.status).toBe(EmailStatus.Sent);
 
             // Both have succeeded
-            expect(await EmailMocker.broadcast.getSucceededCount()).toBe(1);
-            expect(await EmailMocker.broadcast.getFailedCount()).toBe(0);
+            expect(await EmailMocker.getSucceededCount()).toBe(1);
+            expect(await EmailMocker.getFailedCount()).toBe(0);
 
             // Check to header
-            expect(EmailMocker.broadcast.getSucceededEmail(0)).toMatchObject({
+            expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
                 subject: `${darkRed};${expectedContrastColor};${platform.config.name};${platform.config.name}`,
                 html: `${darkRed};${expectedContrastColor};${platform.config.name};${platform.config.name}`,
                 text: `${darkRed};${expectedContrastColor};${platform.config.name};${platform.config.name}`,
             });
         }, 15_000);
+
+        describe('User based replacements', () => {
+            it('The balance is added for existing users', async () => {
+                TestUtils.setEnvironment('domains', {
+                    ...STAMHOOFD.domains,
+                    defaultTransactionalEmail: {
+                        '': 'my-platform.com',
+                    },
+                    defaultBroadcastEmail: {
+                        '': 'broadcast.my-platform.com',
+                    },
+                });
+
+                const organization = await new OrganizationFactory({}).create();
+                const existingUser = await new UserFactory({
+                    organization,
+                }).create();
+
+                // Add outstanding balance items for this user
+                const balanceItem = await new BalanceItemFactory({
+                    userId: existingUser.id,
+                    organizationId: organization.id,
+                    type: BalanceItemType.Other,
+                    amount: 2,
+                    unitPrice: 12_99,
+                    description: 'Test balance item',
+                }).create();
+                await CachedBalance.updateForUsers(organization.id, [existingUser.id]);
+
+                const model = await buildEmail({
+                    organizationId: organization.id,
+                    recipients: [
+                        EmailRecipientStruct.create({
+                            email: existingUser.email,
+                            userId: existingUser.id,
+                        }),
+                    ],
+                    fromAddress: 'custom@customdomain.com',
+                    html: '<p>{{outstandingBalance}}</p>{{balanceTable}}',
+                    subject: '{{outstandingBalance}}',
+                    emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
+                });
+
+                await model.queueForSending(true);
+                await model.refresh();
+
+                // Check if it was sent correctly
+                expect(model.recipientsStatus).toBe(EmailRecipientsStatus.Created);
+                expect(model.emailRecipientsCount).toBe(1);
+                expect(model.status).toBe(EmailStatus.Sent);
+
+                // Both have succeeded
+                expect(await EmailMocker.getSucceededCount()).toBe(1);
+                expect(await EmailMocker.getFailedCount()).toBe(0);
+
+                const expectedAmount = Formatter.price(balanceItem.unitPrice * balanceItem.amount);
+
+                // Check to header
+                expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
+                    subject: `${expectedAmount}`,
+                });
+
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude('<p>' + expectedAmount + '</p>');
+                expect(EmailMocker.getSucceededEmail(0).text).toInclude(expectedAmount);
+
+                // Check if the table is correct
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude('<table');
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude('2 x '); // amount
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude(Formatter.price(12_99)); // unit price
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude('<td>' + expectedAmount); // total price in table
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude('Test balance item'); // description
+
+                expect(EmailMocker.getSucceededEmail(0).text).toInclude('2 x '); // amount
+                expect(EmailMocker.getSucceededEmail(0).text).toInclude(Formatter.price(12_99)); // unit price
+                expect(EmailMocker.getSucceededEmail(0).text).toInclude(expectedAmount); // total price in table
+                expect(EmailMocker.getSucceededEmail(0).text?.toLowerCase()).toInclude('test balance item'); // description
+            }, 15_000);
+
+            it('The balance is zero for unknown users', async () => {
+                TestUtils.setEnvironment('domains', {
+                    ...STAMHOOFD.domains,
+                    defaultTransactionalEmail: {
+                        '': 'my-platform.com',
+                    },
+                    defaultBroadcastEmail: {
+                        '': 'broadcast.my-platform.com',
+                    },
+                });
+
+                const organization = await new OrganizationFactory({}).create();
+
+                const model = await buildEmail({
+                    organizationId: organization.id,
+                    recipients: [
+                        EmailRecipientStruct.create({
+                            email: 'unknown-user@example.com',
+                        }),
+                    ],
+                    fromAddress: 'custom@customdomain.com',
+                    html: '<p>{{outstandingBalance}}</p>{{balanceTable}}',
+                    subject: '{{outstandingBalance}}',
+                    emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
+                });
+
+                await model.queueForSending(true);
+                await model.refresh();
+
+                // Check if it was sent correctly
+                expect(model.recipientsStatus).toBe(EmailRecipientsStatus.Created);
+                expect(model.emailRecipientsCount).toBe(1);
+                expect(model.status).toBe(EmailStatus.Sent);
+
+                // Both have succeeded
+                expect(await EmailMocker.getSucceededCount()).toBe(1);
+                expect(await EmailMocker.getFailedCount()).toBe(0);
+
+                const expectedAmount = Formatter.price(0);
+
+                // Check to header
+                expect(EmailMocker.getSucceededEmail(0)).toMatchObject({
+                    subject: `${expectedAmount}`,
+                });
+
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude('<p>' + expectedAmount + '</p>');
+                expect(EmailMocker.getSucceededEmail(0).text).toInclude(expectedAmount);
+
+                // Check if the table is correct
+                expect(EmailMocker.getSucceededEmail(0).html).not.toInclude('<table');
+                expect(EmailMocker.getSucceededEmail(0).html).not.toInclude(' x '); // amount
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude('<p class="description">' + $t('4c4f6571-f7b5-469d-a16f-b1547b43a610') + '</p>');
+
+                expect(EmailMocker.getSucceededEmail(0).text).not.toInclude(' x '); // amount
+                expect(EmailMocker.getSucceededEmail(0).text?.toLowerCase()).toInclude($t('4c4f6571-f7b5-469d-a16f-b1547b43a610').toLowerCase());
+            }, 15_000);
+
+            it('loginDetails are added for existing users without password', async () => {
+                TestUtils.setEnvironment('domains', {
+                    ...STAMHOOFD.domains,
+                    defaultTransactionalEmail: {
+                        '': 'my-platform.com',
+                    },
+                    defaultBroadcastEmail: {
+                        '': 'broadcast.my-platform.com',
+                    },
+                });
+
+                const organization = await new OrganizationFactory({}).create();
+                const existingUser = await new UserFactory({
+                    organization,
+                    password: null,
+                }).create();
+
+                const model = await buildEmail({
+                    organizationId: organization.id,
+                    recipients: [
+                        EmailRecipientStruct.create({
+                            email: existingUser.email,
+                            userId: existingUser.id,
+                        }),
+                    ],
+                    fromAddress: 'custom@customdomain.com',
+                    html: '{{loginDetails}}',
+                    emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
+                });
+
+                await model.queueForSending(true);
+                await model.refresh();
+
+                // Check if it was sent correctly
+                expect(model.recipientsStatus).toBe(EmailRecipientsStatus.Created);
+                expect(model.emailRecipientsCount).toBe(1);
+                expect(model.status).toBe(EmailStatus.Sent);
+
+                // Both have succeeded
+                expect(await EmailMocker.getSucceededCount()).toBe(1);
+                expect(await EmailMocker.getFailedCount()).toBe(0);
+
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude(
+                    $t('3ab6ddc1-7ddc-4671-95d2-64994a5d36cc', { email: existingUser.email }),
+                );
+                expect(EmailMocker.getSucceededEmail(0).text).toInclude(
+                    $t('3ab6ddc1-7ddc-4671-95d2-64994a5d36cc', { email: existingUser.email }),
+                );
+            }, 15_000);
+
+            it('loginDetails are added for inexisting users', async () => {
+                TestUtils.setEnvironment('domains', {
+                    ...STAMHOOFD.domains,
+                    defaultTransactionalEmail: {
+                        '': 'my-platform.com',
+                    },
+                    defaultBroadcastEmail: {
+                        '': 'broadcast.my-platform.com',
+                    },
+                });
+
+                const organization = await new OrganizationFactory({}).create();
+
+                const model = await buildEmail({
+                    organizationId: organization.id,
+                    recipients: [
+                        EmailRecipientStruct.create({
+                            email: 'unknown@example.com',
+                        }),
+                    ],
+                    fromAddress: 'custom@customdomain.com',
+                    html: '{{loginDetails}}',
+                    emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
+                });
+
+                await model.queueForSending(true);
+                await model.refresh();
+
+                // Check if it was sent correctly
+                expect(model.recipientsStatus).toBe(EmailRecipientsStatus.Created);
+                expect(model.emailRecipientsCount).toBe(1);
+                expect(model.status).toBe(EmailStatus.Sent);
+
+                // Both have succeeded
+                expect(await EmailMocker.getSucceededCount()).toBe(1);
+                expect(await EmailMocker.getFailedCount()).toBe(0);
+
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude(
+                    $t('3ab6ddc1-7ddc-4671-95d2-64994a5d36cc', { email: 'unknown@example.com' }),
+                );
+                expect(EmailMocker.getSucceededEmail(0).text).toInclude(
+                    $t('3ab6ddc1-7ddc-4671-95d2-64994a5d36cc', { email: 'unknown@example.com' }),
+                );
+            }, 15_000);
+
+            it('loginDetails are added for existing users with password', async () => {
+                TestUtils.setEnvironment('domains', {
+                    ...STAMHOOFD.domains,
+                    defaultTransactionalEmail: {
+                        '': 'my-platform.com',
+                    },
+                    defaultBroadcastEmail: {
+                        '': 'broadcast.my-platform.com',
+                    },
+                });
+
+                const organization = await new OrganizationFactory({}).create();
+                const existingUser = await new UserFactory({
+                    organization,
+                    password: 'existing',
+                }).create();
+
+                const model = await buildEmail({
+                    organizationId: organization.id,
+                    recipients: [
+                        EmailRecipientStruct.create({
+                            email: existingUser.email,
+                            userId: existingUser.id,
+                        }),
+                    ],
+                    fromAddress: 'custom@customdomain.com',
+                    html: '{{loginDetails}}',
+                    emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
+                });
+
+                await model.queueForSending(true);
+                await model.refresh();
+
+                // Check if it was sent correctly
+                expect(model.recipientsStatus).toBe(EmailRecipientsStatus.Created);
+                expect(model.emailRecipientsCount).toBe(1);
+                expect(model.status).toBe(EmailStatus.Sent);
+
+                // Both have succeeded
+                expect(await EmailMocker.getSucceededCount()).toBe(1);
+                expect(await EmailMocker.getFailedCount()).toBe(0);
+
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude(
+                    $t('5403b466-98fe-48ac-beff-38acf7c9734d', { email: existingUser.email }),
+                );
+                expect(EmailMocker.getSucceededEmail(0).text).toInclude(
+                    $t('5403b466-98fe-48ac-beff-38acf7c9734d', { email: existingUser.email }),
+                );
+            }, 15_000);
+
+            it('loginDetails include member security keys for existing users without password', async () => {
+                TestUtils.setEnvironment('userMode', 'platform');
+                TestUtils.setEnvironment('domains', {
+                    ...STAMHOOFD.domains,
+                    defaultTransactionalEmail: {
+                        '': 'my-platform.com',
+                    },
+                    defaultBroadcastEmail: {
+                        '': 'broadcast.my-platform.com',
+                    },
+                });
+
+                const organization = await new OrganizationFactory({}).create();
+                const existingUser = await new UserFactory({
+                    organization,
+                    password: null,
+                }).create();
+                const member = await new MemberFactory({
+                    organization,
+                    user: existingUser,
+                }).create();
+                member.details.securityCode = '123456790123456'; // 16 chars
+                await member.save();
+
+                // Add a linked member
+
+                const model = await buildEmail({
+                    organizationId: organization.id,
+                    recipients: [
+                        EmailRecipientStruct.create({
+                            email: existingUser.email,
+                            userId: existingUser.id,
+                        }),
+                    ],
+                    fromAddress: 'custom@customdomain.com',
+                    html: '{{loginDetails}}',
+                    emailType: 'system-test', // Makes sure we don't need to include unsubscribeUrl
+                });
+
+                await model.queueForSending(true);
+                await model.refresh();
+
+                // Check if it was sent correctly
+                expect(model.recipientsStatus).toBe(EmailRecipientsStatus.Created);
+                expect(model.emailRecipientsCount).toBe(1);
+                expect(model.status).toBe(EmailStatus.Sent);
+
+                // Both have succeeded
+                expect(await EmailMocker.getSucceededCount()).toBe(1);
+                expect(await EmailMocker.getFailedCount()).toBe(0);
+
+                expect(EmailMocker.getSucceededEmail(0).html).toInclude(
+                    $t('e2519632-c495-4629-9ddb-334a4f00e272', {
+                        firstName: Formatter.escapeHtml(member.firstName),
+                        securityCode: `<span class="style-inline-code">${Formatter.escapeHtml(Formatter.spaceString(member.details.securityCode ?? '', 4, '-'))}</span>`,
+                    }),
+                );
+                expect(EmailMocker.getSucceededEmail(0).text).toInclude(
+                    $t('e2519632-c495-4629-9ddb-334a4f00e272', {
+                        firstName: member.firstName,
+                        securityCode: Formatter.spaceString(member.details.securityCode ?? '', 4, '-'),
+                    }),
+                );
+            }, 15_000);
+        });
     });
 });