diff --git a/app/services/issues/close_service.rb b/app/services/issues/close_service.rb index c723f6058e9e28bf4512388f38c0389115c4dfdd..baf7974c45d4b90a56badde261871be9fd018ab3 100644 --- a/app/services/issues/close_service.rb +++ b/app/services/issues/close_service.rb @@ -39,7 +39,10 @@ def close_issue(issue, closed_via: nil, notifications: true, system_note: true) issue.update_project_counter_caches track_incident_action(current_user, issue, :incident_closed) - store_first_mentioned_in_commit_at(issue, closed_via) if closed_via.is_a?(MergeRequest) + if closed_via.is_a?(MergeRequest) + store_first_mentioned_in_commit_at(issue, closed_via) + OnboardingProgressService.new(project.namespace).execute(action: :issue_auto_closed) + end delete_milestone_closed_issue_counter_cache(issue.milestone) end diff --git a/spec/services/issues/close_service_spec.rb b/spec/services/issues/close_service_spec.rb index 9076fb11c9b2c9dc6989fe0fb31e31bf2dfac9ff..dc545f57d2364af9b35f0045257c7d21ca7168e1 100644 --- a/spec/services/issues/close_service_spec.rb +++ b/spec/services/issues/close_service_spec.rb @@ -112,10 +112,14 @@ end context "closed by a merge request", :sidekiq_might_not_need_inline do - it 'mentions closure via a merge request' do + subject(:close_issue) do perform_enqueued_jobs do described_class.new(project, user).close_issue(issue, closed_via: closing_merge_request) end + end + + it 'mentions closure via a merge request' do + close_issue email = ActionMailer::Base.deliveries.last @@ -124,12 +128,15 @@ expect(email.body.parts.map(&:body)).to all(include(closing_merge_request.to_reference)) end + it_behaves_like 'records an onboarding progress action', :issue_auto_closed do + let(:namespace) { project.namespace } + end + context 'when user cannot read merge request' do it 'does not mention merge request' do project.project_feature.update_attribute(:repository_access_level, ProjectFeature::DISABLED) - perform_enqueued_jobs do - described_class.new(project, user).close_issue(issue, closed_via: closing_merge_request) - end + + close_issue email = ActionMailer::Base.deliveries.last body_text = email.body.parts.map(&:body).join(" ") @@ -141,13 +148,11 @@ end context 'updating `metrics.first_mentioned_in_commit_at`' do - subject { described_class.new(project, user).close_issue(issue, closed_via: closing_merge_request) } - context 'when `metrics.first_mentioned_in_commit_at` is not set' do it 'uses the first commit authored timestamp' do expected = closing_merge_request.commits.first.authored_date - subject + close_issue expect(issue.metrics.first_mentioned_in_commit_at).to eq(expected) end @@ -159,7 +164,7 @@ end it 'does not update the metrics' do - expect { subject }.not_to change { issue.metrics.first_mentioned_in_commit_at } + expect { close_issue }.not_to change { issue.metrics.first_mentioned_in_commit_at } end end @@ -167,7 +172,7 @@ let(:closing_merge_request) { create(:merge_request, :without_diffs, source_project: project) } it 'does not update the metrics' do - subject + close_issue expect(issue.metrics.first_mentioned_in_commit_at).to be_nil end @@ -206,7 +211,7 @@ end context "valid params" do - def close_issue + subject(:close_issue) do perform_enqueued_jobs do described_class.new(project, user).close_issue(issue) end @@ -290,6 +295,8 @@ def close_issue close_issue end + + it_behaves_like 'does not record an onboarding progress action' end context 'when issue is not confidential' do