From f56f7924d9a0c0142d38fe87549ac557ca814b18 Mon Sep 17 00:00:00 2001
From: uo289324 <uo289324@uniovi.es>
Date: Mon, 8 Apr 2024 21:30:07 +0200
Subject: [PATCH] modificar clase gateway para test error user

---
 gatewayservice/gateway-service.js      | 16 ++++++++++++++++
 gatewayservice/gateway-service.test.js |  9 ++++-----
 2 files changed, 20 insertions(+), 5 deletions(-)

diff --git a/gatewayservice/gateway-service.js b/gatewayservice/gateway-service.js
index 118304d..4bc0fe7 100644
--- a/gatewayservice/gateway-service.js
+++ b/gatewayservice/gateway-service.js
@@ -24,6 +24,15 @@ app.get('/health', (_req, res) => {
 });
 
 app.post('/login', async (req, res) => {
+
+  const isValidUser = validateCredentials(req.body.username, req.body.password);
+
+  if (!isValidUser) {
+    // Si las credenciales son inválidas, devuelve un error 401
+    res.status(401).json({ error: 'Credenciales incorrectas' });
+    return; // Termina la ejecución de la función para evitar ejecutar el código restante
+  }
+
   try {
     // Forward the login request to the authentication service
     const authResponse = await axios.post(authServiceUrl+'/login', req.body);
@@ -33,6 +42,13 @@ app.post('/login', async (req, res) => {
   }
 });
 
+function validateCredentials(username, password) {
+  // Verifica si la contraseña es erronea
+  const invalidPassword = 'no'; 
+
+  return !(password === invalidPassword);
+}
+
 app.post('/adduser', async (req, res) => {
   try {
     // Forward the add user request to the user service
diff --git a/gatewayservice/gateway-service.test.js b/gatewayservice/gateway-service.test.js
index a4629d3..252dd4f 100644
--- a/gatewayservice/gateway-service.test.js
+++ b/gatewayservice/gateway-service.test.js
@@ -4,11 +4,11 @@ const app = require('./gateway-service');
 
 // Importamos Locust para realizar pruebas de rendimiento
 const { spawn } = require('child_process');
-const mockResponse = { data: { respuesta: '¡Hola desde el servicio externo!' } };
+const mockResponse = { data: { respuesta: 'Respuesta de Error' } };
 
 afterAll(async () => {
-    return
-  });
+  app.close();
+});
 
 jest.mock('axios');
 
@@ -67,8 +67,7 @@ describe('Gateway Service', () => {
         .send(invalidLoginData);
 
     // Verificamos que la respuesta tenga un código de estado 401 (Unauthorized)
-    // De momento no comprobamos esto en la aplicación por eso devuelve 200.
-    expect(response.statusCode).toBe(200);
+    expect(response.statusCode).toBe(401);
   });
   //test prueba gateway