spec: more fragile plugins spec fixes
refs #CNVS-19842 Change-Id: Ia7748aa1ce10add55486dcd629bd6a3d4cef97c9 Reviewed-on: https://gerrit.instructure.com/52343 Tested-by: Jenkins Reviewed-by: Dan Minkevitch <dan@instructure.com> Product-Review: Dan Minkevitch <dan@instructure.com> QA-Review: James Williams <jamesw@instructure.com>
This commit is contained in:
parent
0418d4581e
commit
37f4a6a1c5
|
@ -521,8 +521,9 @@ describe "security" do
|
|||
expect(session[:become_user_id]).to be_nil
|
||||
expect(assigns['current_user'].id).to eq @admin.id
|
||||
expect(assigns['real_current_user']).to be_nil
|
||||
expect(PageView.last.user_id).to eq @admin.id
|
||||
expect(PageView.last.real_user_id).to be_nil
|
||||
pv1 = PageView.last
|
||||
expect(pv1.user_id).to eq @admin.id
|
||||
expect(pv1.real_user_id).to be_nil
|
||||
|
||||
post "/users/#{@student.id}/masquerade"
|
||||
assert_response 302
|
||||
|
@ -533,8 +534,9 @@ describe "security" do
|
|||
expect(session[:become_user_id]).to eq @student.id.to_s
|
||||
expect(assigns['current_user'].id).to eq @student.id
|
||||
expect(assigns['real_current_user'].id).to eq @admin.id
|
||||
expect(PageView.last.user_id).to eq @student.id
|
||||
expect(PageView.last.real_user_id).to eq @admin.id
|
||||
pv2 = PageView.all.detect{|pv| pv != pv1}
|
||||
expect(pv2.user_id).to eq @student.id
|
||||
expect(pv2.real_user_id).to eq @admin.id
|
||||
end
|
||||
|
||||
it "should remember the destination with an intervening auth" do
|
||||
|
|
|
@ -22,7 +22,7 @@ require 'lib/data_fixup/set_account_lti_opaque_ids.rb'
|
|||
describe 'DataFixup::SetAccountLtiOpaqueIds' do
|
||||
describe "up" do
|
||||
it "should work" do
|
||||
root_account = Account.default
|
||||
root_account = Account.create!
|
||||
original_guid = root_account.lti_guid
|
||||
expect(original_guid).not_to be_empty
|
||||
|
||||
|
|
Loading…
Reference in New Issue