feat(alerts): Remove "Suspects" UI and endpoint (#16378)

This was used on the Incidents details view, but is no longer used

Author: billyvg

src/sentry/api/endpoints/organization_incident_suspects_index.py

@@ -108,9 +108,6 @@
 from .endpoints.organization_incident_subscription_index import (
     OrganizationIncidentSubscriptionIndexEndpoint,
 )
-from .endpoints.organization_incident_suspects_index import (
-    OrganizationIncidentSuspectsIndexEndpoint,
-)
 from .endpoints.organization_index import OrganizationIndexEndpoint
 from .endpoints.organization_integration_details import OrganizationIntegrationDetailsEndpoint
 from .endpoints.organization_integration_repos import OrganizationIntegrationReposEndpoint
@@ -641,11 +638,6 @@
                     OrganizationIncidentSubscriptionIndexEndpoint.as_view(),
                     name="sentry-api-0-organization-incident-subscription-index",
                 ),
-                url(
-                    r"^(?P<organization_slug>[^\/]+)/incidents/(?P<incident_identifier>[^\/]+)/suspects/$",
-                    OrganizationIncidentSuspectsIndexEndpoint.as_view(),
-                    name="sentry-api-0-organization-incident-suspect-index",
-                ),
                 url(
                     r"^(?P<organization_slug>[^\/]+)/chunk-upload/$",
                     ChunkUploadEndpoint.as_view(),

src/sentry/api/urls.py

@@ -16,7 +16,6 @@ import space from 'app/styles/space';
 import theme from 'app/utils/theme';
 
 import Activity from './activity';
-import Suspects from './suspects';
 
 import {Incident} from '../types';
 
@@ -84,8 +83,6 @@ export default class DetailsBody extends React.Component<Props> {
                 </div>
               )}
             </div>
-
-            <Suspects {...this.props} />
           </PageContent>
         </Sidebar>
       </StyledPageContent>

src/sentry/static/sentry/app/views/incidents/details/body.tsx

@@ -1,18 +0,0 @@
-import {Repository} from './repository';
-import {User} from './user';
-
-export function IncidentSuspectCommit(params = []) {
-  return {
-    data: {
-      repository: Repository(),
-      author: User(),
-      dateCreated: '2019-03-28T01:36:35.457Z',
-      score: 2,
-      message:
-        'feat: Do something to raven/base.py\nvenenatis curae tincidunt feugiat duis parturient metus',
-      id: 'ec85fa0c622c13a09cd27443132711551f45f504',
-      ...params,
-    },
-    type: 'commit',
-  };
-}

src/sentry/static/sentry/app/views/incidents/details/suspects.tsx

@@ -36,18 +36,10 @@ describe('IncidentDetails', function() {
       method: 'POST',
     });
 
-    MockApiClient.addMockResponse({
-      url: '/organizations/org-slug/incidents/123/suspects/',
-      body: [TestStubs.IncidentSuspectCommit()],
-    });
     MockApiClient.addMockResponse({
       url: '/organizations/org-slug/incidents/456/',
       statusCode: 404,
     });
-    MockApiClient.addMockResponse({
-      url: '/organizations/org-slug/incidents/456/suspects/',
-      statusCode: 404,
-    });
     MockApiClient.addMockResponse({
       url: '/organizations/org-slug/incidents/456/activity/',
       statusCode: 404,
@@ -89,15 +81,6 @@ describe('IncidentDetails', function() {
         .at(2)
         .text()
     ).toBe('20');
-
-    expect(wrapper.find('SuspectItem')).toHaveLength(1);
-    expect(
-      wrapper
-        .find('SuspectItem')
-        .at(0)
-        .find('MessageOverflow')
-        .text()
-    ).toBe('feat: Do something to raven/base.py');
   });
 
   it('handles invalid incident', async function() {