refactor: update price field name for staff check (#694)

openedx · Mar 24, 2022 · 1e89b5b · 1e89b5b
1 parent e01b3c0
commit 1e89b5b
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/src/utils/validation.js b/src/utils/validation.js
@@ -100,8 +100,8 @@ const editCourseValidate = (values, props) => {
     if (isSubmittingRun) {
       // naive check to deteremine if track is executive education or not
       const isExecutiveEducation = registeredFields
-        && registeredFields['prices.executive-education']
-        && registeredFields['prices.executive-education'].count;
+        && registeredFields['prices.paid-executive-education']
+        && registeredFields['prices.paid-executive-education'].count;
 
       const runRequiredFields = isExecutiveEducation ? ['transcript_languages'] : ['transcript_languages', 'staff'];
       const runErrors = {};

diff --git a/src/utils/validation.test.js b/src/utils/validation.test.js
@@ -218,7 +218,7 @@ describe('editCourseValidate', () => {
     };
     expect(editCourseValidate(values, { targetRun: unpublishedTargetRun })).toEqual(expectedErrors);
   });
-  it('returns no error on submitting executive education course runs with missing staff', () => {
+  it('returns no error on submitting 2U executive education course runs with missing staff', () => {
     const values = {
       short_description: 'Short',
       full_description: 'Full',
@@ -242,7 +242,7 @@ describe('editCourseValidate', () => {
     expect(editCourseValidate(values, {
       targetRun: unpublishedTargetRun,
       registeredFields: {
-        'prices.executive-education': {
+        'prices.paid-executive-education': {
           count: 1,
         },
       },