chore: Refactor proctoring provider handling in Settings

plugins/course-apps/proctoring/Settings.jsx

@@ -84,10 +84,7 @@ const ProctoringSettings = ({ onClose }) => {
     } else if (name === 'proctoringProvider') {
       const newFormValues = { ...formValues, proctoringProvider: value };
 
-      if (value === 'proctortrack') {
-        setFormValues({ ...newFormValues, createZendeskTickets: false });
-        setShowEscalationEmail(true);
-      } else if (value === 'software_secure') {
+      if (value === 'software_secure') {
         setFormValues({ ...newFormValues, createZendeskTickets: true });
         setShowEscalationEmail(false);
       } else if (isLtiProvider(value)) {

plugins/course-apps/proctoring/Settings.test.jsx

@@ -105,16 +105,6 @@ describe('ProctoredExamSettings', () => {
       await act(async () => render(intlWrapper(<IntlProctoredExamSettings {...defaultProps} />)));
     });
 
-    it('Updates Zendesk ticket field if proctortrack is provider', async () => {
-      await waitFor(() => {
-        screen.getByDisplayValue('mockproc');
-      });
-      const selectElement = screen.getByDisplayValue('mockproc');
-      fireEvent.change(selectElement, { target: { value: 'proctortrack' } });
-      const zendeskTicketInput = screen.getByTestId('createZendeskTicketsNo');
-      expect(zendeskTicketInput.checked).toEqual(true);
-    });
-
     it('Updates Zendesk ticket field if software_secure is provider', async () => {
       await waitFor(() => {
         screen.getByDisplayValue('mockproc');
@@ -250,7 +240,8 @@ describe('ProctoredExamSettings', () => {
         expect(document.activeElement).toEqual(escalationEmailInput);
       });
 
-      it(`Creates an alert when invalid proctoring escalation email is provided with ${provider} selected`, async () => {
+      // Test skipped due to Proctortrack deprecation
+      it.skip(`Creates an alert when invalid proctoring escalation email is provided with ${provider} selected`, async () => {
         await waitFor(() => {
           screen.getByDisplayValue('proctortrack');
         });
@@ -349,7 +340,8 @@ describe('ProctoredExamSettings', () => {
         expect(screen.queryByTestId('escalationEmail')).toBeNull();
       });
 
-      it(`Escalation email Field Show when proctoring backend is switched back to ${provider}`, async () => {
+      // Test skipped due to Proctortrack deprecation
+      it.skip(`Escalation email Field Show when proctoring backend is switched back to ${provider}`, async () => {
         await waitFor(() => {
           screen.getByDisplayValue('proctortrack');
         });
@@ -606,37 +598,6 @@ describe('ProctoredExamSettings', () => {
       expect(submitButton).toHaveAttribute('disabled');
     });
 
-    it('Makes API call successfully with proctoring_escalation_email if proctortrack', async () => {
-      await act(async () => render(intlWrapper(<IntlProctoredExamSettings {...defaultProps} />)));
-      // Make a change to the provider to proctortrack and set the email
-      const selectElement = screen.getByDisplayValue('mockproc');
-      fireEvent.change(selectElement, { target: { value: 'proctortrack' } });
-      const escalationEmail = screen.getByTestId('escalationEmail');
-      expect(escalationEmail.value).toEqual('test@example.com');
-      fireEvent.change(escalationEmail, { target: { value: 'proctortrack@example.com' } });
-      expect(escalationEmail.value).toEqual('proctortrack@example.com');
-      const submitButton = screen.getByTestId('submissionButton');
-      fireEvent.click(submitButton);
-      expect(axiosMock.history.post.length).toBe(1);
-      expect(JSON.parse(axiosMock.history.post[0].data)).toEqual({
-        proctored_exam_settings: {
-          enable_proctored_exams: true,
-          allow_proctoring_opt_out: false,
-          proctoring_provider: 'proctortrack',
-          proctoring_escalation_email: 'proctortrack@example.com',
-          create_zendesk_tickets: false,
-        },
-      });
-
-      await waitFor(() => {
-        const errorAlert = screen.getByTestId('saveSuccess');
-        expect(errorAlert.textContent).toEqual(
-          expect.stringContaining('Proctored exam settings saved successfully.'),
-        );
-        expect(document.activeElement).toEqual(errorAlert);
-      });
-    });
-
     it('Makes API call successfully without proctoring_escalation_email if not proctortrack', async () => {
       await act(async () => render(intlWrapper(<IntlProctoredExamSettings {...defaultProps} />)));
 
@@ -863,7 +824,8 @@ describe('ProctoredExamSettings', () => {
       });
     });
 
-    it('Include Zendesk ticket in post request if user is not an admin', async () => {
+    // Test skipped due to Proctortrack deprecation
+    it.skip('Include Zendesk ticket in post request if user is not an admin', async () => {
       // use non-admin user for test
       const isAdmin = false;
       setupApp(isAdmin);