Skip to content

Commit f61c46a

Browse files
Taimoor  AhmedTaimoor  Ahmed
authored andcommitted
fix: validation errors
1 parent 098d017 commit f61c46a

2 files changed

Lines changed: 2 additions & 3 deletions

File tree

plugins/course-apps/proctoring/Settings.jsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -553,7 +553,8 @@ const ProctoringSettings = ({ onClose }) => {
553553
selectedProvider = proctoredExamSettings.proctoring_provider;
554554
}
555555

556-
const isEscalationEmailRequired = settingsResponse.data.requires_escalation_email_providers.includes(selectedProvider);
556+
const requiresEscalationEmailProvidersList = settingsResponse.data.requires_escalation_email_providers;
557+
const isEscalationEmailRequired = requiresEscalationEmailProvidersList.includes(selectedProvider);
557558
const ltiProviderSelected = proctoringProvidersLti.some(p => p.name === selectedProvider);
558559

559560
if (isEscalationEmailRequired || ltiProviderSelected) {

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

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,7 +105,6 @@ describe('ProctoredExamSettings', () => {
105105
await act(async () => render(intlWrapper(<ProctoredExamSettings {...defaultProps} />)));
106106
});
107107

108-
109108
it('Hides all other fields when enabledProctorExam is false when first loaded', async () => {
110109
cleanup();
111110
// Overrides the handler defined in beforeEach.
@@ -821,6 +820,5 @@ describe('ProctoredExamSettings', () => {
821820
expect(document.activeElement).toEqual(successAlert);
822821
});
823822
});
824-
825823
});
826824
});

0 commit comments

Comments
 (0)