fix missing use case

elastic · May 3, 2023 · 7e22704 · 7e22704
1 parent 2ff49bf
commit 7e22704
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 0 deletions.
diff --git a/...ions_ui/public/application/sections/alerts_table/hooks/use_columns/toggle_column.test.tsx b/...ions_ui/public/application/sections/alerts_table/hooks/use_columns/toggle_column.test.tsx
@@ -47,4 +47,21 @@ describe('toggleColumn', () => {
  ]
  `);
  });
+
+ it('adds a column even if no column is currently shown', async () => {
+ expect(
+ toggleColumn({
+ column: { id: '_id', schema: 'string' },
+ columns: [],
+ defaultColumns: [],
+ })
+ ).toMatchInlineSnapshot(`
+ Array [
+ Object {
+ "id": "_id",
+ "schema": "string",
+ },
+ ]
+ `);
+ });
 });
diff --git a/...rs_actions_ui/public/application/sections/alerts_table/hooks/use_columns/toggle_column.ts b/...rs_actions_ui/public/application/sections/alerts_table/hooks/use_columns/toggle_column.ts
@@ -33,6 +33,10 @@ const insert = ({
  return [...columns.slice(0, defaultIndex), column, ...columns.slice(defaultIndex)];
  }
 
+ if (columns.length === 0) {
+ return [column];
+ }
+
  // if the column isn't shown and it's not part of the default config
  // push it into the second position. Behaviour copied by t_grid, security
  // does this to insert right after the timestamp column