Skip to content

Commit

Permalink
test pipelines tab content
Browse files Browse the repository at this point in the history
  • Loading branch information
darnautov committed Feb 11, 2022
1 parent d9df7aa commit fddd690
Show file tree
Hide file tree
Showing 3 changed files with 25 additions and 5 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@ export const ModelPipelines: FC<ModelPipelinesProps> = ({ pipelines, ingestStats
extraAction={
pipelineDefinition ? (
<EuiButtonEmpty
data-test-subj={`mlTrainedModelPipelineEditButton_${pipelineName}`}
onClick={() => {
const locator = share.url.locators.get('INGEST_PIPELINES_APP_LOCATOR');
if (!locator) return;
Expand All @@ -81,7 +82,7 @@ export const ModelPipelines: FC<ModelPipelinesProps> = ({ pipelines, ingestStats
>
<EuiFlexGrid columns={2}>
{ingestStats?.pipelines ? (
<EuiFlexItem>
<EuiFlexItem data-test-subj={`mlTrainedModelPipelineIngestStats_${pipelineName}`}>
<EuiPanel>
<EuiTitle size={'xxs'}>
<h6>
Expand All @@ -98,7 +99,7 @@ export const ModelPipelines: FC<ModelPipelinesProps> = ({ pipelines, ingestStats
) : null}

{pipelineDefinition ? (
<EuiFlexItem>
<EuiFlexItem data-test-subj={`mlTrainedModelPipelineDefinition_${pipelineName}`}>
<EuiPanel>
<EuiTitle size={'xxs'}>
<h6>
Expand Down
8 changes: 6 additions & 2 deletions x-pack/test/functional/apps/ml/model_management/model_list.ts
Original file line number Diff line number Diff line change
Expand Up @@ -69,7 +69,9 @@ export default function ({ getService }: FtrProviderContext) {
await ml.trainedModelsTable.assertDetailsTabContent();
await ml.trainedModelsTable.assertInferenceConfigTabContent();
await ml.trainedModelsTable.assertStatsTabContent();
await ml.trainedModelsTable.assertPipelinesTabContent();
await ml.trainedModelsTable.assertPipelinesTabContent(true, [
{ pipelineName: `pipeline_${modelWithPipelineData.modelId}`, expectDefinition: true },
]);
});

it('renders expanded row content correctly for model without pipelines', async () => {
Expand Down Expand Up @@ -182,7 +184,9 @@ export default function ({ getService }: FtrProviderContext) {
await ml.trainedModelsTable.assertDetailsTabContent();
await ml.trainedModelsTable.assertInferenceConfigTabContent();
await ml.trainedModelsTable.assertStatsTabContent();
await ml.trainedModelsTable.assertPipelinesTabContent(false);
await ml.trainedModelsTable.assertPipelinesTabContent(true, [
{ pipelineName: `pipeline_${modelWithPipelineData.modelId}`, expectDefinition: false },
]);
});
});
});
Expand Down
17 changes: 16 additions & 1 deletion x-pack/test/functional/services/ml/trained_models_table.ts
Original file line number Diff line number Diff line change
Expand Up @@ -249,8 +249,23 @@ export function TrainedModelsTableProvider({ getService }: FtrProviderContext) {
await this.assertTabContent('stats', expectVisible);
}

public async assertPipelinesTabContent(expectVisible = true) {
public async assertPipelinesTabContent(
expectVisible = true,
pipelinesExpectOptions?: Array<{ pipelineName: string; expectDefinition: boolean }>
) {
await this.assertTabContent('pipelines', expectVisible);

if (Array.isArray(pipelinesExpectOptions)) {
for (const p of pipelinesExpectOptions) {
if (p.expectDefinition) {
await testSubjects.existOrFail(`mlTrainedModelPipelineEditButton_${p.pipelineName}`);
await testSubjects.existOrFail(`mlTrainedModelPipelineDefinition_${p.pipelineName}`);
} else {
await testSubjects.missingOrFail(`mlTrainedModelPipelineEditButton_${p.pipelineName}`);
await testSubjects.missingOrFail(`mlTrainedModelPipelineDefinition_${p.pipelineName}`);
}
}
}
}
})();
}

0 comments on commit fddd690

Please sign in to comment.