From ac6f5b497cbaa586c10c25be0d43efd1dd074f3a Mon Sep 17 00:00:00 2001 From: Hongbo Wu Date: Wed, 1 Feb 2023 13:44:27 +0800 Subject: [PATCH] Accept null value for text in an email --- packages/api/src/routers/svc/emails.ts | 8 +------- packages/api/test/routers/emails.test.ts | 19 +++++++++++++++++-- 2 files changed, 18 insertions(+), 9 deletions(-) diff --git a/packages/api/src/routers/svc/emails.ts b/packages/api/src/routers/svc/emails.ts index b3ce6cefc..f6d1479c5 100644 --- a/packages/api/src/routers/svc/emails.ts +++ b/packages/api/src/routers/svc/emails.ts @@ -36,13 +36,7 @@ interface EmailMessage { } function isEmailMessage(data: any): data is EmailMessage { - return ( - 'from' in data && - 'to' in data && - 'subject' in data && - 'html' in data && - 'text' in data - ) + return 'from' in data && 'to' in data && 'subject' in data } const logger = buildLogger('app.dispatch') diff --git a/packages/api/test/routers/emails.test.ts b/packages/api/test/routers/emails.test.ts index 1b68043b1..f181d06d4 100644 --- a/packages/api/test/routers/emails.test.ts +++ b/packages/api/test/routers/emails.test.ts @@ -125,6 +125,7 @@ describe('Emails Router', () => { }) describe('create', () => { + const url = '/svc/pubsub/emails/save' const html = 'test html' const text = 'test text' const from = 'fake from' @@ -140,12 +141,26 @@ describe('Emails Router', () => { subject, } const res = await request - .post('/svc/pubsub/emails/save') + .post(url) + .set('Authorization', `${authToken}`) + .send(data) + .expect(200) + + expect(res.body.id).not.to.be.undefined + }) + + it('saves the email if body is empty', async () => { + const data = { + from, + to: newsletterEmail, + subject, + } + const res = await request + .post(url) .set('Authorization', `${authToken}`) .send(data) .expect(200) - console.log(res.body) expect(res.body.id).not.to.be.undefined }) })