Tests añadidos

Arquisoft · Apr 11, 2024 · 52c0a71 · 52c0a71
1 parent 0d599b2
commit 52c0a71
Show file tree

Hide file tree

Showing 6 changed files with 106 additions and 20 deletions.
diff --git a/webapp/src/components/Profile/Profile.js b/webapp/src/components/Profile/Profile.js
@@ -55,7 +55,7 @@ const Perfil = (username) => {
                       {t('components.profile.recentGames')}
                     </Heading>
                     <Box overflowX={"scroll"} width={'100%'}>
-                      {userData.games.length > 0 ? (
+                      { userData.games && userData.games.length > 0 ? (
                         <Table variant="simple">
                           <Thead>
                             <Tr>

diff --git a/webapp/src/locales/es.json b/webapp/src/locales/es.json
@@ -131,7 +131,7 @@
       "name": "Nombre de usuario",
       "dateLabel": "Fecha de creación de la cuenta:",
       "ngame": "Partida",
-      "error": "Ha habido un error al intentar cargar las preguntas. Inténtalo más tarde"
+      "error": "Ha habido un error al intentar cargar el perfil del usuario. Inténtalo más tarde"
     },
     "ranking": {
       "loading": "Cargando ...",

diff --git a/webapp/src/pages/Calculadora/Calculadora.test.js b/webapp/src/pages/Calculadora/Calculadora.test.js
@@ -22,7 +22,7 @@ test("renders the game screen", () => {
   expect(screen.getByText(/puntuación/i)).toBeInTheDocument();
 });
 
-test("handles correct answer", () => {
+test("handles correct answer", async () => {
   render(
     <I18nextProvider i18n={i18n}>
       <MemoryRouter>
@@ -78,6 +78,10 @@ test("handles correct answer", () => {
       .trim()
   );
   expect(updatedScore).toBe(initialScore + 2);
+
+  await setTimeout(() => {
+
+  }, 1000);
 });
 
 test("handles incorrect answer", () => {
@@ -102,7 +106,7 @@ test("handles incorrect answer", () => {
   const submitButton = screen.getByRole("button", { name: /enviar/i });
 
   // Enter an incorrect answer and submit
-  fireEvent.change(inputField, { target: { value: 0 } });
+  fireEvent.change(inputField, { target: { value: "a" } });
   fireEvent.click(submitButton);
 
   // Check if the score remains the same
@@ -115,7 +119,7 @@ test("handles incorrect answer", () => {
   expect(updatedScore).toBe(initialScore);
 });
 
-test("handles game over", () => {
+test("handles game over", async () => {
   render(
     <I18nextProvider i18n={i18n}>
       <MemoryRouter>
@@ -131,7 +135,7 @@ test("handles game over", () => {
       .textContent.split(":")[1]
       .trim()
   );
-
+  
   // Get the input field and submit button
   const inputField = screen.getByTitle(/number/i);
   const submitButton = screen.getByRole("button", { name: /enviar/i });
@@ -153,10 +157,10 @@ test("handles game over", () => {
       .textContent.split(":")[1]
       .trim()
   ); 
-
-  screen.getByText(/jugar de nuevo/i).click();
-
-  waitFor(() => {
-    expect(screen.getByText(/¿/i)).toBeInTheDocument();
-  });
+  expect(updatedScore).toBe(initialScore);
+
+  await setTimeout(() => {
+    screen.getByText(/jugar de nuevo/i).click();
+  }, 1000);
+
 });
diff --git a/webapp/src/pages/History/History.js b/webapp/src/pages/History/History.js
@@ -12,7 +12,7 @@ import {
   Divider,
   useColorMode,
 } from "@chakra-ui/react";
-import React, { useEffect, useState, getMemo } from "react";
+import React, { useEffect, useState } from "react";
 import { useTranslation } from "react-i18next";
 import Nav from "../../components/Nav/Nav";
 import Footer from "../../components/Footer/Footer";

diff --git a/webapp/src/pages/History/History.test.js b/webapp/src/pages/History/History.test.js
@@ -0,0 +1,79 @@
+import { render, screen, waitFor } from "@testing-library/react";
+import { MemoryRouter } from "react-router-dom";
+import History from "./History";
+import { I18nextProvider } from "react-i18next";
+import i18n from "../../i18n.js";
+
+describe("History Component", () => {
+  test("renders user profile data after loading", async () => {
+    const mockUserData = {
+      username: "testUser",
+      createdAt: new Date(),
+      games: [
+        {
+          _id: "game1",
+          questions: [
+            {
+              pregunta: "Question 1",
+              respuestas: ["Answer 1", "Answer 2"],
+              correcta: "Answer 1",
+              respuesta: "Answer 2",
+            },
+            {
+              pregunta: "Question 2",
+              respuestas: ["Answer 3", "Answer 4"],
+              correcta: "Answer 3",
+              respuesta: "Answer 3",
+            },
+          ],
+        },
+      ],
+    };
+
+    jest.spyOn(global, "fetch").mockResolvedValueOnce({
+      ok: true,
+      json: () => Promise.resolve(mockUserData),
+    });
+
+    render(
+      <I18nextProvider i18n={i18n}>
+        <MemoryRouter>
+          <History />
+        </MemoryRouter>
+      </I18nextProvider>
+    );
+
+    await waitFor(() => {
+      expect(screen.getByText("testUser")).toBeInTheDocument();
+      expect(
+        screen.getByText(/Fecha de creación de la cuenta/i)
+      ).toBeInTheDocument();
+      expect(screen.getByText("Question 1")).toBeInTheDocument();
+      expect(screen.getByText("Question 2")).toBeInTheDocument();
+      expect(screen.getByText("Answer 1")).toBeInTheDocument();
+      expect(screen.getByText("Answer 2")).toBeInTheDocument();
+      expect(screen.getByText("Answer 3")).toBeInTheDocument();
+      expect(screen.getByText("Answer 4")).toBeInTheDocument();
+    });
+  });
+
+  test("displays an error message when profile loading fails", async () => {
+    jest
+      .spyOn(global, "fetch")
+      .mockRejectedValueOnce(new Error("Failed to fetch"));
+
+    render(
+      <I18nextProvider i18n={i18n}>
+        <MemoryRouter>
+          <History />
+        </MemoryRouter>
+      </I18nextProvider>
+    );
+
+    await waitFor(() => {
+      expect(
+        screen.getByText("Ha habido un error al intentar cargar el perfil del usuario. Inténtalo más tarde")
+      ).toBeInTheDocument();
+    });
+  });
+});
diff --git a/webapp/src/pages/Perfil/Perfil.test.js b/webapp/src/pages/Perfil/Perfil.test.js
@@ -27,7 +27,7 @@ describe("Perfil Component", () => {
       ],
     };
 
-    jest.spyOn(global, "fetch").mockResolvedValueOnce({
+    jest.spyOn(global, "fetch").mockResolvedValue({
       json: jest.fn().mockResolvedValueOnce(mockUserData),
     });
 
@@ -52,9 +52,10 @@ describe("Perfil Component", () => {
   });
 
   test("displays an error message when profile loading fails", async () => {
-    jest
-      .spyOn(global, "fetch")
-      .mockRejectedValueOnce(new Error("Failed to fetch"));
+    jest.spyOn(global, "fetch").mockRejectedValueOnce({
+      ok: false,
+      json: () => Promise.resolve(new Error("Error")),
+    });
 
     render(
       <I18nextProvider i18n={i18n}>
@@ -65,7 +66,7 @@ describe("Perfil Component", () => {
     );
 
     await waitFor(() => {
-      expect(screen.getByText("Error: Failed to fetch")).toBeInTheDocument();
+      expect(screen.getByText("Ha habido un error al intentar cargar la información del usuario. Inténtalo más tarde.")).toBeInTheDocument();
     });
   });
 
@@ -77,7 +78,8 @@ describe("Perfil Component", () => {
     };
 
     jest.spyOn(global, "fetch").mockResolvedValueOnce({
-      json: jest.fn().mockResolvedValueOnce(mockUserData),
+      ok: true,
+      json: () => Promise.resolve(mockUserData),
     });
 
     render(
@@ -104,7 +106,8 @@ test("displays recent calculator game data", async () => {
   };
 
   jest.spyOn(global, "fetch").mockResolvedValueOnce({
-    json: jest.fn().mockResolvedValueOnce(mockUserData),
+    ok: true,
+    json: () => Promise.resolve(mockUserData),
   });
 
   render(