Setup job for web export

closes OFFW-36

test plan:
- you should be able to start a web zip export through
  the console
- this should cause module progression for all
  viewable items
- ensure epub exports still export

Change-Id: I901d491325fba3955af4544d1c1a2b87270cffff
Reviewed-on: https://gerrit.instructure.com/96752
Reviewed-by: Cameron Sutter <csutter@instructure.com>
Tested-by: Jenkins
QA-Review: Nathan Rogowski <nathan@instructure.com>
Product-Review: Jon Willesen <jonw+gerrit@instructure.com>

Author: Ardena

app/controllers/courses_controller.rb

@@ -300,7 +300,7 @@ class CoursesController < ApplicationController
   include CustomSidebarLinksHelper
   include SyllabusHelper
 
-  before_filter :require_user, :only => [:index, :activity_stream, :activity_stream_summary, :effective_due_dates]
+  before_filter :require_user, :only => [:index, :activity_stream, :activity_stream_summary, :effective_due_dates, :offline_web_exports]
   before_filter :require_user_or_observer, :only=>[:user_index]
   before_filter :require_context, :only => [:roster, :locks, :create_file, :ping, :effective_due_dates, :offline_web_exports]
   skip_after_filter :update_enrollment_last_activity_at, only: [:enrollment_invitation, :activity_stream_summary]
@@ -2734,8 +2734,10 @@ def can_change_group_weighting_scheme?
 
   def offline_web_exports
     return render status: 404, template: 'shared/errors/404_message' unless @context.allow_web_export_download?
-    mvmp = MustViewModuleProgressor.new(@current_user, @context)
-    mvmp.make_progress
+    if authorized_action(WebZipExport.new(course: @context), @current_user, :create)
+      @service = EpubExports::CreateService.new(@context, @current_user, :web_zip_export)
+      @service.save
+    end
     @page_title = t('Course Content Downloads')
     render :text => 'Downloads'.html_safe, :layout => true
   end

app/controllers/epub_exports_controller.rb

@@ -146,11 +146,11 @@ def index
   def create
     if authorized_action(EpubExport.new(course: @context), @current_user, :create)
       @course = Course.find(params[:course_id])
-      @service = EpubExports::CreateService.new(@course, @current_user)
+      @service = EpubExports::CreateService.new(@course, @current_user, :epub_export)
       status = @service.save ? 201 : 422
       respond_to do |format|
         format.json do
-          @course.latest_epub_export = @service.epub_export
+          @course.latest_epub_export = @service.offline_export
           render({
             status: status, json: course_epub_export_json(@course)
           })

app/models/course.rb

@@ -219,8 +219,9 @@ def inherited_assessment_question_banks(include_self = false)
   has_many :role_overrides, :as => :context
   has_many :content_migrations, :as => :context
   has_many :content_exports, :as => :context
-  has_many :epub_exports, -> { order("created_at DESC") }
+  has_many :epub_exports, -> { where("type IS NULL").order("created_at DESC") }
   attr_accessor :latest_epub_export
+  has_many :web_zip_exports, -> { where(type: "WebZipExport") }
   has_many :alerts, -> { preload(:criteria) }, as: :context
   has_many :appointment_group_contexts, :as => :context
   has_many :appointment_groups, :through => :appointment_group_contexts

app/models/epub_export.rb

@@ -14,6 +14,8 @@ class EpubExport < ActiveRecord::Base
   has_one :job_progress, as: :context, class_name: 'Progress'
   validates :course_id, :workflow_state, presence: true
 
