Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat (limited to 'spec/services/issues/create_service_spec.rb')
-rw-r--r--spec/services/issues/create_service_spec.rb24
1 files changed, 10 insertions, 14 deletions
diff --git a/spec/services/issues/create_service_spec.rb b/spec/services/issues/create_service_spec.rb
index 9c84242d8ae..94810d6134a 100644
--- a/spec/services/issues/create_service_spec.rb
+++ b/spec/services/issues/create_service_spec.rb
@@ -18,8 +18,8 @@ RSpec.describe Issues::CreateService do
let_it_be(:labels) { create_pair(:label, project: project) }
before_all do
- project.add_maintainer(user)
- project.add_maintainer(assignee)
+ project.add_guest(user)
+ project.add_guest(assignee)
end
let(:opts) do
@@ -78,8 +78,8 @@ RSpec.describe Issues::CreateService do
opts.merge!(title: '')
end
- it 'does not create an incident label prematurely' do
- expect { subject }.not_to change(Label, :count)
+ it 'does not apply an incident label prematurely' do
+ expect { subject }.to not_change(LabelLink, :count).and not_change(Issue, :count)
end
end
end
@@ -88,15 +88,11 @@ RSpec.describe Issues::CreateService do
expect { issue }.to change { project.open_issues_count }.from(0).to(1)
end
- context 'when current user cannot admin issues in the project' do
- let_it_be(:guest) { create(:user) }
-
- before_all do
- project.add_guest(guest)
- end
+ context 'when current user cannot set issue metadata in the project' do
+ let_it_be(:non_member) { create(:user) }
- it 'filters out params that cannot be set without the :admin_issue permission' do
- issue = described_class.new(project: project, current_user: guest, params: opts).execute
+ it 'filters out params that cannot be set without the :set_issue_metadata permission' do
+ issue = described_class.new(project: project, current_user: non_member, params: opts).execute
expect(issue).to be_persisted
expect(issue.title).to eq('Awesome issue')
@@ -107,8 +103,8 @@ RSpec.describe Issues::CreateService do
expect(issue.due_date).to be_nil
end
- it 'creates confidential issues' do
- issue = described_class.new(project: project, current_user: guest, params: { confidential: true }).execute
+ it 'can create confidential issues' do
+ issue = described_class.new(project: project, current_user: non_member, params: { confidential: true }).execute
expect(issue.confidential).to be_truthy
end