Slight revision to logic and added test for wizard with rollup job af…

…ter index pattern changed

x-pack/legacy/plugins/rollup/__jest__/client_integration/helpers/job_clone.helpers.js

@@ -10,13 +10,13 @@ import { JobCreate } from '../../../public/crud_app/sections';
 import { JOB_TO_CLONE } from './constants';
 import { deserializeJob } from '../../../public/crud_app/services';
 
-const initTestBed = registerTestBed(JobCreate, {
-  store: createRollupJobsStore({
-    cloneJob: { job: deserializeJob(JOB_TO_CLONE.jobs[0]) },
-  }),
-});
 
 export const setup = props => {
+  const initTestBed = registerTestBed(JobCreate, {
+    store: createRollupJobsStore({
+      cloneJob: { job: deserializeJob(JOB_TO_CLONE.jobs[0]) },
+    }),
+  });
   const testBed = initTestBed(props);
   const { component } = testBed;
 

x-pack/legacy/plugins/rollup/__jest__/client_integration/job_create_clone.test.js

@@ -4,7 +4,7 @@
  * you may not use this file except in compliance with the Elastic License.
  */
 
-import { setupEnvironment, pageHelpers } from './helpers';
+import { setupEnvironment, pageHelpers, nextTick } from './helpers';
 import { JOB_TO_CLONE, JOB_CLONE_INDEX_PATTERN_CHECK } from './helpers/constants';
 
 jest.mock('ui/new_platform');
@@ -134,4 +134,64 @@ describe('Cloning a rollup job through create job wizard', () => {
       expect(checkedCountActual).toBe(checkedCountExpected);
     });
   });
+
+  it('should correctly reset defaults after index pattern changes', async () => {
+    // 1. Logistics
+
+    // Sanity check for rollup job name, i.e., we are in clone mode.
+    expect(find('rollupJobName').props().value).toBe(jobConfig.id + '-copy');
+
+    // Changing the index pattern value after cloning a rollup job should update a number of values.
+    // On each view of the set up wizard we check for the expected state after this change.
+    form.setInputValue('rollupIndexPattern', 'test');
+    // Fires off a network request.
+    await nextTick();
+
+    const {
+      groups: {
+        date_histogram: dateHistogram
+      },
+    } = jobConfig;
+
+    await actions.clickNextStep();
+
+    // 2. Date Histogram
+
+    expect(exists('rollupJobCreateDateHistogramTitle')).toBe(true);
+    expect(find('rollupJobCreateDateFieldSelect').props().value).toBe(dateHistogram.field);
+
+    await actions.clickNextStep();
+
+    // 3. Terms
+
+    expect(exists('rollupJobCreateTermsTitle')).toBe(true);
+    const { tableCellsValues: tableCellValuesTerms } = table.getMetaData('rollupJobTermsFieldList');
+    expect(tableCellValuesTerms[0][0]).toBe('No terms fields added');
+
+    await actions.clickNextStep();
+
+    // 4. Histogram
+
+    expect(exists('rollupJobCreateHistogramTitle')).toBe(true);
+    const { tableCellsValues: tableCellValuesHisto } = table.getMetaData(
+      'rollupJobHistogramFieldList'
+    );
+
+    expect(tableCellValuesHisto[0][0]).toBe('No histogram fields added');
+
+    await actions.clickNextStep();
+
+    // 5. Metrics
+
+    expect(exists('rollupJobCreateMetricsTitle')).toBe(true);
+    const { rows: metricsRows } = table.getMetaData('rollupJobMetricsFieldList');
+    // Empty placeholder value
+    expect(metricsRows.length).toBe(1);
+
+    // 6. Review
+
+    await actions.clickNextStep();
+
+    expect(exists('rollupJobCreateReviewTitle')).toBe(true);
+  });
 });

x-pack/legacy/plugins/rollup/public/crud_app/sections/job_create/job_create.js

@@ -133,7 +133,6 @@ export class JobCreateUi extends Component {
     const indexPattern = this.getIndexPattern();
     if (indexPattern !== this.getIndexPattern(prevState)) {
 
-
       // If the user hasn't entered anything, then skip validation.
       if (!indexPattern || !indexPattern.trim()) {
         this.setState({
@@ -268,7 +267,6 @@ export class JobCreateUi extends Component {
       indexPatternDateFields.sort();
 
       if (resetDefaults) {
-        const { indexPatternDateFields } = this.state;
         // Whenever the index pattern changes we default to the first date field if there is one.
         this.onFieldsChange(
           {