From 3ef72f25fc642334101262f28e79d3ac6531874d Mon Sep 17 00:00:00 2001
From: Jan Jaroszczak <jan.jaroszczak@binarapps.com>
Date: Wed, 8 May 2024 15:47:27 +0200
Subject: [PATCH] [#937] Missing Test IDs

---
 .../src/components/molecules/AutomatedVotingCard.tsx  |  1 +
 .../src/components/molecules/VoteActionForm.tsx       |  1 +
 .../components/organisms/AutomatedVotingOptions.tsx   |  5 ++++-
 govtool/frontend/src/components/organisms/BgCard.tsx  |  3 ++-
 .../organisms/DashboardCards/DRepDashboardCard.tsx    |  2 +-
 .../DashboardCards/DelegateDashboardCard.tsx          |  2 +-
 .../frontend/src/components/organisms/HomeCards.tsx   |  2 +-
 .../RegisterAsDRepSteps/DRepStorageInformation.tsx    |  1 +
 .../RegisterAsDRepSteps/RegisterAsDRepForm.tsx        | 11 ++++++++++-
 .../RegisterAsDRepSteps/WhatRetirementMeans.tsx       |  1 +
 govtool/frontend/src/components/organisms/TopNav.tsx  |  1 +
 .../organisms/VoteContext/VoteContextText.tsx         |  1 +
 govtool/frontend/src/components/organisms/types.ts    |  1 +
 13 files changed, 26 insertions(+), 6 deletions(-)

diff --git a/govtool/frontend/src/components/molecules/AutomatedVotingCard.tsx b/govtool/frontend/src/components/molecules/AutomatedVotingCard.tsx
index ec1d75171..cc56895d3 100644
--- a/govtool/frontend/src/components/molecules/AutomatedVotingCard.tsx
+++ b/govtool/frontend/src/components/molecules/AutomatedVotingCard.tsx
@@ -50,6 +50,7 @@ export const AutomatedVotingCard = ({
         mt: inProgress || isSelected ? 2 : 0,
         py: 2.25,
       }}
+      data-testid={`${testIdLabel}-card`}
     >
       <Box
         sx={{
diff --git a/govtool/frontend/src/components/molecules/VoteActionForm.tsx b/govtool/frontend/src/components/molecules/VoteActionForm.tsx
index a69da1f37..d2094e141 100644
--- a/govtool/frontend/src/components/molecules/VoteActionForm.tsx
+++ b/govtool/frontend/src/components/molecules/VoteActionForm.tsx
@@ -343,6 +343,7 @@ export const VoteActionForm = ({
                 ? "11.5px"
                 : "14px",
           }}
+          data-testid="provide-context-button"
         >
           {voteContextText
             ? t("govActions.provideNewContextAboutYourVote")
diff --git a/govtool/frontend/src/components/organisms/AutomatedVotingOptions.tsx b/govtool/frontend/src/components/organisms/AutomatedVotingOptions.tsx
index 52772f98d..92fac90e4 100644
--- a/govtool/frontend/src/components/organisms/AutomatedVotingOptions.tsx
+++ b/govtool/frontend/src/components/organisms/AutomatedVotingOptions.tsx
@@ -74,7 +74,10 @@ export const AutomatedVotingOptions = ({
           />
         )}
       </AccordionSummary>
-      <AccordionDetails sx={{ p: { xxs: 2, md: 3 }, pt: { xxs: 0, md: 0 } }}>
+      <AccordionDetails
+        sx={{ p: { xxs: 2, md: 3 }, pt: { xxs: 0, md: 0 } }}
+        data-testid="delegation-options-dropdown"
+      >
         <Box
           sx={{
             display: "flex",
diff --git a/govtool/frontend/src/components/organisms/BgCard.tsx b/govtool/frontend/src/components/organisms/BgCard.tsx
index 5d2a3b1f3..261e52f15 100644
--- a/govtool/frontend/src/components/organisms/BgCard.tsx
+++ b/govtool/frontend/src/components/organisms/BgCard.tsx
@@ -11,6 +11,7 @@ import { BgCardProps } from "./types";
 
 export const BgCard = ({
   actionButtonLabel,
+  actionButtonDataTestId,
   backButtonLabel,
   children,
   isLoadingActionButton,
@@ -51,7 +52,7 @@ export const BgCard = ({
   const renderContinueButton = useMemo(
     () => (
       <Button
-        data-testid="retire-button"
+        data-testid={actionButtonDataTestId ?? "continue-button"}
         disabled={isActionButtonDisabled}
         isLoading={isLoadingActionButton}
         onClick={onClickActionButton}
diff --git a/govtool/frontend/src/components/organisms/DashboardCards/DRepDashboardCard.tsx b/govtool/frontend/src/components/organisms/DashboardCards/DRepDashboardCard.tsx
index 0ac90b61f..6e4435704 100644
--- a/govtool/frontend/src/components/organisms/DashboardCards/DRepDashboardCard.tsx
+++ b/govtool/frontend/src/components/organisms/DashboardCards/DRepDashboardCard.tsx
@@ -144,7 +144,7 @@ export const DRepDashboardCard = ({
     >
       {voter?.isRegisteredAsDRep && !pendingTransaction?.retireAsDrep && (
         <CopyableInfo
-          dataTestId="my-drep-id"
+          dataTestId="dRep-id-display-card-dashboard"
           label={t("dashboard.cards.drep.yourDRepId")}
           sx={{ mt: 1 }}
           value={dRepIDBech32}
diff --git a/govtool/frontend/src/components/organisms/DashboardCards/DelegateDashboardCard.tsx b/govtool/frontend/src/components/organisms/DashboardCards/DelegateDashboardCard.tsx
index f399b2740..9d9b03aba 100644
--- a/govtool/frontend/src/components/organisms/DashboardCards/DelegateDashboardCard.tsx
+++ b/govtool/frontend/src/components/organisms/DashboardCards/DelegateDashboardCard.tsx
@@ -86,7 +86,7 @@ export const DelegateDashboardCard = ({
       buttons: [
         {
           children: t("dashboard.cards.delegation.noDelegationActionButton"),
-          dataTestId: "delegate-button",
+          dataTestId: "view-drep-directory-button",
           onClick: () => navigate(PATHS.dashboardDRepDirectory),
           variant: "contained",
         },
diff --git a/govtool/frontend/src/components/organisms/HomeCards.tsx b/govtool/frontend/src/components/organisms/HomeCards.tsx
index d9cd94db6..8ef9d5384 100644
--- a/govtool/frontend/src/components/organisms/HomeCards.tsx
+++ b/govtool/frontend/src/components/organisms/HomeCards.tsx
@@ -71,7 +71,7 @@ export const HomeCards = () => {
     >
       {/* DELEGATE CARD */}
       <ActionCard
-        dataTestIdFirstButton="delegate-connect-wallet-button"
+        dataTestIdFirstButton="view-drep-directory-button"
         dataTestIdSecondButton="delegate-learn-more-button"
         description={t("home.cards.delegate.description")}
         firstButtonAction={navigateToDRepDirecotry}
diff --git a/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/DRepStorageInformation.tsx b/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/DRepStorageInformation.tsx
index 3588eb1fd..5becfc07a 100644
--- a/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/DRepStorageInformation.tsx
+++ b/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/DRepStorageInformation.tsx
@@ -51,6 +51,7 @@ export const DRepStorageInformation = ({
   return (
     <BgCard
       actionButtonLabel={t("submit")}
+      actionButtonDataTestId="register-button"
       backButtonLabel={t("back")}
       isActionButtonDisabled={isActionButtonDisabled}
       isLoadingActionButton={isRegistrationAsDRepLoading}
diff --git a/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/RegisterAsDRepForm.tsx b/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/RegisterAsDRepForm.tsx
index f6f561bb4..6258f862f 100644
--- a/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/RegisterAsDRepForm.tsx
+++ b/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/RegisterAsDRepForm.tsx
@@ -70,6 +70,7 @@ export const RegisterAsDRepForm = ({
           placeholder={Placeholders.LINK}
           name={`links.${index}.link`}
           rules={Rules.LINK}
+          data-testid="link-input"
         />
       )),
     [errors, links],
@@ -102,6 +103,7 @@ export const RegisterAsDRepForm = ({
         name="dRepName"
         rules={Rules.DREP_NAME}
         placeholder={t("forms.registerAsDRep.dRepNamePlaceholder")}
+        dataTestId="name-input"
       />
       <Spacer y={isMobile ? 5 : 6} />
       <Box textAlign="center">
@@ -119,6 +121,7 @@ export const RegisterAsDRepForm = ({
         name="email"
         placeholder={t("forms.registerAsDRep.emailPlaceholder")}
         rules={Rules.EMAIL}
+        data-testid="email-input"
       />
       <Spacer y={3} />
       <ControlledField.TextArea
@@ -128,6 +131,7 @@ export const RegisterAsDRepForm = ({
         placeholder={t("forms.registerAsDRep.bioPlaceholder")}
         helpfulText={t("forms.registerAsDRep.bioHelpfulText")}
         rules={Rules.BIO}
+        data-testid="bio-input"
       />
       <Spacer y={4} />
       <p
@@ -149,7 +153,12 @@ export const RegisterAsDRepForm = ({
       <Spacer y={3} />
       {renderLinks()}
       {links?.length < MAX_NUMBER_OF_LINKS ? (
-        <Button onClick={addLink} size="extraLarge" variant="text">
+        <Button
+          onClick={addLink}
+          size="extraLarge"
+          variant="text"
+          data-testid="add-link-button"
+        >
           {t("addLink")}
         </Button>
       ) : null}
diff --git a/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/WhatRetirementMeans.tsx b/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/WhatRetirementMeans.tsx
index 5b7db0914..36e6459ac 100644
--- a/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/WhatRetirementMeans.tsx
+++ b/govtool/frontend/src/components/organisms/RegisterAsDRepSteps/WhatRetirementMeans.tsx
@@ -85,6 +85,7 @@ export const WhatRetirementMeans = ({
   return (
     <BgCard
       actionButtonLabel={t("retirement.continue")}
+      actionButtonDataTestId="continue-retirement-button"
       backButtonLabel={t("cancel")}
       isLoadingActionButton={isRetirementLoading}
       onClickActionButton={retireAsDrep}
diff --git a/govtool/frontend/src/components/organisms/TopNav.tsx b/govtool/frontend/src/components/organisms/TopNav.tsx
index ecfed9932..fa34ff4b7 100644
--- a/govtool/frontend/src/components/organisms/TopNav.tsx
+++ b/govtool/frontend/src/components/organisms/TopNav.tsx
@@ -166,6 +166,7 @@ export const TopNav = ({ isConnectButton = true }) => {
                     alt="drawer-icon"
                     src={ICONS.drawerIcon}
                     onClick={openDrawer}
+                    data-testid="open-drawer-button"
                   />
                 )}
               </Box>
diff --git a/govtool/frontend/src/components/organisms/VoteContext/VoteContextText.tsx b/govtool/frontend/src/components/organisms/VoteContext/VoteContextText.tsx
index f241de814..3ac7117e2 100644
--- a/govtool/frontend/src/components/organisms/VoteContext/VoteContextText.tsx
+++ b/govtool/frontend/src/components/organisms/VoteContext/VoteContextText.tsx
@@ -71,6 +71,7 @@ export const VoteContextText = ({
         {...{ control, errors }}
         {...fieldProps}
         isModifiedLayout
+        data-testid="provide-context-input"
       />
     </VoteContextWrapper>
   );
diff --git a/govtool/frontend/src/components/organisms/types.ts b/govtool/frontend/src/components/organisms/types.ts
index 4207dd2db..1cd327aed 100644
--- a/govtool/frontend/src/components/organisms/types.ts
+++ b/govtool/frontend/src/components/organisms/types.ts
@@ -3,6 +3,7 @@ import { Dispatch, SetStateAction } from "react";
 
 export type BgCardProps = {
   actionButtonLabel: string;
+  actionButtonDataTestId?: string;
   backButtonLabel?: string;
   children: React.ReactNode;
   isLoadingActionButton?: boolean;