Handle partial indicator url state

x-pack/plugins/observability/public/pages/slo_edit/components/slo_edit_form.tsx

@@ -62,7 +62,7 @@ export function SloEditForm({ slo }: Props) {
     sloIds: slo?.id ? [slo.id] : undefined,
   });
 
-  const sloFormValuesUrlState = useParseUrlState();
+  const sloFormValuesFromUrlState = useParseUrlState();
   const isAddRuleFlyoutOpen = useAddRuleFlyoutState(isEditMode);
   const [isCreateRuleCheckboxChecked, setIsCreateRuleCheckboxChecked] = useState(true);
 
@@ -73,7 +73,7 @@ export function SloEditForm({ slo }: Props) {
   }, [isEditMode, rules, slo]);
 
   const methods = useForm<CreateSLOForm>({
-    defaultValues: Object.assign({}, SLO_EDIT_FORM_DEFAULT_VALUES, sloFormValuesUrlState),
+    defaultValues: Object.assign({}, SLO_EDIT_FORM_DEFAULT_VALUES, sloFormValuesFromUrlState),
     values: transformSloResponseToCreateSloForm(slo),
     mode: 'all',
   });

x-pack/plugins/observability/public/pages/slo_edit/helpers/process_slo_form_values.test.ts

@@ -0,0 +1,152 @@
+/*
+ * Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one
+ * or more contributor license agreements. Licensed under the Elastic License
+ * 2.0; you may not use this file except in compliance with the Elastic License
+ * 2.0.
+ */
+
+import { transformPartialUrlStateToFormState as transform } from './process_slo_form_values';
+
+describe('Transform Partial URL State into partial State Form', () => {
+  describe('indicators', () => {
+    it('returns null when no indicator type is specified', () => {
+      expect(transform({ indicator: { params: { index: 'my-index' } } })).toEqual(null);
+    });
+
+    it('handles partial APM Availability state', () => {
+      expect(
+        transform({
+          indicator: {
+            type: 'sli.apm.transactionErrorRate',
+            params: {
+              service: 'override-service',
+            },
+          },
+        })
+      ).toEqual({
+        indicator: {
+          type: 'sli.apm.transactionErrorRate',
+          params: {
+            service: 'override-service',
+            environment: '',
+            filter: '',
+            index: '',
+            transactionName: '',
+            transactionType: '',
+          },
+        },
+      });
+    });
+
+    it('handles partial APM Latency state', () => {
+      expect(
+        transform({
+          indicator: {
+            type: 'sli.apm.transactionDuration',
+            params: {
+              service: 'override-service',
+            },
+          },
+        })
+      ).toEqual({
+        indicator: {
+          type: 'sli.apm.transactionDuration',
+          params: {
+            service: 'override-service',
+            environment: '',
+            filter: '',
+            index: '',
+            transactionName: '',
+            transactionType: '',
+            threshold: 250,
+          },
+        },
+      });
+    });
+
+    it('handles partial Custom KQL state', () => {
+      expect(
+        transform({
+          indicator: {
+            type: 'sli.kql.custom',
+            params: {
+              good: "some.override.filter:'foo'",
+              index: 'override-index',
+            },
+          },
+        })
+      ).toEqual({
+        indicator: {
+          type: 'sli.kql.custom',
+          params: {
+            index: 'override-index',
+            timestampField: '',
+            filter: '',
+            good: "some.override.filter:'foo'",
+            total: '',
+          },
+        },
+      });
+    });
+
+    it('handles partial Custom Metric state', () => {
+      expect(
+        transform({
+          indicator: {
+            type: 'sli.metric.custom',
+            params: {
+              index: 'override-index',
+            },
+          },
+        })
+      ).toEqual({
+        indicator: {
+          type: 'sli.metric.custom',
+          params: {
+            index: 'override-index',
+            filter: '',
+            timestampField: '',
+            good: {
+              equation: 'A',
+              metrics: [{ aggregation: 'sum', field: '', name: 'A' }],
+            },
+            total: {
+              equation: 'A',
+              metrics: [{ aggregation: 'sum', field: '', name: 'A' }],
+            },
+          },
+        },
+      });
+    });
+
+    it('handles partial Custom Histogram state', () => {
+      expect(
+        transform({
+          indicator: {
+            type: 'sli.histogram.custom',
+            params: {
+              index: 'override-index',
+            },
+          },
+        })
+      ).toEqual({
+        indicator: {
+          type: 'sli.histogram.custom',
+          params: {
+            index: 'override-index',
+            filter: '',
+            timestampField: '',
+            good: {
+              aggregation: 'value_count',
+              field: '',
+            },
+            total: {
+              aggregation: 'value_count',
+              field: '',
+            },
+          },
+        },
+      });
+    });
+  });
+});

x-pack/plugins/observability/public/pages/slo_edit/helpers/process_slo_form_values.ts

@@ -5,8 +5,17 @@
  * 2.0.
  */
 
