ref(✂️): remove unused useDiscoverSeries (#100296)

The only usage was in its own test, since the usage is currently being
replaced with the `useFetchSpanTimeSeries` hook.

Interestingly, the `DiscoverSeries` type stays behind because it’s used
in a bunch of places, but those will be slowly eliminated as the
`useFetchSpanTimeSeries` hook gets more usage.

Closes ENG-5604

---------

Co-authored-by: George Gritsouk <[email protected]>

Author: TkDodo

static/app/components/charts/chartWidgetLoader-unmocked-imports.spec.tsx

@@ -96,48 +96,6 @@ jest.mock('sentry/views/insights/common/queries/useTopNDiscoverSeries', () => ({
     error: null,
   })),
 }));
-jest.mock('sentry/views/insights/common/queries/useDiscoverSeries', () => ({
-  useSpanSeries: jest.fn(() => ({
-    data: {
-      'epm()': {},
-      'count(span.duration)': mockDiscoverSeries('count(span.duration)'),
-      'avg(span.duration)': mockDiscoverSeries('avg(span.duration)'),
-      'p95(span.duration)': mockDiscoverSeries('p95(span.duration)'),
-      'trace_status_rate(internal_error)': mockDiscoverSeries(
-        'trace_status_rate(internal_error)'
-      ),
-      'cache_miss_rate()': {},
-      'http_response_rate(3)': {},
-      'http_response_rate(4)': {},
-      'http_response_rate(5)': {},
-      'avg(span.self_time)': {},
-      'avg(http.response_content_length)': {},
-      'avg(http.response_transfer_size)': {},
-      'avg(http.decoded_response_content_length)': {},
-      'avg(messaging.message.receive.latency)': {},
-      'performance_score(measurements.score.lcp)': {
-        data: [],
-      },
-      'performance_score(measurements.score.fcp)': {
-        data: [],
-      },
-      'performance_score(measurements.score.cls)': {
-        data: [],
-      },
-      'performance_score(measurements.score.inp)': {
-        data: [],
-      },
-      'performance_score(measurements.score.ttfb)': {
-        data: [],
-      },
-      'count()': {
-        data: [],
-      },
-    },
-    isPending: false,
-    error: null,
-  })),
-}));
 jest.mock('sentry/utils/timeSeries/useFetchEventsTimeSeries', () => ({
   useFetchSpanTimeSeries: jest.fn(({groupBy}: {groupBy?: string[]}) => ({
     data: {

static/app/views/insights/common/components/insightsTimeSeriesWidget.tsx

@@ -41,7 +41,7 @@ import {
   ModalChartContainer,
 } from 'sentry/views/insights/common/components/insightsChartContainer';
 import type {LoadableChartWidgetProps} from 'sentry/views/insights/common/components/widgets/types';
-import type {DiscoverSeries} from 'sentry/views/insights/common/queries/useDiscoverSeries';
+import type {DiscoverSeries} from 'sentry/views/insights/common/queries/types';
 import {convertSeriesToTimeseries} from 'sentry/views/insights/common/utils/convertSeriesToTimeseries';
 import {BASE_FIELD_ALIASES, INGESTION_DELAY} from 'sentry/views/insights/settings';
 import type {SpanFields} from 'sentry/views/insights/types';

static/app/views/insights/common/queries/convertDiscoverTimeseriesResponse.ts

@@ -1,6 +1,6 @@
 import moment from 'moment-timezone';
 
-import type {DiscoverSeries} from './useDiscoverSeries';
+import type {DiscoverSeries} from './types';
 
 const DATE_FORMAT = 'YYYY-MM-DDTHH:mm:ssZ';
 

static/app/views/insights/common/queries/types.ts

@@ -0,0 +1,6 @@
+import type {Series} from 'sentry/types/echarts';
+import {type EventsMetaType} from 'sentry/utils/discover/eventView';
+
+export type DiscoverSeries = Series & {
+  meta: EventsMetaType;
+};