Merge branch...

Merge branch '329235-error-creating-new-jira-issue-from-vulnerability-after-previous-jira-issue-is-deleted' into 'master'

Check for deleted JIRA issue before creating one from Vulnerability

See merge request gitlab-org/gitlab!78323

ee/app/models/vulnerabilities/external_issue_link.rb

@@ -18,5 +18,7 @@ class ExternalIssueLink < ApplicationRecord
                 message: N_('already has a "created" issue link')
               },
               if: :created?
+
+    scope :created_for_vulnerability, -> (vulnerability) { where(vulnerability: vulnerability, link_type: 'created') }
   end
 end

ee/app/services/vulnerability_external_issue_links/create_service.rb

@@ -13,6 +13,9 @@ def initialize(user, vulnerability, external_provider, link_type: Vulnerabilitie
 
     def execute
       return error(['External provider service is not configured to create issues.']) unless external_provider_service&.configured_to_create_issues_from_vulnerabilities?
+
+      remove_links_for_nonexistent_external_issues
+
       return error(external_issue_link.errors.full_messages) unless external_issue_link.valid?
 
       external_issue = create_external_issue
@@ -57,6 +60,14 @@ def prepare_external_issue_link
       )
     end
 
+    def remove_links_for_nonexistent_external_issues
+      Vulnerabilities::ExternalIssueLink.created_for_vulnerability(vulnerability).each do |link|
+        unless external_provider_service.find_issue(link.external_issue_key)
+          link.destroy
+        end
+      end
+    end
+
     def success
       ServiceResponse.success(payload: { record: external_issue_link })
     end

ee/spec/models/vulnerabilities/external_issue_link_spec.rb

@@ -45,4 +45,12 @@
       end
     end
   end
+
+  describe 'created_for_vulnerability' do
+    let_it_be(:external_links) { create_list(:vulnerabilities_external_issue_link, 2, :created) }
+
+    it 'gets external issue links for the specified vulnerability' do
+      expect(described_class.created_for_vulnerability(external_links.first.vulnerability).take).to eq(external_links.first)
+    end
+  end
 end

ee/spec/services/vulnerability_external_issue_links/create_service_spec.rb

@@ -38,10 +38,35 @@
           create(:vulnerabilities_external_issue_link, vulnerability: vulnerability)
         end
 
-        it { is_expected.not_to be_success }
+        context 'and the external issue exists' do
+          let!(:jira_issue) { ExternalIssue.new('JIRA-123', project) }
 
-        it 'returns response with error messages' do
-          expect(subject.message).to eq(['Vulnerability already has a "created" issue link'])
+          before do
+            allow(jira_integration).to receive(:find_issue).and_return(jira_issue)
+          end
+
+          it { is_expected.not_to be_success }
+
+          it 'returns response with error messages' do
+            expect(subject.message).to match_array(['Vulnerability already has a "created" issue link'])
+          end
+        end
+
+        context 'and the external issue does not exist' do
+          let(:jira_issue_id) { nil }
+          let(:errors) { {} }
+          let(:jira_issue) { double(has_errors?: errors.present?, id: jira_issue_id, errors: errors) }
+
+          before do
+            allow(jira_integration).to receive(:create_issue).and_return(jira_issue)
+            allow(jira_integration).to receive(:find_issue).and_return(nil)
+          end
+
+          it 'creates issue using jira service' do
+            expect(jira_integration).to receive(:create_issue).with("Investigate vulnerability: #{vulnerability.title}", kind_of(String), user)
+
+            subject
+          end
         end
       end