CHE-7 Updated Banner button test for editProfile routing

Code-Hammers · Mar 29, 2024 · 3d4a5e1 · 3d4a5e1
1 parent 09877fd
commit 3d4a5e1
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 4 deletions.
diff --git a/client/src/components/Banner/Banner.test.tsx b/client/src/components/Banner/Banner.test.tsx
@@ -74,10 +74,10 @@ describe("Banner Component", () => {
     const optionsButton = screen.getByRole("button", { name: "Options" });
     fireEvent.click(optionsButton);
 
-    const profileOption = screen.getByText("Go to Profile");
+    const profileOption = screen.getByText("Edit Profile");
     fireEvent.click(profileOption);
 
-    expect(mockNavigate).toHaveBeenCalledWith("profile");
+    expect(mockNavigate).toHaveBeenCalledWith("editProfile");
   });
 
   it("handles logout on clicking Logout", () => {

diff --git a/client/src/pages/Profiles/Profiles.tsx b/client/src/pages/Profiles/Profiles.tsx
@@ -17,8 +17,9 @@ const Profiles = (): JSX.Element => {
         <h1 className="text-4xl font-extrabold mb-4">PROFILES</h1>
       </div>
       <div>
-        {profiles.map((profile) => (
-          <ProfileThumb key={profile._id} profile={profile} />
+        {/* TODO Look at better key for this */}
+        {profiles.map((profile, index) => (
+          <ProfileThumb key={index} profile={profile} />
         ))}
       </div>
     </>