Merge pull request #97 from Arquisoft/Fernando-Prueba

Fernando prueba

gatewayservice/gateway-service.js

@@ -78,6 +78,16 @@ app.get('/pregunta', async (req, res) => {
 
 app.get('/updateStats', async (req, res) => {
   const { username, numRespuestasCorrectas, numRespuestasIncorrectas} = req.query;
+
+  if (!username || !numRespuestasCorrectas || !numRespuestasIncorrectas) {
+    return res.status(401).json({ error: 'Faltan parámetros en la solicitud' });
+  }
+
+  if (parseInt(numRespuestasCorrectas) > 10 || parseInt(numRespuestasCorrectas) < 0 ||
+      parseInt(numRespuestasIncorrectas) > 10 || parseInt(numRespuestasIncorrectas) < 0) {
+    return res.status(400).json({ error: 'El número de respuestas incorrectas  o correctas es erróneo' });
+  }
+
   try{  
     const updateStatsResponse = await axios.get(userServiceUrl+ `/updateStats?username=${username}&numRespuestasCorrectas=${numRespuestasCorrectas}&numRespuestasIncorrectas=${numRespuestasIncorrectas}`)
     res.json(updateStatsResponse.data);

gatewayservice/gateway-service.test.js

@@ -49,6 +49,21 @@ describe('Gateway Service', () => {
     expect(response.statusCode).toBe(401);
   });
 
+  it('debería manejar correctamente los errores del servicio de autenticación en /login', async () => {
+    // Simular un error del servicio de autenticación
+    const errorMessage = 'Error al autenticar usuario';
+    axios.post.mockRejectedValueOnce({ response: { status: 500, data: { error: errorMessage } } });
+
+    // Realizar la solicitud POST a /login con datos de usuario válidos
+    const response = await request(app)
+      .post('/login')
+      .send({ username: 'testuser', password: 'testpassword' });
+
+    // Verificar que la respuesta tenga el código de estado esperado (500) y contenga el mensaje de error adecuado
+    expect(response.status).toBe(500);
+    expect(response.body).toEqual({ error: errorMessage });
+  });
+
   // Test /adduser endpoint
   it('deberia añadir usuario correctamente', async () => {
     const response = await request(app)
@@ -82,4 +97,151 @@ describe('Gateway Service', () => {
     expect(response.status).toBe(200);
     expect(response.body).toEqual(mockData);
   });
+
+  // Prueba para la devolución de documentación
+  it('debería devolver la documentación de Swagger correctamente', async () => {
+    const response = await request(app)
+      .get('/api-doc')
+      .redirects(1); // Sigue un máximo de 1 redirección
+
+    // Verificar que la respuesta tenga un código de estado 200 y contenga contenido HTML de la documentación de Swagger
+    expect(response.statusCode).toBe(200);
+    expect(response.text).toContain('Swagger UI');
+  });
+
+  it('debería redirigir correctamente a la documentación de Swagger', async () => {
+    const response = await request(app)
+      .get('/api-doc');
+
+    // Verificar que la respuesta tenga un código de estado 301 y que la cabecera "Location" indique la ubicación de la redirección
+    expect(response.statusCode).toBe(301);
+    expect(response.headers['location']).toBe('/api-doc/');
+  });
+
+  it('debería devolver los datos del usuario correctamente', async () => {
+    // Datos de prueba para simular una solicitud GET a /getUserData
+    const userData = {
+      username: 'testuser',
+      // Otros datos del usuario que esperas recibir
+    };
+
+    // Mock de axios para simular una respuesta exitosa del servicio de usuario
+    axios.get.mockResolvedValueOnce({ data: userData });
+
+    // Realizar la solicitud GET a /getUserData con el nombre de usuario
+    const response = await request(app)
+      .get('/getUserData')
+      .query({ username: 'testuser' });
+
+    // Verificar que la respuesta sea exitosa (código de estado 200) y contenga los datos del usuario
+    expect(response.status).toBe(200);
+    expect(response.body).toEqual(userData);
+  });
+
+  it('debería manejar correctamente los errores del servicio de usuario', async () => {
+    // Simular un error del servicio de usuario
+    const errorMessage = 'Error al obtener datos del usuario';
+    axios.get.mockRejectedValueOnce({ response: { status: 500, data: { error: errorMessage } } });
+
+    // Realizar la solicitud GET a /getUserData con un nombre de usuario
+    const response = await request(app)
+      .get('/getUserData')
+      .query({ username: 'testuser' });
+
+    // Verificar que la respuesta tenga el código de estado esperado (500) y contenga el mensaje de error adecuado
+    expect(response.status).toBe(500);
+    expect(response.body).toEqual({ error: errorMessage });
+  });
+
+  it('debería actualizar las estadísticas del usuario correctamente', async () => {
+    // Datos de prueba para simular una solicitud GET a /updateStats
+    const username = 'testuser';
+    const numRespuestasCorrectas = 5;
+    const numRespuestasIncorrectas = 2;
+    const updateStatsData = { message: 'Estadísticas actualizadas correctamente' };
+
+    // Mock de axios para simular una respuesta exitosa del servicio de usuario
+    axios.get.mockResolvedValueOnce({ data: updateStatsData });
+
+    // Realizar la solicitud GET a /updateStats con los parámetros de las estadísticas
+    const response = await request(app)
+      .get('/updateStats')
+      .query({ 
+        username: username,
+        numRespuestasCorrectas: numRespuestasCorrectas,
+        numRespuestasIncorrectas: numRespuestasIncorrectas
+      });
+
+    // Verificar que la respuesta sea exitosa (código de estado 200) y contenga el mensaje de éxito
+    expect(response.status).toBe(200);
+    expect(response.body).toEqual(updateStatsData);
+  });
+
+  it('debería manejar correctamente los errores del servicio de usuario', async () => {
+    // Simular un error del servicio de usuario
+    const errorMessage = 'Error al actualizar las estadísticas del usuario';
+    axios.get.mockRejectedValueOnce({ response: { status: 500, data: { error: errorMessage } } });
+
+    // Realizar la solicitud GET a /updateStats con los parámetros de las estadísticas
+    const response = await request(app)
+      .get('/updateStats')
+      .query({ 
+        username: 'testuser',
+        numRespuestasCorrectas: 5,
+        numRespuestasIncorrectas: 2
+      });
+
+    // Verificar que la respuesta tenga el código de estado esperado (500) y contenga el mensaje de error adecuado
+    expect(response.status).toBe(500);
+    expect(response.body).toEqual({ error: errorMessage });
+  });
+
+  it('debería devolver un error si los valores de numRespuestasCorrectas o numRespuestasIncorrectas son inválidos', async () => {
+    const response = await request(app)
+      .get('/updateStats')
+      .query({ 
+        username: 'testuser',
+        numRespuestasCorrectas: -1, // Valor negativo, que no debería ser válido
+        numRespuestasIncorrectas: 5,
+      });
+
+    expect(response.status).toBe(400); // Esperamos un error de solicitud incorrecta
+  });
+
+  it('debería devolver un error si falta algún parámetro en la solicitud de updateStats', async () => {
+    const response = await request(app)
+      .get('/updateStats')
+      .query({ 
+        username: 'testuser',
+        // numRespuestasCorrectas o numRespuestasIncorrectas están faltando
+      });
+
+    expect(response.status).toBe(401); // Esperamos un error de solicitud incorrecta
+  });
+
+  /*
+  it('debería manejar correctamente la carga excesiva', async () => {
+      // Realizar múltiples solicitudes simultáneas a endpoints diferentes
+      const promises = [];
+      for (let i = 0; i < 100; i++) {
+        promises.push(request(app).post('/login').send({ username: `user${i}`, password: 'password' }));
+      }
+    
+      // Esperar a que se completen todas las solicitudes
+      const responses = await Promise.all(promises);
+    
+      // Verificar que todas las respuestas tengan el código de estado esperado (200)
+      responses.forEach(response => {
+        expect(response.status).toBe(200);
+      });
+  });
+  */
+
+  it('debería devolver un estado de salud "OK"', async () => {
+    const response = await request(app).get('/health');
+
+    // Verificar que la respuesta tenga el código de estado esperado (200) y contenga un mensaje de estado "OK"
+    expect(response.status).toBe(200);
+    expect(response.body).toEqual({ status: 'OK' });
+  });
 });