[ML] Improve functional tests.

elastic · Jan 20, 2022 · 61ed8e4 · 61ed8e4
1 parent a839fe2
commit 61ed8e4
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/...ions/transform_management/components/edit_transform_flyout/edit_transform_flyout_form.tsx b/...ions/transform_management/components/edit_transform_flyout/edit_transform_flyout_form.tsx
@@ -55,7 +55,9 @@ export const EditTransformFlyoutForm: FC<EditTransformFlyoutFormProps> = ({
   );
 
   const retentionDateFieldOptions = useMemo(() => {
-    return Array.isArray(dateFieldNames) ? dateFieldNames.map((text: string) => ({ text })) : [];
+    return Array.isArray(dateFieldNames)
+      ? dateFieldNames.map((text: string) => ({ text, value: text }))
+      : [];
   }, [dateFieldNames]);
 
   return (

diff --git a/x-pack/test/functional/apps/transform/editing.ts b/x-pack/test/functional/apps/transform/editing.ts
@@ -51,6 +51,8 @@ export default function ({ getService }: FtrProviderContext) {
         transformDescription: 'updated description',
         transformDocsPerSecond: '1000',
         transformFrequency: '10m',
+        transformRetentionPolicyField: 'order_date',
+        transformRetentionPolicyMaxAge: '1d',
         expected: {
           messageText: 'updated transform.',
           retentionPolicy: {
@@ -71,6 +73,8 @@ export default function ({ getService }: FtrProviderContext) {
         transformDescription: 'updated description',
         transformDocsPerSecond: '1000',
         transformFrequency: '10m',
+        transformRetentionPolicyField: 'order_date',
+        transformRetentionPolicyMaxAge: '1d',
         expected: {
           messageText: 'updated transform.',
           retentionPolicy: {
@@ -154,6 +158,9 @@ export default function ({ getService }: FtrProviderContext) {
           await transform.editFlyout.assertTransformEditFlyoutRetentionPolicyFieldSelectValue(
             testData.expected.retentionPolicy.field
           );
+          await transform.editFlyout.setTransformEditFlyoutRetentionPolicyFieldSelectValue(
+            testData.transformRetentionPolicyField
+          );
 
           await transform.editFlyout.assertTransformEditFlyoutInputEnabled(
             'RetentionPolicyMaxAge',
@@ -163,6 +170,10 @@ export default function ({ getService }: FtrProviderContext) {
             'RetentionPolicyMaxAge',
             testData.expected.retentionPolicy.maxAge
           );
+          await transform.editFlyout.setTransformEditFlyoutInputValue(
+            'RetentionPolicyMaxAge',
+            testData.transformRetentionPolicyMaxAge
+          );
         });
 
         it('updates the transform and displays it correctly in the job list', async () => {

diff --git a/x-pack/test/functional/services/transform/edit_flyout.ts b/x-pack/test/functional/services/transform/edit_flyout.ts
@@ -62,7 +62,7 @@ export function TransformEditFlyoutProvider({ getService }: FtrProviderContext)
       );
       expect(actualValue).to.eql(
         expectedValue,
-        `Detector input option value should be '${expectedValue}' (got '${actualValue}')`
+        `Retention policy field option value should be '${expectedValue}' (got '${actualValue}')`
       );
     },