fix: remove and replace sendgrid code in backend for nodemailer and gmail

Author: Clic-stack

email-api/.env.example

@@ -3,6 +3,5 @@ PORT=4000
 DATABASE_URL=postgres://user:password@localhost:5432/your_db
 EMAIL=
 GOOGLE_APP_PASSWORD=
-SENDGRID_API_KEY=
 SECRET_KEY=
 EXPIRE_IN=1d

email-api/src/config/env.js

@@ -6,7 +6,6 @@ export const env = {
   DATABASE_URL: process.env.DATABASE_URL || '',
   EMAIL: process.env.EMAIL || '',
   GOOGLE_APP_PASSWORD: process.env.GOOGLE_APP_PASSWORD || '',
-  SENDGRID_API_KEY: process.env.SENDGRID_API_KEY || '',
   SECRET_KEY: process.env.SECRET_KEY || '',
   EXPIRE_IN: process.env.EXPIRE_IN || '1d'
 }

email-api/src/mails/mailer.js

@@ -1,7 +1,6 @@
-import nodemailer from 'nodemailer' // Cambiamos la librería
+import nodemailer from 'nodemailer' // Se cambia la librería
 import { env } from '../config/env.js'
 
-// Creamos el transportador con la configuración de Gmail
 const transporter = nodemailer.createTransport({
   host: "smtp.gmail.com",
   port: 465,
@@ -15,14 +14,6 @@ const transporter = nodemailer.createTransport({
   }
 });
 
-/*const transporter = nodemailer.createTransport({
-  service: 'gmail',
-  auth: {
-    user: env.EMAIL,
-    pass: env.GOOGLE_APP_PASSWORD, // Aquí irá tu código de 16 letras
-  },
-});*/
-
 export const sendEmail = async ({ to, subject, html }) => {
   const mailOptions = {
     from: env.EMAIL,
@@ -36,29 +27,6 @@ export const sendEmail = async ({ to, subject, html }) => {
     console.log("Email sent via Gmail App Password to:", to);
   } catch (err) {
     console.error("Gmail/Nodemailer error:", err.message);
-    // Importante: No lanzamos el error (throw) para que el registro del usuario 
-    // en el controller no se detenga si el mail falla.
-  }
-}
-
-
-/*import sgMail from '@sendgrid/mail'
-import { env } from '../config/env.js'
-
-sgMail.setApiKey(env.SENDGRID_API_KEY)
-
-export const sendEmail = async ({ to, subject, html }) => {
-  const msg = {
-    to,
-    from: env.EMAIL, // debe ser el correo verificado en SendGrid
-    subject,
-    html
-  }
-  try {
-    await sgMail.send(msg)
-    console.log("Email sent via SendGrid:", to)
-  } catch (err) {
-    console.error("SendGrid error:", err.response?.body || err.message)
-    throw err
+    // Importante: No lanzamos el error (throw) para que el registro del usuario en el controller no se detenga si el mail falla.
   }
-}*/
+}

email-api/tests/user.test.js

@@ -1,13 +1,14 @@
 import request from 'supertest';
 import app from '../src/app.js';
 import sequelize from '../src/db/connect.js';
-//import { User } from '../src/models/user.model.js';
-import sgMail from '@sendgrid/mail';
+import nodemailer from 'nodemailer';
 
-// Mock de SendGrid para no gastar créditos
-jest.mock('@sendgrid/mail', () => ({
-  setApiKey: jest.fn(),
-  send: jest.fn().mockResolvedValue([{ statusCode: 202 }]),
+
+// Mock de Nodemailer para no enviar correos reales durante el test
+jest.mock('nodemailer', () => ({
+  createTransport: jest.fn().mockReturnValue({
+    sendMail: jest.fn().mockResolvedValue(true)
+  })
 }));
 
 const userTest = {
@@ -34,7 +35,7 @@ afterAll(async () => {
 
 describe('User Auth & Management API', () => {
 
-  // 1. Crear Usuario
+  // Crear Usuario
   test('POST /users should create a user and "send" an email', async () => {
     const res = await request(app)
       .post('/users')
@@ -43,11 +44,15 @@ describe('User Auth & Management API', () => {
     expect(res.status).toBe(201);
     expect(res.body.user).toHaveProperty('id');
     expect(res.body.user.email).toBe(userTest.email);
-    expect(sgMail.send).toHaveBeenCalled(); // Verifica que se intentó enviar el mail
+
+    // CAMBIO: Verificamos que el transportador de nodemailer fue llamado
+    const transport = nodemailer.createTransport();
+    expect(transport.sendMail).toHaveBeenCalled(); 
+    
     userId = res.body.user.id;
   });
 
-  // 2. Login Fallido (Sin verificar)
+  // Login Fallido (Sin verificar)
   test('POST /users/login should fail if user is not verified', async () => {
     const res = await request(app)
       .post('/users/login')
@@ -60,7 +65,7 @@ describe('User Auth & Management API', () => {
     expect(res.body.message).toBe('Unverified user');
   });
 
-  // 3. Obtener el código de verificación directamente de la DB para el test
+  // Obtener el código de verificación directamente de la DB para el test
   test('GET /users/verify/:code should verify the user', async () => {
     // Buscamos el código en la base de datos
     const { EmailCode } = await import('../src/models/emailcode.model.js');
@@ -74,7 +79,7 @@ describe('User Auth & Management API', () => {
     expect(res.body.user.isVerified).toBe(true);
   });
 
-  // 4. Login Exitoso (Ya verificado)
+  // Login Exitoso (Ya verificado)
   test('POST /users/login should return a JWT', async () => {
     const res = await request(app)
       .post('/users/login')
@@ -88,7 +93,7 @@ describe('User Auth & Management API', () => {
     token = res.body.token;
   });
 
-  // 5. Ruta Protegida (GET /users/me)
+  // Ruta Protegida (GET /users/me)
   test('GET /users/me should return user data with valid token', async () => {
     const res = await request(app)
       .get('/users/me')
@@ -98,13 +103,13 @@ describe('User Auth & Management API', () => {
     expect(res.body.email).toBe(userTest.email);
   });
 
-  // 6. Prueba de Seguridad (Acceso denegado)
+  // Prueba de Seguridad (Acceso denegado)
   test('GET /users/me should return 401 without token', async () => {
     const res = await request(app).get('/users/me');
     expect(res.status).toBe(401);
   });
 
-  //test para emails
+  // test para emails
   describe('Emails Controller - Auth-Mailer-API', () => {
     test('POST /emails/contact should trigger SendGrid and return 200', async () => {
       const contactData = {
@@ -120,10 +125,12 @@ describe('User Auth & Management API', () => {
 
       expect(res.status).toBe(200);
       expect(res.body.message).toBe('Email sent successfully!');
-      // Verificamos que el mock de SendGrid fue llamado
-      expect(sgMail.send).toHaveBeenCalledWith(expect.objectContaining({
-        to: contactData.email,
-        subject: 'Hi there!'
+      
+      // CAMBIO: Verificamos que Nodemailer recibió los datos correctos
+      const transport = nodemailer.createTransport();
+      expect(transport.sendMail).toHaveBeenCalledWith(expect.objectContaining({
+        to: contactData.email
+        // El subject dependerá de lo definido en el controller
       }));
     });
   });