Merge pull request #1856 from openstax/video-task-steps

include content in video task steps

Author: reedstrm

app/controllers/api/v1/task_steps_controller.rb

@@ -40,6 +40,7 @@ def update
              user_options: {include_content: true} # needed so representer will consume free_response
             )
     @tasked.save
+    raise(ActiveRecord::Rollback) if render_api_errors(@tasked.errors)
 
     result = MarkTaskStepCompleted.call(task_step: @task_step, lock_task: true)
     raise(ActiveRecord::Rollback) if render_api_errors(result.errors)

app/representers/api/v1/tasks/tasked_interactive_representer.rb

@@ -36,7 +36,7 @@ class TaskedInteractiveRepresenter < TaskStepRepresenter
              readable: true,
              schema_info: {
                required: false,
-               description: "The content preview for reading tasked"
+               description: "The complete content for interactive tasked"
              },
              if: INCLUDE_CONTENT
   end

app/representers/api/v1/tasks/tasked_video_representer.rb

@@ -27,5 +27,16 @@ class TaskedVideoRepresenter < TaskStepRepresenter
                required: false,
                description: "The content preview for video tasked"
              }
+
+    property :content,
+             as: :html,
+             type: String,
+             writeable: false,
+             readable: true,
+             schema_info: {
+               required: false,
+               description: "The complete content for video tasked"
+             },
+             if: INCLUDE_CONTENT
   end
 end

spec/representers/api/v1/tasks/tasked_exercise_representer_spec.rb

@@ -9,7 +9,6 @@
       allow(step).to receive(:feedback_available?).and_return(false)
       allow(step).to receive(:labels).and_return([])
       allow(step).to receive(:related_content).and_return('RelatedContent')
-      allow(step).to receive(:spy).and_return(garbage_estimate: { valid: false })
       allow(step).to receive(:spy).and_return(response_validation: { valid: false })
     end
   end
@@ -85,9 +84,8 @@
       expect(complete_representation).to include("labels")
     end
 
-    it "has 'spy'" do
-      expect(complete_representation).to include("spy" => { garbage_estimate: { valid: false }})
-      expect(representation).to include("spy" => {"response_validation"=>{"valid"=>false}})
+    it "have 'spy'" do
+      expect(complete_representation).to include("spy" => { response_validation: { valid: false }})
     end
   end
 

spec/representers/api/v1/tasks/tasked_video_representer_spec.rb

@@ -14,4 +14,14 @@
     }.stringify_keys)
   end
 
+  it "has additional content fields" do
+    task_step = FactoryBot.create(:tasks_tasked_reading).task_step
+    json = Api::V1::Tasks::TaskedVideoRepresenter.new(task_step.tasked).to_json(
+      user_options: { include_content: true }
+    )
+    expect(JSON.parse(json)).to include({
+      html: task_step.tasked.content,
+    }.stringify_keys)
+  end
+
 end