Return undefined if testId if not defined

toptal · Dec 7, 2023 · 43e585e · 43e585e
1 parent db5753c
commit 43e585e
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/packages/picasso-query-builder/src/ValidationErrors/ValidationErrors.tsx b/packages/picasso-query-builder/src/ValidationErrors/ValidationErrors.tsx
@@ -41,8 +41,11 @@ const ValidationErrors = ({
           return (
             <Fragment key={rule}>
               {reasons?.map((reason, index) => (
-                // eslint-disable-next-line react/no-array-index-key
-                <List.Item data-testid={`${testId}-${index}`} key={index}>
+                <List.Item
+                  // eslint-disable-next-line react/no-array-index-key
+                  key={index}
+                  data-testid={testId && `${testId}-${index}`}
+                >
                   {reason.message ?? reason}
                 </List.Item>
               ))}