client/src/__tests__/components/QuestionsDisplay/MultipleChoiceQuestionDisplay/MultipleChoiceQuestionDisplay.test.tsx

@@ -12,23 +12,14 @@ const questions = parse(
     {
         =Choice 1
         ~Choice 2
-    }
-    
-    ::Sample Question 2:: Question stem
-    {
-        =Choice 1
-        =Choice 2
-        ~Choice 3
-    }
-    `) as MultipleChoiceQuestion[];
+    }`) as MultipleChoiceQuestion[];
 
-const questionWithOneCorrectChoice = questions[0];
-const questionWithMultipleCorrectChoices = questions[1];
+const question = questions[0];
 
 describe('MultipleChoiceQuestionDisplay', () => {
     const mockHandleOnSubmitAnswer = jest.fn();
 
-    const TestWrapper = ({ showAnswer, question }: { showAnswer: boolean; question: MultipleChoiceQuestion }) => {
+    const TestWrapper = ({ showAnswer }: { showAnswer: boolean }) => {
         const [showAnswerState, setShowAnswerState] = useState(showAnswer);
 
         const handleOnSubmitAnswer = (answer: AnswerType) => {
@@ -47,41 +38,20 @@ describe('MultipleChoiceQuestionDisplay', () => {
         );
     };
 
-    const twoChoices = questionWithOneCorrectChoice.choices;
-    const threeChoices = questionWithMultipleCorrectChoices.choices;
+    const choices = question.choices;
 
-    test('renders a question (that has only one correct choice) and its choices', () => {
-        render(<TestWrapper showAnswer={false} question={questionWithOneCorrectChoice} />);
+    beforeEach(() => {
+        render(<TestWrapper showAnswer={false} />);
+    });
 
-        expect(screen.getByText(questionWithOneCorrectChoice.formattedStem.text)).toBeInTheDocument();
-        twoChoices.forEach((choice) => {
+    test('renders the question and choices', () => {
+        expect(screen.getByText(question.formattedStem.text)).toBeInTheDocument();
+        choices.forEach((choice) => {
             expect(screen.getByText(choice.formattedText.text)).toBeInTheDocument();
         });
     });
 
-    test('only allows one choice to be selected when question only has one correct answer', () => {
-        render(<TestWrapper showAnswer={false} question={questionWithOneCorrectChoice} />);
-
-        const choiceButton1 = screen.getByText('Choice 1').closest('button');
-        const choiceButton2 = screen.getByText('Choice 2').closest('button');
-
-        if (!choiceButton1 || !choiceButton2) throw new Error('Choice buttons not found');
-
-        // Simulate selecting multiple answers
-        act(() => {
-            fireEvent.click(choiceButton1);
-        });
-        act(() => {
-            fireEvent.click(choiceButton2);
-        });
-
-        // Verify that only the last answer is selected
-        expect(choiceButton1.querySelector('.answer-text.selected')).not.toBeInTheDocument();
-        expect(choiceButton2.querySelector('.answer-text.selected')).toBeInTheDocument();
-    });
-
     test('does not submit when no answer is selected', () => {
-        render(<TestWrapper showAnswer={false} question={questionWithOneCorrectChoice} />);
         const submitButton = screen.getByText('Répondre');
         act(() => {
             fireEvent.click(submitButton);
@@ -91,7 +61,6 @@ describe('MultipleChoiceQuestionDisplay', () => {
     });
 
     test('submits the selected answer', () => {
-        render(<TestWrapper showAnswer={false} question={questionWithOneCorrectChoice} />);
         const choiceButton = screen.getByText('Choice 1').closest('button');
         if (!choiceButton) throw new Error('Choice button not found');
         act(() => {
@@ -106,43 +75,12 @@ describe('MultipleChoiceQuestionDisplay', () => {
         mockHandleOnSubmitAnswer.mockClear();
     });
 
-
-    test('renders a question (that has multiple correct choices) and its choices', () => {
-        render(<TestWrapper showAnswer={false} question={questionWithMultipleCorrectChoices} />);
-        expect(screen.getByText(questionWithMultipleCorrectChoices.formattedStem.text)).toBeInTheDocument();
-        threeChoices.forEach((choice) => {
-            expect(screen.getByText(choice.formattedText.text)).toBeInTheDocument();
-        });
-    });
-
-    test('allows multiple choices to be selected when question has multiple correct answers', () => {
-        render(<TestWrapper showAnswer={false} question={questionWithMultipleCorrectChoices} />);
-        const choiceButton1 = screen.getByText('Choice 1').closest('button');
-        const choiceButton2 = screen.getByText('Choice 2').closest('button');
-        const choiceButton3 = screen.getByText('Choice 3').closest('button');
-
-        if (!choiceButton1 || !choiceButton2 || !choiceButton3) throw new Error('Choice buttons not found');
-
-        act(() => {
-            fireEvent.click(choiceButton1);
-        });
-        act(() => {
-            fireEvent.click(choiceButton2);
-        });
-
-        expect(choiceButton1.querySelector('.answer-text.selected')).toBeInTheDocument();
-        expect(choiceButton2.querySelector('.answer-text.selected')).toBeInTheDocument();
-        expect(choiceButton3.querySelector('.answer-text.selected')).not.toBeInTheDocument(); // didn't click
-    
-    });
-
     test('submits multiple selected answers', () => {
-        render(<TestWrapper showAnswer={false} question={questionWithMultipleCorrectChoices} />);
         const choiceButton1 = screen.getByText('Choice 1').closest('button');
         const choiceButton2 = screen.getByText('Choice 2').closest('button');
-
+    
         if (!choiceButton1 || !choiceButton2) throw new Error('Choice buttons not found');
-
+    
         // Simulate selecting multiple answers
         act(() => {
             fireEvent.click(choiceButton1);
@@ -150,20 +88,19 @@ describe('MultipleChoiceQuestionDisplay', () => {
         act(() => {
             fireEvent.click(choiceButton2);
         });
-
+    
         // Simulate submitting the answers
         const submitButton = screen.getByText('Répondre');
         act(() => {
             fireEvent.click(submitButton);
         });
-
+    
         // Verify that the mockHandleOnSubmitAnswer function is called with both answers
         expect(mockHandleOnSubmitAnswer).toHaveBeenCalledWith(['Choice 1', 'Choice 2']);
         mockHandleOnSubmitAnswer.mockClear();
     });
-
+    
     it('should show ✅ next to the correct answer and ❌ next to the wrong answers when showAnswer is true', async () => {
-        render(<TestWrapper showAnswer={false} question={questionWithOneCorrectChoice} />);
         const choiceButton = screen.getByText('Choice 1').closest('button');
         if (!choiceButton) throw new Error('Choice button not found');
 
@@ -179,17 +116,16 @@ describe('MultipleChoiceQuestionDisplay', () => {
         });
 
         // Wait for the DOM to update
-        const correctAnswer = screen.getByText("Choice 1").closest('button');
-        expect(correctAnswer).toBeInTheDocument();
-        expect(correctAnswer?.textContent).toContain('✅');
+            const correctAnswer = screen.getByText("Choice 1").closest('button');
+            expect(correctAnswer).toBeInTheDocument();
+            expect(correctAnswer?.textContent).toContain('✅');
 
-        const wrongAnswer1 = screen.getByText("Choice 2").closest('button');
-        expect(wrongAnswer1).toBeInTheDocument();
-        expect(wrongAnswer1?.textContent).toContain('❌');
+            const wrongAnswer1 = screen.getByText("Choice 2").closest('button');
+            expect(wrongAnswer1).toBeInTheDocument();
+            expect(wrongAnswer1?.textContent).toContain('❌');
     });
 
-    it('should not show ✅ or ❌ when Répondre button is not clicked', async () => {
-        render(<TestWrapper showAnswer={false} question={questionWithOneCorrectChoice} />);
+    it('should not show ✅ or ❌ when repondre button is not clicked', async () => {
         const choiceButton = screen.getByText('Choice 1').closest('button');
         if (!choiceButton) throw new Error('Choice button not found');
 
@@ -209,5 +145,5 @@ describe('MultipleChoiceQuestionDisplay', () => {
         expect(wrongAnswer1?.textContent).not.toContain('❌');
     });
 
-});
+    });
 

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

@@ -30,12 +30,6 @@ describe('checkIfIsCorrect', () => {
         expect(result).toBe(false);
     });
 
-    test('returns false when all correct answers are selected, but one incorrect is also selected', () => {
-        const answer: AnswerType = ['Answer1', 'Answer2', 'Answer3'];
-        const result = checkIfIsCorrect(answer, 1, mockQuestions);
-        expect(result).toBe(false);
-    });
-
     test('returns false when no answers are selected', () => {
         const answer: AnswerType = [];
         const result = checkIfIsCorrect(answer, 1, mockQuestions);

client/src/__tests__/pages/Student/StudentModeQuiz/StudentModeQuiz.test.tsx

@@ -120,29 +120,27 @@ describe('StudentModeQuiz', () => {
         expect(screen.getByText('Répondre')).toBeInTheDocument();
     });
 
-    // le test suivant est fait dans MultipleChoiceQuestionDisplay.test.tsx
-//     test('allows multiple answers to be selected for a question', async () => {
-//         // Simulate selecting multiple answers
-//         act(() => {
-//             fireEvent.click(screen.getByText('Option A'));
-//         });
-//         act(() => {
-//             fireEvent.click(screen.getByText('Option B'));
-//         });
+    test('allows multiple answers to be selected for a question', async () => {
+        // Simulate selecting multiple answers
+        act(() => {
+            fireEvent.click(screen.getByText('Option A'));
+        });
+        act(() => {
+            fireEvent.click(screen.getByText('Option B'));
+        });
 
-//         // Simulate submitting the answers
-//         act(() => {
-//             fireEvent.click(screen.getByText('Répondre'));
-//         });
+        // Simulate submitting the answers
+        act(() => {
+            fireEvent.click(screen.getByText('Répondre'));
+        });
 
-//         // Verify that the mockSubmitAnswer function is called with both answers
-//         expect(mockSubmitAnswer).toHaveBeenCalledWith(['Option A', 'Option B'], 1);
-
-//         // Verify that the selected answers are displayed as selected
-//         const buttonA = screen.getByRole('button', { name: '✅ A Option A' });
-//         const buttonB = screen.getByRole('button', { name: '✅ B Option B' });
-//         expect(buttonA).toBeInTheDocument();
-//         expect(buttonB).toBeInTheDocument();
-//     });
+        // Verify that the mockSubmitAnswer function is called with both answers
+        expect(mockSubmitAnswer).toHaveBeenCalledWith(['Option A', 'Option B'], 1);
 
+        // Verify that the selected answers are displayed as selected
+        const buttonA = screen.getByRole('button', { name: '✅ A Option A' });
+        const buttonB = screen.getByRole('button', { name: '✅ B Option B' });
+        expect(buttonA).toBeInTheDocument();
+        expect(buttonB).toBeInTheDocument();
+    });
 });

client/src/components/GiftTemplate/templates/MultipleChoiceAnswersTemplate.ts

@@ -17,7 +17,7 @@ interface AnswerWeightOptions extends TemplateOptions {
 export default function MultipleChoiceAnswersTemplate({ choices }: MultipleChoiceAnswerOptions) {
     const id = `id${nanoid(8)}`;
 
-    const isMultipleAnswer = choices.filter(({ isCorrect }) => isCorrect === true).length != 0;
+    const isMultipleAnswer = choices.filter(({ isCorrect }) => isCorrect === true).length === 0;
 
     const prompt = `<span style="${ParagraphStyle(state.theme)}">Choisir une réponse${
         isMultipleAnswer ? ` ou plusieurs` : ``
@@ -32,7 +32,7 @@ export default function MultipleChoiceAnswersTemplate({ choices }: MultipleChoic
             const inputId = `id${nanoid(6)}`;
 
             const isPositiveWeight = (weight != undefined) && (weight > 0);
-            const isCorrectOption = isMultipleAnswer ? isPositiveWeight || isCorrect : isCorrect;
+            const isCorrectOption = isMultipleAnswer ? isPositiveWeight : isCorrect;
 
             return `
         <div class='multiple-choice-answers-container'>

client/src/components/QuestionsDisplay/MultipleChoiceQuestionDisplay/MultipleChoiceQuestionDisplay.tsx

@@ -15,14 +15,7 @@ interface Props {
 
 const MultipleChoiceQuestionDisplay: React.FC<Props> = (props) => {
     const { question, showAnswer, handleOnSubmitAnswer, passedAnswer } = props;
-    console.log('MultipleChoiceQuestionDisplay: passedAnswer', JSON.stringify(passedAnswer));
-
-    const [answer, setAnswer] = useState<AnswerType>(() => {
-        if (passedAnswer && passedAnswer.length > 0) {
-            return passedAnswer;
-        }
-        return [];
-    });
+    const [answer, setAnswer] = useState<AnswerType>(passedAnswer || []);
 
     let disableButton = false;
     if (handleOnSubmitAnswer === undefined) {
@@ -30,31 +23,19 @@ const MultipleChoiceQuestionDisplay: React.FC<Props> = (props) => {
     }
 
     useEffect(() => {
-        console.log('MultipleChoiceQuestionDisplay: passedAnswer', JSON.stringify(passedAnswer));
         if (passedAnswer !== undefined) {
             setAnswer(passedAnswer);
-        } else {
-            setAnswer([]);
         }
-    }, [passedAnswer, question.id]);
+    }, [passedAnswer]);
 
     const handleOnClickAnswer = (choice: string) => {
         setAnswer((prevAnswer) => {
-            console.log(`handleOnClickAnswer -- setAnswer(): prevAnswer: ${prevAnswer}, choice: ${choice}`);
-            const correctAnswersCount = question.choices.filter((c) => c.isCorrect).length;
-
-            if (correctAnswersCount === 1) {
-                // If only one correct answer, replace the current selection
-                return prevAnswer.includes(choice) ? [] : [choice];
+            if (prevAnswer.includes(choice)) {
+                // Remove the choice if it's already selected
+                return prevAnswer.filter((selected) => selected !== choice);
             } else {
-                // Allow multiple selections if there are multiple correct answers
-                if (prevAnswer.includes(choice)) {
-                    // Remove the choice if it's already selected
-                    return prevAnswer.filter((selected) => selected !== choice);
-                } else {
-                    // Add the choice if it's not already selected
-                    return [...prevAnswer, choice];
-                }
+                // Add the choice if it's not already selected
+                return [...prevAnswer, choice];
             }
         });
     };
@@ -69,7 +50,6 @@ const MultipleChoiceQuestionDisplay: React.FC<Props> = (props) => {
             </div>
             <div className="choices-wrapper mb-1">
                 {question.choices.map((choice, i) => {
-                    console.log(`answer: ${answer}, choice: ${choice.formattedText.text}`);
                     const selected = answer.includes(choice.formattedText.text) ? 'selected' : '';
                     return (
                         <div key={choice.formattedText.text + i} className="choice-container">

client/src/components/QuestionsDisplay/TrueFalseQuestionDisplay/TrueFalseQuestionDisplay.tsx

@@ -17,15 +17,6 @@ const TrueFalseQuestionDisplay: React.FC<Props> = (props) => {
     const { question, showAnswer, handleOnSubmitAnswer, passedAnswer } =
         props;
 
-    const [answer, setAnswer] = useState<boolean | undefined>(() => {
-
-        if (passedAnswer && (passedAnswer[0] === true || passedAnswer[0] === false)) {
-            return passedAnswer[0];
-        }
-
-        return undefined;
-    });
-
     let disableButton = false;
     if (handleOnSubmitAnswer === undefined) {
         disableButton = true;
@@ -40,6 +31,15 @@ const TrueFalseQuestionDisplay: React.FC<Props> = (props) => {
         }
     }, [passedAnswer, question.id]);
 
+    const [answer, setAnswer] = useState<boolean | undefined>(() => {
+
+        if (passedAnswer && (passedAnswer[0] === true || passedAnswer[0] === false)) {
+            return passedAnswer[0];
+        }
+
+        return undefined;
+    });
+
     const handleOnClickAnswer = (choice: boolean) => {
         setAnswer(choice);
     };

client/src/pages/Teacher/ManageRoom/useRooms.ts

@@ -55,15 +55,14 @@ export function checkIfIsCorrect(
         (!question.isTrue && simpleAnswerText == 'false')
       );
     } else if (question.type === 'MC') {
-      const correctChoices = question.choices.filter((choice) => choice.isCorrect
+      const correctAnswers = question.choices.filter((choice) => choice.isCorrect
         /* || (choice.weight && choice.weight > 0)*/  // handle weighted answers
       );
       const multipleAnswers = Array.isArray(answer) ? answer : [answer as string];
-      if (correctChoices.length === 0) {
+      if (correctAnswers.length === 0) {
         return false;
       }
-      // check if all (and only) correct choices are in the multipleAnswers array
-      return correctChoices.length === multipleAnswers.length && correctChoices.every(
+      return correctAnswers.every(
         (choice) => multipleAnswers.includes(choice.formattedText.text)
       );
     } else if (question.type === 'Numerical') {