don't enroll users with deleted pseudonyms in sis imports

test plan:
* import the sis .zip package referenced in the ticket
* should have a warning
* should not create an active enrollment in the new course

closes #CNVS-19829

Change-Id: I1f1aa23da7a7a8aab09026bc974945f8f3fc5743
Reviewed-on: https://gerrit.instructure.com/52576
Reviewed-by: Jeremy Stanley <jeremy@instructure.com>
Tested-by: Jenkins
QA-Review: Jahnavi Yetukuri <jyetukuri@instructure.com>
Product-Review: James Williams  <jamesw@instructure.com>

Author: maneframe

lib/sis/enrollment_importer.rb

@@ -195,9 +195,9 @@ def process_batch
                 'TaEnrollment'
               elsif role_name =~ /\Aobserver\z/i
                 if associated_sis_user_id
-                  pseudo = root_account.pseudonyms.where(sis_user_id: associated_sis_user_id).first
-                  if pseudo
-                    associated_user_id = pseudo.user_id
+                  a_pseudo = root_account.pseudonyms.where(sis_user_id: associated_sis_user_id).first
+                  if a_pseudo
+                    associated_user_id = a_pseudo.user_id
                   else
                     @messages << "An enrollment referenced a non-existent associated user #{associated_sis_user_id}"
                     next
@@ -229,7 +229,7 @@ def process_batch
             enrollment.course_section = @section
 
             if status =~ /\Aactive/i
-              if user.workflow_state != 'deleted'
+              if user.workflow_state != 'deleted' && pseudo.workflow_state != 'deleted'
                 enrollment.workflow_state = 'active'
               else
                 enrollment.workflow_state = 'deleted'

spec/lib/sis/csv/enrollment_importer_spec.rb

@@ -751,4 +751,28 @@
     expect(observer.enrollments.first.workflow_state).to eq 'deleted'
   end
 
+  it "should not create enrollments for deleted users" do
+    process_csv_data_cleanly(
+        "course_id,short_name,long_name,account_id,term_id,status",
+        "test_1,TC 101,Test Course 101,,,active"
+    )
+    process_csv_data_cleanly(
+        "user_id,login_id,first_name,last_name,email,status",
+        "student_user,user1,User,Uno,user@example.com,active"
+    )
+    process_csv_data_cleanly(
+        "user_id,login_id,first_name,last_name,email,status",
+        "student_user,user1,User,Uno,user@example.com,deleted"
+    )
+    importer = process_csv_data(
+        "course_id,user_id,role,section_id,status,associated_user_id",
+        "test_1,student_user,student,,active,",
+    )
+    warnings = importer.warnings.map { |r| r.last }
+    expect(warnings).to eq ["Attempted enrolling of deleted user student_user in course test_1"]
+
+    student = Pseudonym.where(:sis_user_id => "student_user").first.user
+    expect(student.enrollments.count).to eq 1
+    expect(student.enrollments.first).to be_deleted
+  end
 end