Commit 4a50dde7 authored by Valery Sizov's avatar Valery Sizov

[Multiple issue asignees] spec/models/issue_spec.rb

parent 799459c3
...@@ -189,10 +189,6 @@ class MergeRequest < ActiveRecord::Base ...@@ -189,10 +189,6 @@ class MergeRequest < ActiveRecord::Base
work_in_progress?(title) ? title : "WIP: #{title}" work_in_progress?(title) ? title : "WIP: #{title}"
end end
def is_being_reassigned?
assignee_id_changed?
end
def update_assignee_cache_counts def update_assignee_cache_counts
# make sure we flush the cache for both the old *and* new assignees(if they exist) # make sure we flush the cache for both the old *and* new assignees(if they exist)
previous_assignee = User.find_by_id(assignee_id_was) if assignee_id_was previous_assignee = User.find_by_id(assignee_id_was) if assignee_id_was
......
...@@ -172,25 +172,6 @@ describe Issue, models: true do ...@@ -172,25 +172,6 @@ describe Issue, models: true do
end end
end end
describe '#is_being_reassigned?' do
it 'returns true if the issue assignee has changed' do
subject.assignee = create(:user)
expect(subject.is_being_reassigned?).to be_truthy
end
it 'returns false if the issue assignee has not changed' do
expect(subject.is_being_reassigned?).to be_falsey
end
end
describe '#is_being_reassigned?' do
it 'returns issues assigned to user' do
user = create(:user)
create_list(:issue, 2, assignees: [user])
expect(Issue.open_for(user).count).to eq 2
end
end
describe '#closed_by_merge_requests' do describe '#closed_by_merge_requests' do
let(:project) { create(:project, :repository) } let(:project) { create(:project, :repository) }
let(:issue) { create(:issue, project: project)} let(:issue) { create(:issue, project: project)}
...@@ -423,7 +404,7 @@ describe Issue, models: true do ...@@ -423,7 +404,7 @@ describe Issue, models: true do
expect(user1.assigned_open_issues_count).to eq(1) expect(user1.assigned_open_issues_count).to eq(1)
expect(user2.assigned_open_issues_count).to eq(0) expect(user2.assigned_open_issues_count).to eq(0)
issue.assignee = user2 issue.assignees = [user2]
issue.save issue.save
expect(user1.assigned_open_issues_count).to eq(0) expect(user1.assigned_open_issues_count).to eq(0)
......
...@@ -317,16 +317,6 @@ describe MergeRequest, models: true do ...@@ -317,16 +317,6 @@ describe MergeRequest, models: true do
end end
end end
describe '#is_being_reassigned?' do
it 'returns true if the merge_request assignee has changed' do
subject.assignee = create(:user)
expect(subject.is_being_reassigned?).to be_truthy
end
it 'returns false if the merge request assignee has not changed' do
expect(subject.is_being_reassigned?).to be_falsey
end
end
describe '#for_fork?' do describe '#for_fork?' do
it 'returns true if the merge request is for a fork' do it 'returns true if the merge request is for a fork' do
subject.source_project = build_stubbed(:empty_project, namespace: create(:group)) subject.source_project = build_stubbed(:empty_project, namespace: create(:group))
......
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