Skip to content

Commit

Permalink
[ML] Adds isRequired where applicable to timeseries_chart props. (ela…
Browse files Browse the repository at this point in the history
…stic#26880)

Adds isRequired to timeseries_chart props to match the minimum required props necessary to render the component without errors reflected in the Minimal initialization test.
  • Loading branch information
walterra committed Dec 11, 2018
1 parent bd110ce commit 8113aa4
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 7 deletions.
14 changes: 7 additions & 7 deletions x-pack/plugins/ml/public/timeseriesexplorer/timeseries_chart.js
Original file line number Diff line number Diff line change
Expand Up @@ -79,19 +79,19 @@ export class TimeseriesChart extends React.Component {
contextAggregationInterval: PropTypes.object,
contextChartData: PropTypes.array,
contextForecastData: PropTypes.array,
contextChartSelected: PropTypes.func,
contextChartSelected: PropTypes.func.isRequired,
detectorIndex: PropTypes.string,
focusAggregationInterval: PropTypes.object,
focusChartData: PropTypes.array,
focusForecastData: PropTypes.array,
modelPlotEnabled: PropTypes.bool,
renderFocusChartOnly: PropTypes.bool,
modelPlotEnabled: PropTypes.bool.isRequired,
renderFocusChartOnly: PropTypes.bool.isRequired,
selectedJob: PropTypes.object,
showForecast: PropTypes.bool,
showModelBounds: PropTypes.bool,
svgWidth: PropTypes.number,
showForecast: PropTypes.bool.isRequired,
showModelBounds: PropTypes.bool.isRequired,
svgWidth: PropTypes.number.isRequired,
swimlaneData: PropTypes.array,
timefilter: PropTypes.object,
timefilter: PropTypes.object.isRequired,
zoomFrom: PropTypes.object,
zoomTo: PropTypes.object
}
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,6 +37,7 @@ function getTimeseriesChartPropsMock() {
return {
contextChartSelected: jest.fn(),
modelPlotEnabled: false,
renderFocusChartOnly: false,
showForecast: true,
showModelBounds: true,
svgWidth: 1600,
Expand Down

0 comments on commit 8113aa4

Please sign in to comment.