From a526cff13a8f5368bde792efa2b5754c7cd689bc Mon Sep 17 00:00:00 2001
From: Gabriel Lee <chihonglee777@gmail.com>
Date: Thu, 12 Sep 2024 13:29:18 +0100
Subject: [PATCH] fix: tests

---
 .../SpaceListItem/SpaceListItem.unit.test.tsx | 13 --------
 .../SpaceListItem.unit.test.tsx.snap          | 33 ++++++++++---------
 .../SpaceRowContent.unit.test.tsx             | 13 ++++++++
 .../SpaceRowContent.unit.test.tsx.snap        | 12 +++----
 4 files changed, 36 insertions(+), 35 deletions(-)

diff --git a/src/components/SpaceListItem/SpaceListItem.unit.test.tsx b/src/components/SpaceListItem/SpaceListItem.unit.test.tsx
index dca3a39887..0ddc730151 100644
--- a/src/components/SpaceListItem/SpaceListItem.unit.test.tsx
+++ b/src/components/SpaceListItem/SpaceListItem.unit.test.tsx
@@ -8,7 +8,6 @@ import Icon from '../Icon';
 import { mountAndWait } from '../../../test/utils';
 import ListItemBase from '../ListItemBase';
 import * as ListContext from '../List/List.utils';
-import { STYLE } from './SpaceListItem.constants';
 import DividerDot from '../DividerDot';
 
 describe('<SpaceListItem />', () => {
@@ -451,18 +450,6 @@ describe('<SpaceListItem />', () => {
       expect(container.filter("[data-test='compact-mode-divider-dot']").length).toEqual(0);
     });
 
-    it('should have provided isNewActivity class when isNewActivity is provided', async () => {
-      expect.assertions(1);
-
-      const isNewActivity = true;
-
-      const element = (await mountAndWait(<SpaceListItem isNewActivity={isNewActivity} />)).find(
-        ListItemBase
-      );
-
-      expect(element.hasClass(STYLE.isNewActivity)).toBe(true);
-    });
-
     it('should have provided unread when isUnread is provided', async () => {
       expect.assertions(1);
 
diff --git a/src/components/SpaceListItem/SpaceListItem.unit.test.tsx.snap b/src/components/SpaceListItem/SpaceListItem.unit.test.tsx.snap
index db84622102..b26b4dc991 100644
--- a/src/components/SpaceListItem/SpaceListItem.unit.test.tsx.snap
+++ b/src/components/SpaceListItem/SpaceListItem.unit.test.tsx.snap
@@ -946,7 +946,7 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isEnterRoom 1`] =
   isEnterRoom={true}
 >
   <ListItemBase
-    className="md-space-list-item-wrapper md-space-list-item-is-new-activity"
+    className="md-space-list-item-wrapper"
     isDisabled={false}
     shape="isPilled"
     size={50}
@@ -960,7 +960,7 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isEnterRoom 1`] =
         isInset={false}
       >
         <li
-          className="md-space-list-item-wrapper md-space-list-item-is-new-activity md-list-item-base-wrapper"
+          className="md-space-list-item-wrapper md-list-item-base-wrapper"
           data-allow-text-select={false}
           data-disabled={false}
           data-interactive={true}
@@ -997,11 +997,11 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isEnterRoom 1`] =
               />
             </ListItemBaseSection>
             <ListItemBaseSection
-              className="md-space-row-content-text-wrapper text-column"
+              className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
               position="middle"
             >
               <div
-                className="md-space-row-content-text-wrapper text-column"
+                className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
                 data-position="middle"
               >
                 <Text
@@ -1192,7 +1192,7 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isMention 1`] = `
   isMention={true}
 >
   <ListItemBase
-    className="md-space-list-item-wrapper md-space-list-item-is-new-activity"
+    className="md-space-list-item-wrapper"
     isDisabled={false}
     shape="isPilled"
     size={50}
@@ -1206,7 +1206,7 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isMention 1`] = `
         isInset={false}
       >
         <li
-          className="md-space-list-item-wrapper md-space-list-item-is-new-activity md-list-item-base-wrapper"
+          className="md-space-list-item-wrapper md-list-item-base-wrapper"
           data-allow-text-select={false}
           data-disabled={false}
           data-interactive={true}
@@ -1243,11 +1243,11 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isMention 1`] = `
               />
             </ListItemBaseSection>
             <ListItemBaseSection
-              className="md-space-row-content-text-wrapper text-column"
+              className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
               position="middle"
             >
               <div
-                className="md-space-row-content-text-wrapper text-column"
+                className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
                 data-position="middle"
               >
                 <Text
@@ -1315,7 +1315,7 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isNewActivity 1`]
   isNewActivity={true}
 >
   <ListItemBase
-    className="md-space-list-item-wrapper md-space-list-item-is-new-activity"
+    className="md-space-list-item-wrapper"
     isDisabled={false}
     shape="isPilled"
     size={50}
