Add specific exception when connected account not found

Author: martmull

packages/twenty-server/src/modules/mail-sender/exceptions/mail-sender.exception.ts

@@ -9,4 +9,5 @@ export class MailSenderException extends CustomException {
 
 export enum MailSenderExceptionCode {
   PROVIDER_NOT_SUPPORTED = 'PROVIDER_NOT_SUPPORTED',
+  CONNECTED_ACCOUNT_NOT_FOUND = 'CONNECTED_ACCOUNT_NOT_FOUND',
 }

packages/twenty-server/src/modules/mail-sender/workflow-actions/send-email.workflow-action.ts

@@ -21,6 +21,7 @@ import {
 } from 'src/modules/mail-sender/exceptions/mail-sender.exception';
 import { GmailClientProvider } from 'src/modules/messaging/message-import-manager/drivers/gmail/providers/gmail-client.provider';
 import { TwentyORMGlobalManager } from 'src/engine/twenty-orm/twenty-orm-global.manager';
+import { isDefined } from 'src/utils/is-defined';
 
 @Injectable()
 export class SendEmailWorkflowAction {
@@ -48,12 +49,17 @@ export class SendEmailWorkflowAction {
         workspaceId,
         'connectedAccount',
       );
-    const connectedAccount = await connectedAccountRepository.findOneOrFail({
-      where: {
-        id: step.settings.connectedAccountId,
-      },
+    const connectedAccount = await connectedAccountRepository.findOneBy({
+      id: step.settings.connectedAccountId,
     });
 
+    if (!isDefined(connectedAccount)) {
+      throw new MailSenderException(
+        `Connected Account '${step.settings.connectedAccountId}' not found`,
+        MailSenderExceptionCode.CONNECTED_ACCOUNT_NOT_FOUND,
+      );
+    }
+
     switch (connectedAccount.provider) {
       case 'google':
         return await this.gmailClientProvider.getGmailClient(connectedAccount);