gradebook2/outcome gradebook student context cards

refs FALCOR-8
closes FALCOR-32, FALCOR-33

Test plan:
  * test gradebook student names with the student context feature flags
    on and off
  * test outcome gradebook student names with the student context
    feature flags on and off

Change-Id: I29e712a82e0eedf06d0fc90792cbaeac7c53fd6a
Reviewed-on: https://gerrit.instructure.com/96293
Reviewed-by: John Corrigan <jcorrigan@instructure.com>
QA-Review: Jeremy Putnam <jeremyp@instructure.com>
Tested-by: Jenkins
Product-Review: Cameron Matheson <cameron@instructure.com>

Author: cmatheson

app/coffeescripts/gradebook2/Gradebook.coffee

@@ -53,6 +53,7 @@ define [
   'jqueryui/sortable'
   'compiled/jquery.kylemenu'
   'compiled/jquery/fixDialogButtons'
+  'jsx/context_cards/StudentContextCardTrigger'
 ], (
   $, _, Backbone, tz, DataLoader, React, ReactDOM, LongTextEditor, KeyboardNavDialog, KeyboardNavTemplate, Slick,
   TotalColumnHeaderView, round, InputFilterView, I18n, GRADEBOOK_TRANSLATIONS, GradeCalculator, UserSettings,
@@ -340,6 +341,8 @@ define [
         I18n.t 'inactive'
 
       student.display_name = RowStudentNameTemplate
+        student_id: student.id
+        course_id: ENV.GRADEBOOK_OPTIONS.context_id
         avatar_url: student.avatar_url
         display_name: displayName
         enrollment_status: enrollmentStatus

app/coffeescripts/gradebook2/OutcomeGradebookGrid.coffee

@@ -7,6 +7,7 @@ define [
   'compiled/util/NumberCompare'
   'jst/gradebook2/outcome_gradebook_cell'
   'jst/gradebook2/outcome_gradebook_student_cell'
+  'jsx/context_cards/StudentContextCardTrigger'
 ], (I18n, $, _, HeaderFilterView, OutcomeColumnView, numberCompare, cellTemplate, studentCellTemplate) ->
 
   ###
@@ -346,7 +347,7 @@ define [
         cellTemplate(score: Math.round(value * 100.0) / 100.0, className: className, masteryScore: outcome.mastery_points)
 
       studentCell: (row, cell, value, columnDef, dataContext) ->
-        studentCellTemplate(value)
+        studentCellTemplate(_.extend value, course_id: ENV.GRADEBOOK_OPTIONS.context_id)
 
       # Public: Create a string class name for the given score.
       #

app/controllers/gradebooks_controller.rb

@@ -305,7 +305,8 @@ def set_js_env
     else
       Setting.get('gradebook2.many_submissions_chunk_size', '10').to_i
     end
-    js_env  :GRADEBOOK_OPTIONS => {
+    js_env STUDENT_CONTEXT_CARDS_ENABLED: @domain_root_account.feature_enabled?(:student_context_cards)
+    js_env :GRADEBOOK_OPTIONS => {
       :chunk_size => chunk_size,
       :assignment_groups_url => api_v1_course_assignment_groups_url(
         @context,
@@ -337,7 +338,7 @@ def set_js_env
       :context_url => named_context_url(@context, :context_url),
       :download_assignment_submissions_url => named_context_url(@context, :context_assignment_submissions_url, "{{ assignment_id }}", :zip => 1),
       :re_upload_submissions_url => named_context_url(@context, :submissions_upload_context_gradebook_url, "{{ assignment_id }}"),
-      :context_id => @context.id,
+      :context_id => @context.id.to_s,
       :context_code => @context.asset_string,
       :context_sis_id => @context.sis_source_id,
       :group_weighting_scheme => @context.group_weighting_scheme,

app/jsx/context_cards/StudentContextCardTrigger.jsx

@@ -0,0 +1,13 @@
+define(["jquery"], ($) => {
+  $(document).on(
+    "click",
+    ".student_context_card_trigger",
+    function(event) {
+      let {student_id, course_id} = $(event.target).data();
+      if (ENV.STUDENT_CONTEXT_CARDS_ENABLED && student_id && course_id) {
+        event.preventDefault();
+        alert(`Student context tray! student=${student_id} course=${course_id}`);
+      }
+    }
+  );
+});

app/views/jst/gradebook2/outcome_gradebook_student_cell.handlebars

@@ -1,5 +1,7 @@
 <div class="outcome-student-cell-content">
   {{>avatar}}
-  <a class="student-grades-list" href="{{grades_html_url}}">{{display_name}}</a>
+  <a class="student-grades-list student_context_card_trigger"
+    data-student_id={{id}} data-course_id={{course_id}}
+    href="{{grades_html_url}}">{{display_name}}</a>
   <div class="student-section">{{section_name}}</div>
 </div>

app/views/jst/gradebook2/row_student_name.handlebars

@@ -1,6 +1,8 @@
 {{>avatar}}
 <div class='student-name'>
-  <a class='student-grades-link' href="{{url}}">{{{display_name}}}</a> {{#if enrollment_status}}<span title={{#t}}"This user is currently not able to access the course"{{/t}} class="label">{{enrollment_status}}</span>{{/if}}
+  <a class='student-grades-link student_context_card_trigger'
+     data-student_id="{{student_id}}" data-course_id="{{course_id}}"
+     href="{{url}}">{{{display_name}}}</a> {{#if enrollment_status}}<span title={{#t}}"This user is currently not able to access the course"{{/t}} class="label">{{enrollment_status}}</span>{{/if}}
 </div>
 <div class="student-placeholder">
   {{#t "student_placeholder"}}Student{{/t}}

spec/controllers/gradebooks_controller_spec.rb

@@ -504,6 +504,27 @@ def check_grades_page(due_at)
       get "show", :course_id => @course.id
       assert_unauthorized
     end
+
+    context "includes student context card info in ENV" do
+      before { user_session(@teacher) }
+
+      it "includes context_id" do
+        get :show, course_id: @course.id
+        context_id = assigns[:js_env][:GRADEBOOK_OPTIONS][:context_id]
+        expect(context_id).to eq @course.id.to_param
+      end
+
+      it "doesn't enable context cards when feature is off" do
+        get :show, course_id: @course.id
+        expect(assigns[:js_env][:STUDENT_CONTEXT_CARDS_ENABLED]).to eq false
+      end
+
+      it "enables context cards when feature is on" do
+        @course.root_account.enable_feature! :student_context_cards
+        get :show, course_id: @course.id
+        expect(assigns[:js_env][:STUDENT_CONTEXT_CARDS_ENABLED]).to eq true
+      end
+    end
   end
 
   describe "GET 'change_gradebook_version'" do

spec/javascripts/jsx/gradebook2/gradebook2Spec.jsx

@@ -1,5 +1,15 @@
-define(['compiled/gradebook2/Gradebook', "underscore"], (Gradebook, _) => {
-  module("addRow");
+define(['compiled/gradebook2/Gradebook',
+  "underscore",
+  "helpers/fakeENV",
+], (Gradebook, _, fakeENV) => {
+  module("addRow", {
+    setup: function() {
+      fakeENV.setup({
+        GRADEBOOK_OPTIONS: { context_id: 1 },
+      });
+    },
+    teardown: () => fakeENV.teardown(),
+  });
 
   test("doesn't add filtered out users", () => {
     const gb = {