+  attr_accessible :course, :user, :content_export
+
   PERCENTAGE_COMPLETE = {
     created: 0,
     exported: 80,

app/models/epub_exports/create_service.rb

@@ -17,30 +17,31 @@
 #
 module EpubExports
   class CreateService
-    def initialize(course, user)
+    def initialize(course, user, export_type)
       @course = course
       @user = user
+      @export_type = export_type
     end
-    attr_reader :course, :user
+    attr_reader :course, :user, :export_type
 
-    def epub_export
-      unless @_epub_export
-        @_epub_export = course.epub_exports.visible_to(user).running.first
-        @_epub_export ||= course.epub_exports.build({
+    def offline_export
+      unless @_offline_export
+        @_offline_export = course.send(export_type.to_s.pluralize).visible_to(user).running.first
+        @_offline_export ||= course.send(export_type.to_s.pluralize).build({
           user: user
         })
       end
-      @_epub_export
+      @_offline_export
     end
 
     def already_running?
-      !epub_export.new_record?
+      !offline_export.new_record?
     end
 
     def save
-      if !already_running? && epub_export.save
+      if !already_running? && offline_export.save
         # Queuing jobs always returns nil, yay
-        epub_export.export
+        offline_export.export
         true
       else
         false

app/models/web_zip_export.rb

@@ -0,0 +1,32 @@
+class WebZipExport < EpubExport
+
+  def export
+    MustViewModuleProgressor.new(user, course).make_progress
+    super
+  end
+
+  def generate
+    job_progress.update_attribute(:completion, PERCENTAGE_COMPLETE[:generating])
+    update_attribute(:workflow_state, 'generating')
+    convert_to_offline_web_zip
+  end
+  handle_asynchronously :generate, priority: Delayed::LOW_PRIORITY, max_attempts: 1
+
+  def convert_to_offline_web_zip
+    begin
+      set_locale
+      # TODO: connect to cameron's patchset
+      # file_path = super
+      file_path = "/Users/mysti/canvas/chaos.mov.zip"
+      I18n.locale = :en
+    rescue => e
+      mark_as_failed
+      raise e
+    end
+
+    create_attachment_from_path!(file_path)
+    mark_as_generated
+    cleanup_file_path!(file_path)
+  end
+  handle_asynchronously :convert_to_offline_web_zip, priority: Delayed::LOW_PRIORITY, max_attempts: 1
+end

db/migrate/20161206230329_add_type_to_epub_exports.rb

@@ -0,0 +1,7 @@
+class AddTypeToEpubExports < ActiveRecord::Migration
+  tag :predeploy
+
+  def change
+    add_column :epub_exports, :type, :string, :limit => 255
+  end
+end

spec/models/epub_exports/create_service_spec.rb

@@ -26,34 +26,34 @@
 
   describe "#save" do
     let_once(:create_service) do
-      EpubExports::CreateService.new(@course, @student)
+      EpubExports::CreateService.new(@course, @student, :epub_export)
     end
 
     it "should send save & export to epub_export" do
-      expect(create_service.epub_export.new_record?).to be_truthy, 'precondition'
-      create_service.epub_export.expects(:export).once.returns(nil)
+      expect(create_service.offline_export.new_record?).to be_truthy, 'precondition'
+      create_service.offline_export.expects(:export).once.returns(nil)
       expect(create_service.save).to be_truthy
-      expect(create_service.epub_export.new_record?).to be_falsey
+      expect(create_service.offline_export.new_record?).to be_falsey
     end
   end
 
-  describe "#epub_export" do
+  describe "#offline_export" do
     context "when user has an active epub_export" do
       before(:once) do
         @epub_export = @course.epub_exports.create(user: @student)
         @epub_export.export_without_send_later
-        @service = EpubExports::CreateService.new(@course, @student)
+        @service = EpubExports::CreateService.new(@course, @student, :epub_export)
       end
 
       it "should return said epub_export" do
-        expect(@service.epub_export).to eq @epub_export
+        expect(@service.offline_export).to eq @epub_export
       end
     end
 
     context "when user has no active epub_exports" do
       it "should return a new epub_export instance" do
-        service = EpubExports::CreateService.new(@course, @student)
-        expect(service.epub_export).to be_new_record
+        service = EpubExports::CreateService.new(@course, @student, :epub_export)
+        expect(service.offline_export).to be_new_record
       end
     end
   end
@@ -62,7 +62,7 @@
     context "when user has an active epub_export" do
       before(:once) do
         @course.epub_exports.create(user: @student).export_without_send_later
-        @service = EpubExports::CreateService.new(@course, @student)
+        @service = EpubExports::CreateService.new(@course, @student, :epub_export)
       end
 
       it "should return true" do
@@ -72,9 +72,21 @@
 
     context "when user doesn't have an active epub_export" do
       it "should return true" do
-        service = EpubExports::CreateService.new(@course, @student)
+        service = EpubExports::CreateService.new(@course, @student, :epub_export)
         expect(service.already_running?).to be_falsey
       end
     end
+
+    context "when user has an active epub_export and starts a web_zip_export" do
+      before(:once) do
+        @epub_export = @course.epub_exports.create(user: @student)
+        @epub_export.export_without_send_later
+        @service = EpubExports::CreateService.new(@course, @student, :web_zip_export)
+      end
+
+      it "should return false" do
+        expect(@service.already_running?).to be_falsey
+      end
+    end
   end
 end

spec/models/web_zip_export_spec.rb

@@ -0,0 +1,28 @@
+require 'spec_helper'
+
+describe WebZipExport do
+  before :once do
+    course_with_teacher(active_all: true)
+    student_in_course(active_all: true)
+  end
+
+  describe "#generate" do
+    let_once(:web_zip_export) do
+      @course.web_zip_exports.create({
+        user: @student
+      }).tap do |web_zip_export|
+        web_zip_export.update_attribute(:workflow_state, 'exported')
+      end
+    end
+
+    it "should update job_progress completion" do
+      web_zip_export.generate_without_send_later
+      expect(web_zip_export.job_progress.completion).to eq WebZipExport::PERCENTAGE_COMPLETE[:generating]
+    end
+
+    it "should set state to generating" do
+      web_zip_export.generate_without_send_later
+      expect(web_zip_export.generating?).to be_truthy
+    end
+  end
+end