Use join for finding SentryIssues

- Also add index to sentry_issue_identifier
- Rename fetch to perform in error tracking services

app/services/error_tracking/base_service.rb

@@ -7,7 +7,7 @@ module ErrorTracking
       return unauthorized if unauthorized
 
       begin
-        response = fetch
+        response = perform
       rescue Sentry::Client::Error => e
         return error(e.message, :bad_request)
       rescue Sentry::Client::MissingKeysError => e
@@ -22,7 +22,7 @@ module ErrorTracking
 
     private
 
-    def fetch
+    def perform
       raise NotImplementedError,
           "#{self.class} does not implement #{__method__}"
     end

app/services/error_tracking/issue_details_service.rb

@@ -4,7 +4,7 @@ module ErrorTracking
   class IssueDetailsService < ErrorTracking::BaseService
     private
 
-    def fetch
+    def perform
       project_error_tracking_setting.issue_details(issue_id: params[:issue_id])
     end
 

app/services/error_tracking/issue_latest_event_service.rb

@@ -4,7 +4,7 @@ module ErrorTracking
   class IssueLatestEventService < ErrorTracking::BaseService
     private
 
-    def fetch
+    def perform
       project_error_tracking_setting.issue_latest_event(issue_id: params[:issue_id])
     end
 

app/services/error_tracking/list_issues_service.rb

@@ -12,7 +12,7 @@ module ErrorTracking
 
     private
 
-    def fetch
+    def perform
       project_error_tracking_setting.list_sentry_issues(
         issue_status: issue_status,
         limit: limit,

app/services/error_tracking/list_projects_service.rb

@@ -12,7 +12,7 @@ module ErrorTracking
 
     private
 
-    def fetch
+    def perform
       project_error_tracking_setting.list_sentry_projects
     end
 

spec/models/sentry_issue_spec.rb

@@ -37,16 +37,4 @@ describe SentryIssue do
 
     it { is_expected.to eq(sentry_issue) }
   end
-
-  describe 'scopes' do
-    describe 'for_identifier' do
-      let!(:sentry_issue) { create(:sentry_issue) }
-      let(:identifier) { sentry_issue.sentry_issue_identifier }
-      let!(:second_sentry_issue) { create(:sentry_issue) }
-
-      subject { described_class.for_identifier(identifier) }
-
-      it { is_expected.to eq(sentry_issue) }
-    end
-  end
 end