Skip to content

Commit

Permalink
Merge pull request #9314 from vikram-raj/bz-1972011
Browse files Browse the repository at this point in the history
Bug 1972011: Fix time range for drag & drop in devconsole monitoring
  • Loading branch information
openshift-merge-robot committed Jul 14, 2021
2 parents c3bc8ad + 900c36c commit fbb4361
Show file tree
Hide file tree
Showing 2 changed files with 22 additions and 0 deletions.
Original file line number Diff line number Diff line change
@@ -1,5 +1,12 @@
import * as React from 'react';
import { useTranslation } from 'react-i18next';
// eslint-disable-next-line @typescript-eslint/ban-ts-ignore
// @ts-ignore
import { useDispatch } from 'react-redux';
import {
monitoringDashboardsSetEndTime,
monitoringDashboardsSetTimespan,
} from '@console/internal/actions/ui';
import { PrometheusGraphLink } from '@console/internal/components/graphs/prometheus-graph';
import { QueryBrowser } from '@console/internal/components/monitoring/query-browser';
import { Humanize } from '@console/internal/components/utils';
Expand Down Expand Up @@ -40,6 +47,14 @@ export const MonitoringDashboardGraph: React.FC<MonitoringDashboardGraphProps> =
endTime,
}) => {
const { t } = useTranslation();
const dispatch = useDispatch();
const onZoom = React.useCallback(
(from, to) => {
dispatch(monitoringDashboardsSetEndTime(to));
dispatch(monitoringDashboardsSetTimespan(to - from));
},
[dispatch],
);
return (
<DashboardCard className="monitoring-dashboards__card odc-monitoring-dashboard-graph">
<DashboardCardHeader>
Expand All @@ -63,6 +78,7 @@ export const MonitoringDashboardGraph: React.FC<MonitoringDashboardGraphProps> =
pollInterval={pollInterval}
fixedEndTime={endTime}
formatSeriesTitle={(labels) => labels.pod}
onZoom={onZoom}
showLegend
/>
</PrometheusGraphLink>
Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import * as React from 'react';
import { shallow } from 'enzyme';
import { TFunction } from 'i18next';
import * as redux from 'react-redux';
import { PrometheusGraphLink } from '@console/internal/components/graphs/prometheus-graph';
import { QueryBrowser } from '@console/internal/components/monitoring/query-browser';
import { monitoringDashboardQueries } from '../../queries';
Expand All @@ -17,6 +18,11 @@ jest.mock('react-i18next', () => {
});

describe('Monitoring Dashboard graph', () => {
// FIXME upgrading redux types is causing many errors at this time
// eslint-disable-next-line @typescript-eslint/ban-ts-ignore
// @ts-ignore
const spyDispatch = jest.spyOn(redux, 'useDispatch');
spyDispatch.mockReturnValue(() => {});
let monitoringDashboardGraphProps: React.ComponentProps<typeof MonitoringDashboardGraph>;

beforeAll(() => {
Expand Down

0 comments on commit fbb4361

Please sign in to comment.