Merge branch 'bvl-ldap-refresh-project-authorizations' into 'master'

Refresh LDAP-users project authorizations on group sync

Closes #2662

See merge request !2145

lib/ee/gitlab/ldap/sync/group.rb

@@ -145,7 +145,7 @@ module EE
               elsif group.last_owner?(user)
                 warn_cannot_remove_last_owner(user, group)
               else
-                group.users.delete(user)
+                group.users.destroy(user)
               end
             end
           end

spec/lib/ee/gitlab/ldap/sync/group_spec.rb

@@ -235,6 +235,15 @@ describe EE::Gitlab::LDAP::Sync::Group, lib: true do
           expect(group.members.find_by(user_id: user.id).access_level)
             .to eq(::Gitlab::Access::OWNER)
         end
+
+        it 'updates projects authorizations' do
+          project = create(:empty_project, namespace: group)
+          group.add_user(user, Gitlab::Access::MASTER)
+
+          sync_group.update_permissions
+
+          expect(project.authorized_users.find_by(id: user.id)).to be_nil
+        end
       end
 
       context 'when the user is the last owner' do