From 5731d4fdd5af3972f0725f8d2317fc2cdb197157 Mon Sep 17 00:00:00 2001
From: Marcella Maki <marcellamaki@gmail.com>
Date: Thu, 25 Aug 2022 16:27:17 -0400
Subject: [PATCH 1/2] Update reference selection for duration for audioVideo
 resources

---
 .../components/edit/CompletionOptions.vue            | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/edit/CompletionOptions.vue b/contentcuration/contentcuration/frontend/channelEdit/components/edit/CompletionOptions.vue
index c25dd7ce60..a137bb0eab 100644
--- a/contentcuration/contentcuration/frontend/channelEdit/components/edit/CompletionOptions.vue
+++ b/contentcuration/contentcuration/frontend/channelEdit/components/edit/CompletionOptions.vue
@@ -4,7 +4,7 @@
     <!-- Layout when practice quizzes are enabled -->
     <VLayout v-if="hideCompletionDropdown" xs6 md6>
       <!-- "Completion" dropdown menu  -->
-      <VFlex xs6 md6 class="completion-container pr-2">
+      <VFlex v-if="!audioVideoResource" xs6 md6 class="completion-container pr-2">
         <VSelect
           ref="completion"
           v-model="completionDropdown"
@@ -201,10 +201,11 @@
             return true;
           }
           return (
-            this.currentCompletionDropdown === CompletionDropdownMap.reference ||
+            (this.currentCompletionDropdown === CompletionDropdownMap.reference &&
+              !this.audioVideoResource) ||
             (this.value.model === CompletionCriteriaModels.REFERENCE &&
               !this.currentCompletionDropdown &&
-              this.audioVideoResource) ||
+              !this.audioVideoResource) ||
             //should be hidden if model is reference and we're getting this from the BE
             this.currentCompletionDropdown === CompletionDropdownMap.determinedByResource
           );
@@ -764,9 +765,8 @@
        */
       selectableDurationOptions() {
         if (
-          (this.kind === ContentKindsNames.DOCUMENT &&
-            this.currentCompletionDropdown === CompletionDropdownMap.completeDuration) ||
-          this.audioVideoResource
+          this.kind === ContentKindsNames.DOCUMENT &&
+          this.currentCompletionDropdown === CompletionDropdownMap.completeDuration
         ) {
           return this.allPossibleDurationOptions.map(model => ({
             value: model.id,

From b55eb93c8a5158cc0d3187790edde3f1c3c30d6a Mon Sep 17 00:00:00 2001
From: Marcella Maki <marcellamaki@gmail.com>
Date: Thu, 25 Aug 2022 16:41:06 -0400
Subject: [PATCH 2/2] Update test

---
 .../components/edit/__tests__/completionOptions.spec.js       | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/contentcuration/contentcuration/frontend/channelEdit/components/edit/__tests__/completionOptions.spec.js b/contentcuration/contentcuration/frontend/channelEdit/components/edit/__tests__/completionOptions.spec.js
index 6166085567..178052522d 100644
--- a/contentcuration/contentcuration/frontend/channelEdit/components/edit/__tests__/completionOptions.spec.js
+++ b/contentcuration/contentcuration/frontend/channelEdit/components/edit/__tests__/completionOptions.spec.js
@@ -202,7 +202,7 @@ describe('CompletionOptions', () => {
         });
       });
       describe(`audio/video`, () => {
-        it(`'Duration dropdown' is not visible and reference hint is visible when 'Reference' is selected`, () => {
+        it(`'Completion dropdown' is not visible and reference hint is visible when 'Reference' is selected`, () => {
           const wrapper = mount(CompletionOptions, {
             propsData: {
               kind: 'audio',
@@ -210,7 +210,7 @@ describe('CompletionOptions', () => {
             },
           });
           expect(wrapper.vm.showReferenceHint).toBe(true);
-          expect(wrapper.find({ ref: 'duration' }).exists()).toBe(false);
+          expect(wrapper.find({ ref: 'completion' }).exists()).toBe(false);
         });
       });
       describe(`exercise`, () => {