Check for empty document in the finding flyout (opensearch-project#841)

* check for empty document Signed-off-by: Amardeepsingh Siglani <[email protected]> * added try catch to avoid page crash for invalid document Signed-off-by: Amardeepsingh Siglani <[email protected]> * added cypress test Signed-off-by: Amardeepsingh Siglani <[email protected]> --------- Signed-off-by: Amardeepsingh Siglani <[email protected]>
riysaxen-amzn · Dec 13, 2023 · 7837a99 · 7837a99
1 parent 7788423
commit 7837a99
Show file tree

Hide file tree

Showing 2 changed files with 44 additions and 5 deletions.
diff --git a/cypress/integration/4_findings.spec.js b/cypress/integration/4_findings.spec.js
@@ -144,9 +144,6 @@ describe('Findings', () => {
     });
   });
 
-  // TODO - upon reaching rules page, trigger appropriate rules detail flyout
-  // see github issue #124 at https://github.com/opensearch-project/security-analytics-dashboards-plugin/issues/124
-
   it('opens rule details flyout when rule name inside accordion drop down is clicked', () => {
     // filter table to show only sample_detector findings
     cy.get(`input[placeholder="Search findings"]`).ospSearch(indexName);
@@ -167,5 +164,26 @@ describe('Findings', () => {
     });
   });
 
+  it('shows document not found warning when the document is empty', () => {
+    cy.deleteIndex(indexName);
+    cy.reload();
+
+    // Wait for page to load
+    cy.waitForPageLoad('findings', {
+      contains: 'Findings',
+    });
+
+    // filter table to show only sample_detector findings
+    cy.get(`input[placeholder="Search findings"]`).ospSearch(indexName);
+
+    // open Finding details flyout via finding id link. cy.wait essential, timeout insufficient.
+    cy.getTableFirstRow('[data-test-subj="view-details-icon"]').then(($el) => {
+      cy.get($el).click({ force: true });
+    });
+
+    // Flyout should show 'Document not found' warning
+    cy.contains('Document not found');
+  });
+
   after(() => cy.cleanUpTests());
 });
diff --git a/public/pages/Findings/components/FindingDetailsFlyout.tsx b/public/pages/Findings/components/FindingDetailsFlyout.tsx
@@ -31,6 +31,7 @@ import {
   EuiTabs,
   EuiTab,
   EuiLoadingContent,
+  EuiEmptyPrompt,
 } from '@elastic/eui';
 import {
   capitalizeFirstLetter,
@@ -306,9 +307,16 @@ export default class FindingDetailsFlyout extends Component<
     const docId = related_doc_ids[0];
     const matchedDocuments = documents.filter((doc) => doc.id === docId);
     const document = matchedDocuments.length > 0 ? matchedDocuments[0].document : '';
+    let formattedDocument = '';
+    try {
+      formattedDocument = document ? JSON.stringify(JSON.parse(document), null, 2) : '';
+    } catch {
+      // no-op
+    }
+
     const { indexPatternId } = this.state;
 
-    return (
+    return document ? (
       <>
         <EuiFlexGroup justifyContent="spaceBetween">
           <EuiFlexItem>
@@ -367,10 +375,23 @@ export default class FindingDetailsFlyout extends Component<
             isCopyable
             data-test-subj={'finding-details-flyout-rule-document'}
           >
-            {JSON.stringify(JSON.parse(document), null, 2)}
+            {formattedDocument}
           </EuiCodeBlock>
         </EuiFormRow>
       </>
+    ) : (
+      <>
+        <EuiTitle size={'s'}>
+          <h3>Documents</h3>
+        </EuiTitle>
+        <EuiSpacer />
+        <EuiEmptyPrompt
+          iconType="alert"
+          iconColor="danger"
+          title={<h2>Document not found</h2>}
+          body={<p>The document that generated this finding could not be loaded.</p>}
+        />
+      </>
     );
   }