From 87e988a9eef1d1374aa441cd45a70e8db42d0227 Mon Sep 17 00:00:00 2001
From: Marcella Maki <marcellamaki@gmail.com>
Date: Sun, 2 Jul 2023 17:24:48 -0400
Subject: [PATCH 1/3] Refactor the default completion and duration content
 completion options to be in utils.js, and make the default data visible in
 the resource panel

---
 .../channelEdit/components/ResourcePanel.vue  |  15 ++-
 .../components/edit/CompletionOptions.vue     |  59 ++--------
 .../frontend/channelEdit/utils.js             | 101 +++++++++++++++++-
 3 files changed, 118 insertions(+), 57 deletions(-)

diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue b/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue
index 8046b9b591..5e249c1556 100644
--- a/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue
+++ b/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue
@@ -532,7 +532,12 @@
   import sortBy from 'lodash/sortBy';
   import { mapActions, mapGetters } from 'vuex';
   import camelCase from 'lodash/camelCase';
-  import { isImportedContent, importedChannelLink, getCompletionCriteriaLabels } from '../utils';
+  import {
+    isImportedContent,
+    importedChannelLink,
+    getCompletionCriteriaLabels,
+    getAudioVideoDefaultDuration,
+  } from '../utils';
   import FilePreview from '../views/files/FilePreview';
   import { ContentLevels, Categories, AccessibilityCategories } from '../../shared/constants';
   import AssessmentItemPreview from './AssessmentItemPreview/AssessmentItemPreview';
