change conclude to completed on sis_export report

closes CNVS-2470

test plan
- have a concluded course
- run sis export{courses, include deleted objects}
- csv should have concluded course as completed

Change-Id: I13242e4e5b9a0ab2e68d3e801a29214ec0af182e
Reviewed-on: https://gerrit.instructure.com/27538
Tested-by: Jenkins <jenkins@instructure.com>
Reviewed-by: Cody Cutrer <cody@instructure.com>
QA-Review: Rob Orton <rob@instructure.com>
Product-Review: Rob Orton <rob@instructure.com>

vendor/plugins/account_reports/lib/canvas/account_reports/sis_exporter.rb

@@ -235,7 +235,7 @@ module Canvas::AccountReports
             #for sis import format 'claimed', 'created', and 'available' are all considered active
             if @sis_format
               if c.workflow_state == 'deleted' || c.workflow_state == 'completed'
-                row << course_state_sub[c.workflow_state]
+                row << c.workflow_state
               else
                 row << 'active'
               end

vendor/plugins/account_reports/spec_canvas/sis_provisioning_reports_spec.rb

@@ -477,6 +477,7 @@ describe "Default Account Reports" do
       end
 
       it "should run the SIS report with sis term and deleted courses" do
+        @course1.complete
         parameters = {}
         parameters["enrollment_term_id"] = "sis_term_id:fall12"
         parameters["include_deleted"] = true
@@ -485,7 +486,7 @@ describe "Default Account Reports" do
 
         parsed.length.should == 2
         parsed[0].should == [@course1.sis_source_id,@course1.course_code,@course1.name,
-                             @sub_account.sis_source_id,@term1.sis_source_id,"active",
+                             @sub_account.sis_source_id,@term1.sis_source_id,"completed",
                              @course1.start_at.iso8601,@course1.conclude_at.iso8601]
         parsed[1].should == ["SIS_COURSE_ID_5","ENG101","Sd Math 100","sub1",
                              "fall12","deleted",nil,nil]