From 3876fc950aa20f0ee4fbe542c71423248b8a2452 Mon Sep 17 00:00:00 2001 From: Rob Orton Date: Mon, 14 Aug 2017 15:38:16 -0600 Subject: [PATCH] spec: test adding cross account admins refs CNVS-37748 Change-Id: Id071caa7a1e216532fb88d9c24bfa2ca9e2e7615 Reviewed-on: https://gerrit.instructure.com/122719 Reviewed-by: Cody Cutrer QA-Review: Tucker McKnight Product-Review: Rob Orton Tested-by: Jenkins --- spec/lib/sis/csv/admin_importer_spec.rb | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/spec/lib/sis/csv/admin_importer_spec.rb b/spec/lib/sis/csv/admin_importer_spec.rb index 4266b0f9615..9745907095e 100644 --- a/spec/lib/sis/csv/admin_importer_spec.rb +++ b/spec/lib/sis/csv/admin_importer_spec.rb @@ -81,4 +81,21 @@ describe SIS::CSV::AdminImporter do expect(@sub_account.account_users.where(user_id: u1).take.workflow_state).to eq 'deleted' expect(@account.account_users.where(user_id: u1).count).to eq 1 end + + it 'should add admins from other root_account' do + account2 = Account.create! + user_with_managed_pseudonym(account: account2, sis_user_id: 'U001') + + before_count = @account.account_users.active.count + + work = SIS::AdminImporter::Work.new(nil, @account, Rails.logger, []) + expect(work).to receive(:root_account_from_id).with('account2').once.and_return(account2) + expect(SIS::AdminImporter::Work).to receive(:new).with(any_args).and_return(work) + + process_csv_data_cleanly( + 'user_id,account_id,role,status,root_account', + 'U001,,AccountAdmin,active,account2' + ) + expect(@account.account_users.active.count).to eq before_count + 1 + end end