debian-mirror-gitlab/spec/models/members/last_group_owner_assigner_spec.rb

# frozen_string_literal: true

require 'spec_helper'

RSpec.describe LastGroupOwnerAssigner do
  describe "#execute" do
    let_it_be(:user, reload: true) { create(:user) }
    let_it_be(:group) { create(:group) }

    let!(:group_member) { group.add_owner(user) }

    subject(:assigner) { described_class.new(group, [group_member]) }

    it "avoids extra database queries utilizing memoization", :aggregate_failures do
      control = ActiveRecord::QueryRecorder.new { assigner.execute }
      count_queries = control.occurrences_by_line_method.first[1][:occurrences].find_all { |i| i.include?('SELECT COUNT') }

      expect(control.count).to be <= 5
      expect(count_queries.count).to eq(0)
    end

    context "when there are unblocked owners" do
      context "with one unblocked owner" do
        specify do
          expect { assigner.execute }.to change(group_member, :last_owner)
                                           .from(nil).to(true)
                                           .and change(group_member, :last_blocked_owner)
                                                  .from(nil).to(false)
        end
      end

      context "with multiple unblocked owners" do
        let_it_be(:unblocked_owner_member) { create(:group_member, :owner, source: group) }

        specify do
          expect { assigner.execute }.to change(group_member, :last_owner)
                                           .from(nil).to(false)
                                           .and change(group_member, :last_blocked_owner)
                                                  .from(nil).to(false)
        end

        it "has many members passed" do
          assigner = described_class.new(group, [unblocked_owner_member, group_member])

          expect { assigner.execute }.to change(group_member, :last_owner)
                                           .from(nil).to(false)
                                           .and change(group_member, :last_blocked_owner)
                                                  .from(nil).to(false)
                                                  .and change(unblocked_owner_member, :last_owner)
                                                         .from(nil).to(false)
                                                         .and change(unblocked_owner_member, :last_blocked_owner)
                                                                .from(nil).to(false)
        end
      end

      context 'with owners from a parent' do
        context 'when top-level group' do
          context 'with group sharing' do
            let!(:subgroup) { create(:group, parent: group) }

            before do
              create(:group_group_link, :owner, shared_group: group, shared_with_group: subgroup)
              create(:group_member, :owner, group: subgroup)
            end

            specify do
              expect { assigner.execute }.to change(group_member, :last_owner)
                .from(nil).to(true)
                .and change(group_member, :last_blocked_owner)
                .from(nil).to(false)
            end
          end
        end

        context 'when subgroup' do
          let!(:subgroup) { create(:group, parent: group) }
          let!(:group_member_2) { subgroup.add_owner(user) }

          subject(:assigner) { described_class.new(subgroup, [group_member_2]) }

          specify do
            expect { assigner.execute }.to change(group_member_2, :last_owner)
              .from(nil).to(false)
              .and change(group_member_2, :last_blocked_owner)
              .from(nil).to(false)
          end
        end
      end
    end

    context "when there are blocked owners" do
      before do
        user.block!
      end

      context "with one blocked owner" do
        specify do
          expect { assigner.execute }.to change(group_member, :last_owner)
                                           .from(nil).to(false)
                                           .and change(group_member, :last_blocked_owner)
                                                  .from(nil).to(true)
        end
      end

      context "with multiple unblocked owners" do
        specify do
          create_list(:group_member, 2, :owner, source: group)

          expect { assigner.execute }.to change(group_member, :last_owner)
                                           .from(nil).to(false)
                                           .and change(group_member, :last_blocked_owner)
                                                  .from(nil).to(false)
        end
      end

      context "with multiple blocked owners" do
        specify do
          create(:group_member, :owner, :blocked, source: group)

          expect { assigner.execute }.to change(group_member, :last_owner)
                                           .from(nil).to(false)
                                           .and change(group_member, :last_blocked_owner)
                                                  .from(nil).to(false)
        end
      end

      context 'with owners from a parent' do
        context 'when top-level group' do
          context 'with group sharing' do
            let!(:subgroup) { create(:group, parent: group) }

            before do
              create(:group_group_link, :owner, shared_group: group, shared_with_group: subgroup)
              create(:group_member, :owner, group: subgroup)
            end

            specify do
              expect { assigner.execute }.to change(group_member, :last_owner)
                .from(nil).to(false)
                .and change(group_member, :last_blocked_owner)
                .from(nil).to(true)
            end
          end
        end

        context 'when subgroup' do
          let!(:subgroup) { create(:group, :nested) }

          let!(:group_member) { subgroup.add_owner(user) }

          subject(:assigner) { described_class.new(subgroup, [group_member]) }

          specify do
            expect { assigner.execute }.to change(group_member, :last_owner)
              .from(nil).to(false)
              .and change(group_member, :last_blocked_owner)
              .from(nil).to(true)
          end

          context 'with two owners' do
            before do
              create(:group_member, :owner, group: subgroup.parent)
            end

            specify do
              expect { assigner.execute }.to change(group_member, :last_owner)
                .from(nil).to(false)
                .and change(group_member, :last_blocked_owner)
                .from(nil).to(false)
            end
          end
        end
      end
    end

    context 'when there are bot members' do
      context 'with a bot owner' do
        specify do
          create(:group_member, :owner, source: group, user: create(:user, :project_bot))

          expect { assigner.execute }.to change(group_member, :last_owner)
            .from(nil).to(true)
            .and change(group_member, :last_blocked_owner)
            .from(nil).to(false)
        end
      end
    end
  end
end