@@ -1329,7 +1329,7 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isNewActivity 1`]
         isInset={false}
       >
         <li
-          className="md-space-list-item-wrapper md-space-list-item-is-new-activity md-list-item-base-wrapper"
+          className="md-space-list-item-wrapper md-list-item-base-wrapper"
           data-allow-text-select={false}
           data-disabled={false}
           data-interactive={true}
@@ -1356,6 +1356,7 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isNewActivity 1`]
           <SpaceRowContent
             isCompact={false}
             isDisabled={false}
+            isNewActivity={true}
           >
             <ListItemBaseSection
               position="start"
@@ -1365,11 +1366,11 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isNewActivity 1`]
               />
             </ListItemBaseSection>
             <ListItemBaseSection
-              className="md-space-row-content-text-wrapper text-column"
+              className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
               position="middle"
             >
               <div
-                className="md-space-row-content-text-wrapper text-column"
+                className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
                 data-position="middle"
               >
                 <Text
@@ -1625,7 +1626,7 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isUnread 1`] = `
   isUnread={true}
 >
   <ListItemBase
-    className="md-space-list-item-wrapper md-space-list-item-is-new-activity"
+    className="md-space-list-item-wrapper"
     isDisabled={false}
     shape="isPilled"
     size={50}
@@ -1639,7 +1640,7 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isUnread 1`] = `
         isInset={false}
       >
         <li
-          className="md-space-list-item-wrapper md-space-list-item-is-new-activity md-list-item-base-wrapper"
+          className="md-space-list-item-wrapper md-list-item-base-wrapper"
           data-allow-text-select={false}
           data-disabled={false}
           data-interactive={true}
@@ -1676,11 +1677,11 @@ exports[`<SpaceListItem /> snapshot should match snapshot with isUnread 1`] = `
               />
             </ListItemBaseSection>
             <ListItemBaseSection
-              className="md-space-row-content-text-wrapper text-column"
+              className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
               position="middle"
             >
               <div
-                className="md-space-row-content-text-wrapper text-column"
+                className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
                 data-position="middle"
               >
                 <Text
diff --git a/src/components/SpaceRowContent/SpaceRowContent.unit.test.tsx b/src/components/SpaceRowContent/SpaceRowContent.unit.test.tsx
index d7fa495ec8..56f10b7134 100644
--- a/src/components/SpaceRowContent/SpaceRowContent.unit.test.tsx
+++ b/src/components/SpaceRowContent/SpaceRowContent.unit.test.tsx
@@ -7,6 +7,7 @@ import Avatar from '../Avatar';
 import Icon from '../Icon';
 import { mountAndWait } from '../../../test/utils';
 import DividerDot from '../DividerDot';
+import ListItemBaseSection from '../ListItemBaseSection';
 
 describe('<SpaceRowContent />', () => {
   describe('snapshot', () => {
@@ -365,6 +366,18 @@ describe('<SpaceRowContent />', () => {
       expect(container.filter("[data-test='compact-mode-divider-dot']").length).toEqual(0);
     });
 
+    it('should have provided isNewActivity class when isNewActivity is provided', async () => {
+      expect.assertions(1);
+
+      const isNewActivity = true;
+
+      const element = (await mountAndWait(<SpaceRowContent isNewActivity={isNewActivity} />))
+        .find(ListItemBaseSection)
+        .find({ position: 'middle' });
+
+      expect(element.hasClass('md-space-row-content-is-new-activity')).toBe(true);
+    });
+
     it('should have provided unread when isUnread is provided', async () => {
       expect.assertions(1);
 
diff --git a/src/components/SpaceRowContent/SpaceRowContent.unit.test.tsx.snap b/src/components/SpaceRowContent/SpaceRowContent.unit.test.tsx.snap
index 26d96e4b37..da6ed9a67d 100644
--- a/src/components/SpaceRowContent/SpaceRowContent.unit.test.tsx.snap
+++ b/src/components/SpaceRowContent/SpaceRowContent.unit.test.tsx.snap
@@ -423,11 +423,11 @@ exports[`<SpaceRowContent /> snapshot should match snapshot with isEnterRoom 1`]
     />
   </ListItemBaseSection>
   <ListItemBaseSection
-    className="md-space-row-content-text-wrapper text-column"
+    className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
     position="middle"
   >
     <div
-      className="md-space-row-content-text-wrapper text-column"
+      className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
       data-position="middle"
     >
       <Text
@@ -571,11 +571,11 @@ exports[`<SpaceRowContent /> snapshot should match snapshot with isMention 1`] =
     />
   </ListItemBaseSection>
   <ListItemBaseSection
-    className="md-space-row-content-text-wrapper text-column"
+    className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
     position="middle"
   >
     <div
-      className="md-space-row-content-text-wrapper text-column"
+      className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
       data-position="middle"
     >
       <Text
@@ -763,11 +763,11 @@ exports[`<SpaceRowContent /> snapshot should match snapshot with isUnread 1`] =
     />
   </ListItemBaseSection>
   <ListItemBaseSection
-    className="md-space-row-content-text-wrapper text-column"
+    className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
     position="middle"
   >
     <div
-      className="md-space-row-content-text-wrapper text-column"
+      className="md-space-row-content-text-wrapper text-column md-space-row-content-is-new-activity"
       data-position="middle"
     >
       <Text