add test ids (#5072) (#5078)

(cherry picked from commit 9c192a3) Signed-off-by: abbyhu2000 <abigailhu2000@gmail.com> Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com> Co-authored-by: Josh Romero <rmerqg@amazon.com>
opensearch-project · Sep 26, 2023 · 5a049c9 · 5a049c9
1 parent 42db9b5
commit 5a049c9
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 2 deletions.
diff --git a/...scover/public/application/components/data_grid/data_grid_table_docview_inspect_button.tsx b/...scover/public/application/components/data_grid/data_grid_table_docview_inspect_button.tsx
@@ -22,6 +22,7 @@ export const DocViewInspectButton = ({ rowIndex }: EuiDataGridCellValueElementPr
         onClick={() => setInspectedHit(isCurrentInspected ? undefined : currentInspected)}
         iconType={isCurrentInspected ? 'minimize' : 'inspect'}
         aria-label={inspectHintMsg}
+        data-test-subj={`docTableExpandToggleColumn-${rowIndex}`}
       />
     </EuiToolTip>
   );

diff --git a/src/plugins/discover/public/application/components/data_grid/data_grid_table_flyout.tsx b/src/plugins/discover/public/application/components/data_grid/data_grid_table_flyout.tsx
@@ -40,7 +40,7 @@ export function DataGridFlyout({
   // TODO: replace EuiLink with doc_view_links registry
   // TODO: Also move the flyout higher in the react tree to prevent redrawing the table component and slowing down page performance
   return (
-    <EuiFlyout onClose={onClose} size="m">
+    <EuiFlyout onClose={onClose} size="m" data-test-subj="documentDetailFlyOut">
       <EuiFlyoutHeader>
         <EuiTitle>
           <h2>Document Details</h2>

diff --git a/src/plugins/discover/public/application/components/doc_viewer_links/doc_viewer_links.tsx b/src/plugins/discover/public/application/components/doc_viewer_links/doc_viewer_links.tsx
@@ -27,7 +27,12 @@ export function DocViewerLinks(renderProps: DocViewLinkRenderProps) {
     <EuiFlexGroup gutterSize="m" justifyContent="flexEnd">
       {listItems.map((item, index) => (
         <EuiFlexItem key={index} grow={false}>
-          <EuiLink href={item.href} target="_blank" style={{ fontWeight: 'normal' }}>
+          <EuiLink
+            href={item.href}
+            target="_blank"
+            style={{ fontWeight: 'normal' }}
+            data-test-subj={item['data-test-subj']}
+          >
             {item.label}
           </EuiLink>
         </EuiFlexItem>