123456789101112131415161718192021222324252627282930313233343536373839404142434445464748495051525354555657585960616263646566676869707172737475767778798081828384858687888990919293949596979899100101102103104105106107108109110111112113114115116117118119120121122123124125126127128129130131132133134135136137138139140141142143144145146147148149150151152153154155156157158159160161162163164165166167168169170171172173174175176177178179180181182183184185186187188189190191192193194195196197198199200201202203204205206207208209210211212213214215216217218219220221222223224225226227228229230231232233234235236237238239240241242243244245246247248249250251252253254255256257258259260261262263264265266267268269270271272273274275276277278279280281282283284285286287288289290291292293294295296297298299300301302303304305306307308309310311312313314315316317318319320321322323324325326327328329330331332333334335336337338339340341342 |
- import {Fragment} from 'react';
- import {initializeData as _initializeData} from 'sentry-test/performance/initializePerformanceData';
- import {render, screen} from 'sentry-test/reactTestingLibrary';
- import {GenericQueryBatcher} from 'sentry/utils/performance/contexts/genericQueryBatcher';
- import {MEPSettingProvider} from 'sentry/utils/performance/contexts/metricsEnhancedSetting';
- import {PerformanceDisplayProvider} from 'sentry/utils/performance/contexts/performanceDisplayContext';
- import {OrganizationContext} from 'sentry/views/organizationContext';
- import WidgetContainer from 'sentry/views/performance/landing/widgets/components/widgetContainer';
- import {PerformanceWidgetSetting} from 'sentry/views/performance/landing/widgets/widgetDefinitions';
- import {PROJECT_PERFORMANCE_TYPE} from 'sentry/views/performance/utils';
- const initializeData = () => {
- const data = _initializeData({
- query: {statsPeriod: '7d', environment: ['prod'], project: [-42]},
- });
- data.eventView.additionalConditions.addFilterValues('transaction.op', ['pageload']);
- return data;
- };
- const BASIC_QUERY_PARAMS = {
- interval: '1h',
- partial: '1',
- query: 'transaction.op:pageload',
- statsPeriod: '14d',
- };
- const WrappedComponent = ({data, ...rest}) => {
- return (
- <OrganizationContext.Provider value={data.organization}>
- <MEPSettingProvider>
- <PerformanceDisplayProvider
- value={{performanceType: PROJECT_PERFORMANCE_TYPE.ANY}}
- >
- <WidgetContainer
- allowedCharts={[
- PerformanceWidgetSetting.TPM_AREA,
- PerformanceWidgetSetting.FAILURE_RATE_AREA,
- PerformanceWidgetSetting.USER_MISERY_AREA,
- ]}
- rowChartSettings={[]}
- forceDefaultChartSetting
- {...data}
- {...rest}
- />
- </PerformanceDisplayProvider>
- </MEPSettingProvider>
- </OrganizationContext.Provider>
- );
- };
- describe('Performance > Widgets > Query Batching', function () {
- let eventStatsMock;
- beforeEach(function () {
- eventStatsMock = MockApiClient.addMockResponse({
- method: 'GET',
- url: `/organizations/org-slug/events-stats/`,
- body: {
- 'tpm()': {
- data: [
- [
- 1636822800,
- [
- {
- count: 30.0,
- },
- ],
- ],
- [
- 1636995600,
- [
- {
- count: 60.1,
- },
- ],
- ],
- ],
- order: 1,
- start: 1636822800,
- end: 1636995600,
- },
- 'user_misery()': {
- data: [
- [
- 1636822800,
- [
- {
- count: 0.02,
- },
- ],
- ],
- [
- 1636995600,
- [
- {
- count: 0.03,
- },
- ],
- ],
- ],
- order: 1,
- start: 1636822800,
- end: 1636995600,
- },
- 'failure_rate()': {
- data: [
- [
- 1636822800,
- [
- {
- count: 0.002,
- },
- ],
- ],
- [
- 1636995600,
- [
- {
- count: 0.001,
- },
- ],
- ],
- ],
- order: 2,
- start: 1636822800,
- end: 1636995600,
- },
- },
- });
- });
- it('EventsRequest based component fires query without provider', async function () {
- const data = initializeData();
- render(
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.TPM_AREA}
- isMEPEnabled={false}
- />,
- {
- organization: data.organization,
- }
- );
- expect(await screen.findByTestId('performance-widget-title')).toBeInTheDocument();
- expect(eventStatsMock).toHaveBeenCalledTimes(1);
- expect(eventStatsMock).toHaveBeenNthCalledWith(
- 1,
- expect.anything(),
- expect.objectContaining({
- query: expect.objectContaining({
- ...BASIC_QUERY_PARAMS,
- yAxis: 'tpm()',
- }),
- })
- );
- });
- it('Multiple EventsRequest based components fire individual queries without provider', async function () {
- const data = initializeData();
- render(
- <Fragment>
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.TPM_AREA}
- isMEPEnabled={false}
- />
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.FAILURE_RATE_AREA}
- isMEPEnabled={false}
- />
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.USER_MISERY_AREA}
- isMEPEnabled={false}
- />
- </Fragment>,
- {
- organization: data.organization,
- }
- );
- expect(await screen.findAllByTestId('performance-widget-title')).toHaveLength(3);
- // Three requests are made
- expect(eventStatsMock).toHaveBeenCalledTimes(3);
- expect(eventStatsMock).toHaveBeenNthCalledWith(
- 1,
- expect.anything(),
- expect.objectContaining({
- query: expect.objectContaining({
- ...BASIC_QUERY_PARAMS,
- yAxis: 'tpm()',
- }),
- })
- );
- });
- it('Multiple EventsRequest based component merge queries with provider', async function () {
- const data = initializeData();
- render(
- <GenericQueryBatcher>
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.TPM_AREA}
- isMEPEnabled={false}
- />
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.FAILURE_RATE_AREA}
- isMEPEnabled={false}
- />
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.USER_MISERY_AREA}
- isMEPEnabled={false}
- />
- </GenericQueryBatcher>,
- {
- organization: data.organization,
- }
- );
- expect(await screen.findAllByTestId('performance-widget-title')).toHaveLength(3);
- expect(eventStatsMock).toHaveBeenNthCalledWith(
- 1,
- expect.anything(),
- expect.objectContaining({
- query: expect.objectContaining({
- ...BASIC_QUERY_PARAMS,
- yAxis: ['tpm()', 'failure_rate()', 'user_misery()'],
- }),
- })
- );
- expect(eventStatsMock).toHaveBeenCalledTimes(1);
- expect(await screen.findAllByTestId('widget-state-has-data')).toHaveLength(3);
- });
- it('Multiple EventsRequest based component merge queries with provider and add MEP', async function () {
- const data = initializeData();
- render(
- <GenericQueryBatcher>
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.TPM_AREA}
- isMEPEnabled
- />
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.FAILURE_RATE_AREA}
- isMEPEnabled
- />
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.USER_MISERY_AREA}
- isMEPEnabled
- />
- </GenericQueryBatcher>,
- {
- organization: data.organization,
- }
- );
- expect(await screen.findAllByTestId('performance-widget-title')).toHaveLength(3);
- expect(eventStatsMock).toHaveBeenNthCalledWith(
- 1,
- expect.anything(),
- expect.objectContaining({
- query: expect.objectContaining({
- ...BASIC_QUERY_PARAMS,
- yAxis: ['tpm()', 'failure_rate()', 'user_misery()'],
- }),
- })
- );
- expect(eventStatsMock).toHaveBeenCalledTimes(1);
- expect(await screen.findAllByTestId('widget-state-has-data')).toHaveLength(3);
- });
- it('Errors work correctly', async function () {
- eventStatsMock = MockApiClient.addMockResponse({
- method: 'GET',
- url: `/organizations/org-slug/events-stats/`,
- statusCode: 404,
- body: {},
- });
- const data = initializeData();
- render(
- <GenericQueryBatcher>
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.TPM_AREA}
- isMEPEnabled={false}
- />
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.FAILURE_RATE_AREA}
- isMEPEnabled={false}
- />
- <WrappedComponent
- data={data}
- defaultChartSetting={PerformanceWidgetSetting.USER_MISERY_AREA}
- isMEPEnabled={false}
- />
- </GenericQueryBatcher>,
- {
- organization: data.organization,
- }
- );
- expect(await screen.findAllByTestId('performance-widget-title')).toHaveLength(3);
- expect(eventStatsMock).toHaveBeenNthCalledWith(
- 1,
- expect.anything(),
- expect.objectContaining({
- query: expect.objectContaining({
- ...BASIC_QUERY_PARAMS,
- yAxis: ['tpm()', 'failure_rate()', 'user_misery()'],
- }),
- })
- );
- expect(eventStatsMock).toHaveBeenCalledTimes(1);
- expect(await screen.findAllByTestId('widget-state-is-errored')).toHaveLength(3);
- });
- });
|