Ember Quiz Stats - File Upload & Formula

Render question statistics for those question types similar to Essay.
File Upload should include a link to download all submissions too.

QuizSerializer was updated to include the submission ZIP download URL.

Closes CNVS-12988

TEST PLAN
---- ----

  - using your data set from https://gerrit.instructure.com/#/c/35112/
    - verify that the score chart renders with the student scores
    - verify that you get a link to Download All Files for FUpload
      questions
    - verify that the "Attempts: x out of Y" reads correctly based on
      the "responses" field and the total participant count,
      respectively
    - tooltips and chart interactivity like that for Essay

Change-Id: I4a77631491b169106e2eb677b21c1f30f3ca9440
Reviewed-on: https://gerrit.instructure.com/35113
Tested-by: Jenkins <jenkins@instructure.com>
Reviewed-by: Jason Madsen <jmadsen@instructure.com>
QA-Review: Caleb Guanzon <cguanzon@instructure.com>
Product-Review: Ahmad Amireh <ahmad@instructure.com>

app/coffeescripts/ember/quizzes/controllers/quiz/statistics/questions/calculated_controller.coffee

@@ -0,0 +1,2 @@
+define [ './essay_controller' ], (Base) ->
+  Base

app/coffeescripts/ember/quizzes/controllers/quiz/statistics/questions/file_upload_controller.coffee

@@ -0,0 +1,2 @@
+define [ './essay_controller' ], (Base) ->
+  Base

app/coffeescripts/ember/quizzes/models/question_statistics.coffee

