Skip to content

Commit

Permalink
Rename to assertions
Browse files Browse the repository at this point in the history
  • Loading branch information
qn895 committed Feb 14, 2023
1 parent 27323ff commit 10d9917
Show file tree
Hide file tree
Showing 8 changed files with 38 additions and 24 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -158,7 +158,7 @@ export default function ({ getService }: FtrProviderContext) {
for (const { fieldName, type: fieldType } of testData.fieldStatsEntries.filter(
(e) => e.isDependentVariableInput
)) {
await ml.dataFrameAnalyticsCreation.clickDependentVariableInputFieldStatTrigger(
await ml.dataFrameAnalyticsCreation.assertFieldStatsFlyoutContentFromDependentVariableInputTrigger(
fieldName,
fieldType
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -330,7 +330,7 @@ export default function ({ getService }: FtrProviderContext) {
for (const { fieldName, type: fieldType } of fieldStatsEntries.filter(
(e) => e.isDependentVariableInput
)) {
await ml.dataFrameAnalyticsCreation.clickDependentVariableInputFieldStatTrigger(
await ml.dataFrameAnalyticsCreation.assertFieldStatsFlyoutContentFromDependentVariableInputTrigger(
fieldName,
fieldType
);
Expand Down Expand Up @@ -363,7 +363,7 @@ export default function ({ getService }: FtrProviderContext) {
for (const { fieldName, type: fieldType } of fieldStatsEntries.filter(
(e) => e.isIncludeFieldInput
)) {
await ml.dataFrameAnalyticsCreation.clickIncludeFieldsInputFieldStatTrigger(
await ml.dataFrameAnalyticsCreation.assertFieldStatFlyoutContentFromIncludeFieldTrigger(
fieldName,
fieldType
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -184,7 +184,7 @@ export default function ({ getService }: FtrProviderContext) {
for (const { fieldName, type: fieldType } of fieldStatsEntries.filter(
(e) => e.isIncludeFieldInput
)) {
await ml.dataFrameAnalyticsCreation.clickIncludeFieldsInputFieldStatTrigger(
await ml.dataFrameAnalyticsCreation.assertFieldStatFlyoutContentFromIncludeFieldTrigger(
fieldName,
fieldType
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -155,7 +155,7 @@ export default function ({ getService }: FtrProviderContext) {
for (const { fieldName, type: fieldType } of fieldStatsEntries.filter(
(e) => e.isDependentVariableInput
)) {
await ml.dataFrameAnalyticsCreation.clickDependentVariableInputFieldStatTrigger(
await ml.dataFrameAnalyticsCreation.assertFieldStatsFlyoutContentFromDependentVariableInputTrigger(
fieldName,
fieldType
);
Expand All @@ -178,7 +178,7 @@ export default function ({ getService }: FtrProviderContext) {
for (const { fieldName, type: fieldType } of fieldStatsEntries.filter(
(e) => e.isIncludeFieldInput
)) {
await ml.dataFrameAnalyticsCreation.clickIncludeFieldsInputFieldStatTrigger(
await ml.dataFrameAnalyticsCreation.assertFieldStatFlyoutContentFromIncludeFieldTrigger(
fieldName,
fieldType
);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -606,7 +606,10 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
: [];

for (const { fieldName, type } of groupByFieldStatsEntries) {
await transform.wizard.clickGroupByInputFieldStatTrigger(fieldName, type);
await transform.wizard.assertFieldStatFlyoutContentFromGroupByInputTrigger(
fieldName,
type
);
}

for (const [index, entry] of testData.groupByEntries.entries()) {
Expand All @@ -626,7 +629,10 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
: [];

for (const { fieldName, type } of aggInputFieldStatsEntries) {
await transform.wizard.clickAggInputFieldStatTrigger(fieldName, type);
await transform.wizard.assertFieldStatFlyoutContentFromAggInputTrigger(
fieldName,
type
);
}

await transform.testExecution.logTestStep('adds the aggregation entries');
Expand Down Expand Up @@ -655,7 +661,10 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
: [];

for (const { fieldName, type } of uniqueKeyInputFieldStatsEntries) {
await transform.wizard.clickUniqueKeysInputFieldStatTrigger(fieldName, type);
await transform.wizard.assertFieldStatsFlyoutContentFromUniqueKeysInputTrigger(
fieldName,
type
);
}

await transform.testExecution.logTestStep('adds unique keys');
Expand All @@ -673,7 +682,10 @@ export default function ({ getService, getPageObjects }: FtrProviderContext) {
: [];

for (const { fieldName, type } of sortFieldInputFieldStatsEntries) {
await transform.wizard.clickSortFieldInputFieldStatTrigger(fieldName, type);
await transform.wizard.assertFieldStatFlyoutContentFromSortFieldInputTrigger(
fieldName,
type
);
}

await transform.testExecution.logTestStep('sets the sort field');
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -204,11 +204,11 @@ export function MachineLearningDataFrameAnalyticsCreationProvider(
});
},

async clickIncludeFieldsInputFieldStatTrigger(
async assertFieldStatFlyoutContentFromIncludeFieldTrigger(
fieldName: string,
fieldType?: 'keyword' | 'date' | 'number'
) {
await mlFieldStatsFlyout.clickFieldStatTrigger(
await mlFieldStatsFlyout.assertFieldStatFlyoutContentFromTrigger(
'mlAnalyticsCreateJobWizardIncludesSelect',
fieldName,
fieldType
Expand Down Expand Up @@ -268,11 +268,11 @@ export function MachineLearningDataFrameAnalyticsCreationProvider(
});
},

async clickDependentVariableInputFieldStatTrigger(
async assertFieldStatsFlyoutContentFromDependentVariableInputTrigger(
fieldName: string,
fieldType?: 'keyword' | 'date' | 'number'
) {
await mlFieldStatsFlyout.clickFieldStatComboBoxTrigger(
await mlFieldStatsFlyout.assertFieldStatFlyoutContentFromComboBoxTrigger(
'mlAnalyticsCreateJobWizardDependentVariableSelect loaded',
fieldName,
fieldType
Expand Down
6 changes: 4 additions & 2 deletions x-pack/test/functional/services/ml/field_stats_flyout.ts
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ export function MachineLearningFieldStatsFlyoutProvider({ getService }: FtrProvi
});
},

async clickFieldStatTrigger(
async assertFieldStatFlyoutContentFromTrigger(
testSubj: string,
fieldName: string,
fieldType?: 'keyword' | 'date' | 'number'
Expand All @@ -76,8 +76,10 @@ export function MachineLearningFieldStatsFlyoutProvider({ getService }: FtrProvi
await testSubjects.existOrFail(`mlFieldStatsFlyoutContent ${fieldName}-title`);
});
await this.assertFieldStatContentByType(testSubj, fieldName, fieldType);
await this.ensureFieldStatsFlyoutClosed();
},
async clickFieldStatComboBoxTrigger(

async assertFieldStatFlyoutContentFromComboBoxTrigger(
parentComboBoxSelector: string,
fieldName: string,
fieldType?: 'keyword' | 'date' | 'number'
Expand Down
16 changes: 8 additions & 8 deletions x-pack/test/functional/services/transform/wizard.ts
Original file line number Diff line number Diff line change
Expand Up @@ -378,11 +378,11 @@ export function TransformWizardProvider({ getService, getPageObjects }: FtrProvi
await this.assertSelectedTransformFunction(transformFunction);
},

async clickUniqueKeysInputFieldStatTrigger(
async assertFieldStatsFlyoutContentFromUniqueKeysInputTrigger(
fieldName: string,
fieldType?: 'keyword' | 'date' | 'number'
) {
await ml.commonFieldStatsFlyout.clickFieldStatComboBoxTrigger(
await ml.commonFieldStatsFlyout.assertFieldStatFlyoutContentFromComboBoxTrigger(
'transformWizardUniqueKeysSelector',
fieldName,
fieldType
Expand Down Expand Up @@ -416,11 +416,11 @@ export function TransformWizardProvider({ getService, getPageObjects }: FtrProvi
]);
},

async clickSortFieldInputFieldStatTrigger(
async assertFieldStatFlyoutContentFromSortFieldInputTrigger(
fieldName: string,
fieldType?: 'keyword' | 'date' | 'number'
) {
await ml.commonFieldStatsFlyout.clickFieldStatComboBoxTrigger(
await ml.commonFieldStatsFlyout.assertFieldStatFlyoutContentFromComboBoxTrigger(
'transformWizardSortFieldSelector',
fieldName,
fieldType
Expand Down Expand Up @@ -448,11 +448,11 @@ export function TransformWizardProvider({ getService, getPageObjects }: FtrProvi
await this.assertSortFieldInputValue(identificator);
},

async clickGroupByInputFieldStatTrigger(
async assertFieldStatFlyoutContentFromGroupByInputTrigger(
fieldName: string,
fieldType?: 'keyword' | 'date' | 'number'
) {
await ml.commonFieldStatsFlyout.clickFieldStatComboBoxTrigger(
await ml.commonFieldStatsFlyout.assertFieldStatFlyoutContentFromComboBoxTrigger(
'transformGroupBySelection',
fieldName,
fieldType
Expand Down Expand Up @@ -518,11 +518,11 @@ export function TransformWizardProvider({ getService, getPageObjects }: FtrProvi
} > comboBoxInput`;
},

async clickAggInputFieldStatTrigger(
async assertFieldStatFlyoutContentFromAggInputTrigger(
fieldName: string,
fieldType?: 'keyword' | 'date' | 'number'
) {
await ml.commonFieldStatsFlyout.clickFieldStatComboBoxTrigger(
await ml.commonFieldStatsFlyout.assertFieldStatFlyoutContentFromComboBoxTrigger(
'transformAggregationSelection',
fieldName,
fieldType
Expand Down

0 comments on commit 10d9917

Please sign in to comment.