diff options
author | Jarka Kadlecová <jarka@gitlab.com> | 2018-07-26 17:53:50 +0300 |
---|---|---|
committer | Jarka Kadlecová <jarka@gitlab.com> | 2018-07-30 14:29:18 +0300 |
commit | 7934b91311a70d994c6700201979c6673160fd01 (patch) | |
tree | f2a318eaadeeb644bf108dcffaf546d45260b208 /spec/services/todos | |
parent | 501fb04ec65cadcd7dddc6376546db8d8f7f123c (diff) |
Fix removing todos for confidential issues
- dont remove todos for authos & assignees
- remove todos for project guests
Diffstat (limited to 'spec/services/todos')
-rw-r--r-- | spec/services/todos/destroy/confidential_issue_service_spec.rb | 12 | ||||
-rw-r--r-- | spec/services/todos/destroy/entity_leave_service_spec.rb | 38 |
2 files changed, 43 insertions, 7 deletions
diff --git a/spec/services/todos/destroy/confidential_issue_service_spec.rb b/spec/services/todos/destroy/confidential_issue_service_spec.rb index 5c214df49bc..54d1d7e83f1 100644 --- a/spec/services/todos/destroy/confidential_issue_service_spec.rb +++ b/spec/services/todos/destroy/confidential_issue_service_spec.rb @@ -3,16 +3,23 @@ require 'spec_helper' describe Todos::Destroy::ConfidentialIssueService do let(:project) { create(:project, :public) } let(:user) { create(:user) } + let(:author) { create(:user) } + let(:assignee) { create(:user) } + let(:guest) { create(:user) } let(:project_member) { create(:user) } - let(:issue) { create(:issue, project: project) } + let(:issue) { create(:issue, project: project, author: author, assignees: [assignee]) } let!(:todo_issue_non_member) { create(:todo, user: user, target: issue, project: project) } let!(:todo_issue_member) { create(:todo, user: project_member, target: issue, project: project) } + let!(:todo_issue_author) { create(:todo, user: author, target: issue, project: project) } + let!(:todo_issue_asignee) { create(:todo, user: assignee, target: issue, project: project) } + let!(:todo_issue_guest) { create(:todo, user: guest, target: issue, project: project) } let!(:todo_another_non_member) { create(:todo, user: user, project: project) } describe '#execute' do before do project.add_developer(project_member) + project.add_guest(guest) end subject { described_class.new(issue.id).execute } @@ -23,9 +30,10 @@ describe Todos::Destroy::ConfidentialIssueService do end it 'removes issue todos for a user who is not a project member' do - expect { subject }.to change { Todo.count }.from(3).to(2) + expect { subject }.to change { Todo.count }.from(6).to(4) expect(user.todos).to match_array([todo_another_non_member]) + expect(author.todos).to match_array([todo_issue_author]) expect(project_member.todos).to match_array([todo_issue_member]) end end diff --git a/spec/services/todos/destroy/entity_leave_service_spec.rb b/spec/services/todos/destroy/entity_leave_service_spec.rb index e5673383df8..52175ed9032 100644 --- a/spec/services/todos/destroy/entity_leave_service_spec.rb +++ b/spec/services/todos/destroy/entity_leave_service_spec.rb @@ -29,13 +29,41 @@ describe Todos::Destroy::EntityLeaveService do end context 'when project is not private' do - before do - group.update!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) - project.update!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) + context 'when a user is not an author of confidential issue' do + before do + group.update!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) + project.update!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) + end + + it 'removes only confidential issues todos' do + expect { subject }.to change { Todo.count }.from(3).to(2) + end end - it 'removes only confidential issues todos' do - expect { subject }.to change { Todo.count }.from(3).to(2) + context 'when a user is an author of confidential issue' do + before do + issue.update!(author: user) + + group.update!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) + project.update!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) + end + + it 'removes only confidential issues todos' do + expect { subject }.not_to change { Todo.count } + end + end + + context 'when a user is an assignee of confidential issue' do + before do + issue.assignees << user + + group.update!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) + project.update!(visibility_level: Gitlab::VisibilityLevel::INTERNAL) + end + + it 'removes only confidential issues todos' do + expect { subject }.not_to change { Todo.count } + end end end end |