@@ -31,7 +31,8 @@ define [
     correctMiddleStudentCount: attr()
     correctBottomStudentCount: attr()
 
-    speedGraderUrl: Em.computed.alias('quizStatistics.quiz.speedGraderUrl').readOnly()
+    speedGraderUrl: alias('quizStatistics.quiz.speedGraderUrl').readOnly()
+    quizSubmissionsZipUrl: alias('quizStatistics.quiz.quizSubmissionsZipUrl').readOnly()
 
     # Helper for calculating the ratio of correct responses for this question.
     #
@@ -61,6 +62,10 @@ define [
           'fill_in_multiple_blanks'
         when 'essay_question'
           'essay'
+        when 'file_upload_question'
+          'file_upload'
+        when 'calculated_question'
+          'calculated'
         else
           'generic'
     ).property('questionType')

app/coffeescripts/ember/quizzes/models/quiz.coffee

@@ -124,6 +124,7 @@ define [
     quizSubmissions: alias('studentQuizSubmissions')
     takeable: attr()
     takeQuizUrl: attr()
+    quizSubmissionsZipUrl: attr()
 
   Quiz.SORT_LAST = 'ZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZZ'
 

app/coffeescripts/ember/quizzes/templates/quiz/statistics/questions/calculated.hbs

@@ -0,0 +1,7 @@
+<header>
+  {{partial "quiz/statistics/questions/essay/header_contents"}}
+</header>
+
+<div>
+  {{partial "quiz/statistics/questions/essay/charts"}}
+</div>

app/coffeescripts/ember/quizzes/templates/quiz/statistics/questions/essay.hbs

@@ -5,19 +5,9 @@
     </a>
   </aside>
 
-  <span class="question-attempts">{{attemptsLabel}}</span>
+  {{partial "quiz/statistics/questions/essay/header_contents"}}
-
-  <div class="question-text">
-    {{{questionText}}}
-  </div>
 </header>
 
 <div>
-  <section class="correct-answer-ratio-section">
+  {{partial "quiz/statistics/questions/essay/charts"}}
-    {{render "quiz/statistics/questions/multiple_choice/correct_pie" controller}}
-  </section>
-
-  <section class="essay-score-chart-section">
-    {{render "quiz/statistics/questions/essay/score_chart" controller}}
-  </section>
 </div>

app/coffeescripts/ember/quizzes/templates/quiz/statistics/questions/essay/_charts.hbs

@@ -0,0 +1,7 @@
+<section class="correct-answer-ratio-section">
+  {{render "quiz/statistics/questions/multiple_choice/correct_pie" controller}}
+</section>
+
+<section class="essay-score-chart-section">
+  {{render "quiz/statistics/questions/essay/score_chart" controller}}
+</section>

app/coffeescripts/ember/quizzes/templates/quiz/statistics/questions/essay/_header_contents.hbs

@@ -0,0 +1,5 @@
+<span class="question-attempts">{{attemptsLabel}}</span>
+
+<div class="question-text">
+  {{{questionText}}}
+</div>

app/coffeescripts/ember/quizzes/templates/quiz/statistics/questions/file_upload.hbs

@@ -0,0 +1,13 @@
+<header>
+  <aside class="pull-right">
+    <a {{bind-attr href=quizSubmissionsZipUrl}} target="_blank">
+      {{t 'download_submissions' 'Download All Files'}}
+    </a>
+  </aside>
+
+  {{partial "quiz/statistics/questions/essay/header_contents"}}
+</header>
+
+<div>
+  {{partial "quiz/statistics/questions/essay/charts"}}
+</div>

app/coffeescripts/ember/quizzes/views/quiz/statistics/questions/calculated_view.coffee

@@ -0,0 +1,2 @@
+define [ '../questions_view' ], (Base) ->
+  Base

app/coffeescripts/ember/quizzes/views/quiz/statistics/questions/essay_view.coffee

@@ -1,2 +1,2 @@
-define [ '../questions_view' ], (BaseView) ->
+define [ '../questions_view' ], (Base) ->
-  BaseView
+  Base

app/coffeescripts/ember/quizzes/views/quiz/statistics/questions/file_upload_view.coffee

@@ -0,0 +1,2 @@
+define [ '../questions_view' ], (Base) ->
+  Base

app/coffeescripts/ember/quizzes/views/quiz/statistics/questions/multiple_choice_view.coffee

@@ -1,2 +1,2 @@
-define [ '../questions_view' ], (BaseView) ->
+define [ '../questions_view' ], (Base) ->
-  BaseView
+  Base

app/coffeescripts/ember/quizzes/views/quiz/statistics/questions/short_answer_view.coffee

@@ -1,2 +1,2 @@
-define [ '../questions_view' ], (BaseView) ->
+define [ '../questions_view' ], (Base) ->
-  BaseView.extend {}
+  Base

app/serializers/quizzes/quiz_serializer.rb

@@ -17,7 +17,7 @@ module Quizzes
                 :require_lockdown_browser_monitor, :lockdown_browser_monitor_data,
                 :speed_grader_url, :permissions, :quiz_reports_url, :quiz_statistics_url,
                 :message_students_url, :quiz_submission_html_url, :section_count,
-                :take_quiz_url, :takeable
+                :take_quiz_url, :takeable, :quiz_submissions_zip_url
 
     def_delegators :@controller,
       :api_v1_course_assignment_group_url,
@@ -29,7 +29,8 @@ module Quizzes
       :course_quiz_submission_html_url,
       :api_v1_course_quiz_submission_users_url,
       :api_v1_course_quiz_submission_users_message_url,
-      :course_quiz_take_url
+      :course_quiz_take_url,
+      :course_quiz_quiz_submissions_url
 
    def_delegators :@object,
      :context,
@@ -145,6 +146,8 @@ module Quizzes
         when :submitted_students, :unsubmitted_students then user_may_grade?
         when :quiz_submission then accepts_jsonapi?
         when :quiz_submission_html_url then accepts_jsonapi?
+        when :quiz_submissions_zip_url then
+          accepts_jsonapi? && user_may_grade? && has_file_uploads?
         else true
         end
       end
@@ -211,6 +214,10 @@ module Quizzes
       !!(accepts_jsonapi? || stringify_json_ids?)
     end
 
+    def quiz_submissions_zip_url
+      course_quiz_quiz_submissions_url(quiz.context, quiz.id, zip: 1)
+    end
+
     private
 
     def show_speedgrader?
@@ -260,5 +267,8 @@ module Quizzes
       !!submission_for_current_user
     end
 
+    def has_file_uploads?
+      quiz.has_file_upload_question?
+    end
   end
 end

gems/canvas_quiz_statistics/lib/canvas_quiz_statistics/analyzers/file_upload.rb

@@ -22,7 +22,7 @@ module CanvasQuizStatistics::Analyzers
     private
 
     def answer_present?(response)
-      response.has_key?(:attachment_ids) && response[:attachment_ids].any?
+      (response[:attachment_ids] || []).any?
     end
   end
 end

gems/canvas_quiz_statistics/spec/canvas_quiz_statistics/analyzers/file_upload_spec.rb

@@ -12,6 +12,7 @@ describe CanvasQuizStatistics::Analyzers::FileUpload do
     it 'should count students who have uploaded an attachment' do
       subject.run([
         {},
+        { attachment_ids: nil },
         { attachment_ids: [] },
         { attachment_ids: ['1'] }
       ])[:responses].should == 1

spec/serializers/quizzes/quiz_serializer_spec.rb

@@ -354,6 +354,35 @@ describe Quizzes::QuizSerializer do
     end
   end
 
+  describe "quiz_submissions_zip_url" do
+    it "includes a url to download all files" do
+      controller.expects(:accepts_jsonapi?).at_least_once.returns true
+      serializer.expects(:user_may_grade?).at_least_once.returns true
+      serializer.expects(:has_file_uploads?).at_least_once.returns true
+      serializer.as_json[:quiz][:quiz_submissions_zip_url].should ==
+        'http://example.com/courses/1/quizzes/1/submissions?zip=1'
+    end
+
+    it "doesn't if it's not a JSON-API request" do
+      controller.expects(:accepts_jsonapi?).at_least_once.returns false
+      serializer.expects(:user_may_grade?).at_least_once.returns true
+      serializer.as_json[:quiz].should_not have_key :quiz_submissions_zip_url
+    end
+
+    it "doesn't if the user may not grade" do
+      controller.expects(:accepts_jsonapi?).at_least_once.returns true
+      serializer.expects(:user_may_grade?).at_least_once.returns false
+      serializer.as_json[:quiz].should_not have_key :quiz_submissions_zip_url
+    end
+
+    it "doesn't if the quiz has no file upload questions" do
+      controller.expects(:accepts_jsonapi?).at_least_once.returns true
+      serializer.expects(:user_may_grade?).at_least_once.returns true
+      serializer.expects(:has_file_uploads?).at_least_once.returns false
+      serializer.as_json[:quiz].should_not have_key :quiz_submissions_zip_url
+    end
+  end
+
   describe "permissions" do
     it "serializes permissions" do
       serializer.as_json[:quiz][:permissions].should == {