2019-12-21 20:55:43 +05:30
|
|
|
# frozen_string_literal: true
|
|
|
|
|
2018-03-27 19:54:05 +05:30
|
|
|
require 'spec_helper'
|
|
|
|
|
2020-06-23 00:09:42 +05:30
|
|
|
RSpec.describe UserRecentEventsFinder do
|
2020-07-28 23:09:34 +05:30
|
|
|
let_it_be(:project_owner, reload: true) { create(:user) }
|
|
|
|
let_it_be(:current_user, reload: true) { create(:user) }
|
2021-03-11 19:13:27 +05:30
|
|
|
let_it_be(:private_project) { create(:project, :private, creator: project_owner) }
|
|
|
|
let_it_be(:internal_project) { create(:project, :internal, creator: project_owner) }
|
|
|
|
let_it_be(:public_project) { create(:project, :public, creator: project_owner) }
|
2018-06-27 16:04:02 +05:30
|
|
|
let!(:private_event) { create(:event, project: private_project, author: project_owner) }
|
|
|
|
let!(:internal_event) { create(:event, project: internal_project, author: project_owner) }
|
|
|
|
let!(:public_event) { create(:event, project: public_project, author: project_owner) }
|
2021-03-11 19:13:27 +05:30
|
|
|
let_it_be(:issue) { create(:issue, project: public_project) }
|
2021-09-30 23:02:18 +05:30
|
|
|
|
2020-09-03 11:15:55 +05:30
|
|
|
let(:limit) { nil }
|
|
|
|
let(:params) { { limit: limit } }
|
2018-03-27 19:54:05 +05:30
|
|
|
|
2021-03-11 19:13:27 +05:30
|
|
|
subject(:finder) { described_class.new(current_user, project_owner, nil, params) }
|
2018-03-27 19:54:05 +05:30
|
|
|
|
|
|
|
describe '#execute' do
|
2018-11-20 20:47:30 +05:30
|
|
|
context 'when profile is public' do
|
|
|
|
it 'returns all the events' do
|
|
|
|
expect(finder.execute).to include(private_event, internal_event, public_event)
|
2018-06-27 16:04:02 +05:30
|
|
|
end
|
2018-03-27 19:54:05 +05:30
|
|
|
end
|
|
|
|
|
2018-11-20 20:47:30 +05:30
|
|
|
context 'when profile is private' do
|
|
|
|
it 'returns no event' do
|
|
|
|
allow(Ability).to receive(:allowed?).and_call_original
|
|
|
|
allow(Ability).to receive(:allowed?).with(current_user, :read_user_profile, project_owner).and_return(false)
|
2018-03-27 19:54:05 +05:30
|
|
|
|
|
|
|
expect(finder.execute).to be_empty
|
|
|
|
end
|
|
|
|
end
|
2018-06-27 16:04:02 +05:30
|
|
|
|
2018-11-20 20:47:30 +05:30
|
|
|
it 'does not include the events if the user cannot read cross project' do
|
2019-02-15 15:39:39 +05:30
|
|
|
allow(Ability).to receive(:allowed?).and_call_original
|
2018-11-20 20:47:30 +05:30
|
|
|
expect(Ability).to receive(:allowed?).with(current_user, :read_cross_project) { false }
|
2019-02-15 15:39:39 +05:30
|
|
|
|
2018-11-20 20:47:30 +05:30
|
|
|
expect(finder.execute).to be_empty
|
2018-06-27 16:04:02 +05:30
|
|
|
end
|
2020-07-28 23:09:34 +05:30
|
|
|
|
2021-03-11 19:13:27 +05:30
|
|
|
context 'events from multiple users' do
|
|
|
|
let_it_be(:second_user, reload: true) { create(:user) }
|
|
|
|
let_it_be(:private_project_second_user) { create(:project, :private, creator: second_user) }
|
2021-09-30 23:02:18 +05:30
|
|
|
|
2021-03-11 19:13:27 +05:30
|
|
|
let(:internal_project_second_user) { create(:project, :internal, creator: second_user) }
|
|
|
|
let(:public_project_second_user) { create(:project, :public, creator: second_user) }
|
|
|
|
let!(:private_event_second_user) { create(:event, project: private_project_second_user, author: second_user) }
|
|
|
|
let!(:internal_event_second_user) { create(:event, project: internal_project_second_user, author: second_user) }
|
|
|
|
let!(:public_event_second_user) { create(:event, project: public_project_second_user, author: second_user) }
|
|
|
|
|
|
|
|
it 'includes events from all users', :aggregate_failures do
|
|
|
|
events = described_class.new(current_user, [project_owner, second_user], nil, params).execute
|
|
|
|
|
|
|
|
expect(events).to include(private_event, internal_event, public_event)
|
|
|
|
expect(events).to include(private_event_second_user, internal_event_second_user, public_event_second_user)
|
|
|
|
expect(events.size).to eq(6)
|
|
|
|
end
|
|
|
|
|
2022-03-02 08:16:31 +05:30
|
|
|
context 'selected events' do
|
|
|
|
let!(:push_event) { create(:push_event, project: public_project, author: project_owner) }
|
|
|
|
let!(:push_event_second_user) { create(:push_event, project: public_project_second_user, author: second_user) }
|
|
|
|
|
|
|
|
it 'only includes selected events (PUSH) from all users', :aggregate_failures do
|
|
|
|
event_filter = EventFilter.new(EventFilter::PUSH)
|
|
|
|
events = described_class.new(current_user, [project_owner, second_user], event_filter, params).execute
|
|
|
|
|
|
|
|
expect(events).to contain_exactly(push_event, push_event_second_user)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2021-03-11 19:13:27 +05:30
|
|
|
it 'does not include events from users with private profile', :aggregate_failures do
|
|
|
|
allow(Ability).to receive(:allowed?).and_call_original
|
|
|
|
allow(Ability).to receive(:allowed?).with(current_user, :read_user_profile, second_user).and_return(false)
|
|
|
|
|
|
|
|
events = described_class.new(current_user, [project_owner, second_user], nil, params).execute
|
|
|
|
|
2022-03-02 08:16:31 +05:30
|
|
|
expect(events).to contain_exactly(private_event, internal_event, public_event)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with pagination params' do
|
|
|
|
using RSpec::Parameterized::TableSyntax
|
|
|
|
|
|
|
|
where(:limit, :offset, :ordered_expected_events) do
|
|
|
|
nil | nil | lazy { [public_event_second_user, internal_event_second_user, private_event_second_user, public_event, internal_event, private_event] }
|
|
|
|
2 | nil | lazy { [public_event_second_user, internal_event_second_user] }
|
|
|
|
nil | 4 | lazy { [internal_event, private_event] }
|
|
|
|
2 | 2 | lazy { [private_event_second_user, public_event] }
|
|
|
|
end
|
|
|
|
|
|
|
|
with_them do
|
|
|
|
let(:params) { { limit: limit, offset: offset }.compact }
|
|
|
|
|
|
|
|
it 'returns paginated events sorted by id (DESC)' do
|
|
|
|
events = described_class.new(current_user, [project_owner, second_user], nil, params).execute
|
|
|
|
|
|
|
|
expect(events).to eq(ordered_expected_events)
|
|
|
|
end
|
|
|
|
end
|
2021-03-11 19:13:27 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'filter activity events' do
|
|
|
|
let!(:push_event) { create(:push_event, project: public_project, author: project_owner) }
|
|
|
|
let!(:merge_event) { create(:event, :merged, project: public_project, author: project_owner) }
|
|
|
|
let!(:issue_event) { create(:event, :closed, project: public_project, target: issue, author: project_owner) }
|
|
|
|
let!(:comment_event) { create(:event, :commented, project: public_project, author: project_owner) }
|
|
|
|
let!(:wiki_event) { create(:wiki_page_event, project: public_project, author: project_owner) }
|
|
|
|
let!(:design_event) { create(:design_event, project: public_project, author: project_owner) }
|
|
|
|
let!(:team_event) { create(:event, :joined, project: public_project, author: project_owner) }
|
|
|
|
|
|
|
|
it 'includes all events', :aggregate_failures do
|
|
|
|
event_filter = EventFilter.new(EventFilter::ALL)
|
|
|
|
events = described_class.new(current_user, project_owner, event_filter, params).execute
|
|
|
|
|
|
|
|
expect(events).to include(private_event, internal_event, public_event)
|
|
|
|
expect(events).to include(push_event, merge_event, issue_event, comment_event, wiki_event, design_event, team_event)
|
|
|
|
expect(events.size).to eq(10)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'only includes push events', :aggregate_failures do
|
|
|
|
event_filter = EventFilter.new(EventFilter::PUSH)
|
|
|
|
events = described_class.new(current_user, project_owner, event_filter, params).execute
|
|
|
|
|
|
|
|
expect(events).to include(push_event)
|
|
|
|
expect(events.size).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'only includes merge events', :aggregate_failures do
|
|
|
|
event_filter = EventFilter.new(EventFilter::MERGED)
|
|
|
|
events = described_class.new(current_user, project_owner, event_filter, params).execute
|
|
|
|
|
|
|
|
expect(events).to include(merge_event)
|
|
|
|
expect(events.size).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'only includes issue events', :aggregate_failures do
|
|
|
|
event_filter = EventFilter.new(EventFilter::ISSUE)
|
|
|
|
events = described_class.new(current_user, project_owner, event_filter, params).execute
|
|
|
|
|
|
|
|
expect(events).to include(issue_event)
|
|
|
|
expect(events.size).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'only includes comments events', :aggregate_failures do
|
|
|
|
event_filter = EventFilter.new(EventFilter::COMMENTS)
|
|
|
|
events = described_class.new(current_user, project_owner, event_filter, params).execute
|
|
|
|
|
|
|
|
expect(events).to include(comment_event)
|
|
|
|
expect(events.size).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'only includes wiki events', :aggregate_failures do
|
|
|
|
event_filter = EventFilter.new(EventFilter::WIKI)
|
|
|
|
events = described_class.new(current_user, project_owner, event_filter, params).execute
|
|
|
|
|
|
|
|
expect(events).to include(wiki_event)
|
|
|
|
expect(events.size).to eq(1)
|
|
|
|
end
|
2020-07-28 23:09:34 +05:30
|
|
|
|
|
|
|
it 'only includes design events', :aggregate_failures do
|
2021-03-11 19:13:27 +05:30
|
|
|
event_filter = EventFilter.new(EventFilter::DESIGNS)
|
|
|
|
events = described_class.new(current_user, project_owner, event_filter, params).execute
|
|
|
|
|
|
|
|
expect(events).to include(design_event)
|
|
|
|
expect(events.size).to eq(1)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'only includes team events', :aggregate_failures do
|
|
|
|
event_filter = EventFilter.new(EventFilter::TEAM)
|
|
|
|
events = described_class.new(current_user, project_owner, event_filter, params).execute
|
|
|
|
|
|
|
|
expect(events).to include(private_event, internal_event, public_event, team_event)
|
|
|
|
expect(events.size).to eq(4)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
describe 'issue activity events' do
|
|
|
|
let(:issue) { create(:issue, project: public_project) }
|
|
|
|
let(:note) { create(:note_on_issue, noteable: issue, project: public_project) }
|
|
|
|
let!(:event_a) { create(:event, :commented, target: note, author: project_owner) }
|
|
|
|
let!(:event_b) { create(:event, :closed, target: issue, author: project_owner) }
|
|
|
|
|
|
|
|
it 'includes all issue related events', :aggregate_failures do
|
2020-07-28 23:09:34 +05:30
|
|
|
events = finder.execute
|
|
|
|
|
|
|
|
expect(events).to include(event_a)
|
|
|
|
expect(events).to include(event_b)
|
|
|
|
end
|
|
|
|
end
|
2020-09-03 11:15:55 +05:30
|
|
|
|
|
|
|
context 'limits' do
|
|
|
|
before do
|
|
|
|
stub_const("#{described_class}::DEFAULT_LIMIT", 1)
|
|
|
|
stub_const("#{described_class}::MAX_LIMIT", 3)
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when limit is not set' do
|
|
|
|
it 'returns events limited to DEFAULT_LIMIT' do
|
|
|
|
expect(finder.execute.size).to eq(described_class::DEFAULT_LIMIT)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when limit is set' do
|
|
|
|
let(:limit) { 2 }
|
|
|
|
|
|
|
|
it 'returns events limited to specified limit' do
|
|
|
|
expect(finder.execute.size).to eq(limit)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when limit is set to a number that exceeds maximum limit' do
|
|
|
|
let(:limit) { 4 }
|
|
|
|
|
|
|
|
before do
|
|
|
|
create(:event, project: public_project, author: project_owner)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns events limited to MAX_LIMIT' do
|
|
|
|
expect(finder.execute.size).to eq(described_class::MAX_LIMIT)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
2018-03-27 19:54:05 +05:30
|
|
|
end
|
|
|
|
end
|