feat: made datatestids mandatory on readablemessages

ts/components/conversation/message/message-content/MessageContentWithStatus.tsx

@@ -29,7 +29,7 @@ type Props = {
   messageId: string;
   ctxMenuID: string;
   isDetailView?: boolean;
-  dataTestId?: string;
+  dataTestId: string;
   enableReactions: boolean;
 };
 
@@ -124,7 +124,7 @@ export const MessageContentWithStatuses = (props: Props) => {
         role={'button'}
         onClick={onClickOnMessageOuterContainer}
         onDoubleClickCapture={onDoubleClickReplyToMessage}
-        data-testid={dataTestId}
+        dataTestId={dataTestId}
       >
         <MessageAvatar
           messageId={messageId}

ts/components/conversation/message/message-item/DataExtractionNotification.tsx

@@ -19,6 +19,7 @@ export const DataExtractionNotification = (props: PropsForDataExtractionNotifica
   return (
     <ExpirableReadableMessage
       messageId={messageId}
+      dataTestId="data-extraction-notification"
       key={`readable-message-${messageId}`}
       isCentered
     >

ts/components/conversation/message/message-item/ReadableMessage.tsx

@@ -26,8 +26,8 @@ import {
   getShowScrollButton,
   getYoungestMessageId,
 } from '../../../../state/selectors/conversations';
-import { useSelectedConversationKey } from '../../../../state/selectors/selectedConversation';
 import { getIsAppFocused } from '../../../../state/selectors/section';
+import { useSelectedConversationKey } from '../../../../state/selectors/selectedConversation';
 import { ScrollToLoadedMessageContext } from '../../SessionMessagesListContainer';
 
 export type ReadableMessageProps = {
@@ -39,7 +39,7 @@ export type ReadableMessageProps = {
   onClick?: MouseEventHandler<HTMLElement>;
   onDoubleClickCapture?: MouseEventHandler<HTMLElement>;
   role?: AriaRole;
-  dataTestId?: string;
+  dataTestId: string;
   onContextMenu?: (e: React.MouseEvent<HTMLElement>) => void;
 };
 
@@ -207,8 +207,7 @@ export const ReadableMessage = (props: ReadableMessageProps) => {
       onDoubleClickCapture={onDoubleClickCapture}
       role={role}
       key={`inview-msg-${messageId}`}
-      // TODO We will need to update the integration tests to use that new value, or update the values given in the `dataTestId` props to match what they expect
+      data-testid={dataTestId}
-      data-testid={dataTestId || 'readable-message'}
     >
       {props.children}
     </InView>

ts/components/conversation/message/message-item/notification-bubble/CallNotification.tsx

@@ -62,6 +62,7 @@ export const CallNotification = (props: PropsForCallNotification) => {
     <ExpirableReadableMessage
       messageId={messageId}
       key={`readable-message-${messageId}`}
+      dataTestId={`call-notification-${notificationType}`}
       isCentered={true}
     >
       <NotificationBubble