From f5d7fc2050910ed7cc1ab0747604bb7e66227050 Mon Sep 17 00:00:00 2001
From: 0x1eef <0x1eef@protonmail.com>
Date: Fri, 5 Apr 2024 15:44:49 -0300
Subject: [PATCH] Fix tests

---
 src/js/components/ErrorRenderer.tsx |  3 ++-
 test/ErrorRenderer.test.tsx         |  6 ++++--
 test/ResponseRenderer.test.tsx      | 11 +++--------
 3 files changed, 9 insertions(+), 11 deletions(-)

diff --git a/src/js/components/ErrorRenderer.tsx b/src/js/components/ErrorRenderer.tsx
index eed262b..972266c 100644
--- a/src/js/components/ErrorRenderer.tsx
+++ b/src/js/components/ErrorRenderer.tsx
@@ -7,7 +7,8 @@ export function ErrorRenderer({ error }: { error: Error }) {
     <div data-testid="error" className="w-full h-48">
       <div className="flex flex-col h-5/6 justify-center w-3/4 m-auto"></div>
       <Footer>
-        <span className="font-bold">{t("error")}</span>: {error.message}
+        <span className="font-bold">{t("error")}</span>:
+        <span data-testid="error-message">{error.message}</span>
       </Footer>
     </div>
   );
diff --git a/test/ErrorRenderer.test.tsx b/test/ErrorRenderer.test.tsx
index 4003d9d..d530922 100644
--- a/test/ErrorRenderer.test.tsx
+++ b/test/ErrorRenderer.test.tsx
@@ -18,7 +18,9 @@ describe("ErrorRenderer.tsx", () => {
   });
 
   test("an error is rendered", () => {
-    expect(screen.getByText("Error")).toBeInTheDocument();
-    expect(screen.getByText(error.message)).toBeInTheDocument();
+    const { getByTestId, getByText } = screen;
+    const span = getByTestId("error-message");
+    expect(getByText("Error")).toBeInTheDocument();
+    expect(span.textContent).toEqual(error.message);
   });
 });
diff --git a/test/ResponseRenderer.test.tsx b/test/ResponseRenderer.test.tsx
index 4e4eaa9..ee75f07 100644
--- a/test/ResponseRenderer.test.tsx
+++ b/test/ResponseRenderer.test.tsx
@@ -39,14 +39,9 @@ describe("ResponseRenderer.tsx", () => {
       expect(screen.getByText(response.ISP)).toBeInTheDocument();
     });
 
-    test("a city is rendered", () => {
-      expect(screen.getByText("City")).toBeInTheDocument();
-      expect(screen.getByText(response.City)).toBeInTheDocument();
-    });
-
-    test("a country is rendered", () => {
-      expect(screen.getByText("Country")).toBeInTheDocument();
-      expect(screen.getByText(response.Country)).toBeInTheDocument();
+    test("a location is rendered", () => {
+      expect(screen.getByText("Location")).toBeInTheDocument();
+      expect(screen.getByText(`${response.City}, ${response.Country}`)).toBeInTheDocument();
     });
 
     test("isTorExitNode is rendered as No", () => {