Add Blueprint Course course setting

Closes MC-40

Test Plan:
* Go to any course's settings page with the
  Master Course's feature flag disabled
* There shouldn't be a check box to mark the course
  as a Blueprint Course
* Enable the course's account's Master Courses feature
  flag
* Go back to the course's settings page
* There should now be a checkbox for "Blueprint Courses" for
  users with `manage courses` permissions
* If the user doesn't have the `manage courses` permission, then
  there should just be a label in place of the check
  box reading `Yes` or `No` depending on whether or not the setting
  is enabled.

Change-Id: I3de1565acb10b8e4438f7a48e231d6b58aa0be55
Reviewed-on: https://gerrit.instructure.com/99283
Reviewed-by: James Williams  <jamesw@instructure.com>
Tested-by: Jenkins
QA-Review: Heath Hales <hhales@instructure.com>
Product-Review: Matt Goodwin <mattg@instructure.com>

Author: Dan Minkevitch

app/controllers/courses_controller.rb

@@ -2267,6 +2267,12 @@ def update
       params_for_update[:conclude_at] = params[:course].delete(:end_at) if api_request? && params[:course].has_key?(:end_at)
       @default_wiki_editing_roles_was = @course.default_wiki_editing_roles
 
+      if params[:course].has_key?(:master_course)
+        master_course = value_to_boolean(params[:course].delete(:master_course))
+        action = master_course ? "set" : "remove"
+        MasterCourses::MasterTemplate.send("#{action}_as_master_course", @course)
+      end
+
       @course.attributes = params_for_update
 
       if params[:course][:course_visibility].present?

app/models/course.rb

@@ -28,7 +28,7 @@ class Course < ActiveRecord::Base
   include ContentLicenses
   include TurnitinID
 
-  attr_accessor :teacher_names
+  attr_accessor :teacher_names, :master_course
   attr_writer :student_count, :primary_enrollment_type, :primary_enrollment_role_id, :primary_enrollment_rank, :primary_enrollment_state, :primary_enrollment_date, :invitation
 
   time_zone_attribute :time_zone

app/models/master_courses/master_template.rb

@@ -49,7 +49,16 @@ def self.is_master_course?(course_id)
 
   def self.set_as_master_course(course)
     self.unique_constraint_retry do
-      course.master_course_templates.active.for_full_course.first_or_create
+      template = course.master_course_templates.for_full_course.first_or_create
+      template.undestroy unless template.active?
+      template
+    end
+  end
+
+  def self.remove_as_master_course(course)
+    self.unique_constraint_retry do
+      template = course.master_course_templates.active.for_full_course.first
+      template.destroy && template if template.present?
     end
   end
 

app/views/courses/settings.html.erb

@@ -8,9 +8,12 @@
 
   can_manage_courses = can_do(@context.account, @current_user, :manage_courses)
   can_rename_course = can_manage_courses || (can_manage && !@context.root_account.settings[:prevent_course_renaming_by_teachers])
+  can_manage_master_courses = can_manage_courses && can_do(@context.account, @current_user, :manage_master_courses)
   has_multiple_sections = @context.course_sections.active.count > 1
 
   can_see_users = can_do(@context, @current_user, :read_roster, :manage_students, :manage_admin_users)
+
+  is_master_course = MasterCourses::MasterTemplate.is_master_course?(@context)
 %>
 
 <% content_for :right_side do %>
@@ -97,6 +100,19 @@
           <% end %>
         </td>
       </tr>
+      <% if @context.root_account.feature_enabled?(:master_courses) %>
+      <tr>
+        <td class="form-label nobr"><%= f.blabel :blueprint_course, :en => "Blueprint Course" %></td>
+        <td class="nobr">
+          <% if can_manage_master_courses %>
+            <%= f.check_box :master_course, :checked => is_master_course %>
+            <%= f.label :master_course, :en => "Enable course as a Blueprint Course" %>
+          <% else %>
+            <span id="course_master_course"><%= is_master_course ? t("Yes") : t("No") %></span>
+          <% end %>
+        </td>
+      </tr>
+      <% end %>
       <tr>
         <td><%= f.blabel :time_zone, :en => "Time Zone" %></td>
         <td>

spec/models/master_courses/master_template_spec.rb

@@ -15,18 +15,40 @@
       expect(MasterCourses::MasterTemplate.full_template_for(@course)).to eq template
     end
 
-    it "should ignore deleted templates" do
+    it "should restore deleted templates" do
       template = MasterCourses::MasterTemplate.set_as_master_course(@course)
       template.destroy!
 
       expect(template).to be_deleted
 
       template2 = MasterCourses::MasterTemplate.set_as_master_course(@course)
-      expect(template2).to_not eq template
+      expect(template2).to eq template
       expect(template2).to be_active
     end
   end
 
+  describe "remove_as_master_course" do
+    it "should remove a template from a course" do
+      template = MasterCourses::MasterTemplate.set_as_master_course(@course)
+      expect(template.workflow_state).to eq "active"
+      expect(template.active?).to eq true
+
+      expect{MasterCourses::MasterTemplate.remove_as_master_course(@course)}.to change{template.reload.workflow_state}.from("active").to("deleted")
+      expect(MasterCourses::MasterTemplate.full_template_for(@course)).to be_nil
+    end
+
+    it "should ignore deleted templates" do
+      template = MasterCourses::MasterTemplate.set_as_master_course(@course)
+      template.destroy
+      expect(template).to be_deleted
+
+      MasterCourses::MasterTemplate.remove_as_master_course(@course)
+
+      template.expects(:destroy).never
+      expect(template).to be_deleted
+    end
+  end
+
   describe "is_master_course?" do
     def check
       MasterCourses::MasterTemplate.is_master_course?(@course)