From 1a25f33c000c148bc13e5023a3b41b30f90ea007 Mon Sep 17 00:00:00 2001 From: Eric Briscoe Date: Wed, 9 Nov 2022 12:18:55 -0800 Subject: [PATCH] Adding comment in test file to clarify appendDataToMenu test purpose based on PR question --- .../Table/cell-renderers/ActionCell/ActionCell.test.tsx | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/superset-frontend/src/components/Table/cell-renderers/ActionCell/ActionCell.test.tsx b/superset-frontend/src/components/Table/cell-renderers/ActionCell/ActionCell.test.tsx index f55932d9e86c..5da7453aa9d7 100644 --- a/superset-frontend/src/components/Table/cell-renderers/ActionCell/ActionCell.test.tsx +++ b/superset-frontend/src/components/Table/cell-renderers/ActionCell/ActionCell.test.tsx @@ -39,6 +39,10 @@ test('renders with default props', async () => { expect(clickHandler).toHaveBeenCalled(); }); +/** + * Validate that the appendDataToMenu utility function used within the + * Action cell menu rendering works as expected + */ test('appendDataToMenu utility', () => { exampleMenuOptions.forEach(item => expect(item?.row).toBeUndefined()); const modifiedMenuOptions = appendDataToMenu(exampleMenuOptions, exampleRow);