-import { CreateSLOInput, SLOWithSummaryResponse, UpdateSLOInput } from '@kbn/slo-schema';
+import { CreateSLOInput, Indicator, SLOWithSummaryResponse, UpdateSLOInput } from '@kbn/slo-schema';
+import { assertNever } from '@kbn/std';
+import { RecursivePartial } from '@kbn/utility-types';
 import { toDuration } from '../../../utils/slo/duration';
+import {
+  APM_AVAILABILITY_DEFAULT_VALUES,
+  APM_LATENCY_DEFAULT_VALUES,
+  CUSTOM_KQL_DEFAULT_VALUES,
+  CUSTOM_METRIC_DEFAULT_VALUES,
+  HISTOGRAM_DEFAULT_VALUES,
+} from '../constants';
 import { CreateSLOForm } from '../types';
 
 export function transformSloResponseToCreateSloForm(
@@ -91,21 +100,50 @@ export function transformValuesToUpdateSLOInput(values: CreateSLOForm): UpdateSL
   };
 }
 
-export function transformPartialCreateSLOInputToPartialCreateSLOForm(
-  values: Partial<CreateSLOInput>
-): Partial<CreateSLOForm> {
-  return {
-    ...values,
-    ...(values.objective && {
-      objective: {
-        target: values.objective.target * 100,
-        ...(values.objective.timesliceTarget && {
-          timesliceTarget: values.objective.timesliceTarget * 100,
-        }),
-        ...(values.objective.timesliceWindow && {
-          timesliceWindow: String(toDuration(values.objective.timesliceWindow).value),
-        }),
-      },
-    }),
-  };
+function transformPartialIndicatorState(
+  indicator?: RecursivePartial<Indicator>
+): Indicator | undefined {
+  if (indicator === undefined || indicator.type === undefined) return undefined;
+
+  const indicatorType = indicator.type;
+  switch (indicatorType) {
+    case 'sli.apm.transactionDuration':
+      return {
+        type: 'sli.apm.transactionDuration' as const,
+        params: Object.assign({}, APM_LATENCY_DEFAULT_VALUES.params, indicator.params ?? {}),
+      };
+    case 'sli.apm.transactionErrorRate':
+      return {
+        type: 'sli.apm.transactionErrorRate' as const,
+        params: Object.assign({}, APM_AVAILABILITY_DEFAULT_VALUES.params, indicator.params ?? {}),
+      };
+    case 'sli.histogram.custom':
+      return {
+        type: 'sli.histogram.custom' as const,
+        params: Object.assign({}, HISTOGRAM_DEFAULT_VALUES.params, indicator.params ?? {}),
+      };
+    case 'sli.kql.custom':
+      return {
+        type: 'sli.kql.custom' as const,
+        params: Object.assign({}, CUSTOM_KQL_DEFAULT_VALUES.params, indicator.params ?? {}),
+      };
+    case 'sli.metric.custom':
+      return {
+        type: 'sli.metric.custom' as const,
+        params: Object.assign({}, CUSTOM_METRIC_DEFAULT_VALUES.params, indicator.params ?? {}),
+      };
+    default:
+      assertNever(indicatorType);
+  }
+}
+
+export function transformPartialUrlStateToFormState(
+  values: RecursivePartial<Pick<CreateSLOInput, 'indicator'>>
+): Partial<CreateSLOForm> | {} {
+  const state: Partial<CreateSLOForm> = {};
+
+  const parsedIndicator = transformPartialIndicatorState(values.indicator);
+  if (parsedIndicator !== undefined) state.indicator = parsedIndicator;
+
+  return state;
 }

x-pack/plugins/observability/public/pages/slo_edit/hooks/use_parse_url_state.ts

@@ -7,8 +7,9 @@
 
 import { createKbnUrlStateStorage } from '@kbn/kibana-utils-plugin/public';
 import { CreateSLOInput } from '@kbn/slo-schema';
+import { RecursivePartial } from '@kbn/utility-types';
 import { useHistory } from 'react-router-dom';
-import { transformPartialCreateSLOInputToPartialCreateSLOForm } from '../helpers/process_slo_form_values';
+import { transformPartialUrlStateToFormState } from '../helpers/process_slo_form_values';
 import { CreateSLOForm } from '../types';
 
 export function useParseUrlState(): Partial<CreateSLOForm> | null {
@@ -19,7 +20,7 @@ export function useParseUrlState(): Partial<CreateSLOForm> | null {
     useHashQuery: false,
   });
 
-  const urlParams = urlStateStorage.get<Partial<CreateSLOInput>>('_a');
+  const urlParams = urlStateStorage.get<RecursivePartial<CreateSLOInput>>('_a');
 
-  return !!urlParams ? transformPartialCreateSLOInputToPartialCreateSLOForm(urlParams) : null;
+  return !!urlParams ? transformPartialUrlStateToFormState(urlParams) : null;
 }