Merge 182402e621 into ee7a7a0544

client/src/__tests__/pages/ManageRoom/ManageRoom.test.tsx

@@ -294,75 +294,5 @@ describe('ManageRoom', () => {
             expect(screen.queryByText('Student 1')).not.toBeInTheDocument();
         });
     });
-
-    test('handles user joined and leaves but name still seeable', async () => {
-        await act(async () => {
-            render(
-                <MemoryRouter>
-                    <ManageRoom />
-                </MemoryRouter>
-            );
-        });
-
-        await act(async () => {
-            const createSuccessCallback = (mockSocket.on as jest.Mock).mock.calls.find(call => call[0] === 'create-success')[1];
-            createSuccessCallback('Test Room');
-        });
-
-        await act(async () => {
-            const userJoinedCallback = (mockSocket.on as jest.Mock).mock.calls.find(call => call[0] === 'user-joined')[1];
-            userJoinedCallback(mockStudents[0]);
-        });
-        await act(async () => {
-            const userJoinedCallback = (mockSocket.on as jest.Mock).mock.calls.find(call => call[0] === 'user-joined')[1];
-            userJoinedCallback(mockStudents[1]);
-        });
-
-        await waitFor(() => {
-            expect(screen.getByText('Student 1')).toBeInTheDocument();
-
-        });
-        await waitFor(() => {
-            expect(screen.getByText('Student 2')).toBeInTheDocument();
-
-        });
-
-        const launchButton = screen.getByText('Lancer');
-        fireEvent.click(launchButton);
-
-        const rythmeButton = screen.getByText('Rythme du professeur');
-        fireEvent.click(rythmeButton);
-
-        const secondLaunchButton = screen.getAllByText('Lancer');
-        fireEvent.click(secondLaunchButton[1]);
-
-        await waitFor(() => {
-            expect(screen.getByText('2/60')).toBeInTheDocument();
-
-        });
-
-        await act(async () => {
-            const userDisconnectedCallback = (mockSocket.on as jest.Mock).mock.calls.find(call => call[0] === 'user-disconnected')[1];
-            userDisconnectedCallback(mockStudents[1].id);
-        });
-
-
-        await waitFor(() => {
-            expect(screen.getByText('Résultats du quiz')).toBeInTheDocument();
-
-        });
-        const toggleUsernamesSwitch = screen.getByLabelText('Afficher les noms');
-
-        // Toggle the display of usernames back
-        fireEvent.click(toggleUsernamesSwitch);
-
-        expect(screen.getByText('Student 1')).toBeInTheDocument();
-
-        expect(screen.getByText('Student 2')).toBeInTheDocument();
-
-        await waitFor(() => {
-            expect(screen.getByText('1/60')).toBeInTheDocument();
-        });
-    });
 });
 

client/src/pages/Teacher/ManageRoom/ManageRoom.tsx

@@ -24,8 +24,8 @@ import { checkIfIsCorrect } from './useRooms';
 const ManageRoom: React.FC = () => {
     const navigate = useNavigate();
     const [socket, setSocket] = useState<Socket | null>(null);
-    const [connectedStudents, setConnectedStudents] = useState<StudentType[]>([]);
+    const [students, setStudents] = useState<StudentType[]>([]);
-    const [totalStudents, setTotalStudents] = useState<StudentType[]>([]);
+    const [allStudents, setAllStudents] = useState<StudentType[]>([]);
     const { quizId = '', roomName = '' }  = useParams<{ quizId: string, roomName: string }>();
     const [quizQuestions, setQuizQuestions] = useState<QuestionType[] | undefined>();
     const [quiz, setQuiz] = useState<QuizType | null>(null);
@@ -41,8 +41,8 @@ const ManageRoom: React.FC = () => {
     useEffect(() => {
         if (newlyConnectedUser) {
             console.log(`Handling newly connected user: ${newlyConnectedUser.name}`);
-            setConnectedStudents((prevStudents) => [...prevStudents, newlyConnectedUser]);
+            setStudents((prevStudents) => [...prevStudents, newlyConnectedUser]);
-            setTotalStudents((prevStudents) => [...prevStudents, newlyConnectedUser]);
+            setAllStudents((prevStudents) => [...prevStudents, newlyConnectedUser]);
 
             // only send nextQuestion if the quiz has started
             if (!quizStarted) {
@@ -130,8 +130,8 @@ const ManageRoom: React.FC = () => {
             setSocket(null);
             setQuizQuestions(undefined);
             setCurrentQuestion(undefined);
-            setConnectedStudents(new Array<StudentType>());
+            setStudents(new Array<StudentType>());
-            setTotalStudents(new Array<StudentType>());
+            setAllStudents(new Array<StudentType>());
 
         }
     };
@@ -176,7 +176,7 @@ const ManageRoom: React.FC = () => {
 
         socket.on('user-disconnected', (userId: string) => {
             console.log(`Student left: id = ${userId}`);
-            setConnectedStudents((prevUsers) => prevUsers.filter((user) => user.id !== userId));
+            setStudents((prevUsers) => prevUsers.filter((user) => user.id !== userId));
         });
 
         setSocket(socket);
@@ -197,7 +197,7 @@ const ManageRoom: React.FC = () => {
                 }
 
                 // Update the students state using the functional form of setStudents
-                setConnectedStudents((prevStudents) => {
+                setStudents((prevStudents) => {
                     console.log('Current students:');
                     prevStudents.forEach((student) => {
                         console.log(student.name);
@@ -244,7 +244,7 @@ const ManageRoom: React.FC = () => {
                     }
                     return updatedStudents;
                 });
-                setTotalStudents((prevStudents) => {
+                setAllStudents((prevStudents) => {
                     let foundStudent = false;
                     const updatedStudents = prevStudents.map((student) => {
                         console.log(`Comparing ${student.id} to ${idUser}`);
@@ -299,12 +299,10 @@ const ManageRoom: React.FC = () => {
         if (nextQuestionIndex === undefined || nextQuestionIndex > quizQuestions.length - 1) return;
 
         setCurrentQuestion(quizQuestions[nextQuestionIndex]);
-        webSocketService.nextQuestion({
+        webSocketService.nextQuestion({roomName: formattedRoomName, 
-            roomName: formattedRoomName,
                                        questions: quizQuestions, 
                                        questionIndex: nextQuestionIndex, 
-            isLaunch: false
+                                       isLaunch: false});
-        });
     };
 
     const previousQuestion = () => {
@@ -436,7 +434,7 @@ const ManageRoom: React.FC = () => {
                             style={{ display: "flex", justifyContent: "flex-end" }}
                         >
                             <GroupIcon style={{ marginRight: '5px' }} />
-                            {connectedStudents.length}/60
+                            {students.length}/60
                         </div>
                     )}
                 </div>
@@ -482,7 +480,7 @@ const ManageRoom: React.FC = () => {
                                     socket={socket}
                                     questions={quizQuestions}
                                     showSelectedQuestion={showSelectedQuestion}
-                                    students={totalStudents}
+                                    students={allStudents}
                                 ></LiveResultsComponent>
                             </div>
                         </div>
@@ -518,7 +516,7 @@ const ManageRoom: React.FC = () => {
                     </div>
                 ) : (
                     <StudentWaitPage
-                        students={connectedStudents}
+                        students={students}
                         launchQuiz={launchQuiz}
                         setQuizMode={setQuizMode}
                     />