refactor: migrate SmsFactory to Typescript

src/app/models/form.server.model.ts

@@ -92,7 +92,7 @@ export interface IFormModel extends Model<IFormSchema> {
   deactivateById(formId: string): Promise<IFormSchema | null>
 }
 
-type IEncryptedFormModel = Model<IEncryptedFormSchema>
+type IEncryptedFormModel = Model<IEncryptedFormSchema> & IFormModel
 
 const EncryptedFormSchema = new Schema<IEncryptedFormSchema>({
   publicKey: {
@@ -101,7 +101,7 @@ const EncryptedFormSchema = new Schema<IEncryptedFormSchema>({
   },
 })
 
-type IEmailFormModel = Model<IEmailFormSchema>
+type IEmailFormModel = Model<IEmailFormSchema> & IFormModel
 
 const EmailFormSchema = new Schema<IEmailFormSchema>({
   emails: {

src/app/modules/user/user.controller.ts

@@ -5,7 +5,7 @@ import { StatusCodes } from 'http-status-codes'
 
 import { createLoggerWithLabel } from '../../../config/logger'
 import { IPopulatedUser } from '../../../types'
-import SmsFactory from '../../factories/sms.factory'
+import { SmsFactory } from '../../services/sms/sms.factory'
 import { ApplicationError } from '../core/core.errors'
 
 import {

src/app/modules/verification/verification.factory.ts

@@ -1,8 +1,7 @@
 import { RequestHandler } from 'express'
 import { StatusCodes } from 'http-status-codes'
 
-import featureManager from '../../../config/feature-manager'
-import { FeatureNames } from '../../../config/feature-manager/types'
+import featureManager, { FeatureNames } from '../../../config/feature-manager'
 
 import * as verification from './verification.controller'
 

src/app/modules/verification/verification.service.ts

@@ -12,10 +12,10 @@ import {
   IVerificationFieldSchema,
   IVerificationSchema,
 } from '../../../types'
-import smsFactory from '../../factories/sms.factory'
 import getFormModel from '../../models/form.server.model'
 import getVerificationModel from '../../models/verification.server.model'
 import MailService from '../../services/mail.service'
+import { SmsFactory } from '../../services/sms/sms.factory'
 import { generateOtp } from '../../utils/otp'
 
 import { ITransaction } from './verification.types'
@@ -257,17 +257,15 @@ const sendOTPForField = async (
   field: IVerificationFieldSchema,
   recipient: string,
   otp: string,
-): Promise<void> => {
+): Promise<unknown> => {
   const { fieldType } = field
   switch (fieldType) {
     case 'mobile':
       // call sms - it should validate the recipient
-      await smsFactory.sendVerificationOtp(recipient, otp, formId)
-      break
+      return SmsFactory.sendVerificationOtp(recipient, otp, formId)
     case 'email':
       // call email - it should validate the recipient
-      await MailService.sendVerificationOtp(recipient, otp)
-      break
+      return MailService.sendVerificationOtp(recipient, otp)
     default:
       throw new Error(`sendOTPForField: ${fieldType} is unsupported`)
   }

src/app/services/sms/__tests__/sms.factory.spec.ts

@@ -0,0 +1,127 @@
+import Twilio from 'twilio'
+
+import {
+  FeatureNames,
+  ISms,
+  RegisteredFeature,
+} from 'src/config/feature-manager'
+
+import { createSmsFactory } from '../sms.factory'
+import * as SmsService from '../sms.service'
+import { TwilioConfig } from '../sms.types'
+
+// This is hoisted and thus a const cannot be passed in.
+jest.mock('twilio', () =>
+  jest.fn().mockImplementation(() => ({
+    mocked: 'this is mocked',
+  })),
+)
+
+const MOCKED_TWILIO = ({
+  mocked: 'this is mocked',
+} as unknown) as Twilio.Twilio
+
+describe('sms.factory', () => {
+  beforeEach(() => jest.clearAllMocks())
+
+  describe('sms feature disabled', () => {
+    const MOCK_DISABLED_SMS_FEATURE: RegisteredFeature<FeatureNames.Sms> = {
+      isEnabled: false,
+      props: {} as ISms,
+    }
+
+    const SmsFactory = createSmsFactory(MOCK_DISABLED_SMS_FEATURE)
+
+    it('should throw error when invoking sendAdminContactOtp', () => {
+      // Act
+      const invocation = () =>
+        SmsFactory.sendAdminContactOtp('anything', 'anything', 'anything')
+
+      // Assert
+      expect(invocation).toThrowError(
+        'sendAdminContactOtp: SMS feature must be enabled in Feature Manager first',
+      )
+    })
+
+    it('should throw error when invoking sendVerificationOtp', () => {
+      // Act
+      const invocation = () =>
+        SmsFactory.sendVerificationOtp('anything', 'anything', 'anything')
+
+      // Assert
+      expect(invocation).toThrowError(
+        'sendVerificationOtp: SMS feature must be enabled in Feature Manager first',
+      )
+    })
+  })
+
+  describe('sms feature enabled', () => {
+    const MOCK_ENABLED_SMS_FEATURE: RegisteredFeature<FeatureNames.Sms> = {
+      isEnabled: true,
+      props: {
+        twilioAccountSid: 'ACrandomTwilioSid',
+        twilioApiKey: 'SKrandomTwilioAPIKEY',
+        twilioApiSecret: 'this is a super secret',
+        twilioMsgSrvcSid: 'formsg-is-great-pleasehelpme',
+      },
+    }
+
+    const SmsFactory = createSmsFactory(MOCK_ENABLED_SMS_FEATURE)
+
+    it('should call SmsService counterpart when invoking sendAdminContactOtp', async () => {
+      // Arrange
+      const serviceContactSpy = jest
+        .spyOn(SmsService, 'sendAdminContactOtp')
+        .mockResolvedValue(true)
+
+      const mockArguments: Parameters<typeof SmsFactory.sendAdminContactOtp> = [
+        'mockRecipient',
+        'mockOtp',
+        'mockFormId',
+      ]
+
+      // Act
+      await SmsFactory.sendAdminContactOtp(...mockArguments)
+
+      // Assert
+      const expectedTwilioConfig: TwilioConfig = {
+        msgSrvcSid: MOCK_ENABLED_SMS_FEATURE.props.twilioMsgSrvcSid,
+        client: MOCKED_TWILIO,
+      }
+
+      expect(serviceContactSpy).toHaveBeenCalledTimes(1)
+      expect(serviceContactSpy).toHaveBeenCalledWith(
+        ...mockArguments,
+        expectedTwilioConfig,
+      )
+    })
+
+    it('should call SmsService counterpart when invoking sendVerificationOtp', async () => {
+      // Arrange
+      const serviceVfnSpy = jest
+        .spyOn(SmsService, 'sendVerificationOtp')
+        .mockResolvedValue(true)
+
+      const mockArguments: Parameters<typeof SmsFactory.sendAdminContactOtp> = [
+        'mockRecipient',
+        'mockOtp',
+        'mockUserId',
+      ]
+
+      // Act
+      await SmsFactory.sendVerificationOtp(...mockArguments)
+
+      // Assert
+      const expectedTwilioConfig: TwilioConfig = {
+        msgSrvcSid: MOCK_ENABLED_SMS_FEATURE.props.twilioMsgSrvcSid,
+        client: MOCKED_TWILIO,
+      }
+
+      expect(serviceVfnSpy).toHaveBeenCalledTimes(1)
+      expect(serviceVfnSpy).toHaveBeenCalledWith(
+        ...mockArguments,
+        expectedTwilioConfig,
+      )
+    })
+  })
+})