Improvements on CI

.github/workflows/node.js.yml

@@ -14,10 +14,18 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
+
+      - name: Cache dependencies
+        uses: actions/cache@v2
+        with:
+          path: ~/.npm
+          key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
+          restore-keys: |
+            ${{ runner.os }}-node-
 
       - name: Set up Node.js
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
           node-version: "20"
 
@@ -47,10 +55,18 @@ jobs:
 
     steps:
       - name: Checkout code
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
+
+      - name: Cache dependencies
+        uses: actions/cache@v2
+        with:
+          path: ~/.npm
+          key: ${{ runner.os }}-node-${{ hashFiles('**/package-lock.json') }}
+          restore-keys: |
+            ${{ runner.os }}-node-
 
       - name: Set up Node.js
-        uses: actions/setup-node@v3
+        uses: actions/setup-node@v4
         with:
           node-version: "20"
 
@@ -63,7 +79,12 @@ jobs:
 
       - name: Initialize database
         run: |
-          docker exec -i $(docker ps -q --filter "ancestor=mysql:8.0") mysql -uroot -prootpassword test < ./dbscripts/mysql.sql
+          if docker exec -i $(docker ps -q --filter "ancestor=mysql:8.0") mysql -uroot -prootpassword test < ./dbscripts/mysql.sql; then
+            echo "Database initialized successfully";
+          else
+            echo "Database initialization failed";
+            exit 1;
+          fi
 
       - name: Run tests
         env:

src/loaders/sequelize.ts

@@ -15,8 +15,7 @@ const sequelizeLoader = async () => {
       dialect: "mysql",
     }
   );
-  console.log(config.database.mysql);
-
+
   await sequelize.authenticate();
 
 

tests/unit/controllers/authController.spec.ts

@@ -17,7 +17,6 @@ describe('Auth Controller', () => {
             .post('/api/auth/login')
             .send({});
 
-        console.log(response.body);
 
         expect(response.status).toBe(400);
         expect(response.body).toHaveProperty('message', "email is required, contrasena is required");
@@ -39,7 +38,6 @@ describe('Auth Controller', () => {
             .post('/api/auth/register')
             .send(user);
 
-        console.log(response.body);
 
         expect(response.status).toBe(200);
         expect(response.body).toHaveProperty('message', 'Registro exitoso');
@@ -51,7 +49,6 @@ describe('Auth Controller', () => {
             .post('/api/auth/login')
             .send({ email: user.email, contrasena: user.contrasena });
 
-        console.log(response.body);
 
         expect(response.status).toBe(200);
         expect(response.body).toHaveProperty('message', 'Inicio de sesion exitoso');
@@ -68,7 +65,6 @@ describe('Auth Controller', () => {
                 contrasena: '22222222222275f6tuyibinj',
             });
 
-        console.log(response.body);
 
         expect(response.status).toBe(400);
         expect(response).toHaveProperty('error');
@@ -99,8 +95,6 @@ describe('Auth Controller', () => {
                 contrasena: 'testpassword',
             });
 
-        console.log(response.body);
-
         expect(response.status).toBe(400);
         expect(response).toHaveProperty('error');
         expect(response.body).toHaveProperty('message', 'email must be a valid email');