Commit 4b5a2f75 authored by Małgorzata Ksionek's avatar Małgorzata Ksionek

Fix naming in specs

parent 55c74619
......@@ -115,7 +115,7 @@ RSpec.describe Groups::OmniauthCallbacksController do
end
end
context 'when used to be a member of a group' do
context 'when user used to be a member of a group' do
before do
user.provisioned_by_group = group
user.save!
......@@ -148,11 +148,11 @@ RSpec.describe Groups::OmniauthCallbacksController do
expect(flash[:notice]).to eq('Login to a GitLab account to link with your SAML identity')
end
it 'adds linked identity' do
it 'does not add linked identity' do
expect { post provider, params: { group_id: group } }.not_to change(linked_accounts, :count)
end
it 'adds group membership' do
it 'does not add group membership' do
expect { post provider, params: { group_id: group } }.not_to change { group.members.count }
end
end
......
......@@ -15,7 +15,7 @@ RSpec.describe 'Groups > Members > Leave group' do
end
context 'with block_password_auth_for_saml_users feature flag switched on' do
it 'guest provisoned by this group leaves the group' do
it 'guest provisoned by this group leaves the group and is signed off' do
group.add_guest(user)
group.add_owner(other_user)
......@@ -26,7 +26,7 @@ RSpec.describe 'Groups > Members > Leave group' do
expect(current_path).to eq(new_user_session_path)
end
it 'guest leaves the group by url param', :js do
it 'guest leaves the group by url param and is signed off', :js do
group.add_guest(user)
group.add_owner(other_user)
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment