Azure Monitor: Select all Event Types by default (#77603)
This commit is contained in:
+1
-2
@@ -24,11 +24,10 @@ describe('TraceTypeField', () => {
|
|||||||
...props.query,
|
...props.query,
|
||||||
azureTraces: {
|
azureTraces: {
|
||||||
...props.query.azureTraces,
|
...props.query.azureTraces,
|
||||||
traceTypes: [],
|
|
||||||
},
|
},
|
||||||
};
|
};
|
||||||
|
|
||||||
render(<TraceTypeField {...props} query={query} />);
|
render(<TraceTypeField {...props} query={query} />);
|
||||||
expect(screen.getByText('Choose event types')).toBeInTheDocument();
|
|
||||||
|
|
||||||
const menu = screen.getByLabelText(selectors.components.queryEditor.tracesQueryEditor.traceTypes.select);
|
const menu = screen.getByLabelText(selectors.components.queryEditor.tracesQueryEditor.traceTypes.select);
|
||||||
openMenu(menu);
|
openMenu(menu);
|
||||||
|
|||||||
+12
-1
@@ -30,12 +30,23 @@ const TraceTypeField = ({ query, variableOptionGroup, onQueryChange }: AzureQuer
|
|||||||
|
|
||||||
const options = useMemo(() => [...tables, variableOptionGroup], [tables, variableOptionGroup]);
|
const options = useMemo(() => [...tables, variableOptionGroup], [tables, variableOptionGroup]);
|
||||||
|
|
||||||
|
// Select all trace event ypes by default
|
||||||
|
const getDefaultOptions = () => {
|
||||||
|
const allEventTypes = tables.map((t) => t.value);
|
||||||
|
const defaultQuery = setTraceTypes(query, allEventTypes);
|
||||||
|
onQueryChange(defaultQuery);
|
||||||
|
return allEventTypes;
|
||||||
|
};
|
||||||
|
|
||||||
return (
|
return (
|
||||||
<Field label="Event Type">
|
<Field label="Event Type">
|
||||||
<MultiSelect
|
<MultiSelect
|
||||||
placeholder="Choose event types"
|
placeholder="Choose event types"
|
||||||
inputId="azure-monitor-traces-type-field"
|
inputId="azure-monitor-traces-type-field"
|
||||||
value={findOptions([...tables, ...variableOptionGroup.options], query.azureTraces?.traceTypes ?? [])}
|
value={findOptions(
|
||||||
|
[...tables, ...variableOptionGroup.options],
|
||||||
|
query.azureTraces?.traceTypes ?? getDefaultOptions()
|
||||||
|
)}
|
||||||
onChange={handleChange}
|
onChange={handleChange}
|
||||||
options={options}
|
options={options}
|
||||||
allowCustomValue
|
allowCustomValue
|
||||||
|
|||||||
Reference in New Issue
Block a user