diff options
Diffstat (limited to 'spec/services/merge_requests/after_create_service_spec.rb')
-rw-r--r-- | spec/services/merge_requests/after_create_service_spec.rb | 23 |
1 files changed, 7 insertions, 16 deletions
diff --git a/spec/services/merge_requests/after_create_service_spec.rb b/spec/services/merge_requests/after_create_service_spec.rb index f2823b1f0c7..50a3d49d4a3 100644 --- a/spec/services/merge_requests/after_create_service_spec.rb +++ b/spec/services/merge_requests/after_create_service_spec.rb @@ -4,6 +4,7 @@ require 'spec_helper' RSpec.describe MergeRequests::AfterCreateService, feature_category: :code_review_workflow do let_it_be(:merge_request) { create(:merge_request) } + let(:project) { merge_request.project } subject(:after_create_service) do described_class.new(project: merge_request.target_project, current_user: merge_request.author) @@ -68,6 +69,12 @@ RSpec.describe MergeRequests::AfterCreateService, feature_category: :code_review execute_service end + it 'executes hooks with default action' do + expect(project).to receive(:execute_hooks) + + execute_service + end + it_behaves_like 'records an onboarding progress action', :merge_request_created do let(:namespace) { merge_request.target_project.namespace } end @@ -143,22 +150,6 @@ RSpec.describe MergeRequests::AfterCreateService, feature_category: :code_review expect { execute_service }.to change { counter.read(:create) }.by(1) end - context 'with a milestone' do - let(:milestone) { create(:milestone, project: merge_request.target_project) } - - before do - merge_request.update!(milestone_id: milestone.id) - end - - it 'deletes the cache key for milestone merge request counter', :use_clean_rails_memory_store_caching do - expect_next_instance_of(Milestones::MergeRequestsCountService, milestone) do |service| - expect(service).to receive(:delete_cache).and_call_original - end - - execute_service - end - end - context 'todos' do it 'does not creates todos' do attributes = { |