fix: the missing role and unnecessary test

incubyte · Aug 8, 2023 · 92672c3 · 92672c3
1 parent a852f05
commit 92672c3
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 5 deletions.
diff --git a/frontend/src/components/home/LeaderBoard.test.tsx b/frontend/src/components/home/LeaderBoard.test.tsx
@@ -174,7 +174,6 @@ describe("should render the leaderBoard component", () => {
       expect(leaderBoardContainer2).toBeInTheDocument();
       expect(table).toBeInTheDocument();
       expect(table).toHaveTextContent("Email");
-      expect(table).toHaveTextContent("Rank");
       expect(table).toHaveTextContent("Role");
       expect(table).toHaveTextContent("Credits");
     });

diff --git a/frontend/src/components/home/LeaderBoard.tsx b/frontend/src/components/home/LeaderBoard.tsx
@@ -165,7 +165,7 @@ const LeaderBoard = () => {
         <h1 data-testid="leaderBoardTitle" className="courseHeading">
           Leader Board
         </h1>
-        <div className="courseContainer" data-testid="container2">
+        <div className="courseContainer" data-testid="container2" role="leaderBoard">
           <table data-testid="container2 table" className="courseTable">
             <thead className="courseTableHead" data-testid="tableHeading">
               <tr>
@@ -188,9 +188,8 @@ const LeaderBoard = () => {
                 return (
                   <tr
                     key={index}
-                    className={`${
-                      index % 2 === 1 ? "bg-gray-100" : "bg-white"
-                    }`}
+                    className={`${index % 2 === 1 ? "bg-gray-100" : "bg-white"
+                      }`}
                     role="row"
                   >
                     <td className="courseTableRows">{index + 1}</td>