@@ -614,6 +619,9 @@
         return getCompletionCriteriaLabels(this.node).completion;
       },
       duration() {
+        if (this.isAudioVideo) {
+          return getAudioVideoDefaultDuration(this.files);
+        }
         return getCompletionCriteriaLabels(this.node).duration;
       },
       files() {
@@ -665,6 +673,11 @@
       isResource() {
         return !this.isTopic && !this.isExercise;
       },
+      isAudioVideo() {
+        return (
+          this.node.kind === ContentKindsNames.AUDIO || this.node.kind === ContentKindsNames.VIDEO
+        );
+      },
       isImported() {
         return isImportedContent(this.node);
       },
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/edit/CompletionOptions.vue b/contentcuration/contentcuration/frontend/channelEdit/components/edit/CompletionOptions.vue
index 6840c106bc..bc16858092 100644
--- a/contentcuration/contentcuration/frontend/channelEdit/components/edit/CompletionOptions.vue
+++ b/contentcuration/contentcuration/frontend/channelEdit/components/edit/CompletionOptions.vue
@@ -100,6 +100,12 @@
 <script>
 
   import get from 'lodash/get';
+  import {
+    defaultCompletionCriteriaModels,
+    defaultCompletionCriteriaThresholds,
+    CompletionOptionsDropdownMap,
+    completionCriteriaToDropdownMap,
+  } from '../../utils';
   import MasteryCriteriaMofNFields from './MasteryCriteriaMofNFields';
   import ActivityDuration from './ActivityDuration.vue';
   import MasteryCriteriaGoal from './MasteryCriteriaGoal';
@@ -125,59 +131,6 @@
   const DEFAULT_SHORT_ACTIVITY = 600;
   const DEFAULT_LONG_ACTIVITY = 3000;
 
-  const defaultCompletionCriteriaModels = {
-    [ContentKindsNames.VIDEO]: CompletionCriteriaModels.TIME,
-    [ContentKindsNames.AUDIO]: CompletionCriteriaModels.TIME,
-    [ContentKindsNames.DOCUMENT]: CompletionCriteriaModels.PAGES,
-    [ContentKindsNames.H5P]: CompletionCriteriaModels.DETERMINED_BY_RESOURCE,
-    [ContentKindsNames.HTML5]: CompletionCriteriaModels.APPROX_TIME,
-    [ContentKindsNames.EXERCISE]: CompletionCriteriaModels.MASTERY,
-  };
-
-  const defaultCompletionCriteriaThresholds = {
-    // Audio and Video threshold defaults are dynamic based
-    // on the duration of the file itself.
-    [ContentKindsNames.DOCUMENT]: '100%',
-    [ContentKindsNames.HTML5]: 300,
-    // We cannot set an automatic default threshold for exercises.
-  };
-
-  const completionCriteriaToDropdownMap = {
-    [CompletionCriteriaModels.TIME]: CompletionDropdownMap.completeDuration,
-    [CompletionCriteriaModels.APPROX_TIME]: CompletionDropdownMap.completeDuration,
-    [CompletionCriteriaModels.PAGES]: CompletionDropdownMap.allContent,
-    [CompletionCriteriaModels.DETERMINED_BY_RESOURCE]: CompletionDropdownMap.determinedByResource,
-    [CompletionCriteriaModels.MASTERY]: CompletionDropdownMap.goal,
-    [CompletionCriteriaModels.REFERENCE]: CompletionDropdownMap.reference,
-  };
-
-  const CompletionOptionsDropdownMap = {
-    [ContentKindsNames.DOCUMENT]: [
-      CompletionDropdownMap.allContent,
-      CompletionDropdownMap.completeDuration,
-      CompletionDropdownMap.reference,
-    ],
-    [ContentKindsNames.EXERCISE]: [CompletionDropdownMap.goal, CompletionDropdownMap.practiceQuiz],
-    [ContentKindsNames.HTML5]: [
-      CompletionDropdownMap.completeDuration,
-      CompletionDropdownMap.determinedByResource,
-      CompletionDropdownMap.reference,
-    ],
-    [ContentKindsNames.H5P]: [
-      CompletionDropdownMap.determinedByResource,
-      CompletionDropdownMap.completeDuration,
-      CompletionDropdownMap.reference,
-    ],
-    [ContentKindsNames.VIDEO]: [
-      CompletionDropdownMap.completeDuration,
-      CompletionDropdownMap.reference,
-    ],
-    [ContentKindsNames.AUDIO]: [
-      CompletionDropdownMap.completeDuration,
-      CompletionDropdownMap.reference,
-    ],
-  };
-
   export default {
     name: 'CompletionOptions',
     components: {
diff --git a/contentcuration/contentcuration/frontend/channelEdit/utils.js b/contentcuration/contentcuration/frontend/channelEdit/utils.js
index a475d4e67f..3a275a7362 100644
--- a/contentcuration/contentcuration/frontend/channelEdit/utils.js
+++ b/contentcuration/contentcuration/frontend/channelEdit/utils.js
@@ -1,13 +1,73 @@
 import translator from './translator';
 import { RouteNames } from './constants';
+import { ContentKindsNames } from 'shared/leUtils/ContentKinds';
 import { MasteryModelsNames } from 'shared/leUtils/MasteryModels';
 import { metadataStrings, constantStrings } from 'shared/mixins';
 import {
   AssessmentItemTypes,
   CompletionCriteriaModels,
   SHORT_LONG_ACTIVITY_MIDPOINT,
+  CompletionDropdownMap,
 } from 'shared/constants';
 
+// The constant mapping below is used to set
+// default completion criteria and durations
+// both as initial values in the edit modal, and
+// to ensure backwards compatibility for contentnodes
+// that were added before this was in place
+export const defaultCompletionCriteriaModels = {
+  [ContentKindsNames.VIDEO]: CompletionCriteriaModels.TIME,
+  [ContentKindsNames.AUDIO]: CompletionCriteriaModels.TIME,
+  [ContentKindsNames.DOCUMENT]: CompletionCriteriaModels.PAGES,
+  [ContentKindsNames.H5P]: CompletionCriteriaModels.DETERMINED_BY_RESOURCE,
+  [ContentKindsNames.HTML5]: CompletionCriteriaModels.APPROX_TIME,
+  [ContentKindsNames.EXERCISE]: CompletionCriteriaModels.MASTERY,
+};
+
+export const defaultCompletionCriteriaThresholds = {
+  // Audio and Video threshold defaults are dynamic based
+  // on the duration of the file itself.
+  [ContentKindsNames.DOCUMENT]: '100%',
+  [ContentKindsNames.HTML5]: 300,
+  // We cannot set an automatic default threshold for exercises.
+};
+
+export const completionCriteriaToDropdownMap = {
+  [CompletionCriteriaModels.TIME]: CompletionDropdownMap.completeDuration,
+  [CompletionCriteriaModels.APPROX_TIME]: CompletionDropdownMap.completeDuration,
+  [CompletionCriteriaModels.PAGES]: CompletionDropdownMap.allContent,
+  [CompletionCriteriaModels.DETERMINED_BY_RESOURCE]: CompletionDropdownMap.determinedByResource,
+  [CompletionCriteriaModels.MASTERY]: CompletionDropdownMap.goal,
+  [CompletionCriteriaModels.REFERENCE]: CompletionDropdownMap.reference,
+};
+
+export const CompletionOptionsDropdownMap = {
+  [ContentKindsNames.DOCUMENT]: [
+    CompletionDropdownMap.allContent,
+    CompletionDropdownMap.completeDuration,
+    CompletionDropdownMap.reference,
+  ],
+  [ContentKindsNames.EXERCISE]: [CompletionDropdownMap.goal, CompletionDropdownMap.practiceQuiz],
+  [ContentKindsNames.HTML5]: [
+    CompletionDropdownMap.completeDuration,
+    CompletionDropdownMap.determinedByResource,
+    CompletionDropdownMap.reference,
+  ],
+  [ContentKindsNames.H5P]: [
+    CompletionDropdownMap.determinedByResource,
+    CompletionDropdownMap.completeDuration,
+    CompletionDropdownMap.reference,
+  ],
+  [ContentKindsNames.VIDEO]: [
+    CompletionDropdownMap.completeDuration,
+    CompletionDropdownMap.reference,
+  ],
+  [ContentKindsNames.AUDIO]: [
+    CompletionDropdownMap.completeDuration,
+    CompletionDropdownMap.reference,
+  ],
+};
+
 /**
  * Get correct answer index/indices out of an array of answer objects.
  * @param {String} questionType single/multiple selection, true/false, input question
@@ -178,6 +238,36 @@ export function secondsToHms(seconds) {
   return hms;
 }
 
+/**
+ * Set a default completion model for audiovisual content
+ * equal to the file length in seconds (parsed), or a fallback placeholder
+ *
+ * @param {Object} node
+ * @returns {String|Number} {from options in the map above}
+ */
+export function getAudioVideoDefaultDuration(files) {
+  return files && files[0] ? secondsToHms(files[0].duration) : '-';
+}
+
+export function generateDefaultThreshold(node) {
+  if (node.kind !== ContentKindsNames.AUDIO || node.kind !== ContentKindsNames.VIDEO) {
+    return defaultCompletionCriteriaThresholds[node.kind];
+  }
+}
+
+/**
+ * Set a default completion model for non-audiovisual content
+ * the default completion for audio video is the duration of the file length
+ *
+ * @param {Object} node
+ * @returns {String|Number} {from options in the map above}
+ */
+export function generateDefaultModel(node) {
+  if (node.kind !== ContentKindsNames.AUDIO || node.kind !== ContentKindsNames.VIDEO) {
+    return defaultCompletionCriteriaThresholds[node.kind];
+  }
+}
+
 /**
  * Gathers data from a content node related to its completion.
  *
@@ -190,15 +280,18 @@ export function getCompletionDataFromNode(node) {
     return;
   }
 
-  const { model, threshold } = node.extra_fields?.options?.completion_criteria || {};
-  const masteryModel = threshold?.mastery_model;
+  const completionCriteria = node.extra_fields?.options?.completion_criteria
+    ? JSON.parse(JSON.stringify(node.extra_fields?.options?.completion_criteria))
+    : null;
+  const threshold = completionCriteria?.threshold || generateDefaultThreshold(node);
+  const model = completionCriteria?.model || defaultCompletionCriteriaModels[node.kind];
   const suggestedDurationType = node.extra_fields?.suggested_duration_type;
   const suggestedDuration = node.suggested_duration;
 
   return {
     completionModel: model,
     completionThreshold: threshold,
-    masteryModel,
+    masteryModel: threshold?.mastery_model,
     suggestedDurationType,
     suggestedDuration,
   };
@@ -252,6 +345,8 @@ export function getCompletionCriteriaLabels(node) {
       labels.completion = metadataStrings.$tr('completeDuration');
       if (suggestedDuration) {
         labels.duration = secondsToHms(suggestedDuration);
+      } else {
+        getAudioVideoDefaultDuration(node);
       }
       break;
 

From b6e295fe4c6bad28f66b9f6114ca20cb8990f894 Mon Sep 17 00:00:00 2001
From: Marcella Maki <marcellamaki@gmail.com>
Date: Mon, 3 Jul 2023 16:41:05 -0400
Subject: [PATCH 2/3] Add tests, PR feedback

---
 .../channelEdit/__tests__/utils.spec.js       | 71 +++++++++++++++++++
 .../channelEdit/components/ResourcePanel.vue  | 17 +----
 .../frontend/channelEdit/utils.js             | 34 +++++----
 3 files changed, 89 insertions(+), 33 deletions(-)

diff --git a/contentcuration/contentcuration/frontend/channelEdit/__tests__/utils.spec.js b/contentcuration/contentcuration/frontend/channelEdit/__tests__/utils.spec.js
index 1ea5215eb8..ac4ef7539e 100644
--- a/contentcuration/contentcuration/frontend/channelEdit/__tests__/utils.spec.js
+++ b/contentcuration/contentcuration/frontend/channelEdit/__tests__/utils.spec.js
@@ -8,6 +8,7 @@ import {
   importedChannelLink,
   secondsToHms,
   getCompletionCriteriaLabels,
+  getCompletionDataFromNode,
 } from '../utils';
 import router from '../router';
 import { RouteNames } from '../constants';
@@ -475,6 +476,76 @@ describe('channelEdit utils', () => {
   });
 
   describe(`getCompletionCriteriaLabels`, () => {
+    describe(`setting default values for completion and duration`, () => {
+      describe(`for audio and video content`, () => {
+        it(`returns 'When time spent is equal to duration' completion label and  duration label equal to the file length in hh:mm:ss format`, () => {
+          expect(
+            getCompletionCriteriaLabels(
+              {
+                extra_fields: {
+                  options: {},
+                },
+                kind: 'audio',
+              },
+              [{ duration: 100 }]
+            )
+          ).toEqual({
+            completion: 'When time spent is equal to duration',
+            duration: '01:40',
+          });
+        });
+        it(`returns 'When time spent is equal to duration' completion label and  duration label equal to the file length in hh:mm:ss format`, () => {
+          expect(
+            getCompletionCriteriaLabels(
+              {
+                extra_fields: {
+                  options: {},
+                },
+                kind: 'video',
+              },
+              [{ duration: 100 }]
+            )
+          ).toEqual({
+            completion: 'When time spent is equal to duration',
+            duration: '01:40',
+          });
+        });
+      });
+      describe(`for documents`, () => {
+        it(`returns 'Viewed in its entirety' completion label and empty duration label`, () => {
+          expect(
+            getCompletionCriteriaLabels(
+              {
+                extra_fields: {
+                  options: {},
+                },
+                kind: 'document',
+              },
+              []
+            )
+          ).toEqual({
+            completion: 'Viewed in its entirety',
+            duration: '-',
+          });
+        });
+      });
+      describe(`for exercises`, () => {
+        it(`sets the Completion Criteria model to 'mastery'`, () => {
+          expect(
+            getCompletionDataFromNode(
+              {
+                extra_fields: {
+                  options: {},
+                },
+                kind: 'exercise',
+              },
+              []
+            ).completionModel
+          ).toEqual('mastery');
+        });
+      });
+    });
+
     describe(`for 'reference' completion criteria`, () => {
       it(`returns 'Reference material' completion label and empty duration label`, () => {
         expect(
diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue b/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue
index 5e249c1556..bd62bea98b 100644
--- a/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue
+++ b/contentcuration/contentcuration/frontend/channelEdit/components/ResourcePanel.vue
@@ -532,12 +532,7 @@
   import sortBy from 'lodash/sortBy';
   import { mapActions, mapGetters } from 'vuex';
   import camelCase from 'lodash/camelCase';
-  import {
-    isImportedContent,
-    importedChannelLink,
-    getCompletionCriteriaLabels,
-    getAudioVideoDefaultDuration,
-  } from '../utils';
+  import { isImportedContent, importedChannelLink, getCompletionCriteriaLabels } from '../utils';
   import FilePreview from '../views/files/FilePreview';
   import { ContentLevels, Categories, AccessibilityCategories } from '../../shared/constants';
   import AssessmentItemPreview from './AssessmentItemPreview/AssessmentItemPreview';
@@ -619,10 +614,7 @@
         return getCompletionCriteriaLabels(this.node).completion;
       },
       duration() {
-        if (this.isAudioVideo) {
-          return getAudioVideoDefaultDuration(this.files);
-        }
-        return getCompletionCriteriaLabels(this.node).duration;
+        return getCompletionCriteriaLabels(this.node, this.files).duration;
       },
       files() {
         return sortBy(this.getContentNodeFiles(this.nodeId), f => f.preset.order);
@@ -673,11 +665,6 @@
       isResource() {
         return !this.isTopic && !this.isExercise;
       },
-      isAudioVideo() {
-        return (
-          this.node.kind === ContentKindsNames.AUDIO || this.node.kind === ContentKindsNames.VIDEO
-        );
-      },
       isImported() {
         return isImportedContent(this.node);
       },
diff --git a/contentcuration/contentcuration/frontend/channelEdit/utils.js b/contentcuration/contentcuration/frontend/channelEdit/utils.js
index 3a275a7362..30ad2196b5 100644
--- a/contentcuration/contentcuration/frontend/channelEdit/utils.js
+++ b/contentcuration/contentcuration/frontend/channelEdit/utils.js
@@ -239,30 +239,24 @@ export function secondsToHms(seconds) {
 }
 
 /**
- * Set a default completion model for audiovisual content
+ * Set a default duration for audiovisual content
  * equal to the file length in seconds (parsed), or a fallback placeholder
  *
- * @param {Object} node
- * @returns {String|Number} {from options in the map above}
+ * @param {Array} files
+ * @returns {String|Number} {human-readable duration}
  */
 export function getAudioVideoDefaultDuration(files) {
   return files && files[0] ? secondsToHms(files[0].duration) : '-';
 }
 
-export function generateDefaultThreshold(node) {
-  if (node.kind !== ContentKindsNames.AUDIO || node.kind !== ContentKindsNames.VIDEO) {
-    return defaultCompletionCriteriaThresholds[node.kind];
-  }
-}
-
 /**
- * Set a default completion model for non-audiovisual content
- * the default completion for audio video is the duration of the file length
+ * Set a default completion threshold for audiovisual content
+ * equal to the file's duration
  *
  * @param {Object} node
- * @returns {String|Number} {from options in the map above}
+ * @returns {String|Number} {human-readable threshold}
  */
-export function generateDefaultModel(node) {
+export function generateDefaultThreshold(node) {
   if (node.kind !== ContentKindsNames.AUDIO || node.kind !== ContentKindsNames.VIDEO) {
     return defaultCompletionCriteriaThresholds[node.kind];
   }
@@ -319,8 +313,10 @@ export function isLongActivity(node) {
  * @param {Object} node
  * @returns {Object} { completion, duration }
  */
-export function getCompletionCriteriaLabels(node) {
-  if (!node) {
+export function getCompletionCriteriaLabels(node = {}, files = []) {
+  console.log('labels');
+  if (!node && !files) {
+    console.log('none');
     return;
   }
 
@@ -336,6 +332,8 @@ export function getCompletionCriteriaLabels(node) {
     duration: '-',
   };
 
+  console.log('filesss', files);
+
   switch (completionModel) {
     case CompletionCriteriaModels.REFERENCE:
       labels.completion = metadataStrings.$tr('reference');
@@ -343,10 +341,10 @@ export function getCompletionCriteriaLabels(node) {
 
     case CompletionCriteriaModels.TIME:
       labels.completion = metadataStrings.$tr('completeDuration');
-      if (suggestedDuration) {
+      if (node.kind === ContentKindsNames.AUDIO || node.kind === ContentKindsNames.VIDEO) {
+        labels.duration = getAudioVideoDefaultDuration(files);
+      } else if (suggestedDuration) {
         labels.duration = secondsToHms(suggestedDuration);
-      } else {
-        getAudioVideoDefaultDuration(node);
       }
       break;
 

From a47d924feb79abd940c96d643d572cf6519d70e0 Mon Sep 17 00:00:00 2001
From: Marcella Maki <marcellamaki@gmail.com>
Date: Tue, 4 Jul 2023 08:36:35 -0400
Subject: [PATCH 3/3] remove stray logs

---
 contentcuration/contentcuration/frontend/channelEdit/utils.js | 4 ----
 1 file changed, 4 deletions(-)

diff --git a/contentcuration/contentcuration/frontend/channelEdit/utils.js b/contentcuration/contentcuration/frontend/channelEdit/utils.js
index 30ad2196b5..15ae9413a5 100644
--- a/contentcuration/contentcuration/frontend/channelEdit/utils.js
+++ b/contentcuration/contentcuration/frontend/channelEdit/utils.js
@@ -314,9 +314,7 @@ export function isLongActivity(node) {
  * @returns {Object} { completion, duration }
  */
 export function getCompletionCriteriaLabels(node = {}, files = []) {
-  console.log('labels');
   if (!node && !files) {
-    console.log('none');
     return;
   }
 
@@ -332,8 +330,6 @@ export function getCompletionCriteriaLabels(node = {}, files = []) {
     duration: '-',
   };
 
-  console.log('filesss', files);
-
   switch (completionModel) {
     case CompletionCriteriaModels.REFERENCE:
       labels.completion = metadataStrings.$tr('reference');