From e41efba0cddda1c55a7c9b2dce3c0ea9b58af86c Mon Sep 17 00:00:00 2001 From: Zachary Hancock Date: Mon, 13 Nov 2023 09:14:54 -0500 Subject: [PATCH] feat: opt out is not supported by lti proctoring (#673) This toggle does nothing if an LTI tool is selected. We should hide it in that case. --- src/pages-and-resources/proctoring/Settings.jsx | 3 ++- src/pages-and-resources/proctoring/Settings.test.jsx | 1 + 2 files changed, 3 insertions(+), 1 deletion(-) diff --git a/src/pages-and-resources/proctoring/Settings.jsx b/src/pages-and-resources/proctoring/Settings.jsx index a4523902a..ecd38769d 100644 --- a/src/pages-and-resources/proctoring/Settings.jsx +++ b/src/pages-and-resources/proctoring/Settings.jsx @@ -350,7 +350,7 @@ const ProctoringSettings = ({ intl, onClose }) => { )} {/* ALLOW OPTING OUT OF PROCTORED EXAMS */} - { isEdxStaff && formValues.enableProctoredExams && ( + { isEdxStaff && formValues.enableProctoredExams && !isLtiProvider(formValues.proctoringProvider) && (
@@ -358,6 +358,7 @@ const ProctoringSettings = ({ intl, onClose }) => { diff --git a/src/pages-and-resources/proctoring/Settings.test.jsx b/src/pages-and-resources/proctoring/Settings.test.jsx index 9e5e6e8ca..eacf840a6 100644 --- a/src/pages-and-resources/proctoring/Settings.test.jsx +++ b/src/pages-and-resources/proctoring/Settings.test.jsx @@ -197,6 +197,7 @@ describe('ProctoredExamSettings', () => { fireEvent.change(selectElement, { target: { value: 'test_lti' } }); }); expect(screen.queryByTestId('escalationEmail')).toBeNull(); + expect(screen.queryByTestId('allowOptingOutRadio')).toBeNull(); expect(screen.queryByTestId('createZendeskTicketsYes')).toBeNull(); expect(screen.queryByTestId('createZendeskTicketsNo')).toBeNull(); });