Commit 5a8eddc7 authored by James Lopez's avatar James Lopez

update spec

parent 337f5dcf
...@@ -99,7 +99,7 @@ module EE ...@@ -99,7 +99,7 @@ module EE
def dns_for_group_cn(group_cn) def dns_for_group_cn(group_cn)
if config.group_base.blank? if config.group_base.blank?
logger.debug { "No `group_base` configured for '#{provider}' provider and group link CN #{group_link.cn}. Skipping" } logger.debug { "No `group_base` configured for '#{provider}' provider and group link CN #{group_cn}. Skipping" }
return nil return nil
end end
......
...@@ -419,7 +419,7 @@ describe EE::Gitlab::LDAP::Sync::Group do ...@@ -419,7 +419,7 @@ describe EE::Gitlab::LDAP::Sync::Group do
it 'adds new members and sets ldap attribute to true' do it 'adds new members and sets ldap attribute to true' do
sync_group.update_permissions sync_group.update_permissions
expect(group.members.pluck(:user_id)).to include(user.id) expect(group.users).to include(user)
expect(group.members.find_by(user_id: user.id).ldap?).to be_truthy expect(group.members.find_by(user_id: user.id).ldap?).to be_truthy
end end
...@@ -428,7 +428,7 @@ describe EE::Gitlab::LDAP::Sync::Group do ...@@ -428,7 +428,7 @@ describe EE::Gitlab::LDAP::Sync::Group do
sync_group.update_permissions sync_group.update_permissions
expect(group.members.pluck(:user_id)).to include(user.id) expect(group.users).to include(user)
end end
end end
end end
......
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