chore(RHIF-180): Supress data in sentry/hotjar

RedHatInsights · Mar 14, 2023 · 2aad220 · 2aad220
1 parent a82ed4f
commit 2aad220
Show file tree

Hide file tree

Showing 12 changed files with 33 additions and 23 deletions.
diff --git a/src/Utilities/DeleteModal.js b/src/Utilities/DeleteModal.js
@@ -27,7 +27,7 @@ const DeleteModal = ({ handleModalToggle, isModalOpen, currentSytems, onConfirm
     return <Modal
         variant="small"
         title="Remove from inventory"
-        className="ins-c-inventory__table--remove"
+        className="ins-c-inventory__table--remove sentry-mask data-hj-suppress"
         ouiaId="inventory-delete-modal"
         isOpen={isModalOpen}
         onClose={() => handleModalToggle(false)}

diff --git a/src/Utilities/__snapshots__/DeleteModal.test.js.snap b/src/Utilities/__snapshots__/DeleteModal.test.js.snap
@@ -7,7 +7,7 @@ exports[`EntityTable DOM should render correctly 1`] = `
   aria-describedby=""
   aria-label=""
   aria-labelledby=""
-  className="ins-c-inventory__table--remove"
+  className="ins-c-inventory__table--remove sentry-mask data-hj-suppress"
   hasNoBodyWrapper={false}
   isOpen={false}
   onClose={[Function]}
@@ -103,7 +103,7 @@ exports[`EntityTable DOM should render correctly with multiple systems - count 1
   aria-describedby=""
   aria-label=""
   aria-labelledby=""
-  className="ins-c-inventory__table--remove"
+  className="ins-c-inventory__table--remove sentry-mask data-hj-suppress"
   hasNoBodyWrapper={false}
   isOpen={true}
   onClose={[Function]}
@@ -200,7 +200,7 @@ exports[`EntityTable DOM should render correctly with multiple systems - count 2
   aria-describedby=""
   aria-label=""
   aria-labelledby=""
-  className="ins-c-inventory__table--remove"
+  className="ins-c-inventory__table--remove sentry-mask data-hj-suppress"
   hasNoBodyWrapper={false}
   isOpen={true}
   onClose={[Function]}
@@ -297,7 +297,7 @@ exports[`EntityTable DOM should render correctly with one system 1`] = `
   aria-describedby=""
   aria-label=""
   aria-labelledby=""
-  className="ins-c-inventory__table--remove"
+  className="ins-c-inventory__table--remove sentry-mask data-hj-suppress"
   hasNoBodyWrapper={false}
   isOpen={true}
   onClose={[Function]}
@@ -394,7 +394,7 @@ exports[`EntityTable DOM should render correctly with one system 2`] = `
   aria-describedby=""
   aria-label=""
   aria-labelledby=""
-  className="ins-c-inventory__table--remove"
+  className="ins-c-inventory__table--remove sentry-mask data-hj-suppress"
   hasNoBodyWrapper={false}
   isOpen={true}
   onClose={[Function]}

diff --git a/src/components/GeneralInfo/SystemCard/SystemCard.js b/src/components/GeneralInfo/SystemCard/SystemCard.js
@@ -170,6 +170,7 @@ class SystemCardCore extends Component {
                     inputOuiaId="input-edit-display-name"
                     onCancel={ this.onCancel }
                     onSubmit={ this.onSubmit(setDisplayName, entity && entity.display_name) }
+                    className ='sentry-mask data-hj-suppress'
                 />
                 <TextInputModal
                     isOpen={ isAnsibleHostModalOpen }
@@ -182,6 +183,7 @@ class SystemCardCore extends Component {
                     inputOuiaId="input-edit-ansible-name"
                     onCancel={ this.onCancel }
                     onSubmit={ this.onSubmit(setAnsibleHost, entity && this.getAnsibleHost()) }
+                    className ='sentry-mask data-hj-suppress'
                 />
             </Fragment>
         );

diff --git a/src/components/InventoryDetail/TopBar.js b/src/components/InventoryDetail/TopBar.js
@@ -144,7 +144,8 @@ const TopBar = ({
                             variant: 'warning',
                             title: 'Delete operation initiated',
                             description: `Removal of ${entity.display_name} started.`,
-                            dismissable: false
+                            dismissable: false,
+                            className: 'sentry-mask data-hj-suppress'
                         });
                         deleteEntity(
                             [entity.id],

diff --git a/src/components/InventoryTable/TitleColumn.js b/src/components/InventoryTable/TitleColumn.js
@@ -32,7 +32,7 @@ const onRowClick = (event, key, { loaded, onRowClick: rowClick, noDetail }) => {
  * @param {*} props additional props passed from `EntityTable` - holds any props passed to inventory table.
  */
 const TitleColumn = (data, id, item, props) => (
-    <div className="ins-composed-col">
+    <div className="ins-composed-col sentry-mask data-hj-suppress">
         <div key="os_release">{item?.os_release}</div>
         <div key="data" className={props?.noDetail ? 'ins-m-nodetail' : ''}>
             { props?.noDetail ?

diff --git a/src/components/InventoryTable/__snapshots__/EntityTable.test.js.snap b/src/components/InventoryTable/__snapshots__/EntityTable.test.js.snap
@@ -102,7 +102,7 @@ exports[`EntityTable DOM should render correctly - compact 1`] = `
         data-label="One"
       >
         <div
-          class="ins-composed-col"
+          class="ins-composed-col sentry-mask data-hj-suppress"
         >
           <div />
           <div
@@ -283,7 +283,7 @@ exports[`EntityTable DOM should render correctly - is expandable 1`] = `
         "cells": Array [
           <React.Fragment>
             <div
-              className="ins-composed-col"
+              className="ins-composed-col sentry-mask data-hj-suppress"
             >
               <div />
               <div
@@ -468,7 +468,7 @@ exports[`EntityTable DOM should render correctly - with actions 1`] = `
         "cells": Array [
           <React.Fragment>
             <div
-              className="ins-composed-col"
+              className="ins-composed-col sentry-mask data-hj-suppress"
             >
               <div />
               <div
@@ -601,7 +601,7 @@ exports[`EntityTable DOM should render correctly - without checkbox 1`] = `
         "cells": Array [
           <React.Fragment>
             <div
-              className="ins-composed-col"
+              className="ins-composed-col sentry-mask data-hj-suppress"
             >
               <div />
               <div
@@ -737,7 +737,7 @@ exports[`EntityTable DOM should render correctly 1`] = `
         data-label="One"
       >
         <div
-          class="ins-composed-col"
+          class="ins-composed-col sentry-mask data-hj-suppress"
         >
           <div />
           <div
@@ -808,7 +808,7 @@ exports[`EntityTable DOM sort by should render correctly - is expandable 1`] = `
         "cells": Array [
           <React.Fragment>
             <div
-              className="ins-composed-col"
+              className="ins-composed-col sentry-mask data-hj-suppress"
             >
               <div />
               <div
@@ -887,7 +887,7 @@ exports[`EntityTable DOM sort by should render correctly - without checkbox 1`]
         "cells": Array [
           <React.Fragment>
             <div
-              className="ins-composed-col"
+              className="ins-composed-col sentry-mask data-hj-suppress"
             >
               <div />
               <div
@@ -967,7 +967,7 @@ exports[`EntityTable DOM sort by should render correctly 1`] = `
         "cells": Array [
           <React.Fragment>
             <div
-              className="ins-composed-col"
+              className="ins-composed-col sentry-mask data-hj-suppress"
             >
               <div />
               <div

diff --git a/src/components/InventoryTable/__snapshots__/TitleColumn.test.js.snap b/src/components/InventoryTable/__snapshots__/TitleColumn.test.js.snap
@@ -3,7 +3,7 @@
 exports[`TitleColumn should render correctly no detail with data 1`] = `
 <Cmp>
   <div
-    className="ins-composed-col"
+    className="ins-composed-col sentry-mask data-hj-suppress"
   >
     <div
       key="os_release"
@@ -23,7 +23,7 @@ exports[`TitleColumn should render correctly no detail with data 1`] = `
 exports[`TitleColumn should render correctly with NO data 1`] = `
 <Cmp>
   <div
-    className="ins-composed-col"
+    className="ins-composed-col sentry-mask data-hj-suppress"
   >
     <div
       key="os_release"
@@ -45,7 +45,7 @@ exports[`TitleColumn should render correctly with NO data 1`] = `
 exports[`TitleColumn should render correctly with data 1`] = `
 <Cmp>
   <div
-    className="ins-composed-col"
+    className="ins-composed-col sentry-mask data-hj-suppress"
   >
     <div
       key="os_release"

diff --git a/src/routes/InventoryTable.js b/src/routes/InventoryTable.js
@@ -260,7 +260,8 @@ const Inventory = ({
                         variant: 'warning',
                         title: 'Delete operation initiated',
                         description: `Removal of ${displayName} started.`,
-                        dismissable: false
+                        dismissable: false,
+                        className: 'sentry-mask data-hj-suppress'
                     }));
                     dispatch(reloadWrapper(actions.deleteEntity(removeSystems, displayName), () => onRefresh({ filters })));
                     handleModalToggle(false);

diff --git a/src/routes/InventoryTable.test.js b/src/routes/InventoryTable.test.js
@@ -206,6 +206,7 @@ describe('InventoryTable', () => {
 
         expect(actions[0]).toEqual({
             payload: {
+                className: 'sentry-mask data-hj-suppress',
                 description: 'Removal of RHIQE.31ea86a9-a439-4422-9516-27c879057535.test started.',
                 dismissable: false,
                 id: 'remove-initiated',
@@ -219,6 +220,7 @@ describe('InventoryTable', () => {
             meta: {
                 notifications: {
                     fulfilled: {
+                        className: 'sentry-mask data-hj-suppress',
                         description: 'RHIQE.31ea86a9-a439-4422-9516-27c879057535.test has been successfully removed.',
                         dismissable: true,
                         title: 'Delete operation finished',

diff --git a/src/store/actions.js b/src/store/actions.js
@@ -12,7 +12,8 @@ export const deleteEntity = (systems, displayName) => ({
                 variant: 'success',
                 title: 'Delete operation finished',
                 description: `${displayName} has been successfully removed.`,
-                dismissable: true
+                dismissable: true,
+                className: 'sentry-mask data-hj-suppress'
             }
         },
         systems
@@ -43,7 +44,8 @@ export const editDisplayName = (id, value, origValue) => ({
             fulfilled: {
                 variant: 'success',
                 title: `Display name for entity with ID ${id} has been changed to ${value}`,
-                dismissable: true
+                dismissable: true,
+                className: 'sentry-mask data-hj-suppress'
             }
         }
     }

diff --git a/src/store/actions.test.js b/src/store/actions.test.js
@@ -30,6 +30,7 @@ describe('editDisplayName', () => {
             origValue: undefined,
             notifications: {
                 fulfilled: {
+                    className: 'sentry-mask data-hj-suppress',
                     variant: 'success',
                     title: 'Display name for entity with ID 4 has been changed to test-value',
                     dismissable: true

diff --git a/src/store/inventory-actions.js b/src/store/inventory-actions.js
@@ -198,7 +198,8 @@ export const deleteEntity = (systems, displayName) => ({
                 variant: 'success',
                 title: 'Delete operation finished',
                 description: `${displayName} has been successfully removed.`,
-                dismissable: true
+                dismissable: true,
+                className: 'sentry-mask data-hj-suppress'
             }
         },
         systems