f1479b56b7
In 8278b763d9
the default behaviour of annotation
has changes, which was causing a lot of noise in diffs. We decided in #17382
that it is better to get rid of the whole annotate gem, and instead let people
look at schema.rb for the columns in a table.
Fixes: #17382
151 lines
5 KiB
Ruby
151 lines
5 KiB
Ruby
require 'spec_helper'
|
|
|
|
describe Event, models: true do
|
|
describe "Associations" do
|
|
it { is_expected.to belong_to(:project) }
|
|
it { is_expected.to belong_to(:target) }
|
|
end
|
|
|
|
describe "Respond to" do
|
|
it { is_expected.to respond_to(:author_name) }
|
|
it { is_expected.to respond_to(:author_email) }
|
|
it { is_expected.to respond_to(:issue_title) }
|
|
it { is_expected.to respond_to(:merge_request_title) }
|
|
it { is_expected.to respond_to(:commits) }
|
|
end
|
|
|
|
describe 'Callbacks' do
|
|
describe 'after_create :reset_project_activity' do
|
|
let(:project) { create(:project) }
|
|
|
|
context "project's last activity was less than 5 minutes ago" do
|
|
it 'does not update project.last_activity_at if it has been touched less than 5 minutes ago' do
|
|
create_event(project, project.owner)
|
|
project.update_column(:last_activity_at, 5.minutes.ago)
|
|
project_last_activity_at = project.last_activity_at
|
|
|
|
create_event(project, project.owner)
|
|
|
|
expect(project.last_activity_at).to eq(project_last_activity_at)
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "Push event" do
|
|
before do
|
|
project = create(:project)
|
|
@user = project.owner
|
|
@event = create_event(project, @user)
|
|
end
|
|
|
|
it { expect(@event.push?).to be_truthy }
|
|
it { expect(@event.visible_to_user?).to be_truthy }
|
|
it { expect(@event.tag?).to be_falsey }
|
|
it { expect(@event.branch_name).to eq("master") }
|
|
it { expect(@event.author).to eq(@user) }
|
|
end
|
|
|
|
describe '#visible_to_user?' do
|
|
let(:project) { create(:empty_project, :public) }
|
|
let(:non_member) { create(:user) }
|
|
let(:member) { create(:user) }
|
|
let(:author) { create(:author) }
|
|
let(:assignee) { create(:user) }
|
|
let(:admin) { create(:admin) }
|
|
let(:issue) { create(:issue, project: project, author: author, assignee: assignee) }
|
|
let(:confidential_issue) { create(:issue, :confidential, project: project, author: author, assignee: assignee) }
|
|
let(:note_on_issue) { create(:note_on_issue, noteable: issue, project: project) }
|
|
let(:note_on_confidential_issue) { create(:note_on_issue, noteable: confidential_issue, project: project) }
|
|
let(:event) { Event.new(project: project, target: target, author_id: author.id) }
|
|
|
|
before do
|
|
project.team << [member, :developer]
|
|
end
|
|
|
|
context 'issue event' do
|
|
context 'for non confidential issues' do
|
|
let(:target) { issue }
|
|
|
|
it { expect(event.visible_to_user?(non_member)).to eq true }
|
|
it { expect(event.visible_to_user?(author)).to eq true }
|
|
it { expect(event.visible_to_user?(assignee)).to eq true }
|
|
it { expect(event.visible_to_user?(member)).to eq true }
|
|
it { expect(event.visible_to_user?(admin)).to eq true }
|
|
end
|
|
|
|
context 'for confidential issues' do
|
|
let(:target) { confidential_issue }
|
|
|
|
it { expect(event.visible_to_user?(non_member)).to eq false }
|
|
it { expect(event.visible_to_user?(author)).to eq true }
|
|
it { expect(event.visible_to_user?(assignee)).to eq true }
|
|
it { expect(event.visible_to_user?(member)).to eq true }
|
|
it { expect(event.visible_to_user?(admin)).to eq true }
|
|
end
|
|
end
|
|
|
|
context 'note event' do
|
|
context 'on non confidential issues' do
|
|
let(:target) { note_on_issue }
|
|
|
|
it { expect(event.visible_to_user?(non_member)).to eq true }
|
|
it { expect(event.visible_to_user?(author)).to eq true }
|
|
it { expect(event.visible_to_user?(assignee)).to eq true }
|
|
it { expect(event.visible_to_user?(member)).to eq true }
|
|
it { expect(event.visible_to_user?(admin)).to eq true }
|
|
end
|
|
|
|
context 'on confidential issues' do
|
|
let(:target) { note_on_confidential_issue }
|
|
|
|
it { expect(event.visible_to_user?(non_member)).to eq false }
|
|
it { expect(event.visible_to_user?(author)).to eq true }
|
|
it { expect(event.visible_to_user?(assignee)).to eq true }
|
|
it { expect(event.visible_to_user?(member)).to eq true }
|
|
it { expect(event.visible_to_user?(admin)).to eq true }
|
|
end
|
|
end
|
|
end
|
|
|
|
describe '.limit_recent' do
|
|
let!(:event1) { create(:closed_issue_event) }
|
|
let!(:event2) { create(:closed_issue_event) }
|
|
|
|
describe 'without an explicit limit' do
|
|
subject { Event.limit_recent }
|
|
|
|
it { is_expected.to eq([event2, event1]) }
|
|
end
|
|
|
|
describe 'with an explicit limit' do
|
|
subject { Event.limit_recent(1) }
|
|
|
|
it { is_expected.to eq([event2]) }
|
|
end
|
|
end
|
|
|
|
def create_event(project, user, attrs = {})
|
|
data = {
|
|
before: Gitlab::Git::BLANK_SHA,
|
|
after: "0220c11b9a3e6c69dc8fd35321254ca9a7b98f7e",
|
|
ref: "refs/heads/master",
|
|
user_id: user.id,
|
|
user_name: user.name,
|
|
repository: {
|
|
name: project.name,
|
|
url: "localhost/rubinius",
|
|
description: "",
|
|
homepage: "localhost/rubinius",
|
|
private: true
|
|
}
|
|
}
|
|
|
|
Event.create({
|
|
project: project,
|
|
action: Event::PUSHED,
|
|
data: data,
|
|
author_id: user.id
|
|
}.merge(attrs))
|
|
end
|
|
end
|