fix: set sendMail env vars optional

Author: nfroidure

src/commands/printEnv.test.ts

@@ -15,7 +15,9 @@ describe('printEnvCommand', () => {
     // See : https://stackoverflow.com/questions/56349619/ts2352-declare-object-with-dynamic-properties-and-one-property-with-specific-t
     const printEnvCommand = await initPrintEnvCommand({
       log,
-      ENV: { NODE_ENV: NodeEnv.Test },
+      ENV: {
+        NODE_ENV: NodeEnv.Test,
+      },
     });
     const result = await printEnvCommand({
       command: 'whook',

src/services/sendMail.ts

@@ -7,7 +7,7 @@ import { type JsonObject } from 'type-fest';
 import { YError } from 'yerror';
 
 export type SendMailEnvVars = {
-  SMTP_CONNECTION_URL: string;
+  SMTP_CONNECTION_URL?: string;
 };
 export type SendMailDependencies = {
   ENV: SendMailEnvVars;
@@ -19,6 +19,10 @@ async function initSendMail({
   ENV,
   log = noop,
 }: SendMailDependencies): Promise<SendMailService> {
+  if (!ENV.SMTP_CONNECTION_URL) {
+    throw new YError('E_NO_SMTP_URL');
+  }
+
   const transporter = createTransport({
     host: new URL(ENV.SMTP_CONNECTION_URL).hostname,
     port: parseInt(new URL(ENV.SMTP_CONNECTION_URL).port, 10),