From 323dba620d4a01e9ac14a42a275048e8890f8195 Mon Sep 17 00:00:00 2001
From: Michael Kaye <1917473+michaelkaye@users.noreply.github.com>
Date: Wed, 19 Apr 2023 13:47:05 +0100
Subject: [PATCH] Add a range of data-testid tags to be able to cleanly
 identify parts of the application.

---
 src/home/JoinExistingCallModal.tsx | 2 +-
 src/home/UnauthenticatedView.tsx   | 4 +++-
 src/icons/VideoMuted.svg           | 2 +-
 src/room/LobbyView.tsx             | 1 +
 src/room/VideoPreview.tsx          | 2 +-
 src/video-grid/VideoTile.tsx       | 5 +++--
 6 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/src/home/JoinExistingCallModal.tsx b/src/home/JoinExistingCallModal.tsx
index 0beae60..02d7cd2 100644
--- a/src/home/JoinExistingCallModal.tsx
+++ b/src/home/JoinExistingCallModal.tsx
@@ -43,7 +43,7 @@ export function JoinExistingCallModal({ onJoin, onClose, ...rest }: Props) {
         <p>{t("This call already exists, would you like to join?")}</p>
         <FieldRow rightAlign className={styles.buttons}>
           <Button onPress={onClose}>{t("No")}</Button>
-          <Button onPress={onJoin}>{t("Yes, join call")}</Button>
+          <Button onPress={onJoin} data-testid="home_joinExistingRoom">{t("Yes, join call")}</Button>
         </FieldRow>
       </ModalContent>
     </Modal>
diff --git a/src/home/UnauthenticatedView.tsx b/src/home/UnauthenticatedView.tsx
index c31637b..be20d70 100644
--- a/src/home/UnauthenticatedView.tsx
+++ b/src/home/UnauthenticatedView.tsx
@@ -142,6 +142,7 @@ export const UnauthenticatedView: FC = () => {
                 type="text"
                 required
                 autoComplete="off"
+                data-testid="home_callName"
               />
             </FieldRow>
             <FieldRow>
@@ -152,6 +153,7 @@ export const UnauthenticatedView: FC = () => {
                 placeholder={t("Display name")}
                 type="text"
                 required
+                data-testid="home_displayName"
                 autoComplete="off"
               />
             </FieldRow>
@@ -171,7 +173,7 @@ export const UnauthenticatedView: FC = () => {
                 <ErrorMessage error={error} />
               </FieldRow>
             )}
-            <Button type="submit" size="lg" disabled={loading}>
+            <Button type="submit" size="lg" disabled={loading} data-testid="home_go">
               {loading ? t("Loading…") : t("Go")}
             </Button>
             <div id={recaptchaId} />
diff --git a/src/icons/VideoMuted.svg b/src/icons/VideoMuted.svg
index 188ed08..20ecf0f 100644
--- a/src/icons/VideoMuted.svg
+++ b/src/icons/VideoMuted.svg
@@ -1,4 +1,4 @@
-<svg width="14" height="14" viewBox="0 0 14 14" fill="none" xmlns="http://www.w3.org/2000/svg">
+<svg data-testid="videoTile_muted" width="14" height="14" viewBox="0 0 14 14" fill="none" xmlns="http://www.w3.org/2000/svg">
 <path d="M0.20333 0.963373C0.474437 0.690007 0.913989 0.690007 1.1851 0.963373L11.5983 11.4633C11.8694 11.7367 11.8694 12.1799 11.5983 12.4533C11.3272 12.7267 10.8876 12.7267 10.6165 12.4533L0.20333 1.95332C-0.0677768 1.67995 -0.0677768 1.23674 0.20333 0.963373Z" fill="white"/>
 <path d="M0.418261 3.63429C0.226267 3.95219 0.115674 4.32557 0.115674 4.725V9.85832C0.115674 11.0181 1.0481 11.9583 2.19831 11.9583H8.65411L0.447396 3.66596C0.437225 3.65568 0.427513 3.64511 0.418261 3.63429Z" fill="white"/>
 <path d="M9.95036 4.725V8.33212L4.30219 2.625H7.86772C9.01793 2.625 9.95036 3.5652 9.95036 4.725Z" fill="white"/>
diff --git a/src/room/LobbyView.tsx b/src/room/LobbyView.tsx
index a1a6961..2e726cc 100644
--- a/src/room/LobbyView.tsx
+++ b/src/room/LobbyView.tsx
@@ -137,6 +137,7 @@ export function LobbyView({
               size="lg"
               disabled={state !== GroupCallState.LocalCallFeedInitialized}
               onPress={onEnter}
+              data-testid="lobby_joinCall"
             >
               Join call now
             </Button>
diff --git a/src/room/VideoPreview.tsx b/src/room/VideoPreview.tsx
index 0e92cc5..6325bda 100644
--- a/src/room/VideoPreview.tsx
+++ b/src/room/VideoPreview.tsx
@@ -64,7 +64,7 @@ export function VideoPreview({
 
   return (
     <div className={styles.preview} ref={previewRef}>
-      <video ref={videoRef} muted playsInline disablePictureInPicture />
+      <video ref={videoRef} muted playsInline disablePictureInPicture data-testid="preview_video" />
       {state === GroupCallState.LocalCallFeedUninitialized && (
         <Body fontWeight="semiBold" className={styles.cameraPermissions}>
           {t("Camera/microphone permissions needed to join the call.")}
diff --git a/src/video-grid/VideoTile.tsx b/src/video-grid/VideoTile.tsx
index eea617f..804a036 100644
--- a/src/video-grid/VideoTile.tsx
+++ b/src/video-grid/VideoTile.tsx
@@ -154,6 +154,7 @@ export const VideoTile = forwardRef<HTMLElement, Props>(
           "--tileHeight": height?.to((h) => `${h}px`),
         }}
         ref={ref as ForwardedRef<HTMLDivElement>}
+        data-testid='videoTile'
         {...rest}
       >
         {toolbarButtons.length > 0 && !maximised && (
@@ -180,10 +181,10 @@ export const VideoTile = forwardRef<HTMLElement, Props>(
                 audioMuted && !videoMuted && !speaking && <MicMutedIcon />
               }
               {videoMuted && <VideoMutedIcon />}
-              <span title={caption}>{caption}</span>
+              <span data-testid="videoTile_caption" title={caption}>{caption}</span>
             </div>
           ))}
-        <video ref={mediaRef} playsInline disablePictureInPicture />
+        <video data-testid="videoTile_video" ref={mediaRef} playsInline disablePictureInPicture />
       </animated.div>
     );
   }