fix: update test IDs for budget discussion form components and related tests

Author: kneerose

tests/govtool-frontend/playwright/lib/pages/budgetDiscussionSubmissionPage.ts

@@ -51,32 +51,27 @@ export default class BudgetDiscussionSubmissionPage {
   readonly verifyIdentityBtn = this.page.getByTestId("verify-identity-button");
   readonly saveDraftBtn = this.page.getByTestId("draft-button");
   readonly submitBtn = this.page.getByTestId("submit-button");
-  readonly countryOfIncorporationBtn = this.page.getByTestId(
-    "country-of-incorporation"
-  );
 
-  readonly agreeCheckbox = this.page.getByLabel(
-    "I agree to the information in"
-  ); //BUG missing test Ids
+  readonly agreeCheckbox = this.page.getByTestId("agree-checkbox");
   readonly submitCheckbox = this.page.getByTestId("submit-checkbox");
 
   // input
   readonly linkTextInput = this.page.getByTestId("link-0-text-input");
   readonly linkUrlInput = this.page.getByTestId("link-0-url-input");
 
   // proposal-ownership
-  readonly companyNameInput = this.page.getByLabel("Company Name *"); //BUG missing test Ids
-  readonly companyDomainNameInput = this.page.getByLabel(
-    "Company Domain Name *"
-  ); //BUG missing test Ids
-  readonly groupNameInput = this.page.getByLabel("Group Name *"); //BUG missing test Ids
-  readonly groupTypeInput = this.page.getByLabel("Type of Group *"); //BUG missing test Ids
-  readonly keyInformationOfGroupInput = this.page.getByLabel(
-    "Key Information to Identify"
-  ); //BUG missing test Ids
-  readonly contactDetailsInput = this.page.getByLabel(
-    "Please provide your preferred"
-  ); //BUG missing test Ids
+  readonly companyNameInput = this.page.getByTestId("company-name-input");
+  readonly companyDomainNameInput = this.page.getByTestId(
+    "company-domain-input"
+  );
+  readonly groupNameInput = this.page.getByTestId("group-name-input");
+  readonly groupTypeInput = this.page.getByTestId("group-type-input");
+  readonly keyInformationOfGroupInput = this.page.getByTestId(
+    "group-identity-information-input"
+  );
+  readonly contactDetailsInput = this.page.getByTestId(
+    "provide-preferred-input"
+  );
 
   // problem-statements
   readonly problemStatementInput = this.page.getByTestId(
@@ -93,51 +88,47 @@ export default class BudgetDiscussionSubmissionPage {
   ); // BUG missing test Ids
 
   // proposal-details
-  readonly proposalNameInput = this.page.getByLabel(
-    "What is your proposed name to"
-  ); //BUG missing testId
+  readonly proposalNameInput = this.page.getByTestId("proposal-name-input");
   readonly proposalDescriptionInput = this.page.getByTestId(
     "proposal-description-input"
   );
   readonly proposalKeyDependenciesInput = this.page.getByTestId(
     "key-dependencies-input"
   );
-  readonly proposalMaintainAndSupportInput = this.page.getByLabel(
-    "How will this proposal be"
-  ); //BUG missing testId
+  readonly proposalMaintainAndSupportInput = this.page.getByTestId(
+    "proposal-maintain-and-support-input"
+  );
   readonly milestonesInput = this.page.getByTestId(
     "key-proposal-deliverables-input"
   );
   readonly teamSizeAndDurationInput = this.page.getByTestId(
     "resourcing-duration-estimates-input"
   );
-  readonly previousExperienceInput = this.page.getByLabel(
-    "Please provide previous"
-  ); //BUG missing testId
-  readonly otherDescriptionInput = this.page.getByLabel(
-    "Please describe what you have"
+  readonly previousExperienceInput = this.page.getByTestId(
+    "proposal-previous-experience-input"
+  );
+  readonly otherDescriptionInput = this.page.getByTestId(
+    "other-contract-description"
   );
 
   // costing
-  readonly adaAmountInput = this.page.getByLabel("ADA Amount *"); //BUG missing test Ids
-  readonly usaToAdaCnversionRateInput = this.page.getByLabel(
-    "USD to ADA Conversion Rate *"
-  ); //BUG missing test Ids
-  readonly preferredCurrencyInput = this.page.getByLabel(
-    "Amount in preferred currency *"
+  readonly adaAmountInput = this.page.getByTestId("ada-amount-input");
+  readonly usaToAdaCnversionRateInput = this.page.getByTestId(
+    "usd-ada-conversion-input"
+  );
+  readonly preferredCurrencyInput = this.page.getByTestId(
+    "preferred-currency-amount-input"
   );
   readonly costBreakdownInput = this.page.getByTestId("cost-breakdown-input");
   readonly venderDetailsInput = this.page.getByLabel("Please provide further"); //BUG missing test Ids
 
   // select
-  readonly beneficiaryCountrySelect = this.page.getByTestId(
-    "beneficiary-country-of-residence"
-  );
-  readonly beneficiaryNationalitySelect = this.page.getByTestId(
-    "beneficiary-nationality"
-  );
-
-  readonly companyTypeSelect = this.page.getByTestId("beneficiary-type");
+  readonly proposalCommittee = this.page
+    .getByLabel("Is this proposal being")
+    .first(); // BUG incorrect position of testId
+  readonly countryOfIncorporationBtn = this.page
+    .getByLabel("Country of Incorporation *")
+    .first(); // BUG incorrect position of testId
   readonly publicChampionSelect = this.page.getByTestId(
     "proposal-public-champion"
   );
@@ -234,9 +225,9 @@ export default class BudgetDiscussionSubmissionPage {
 
   // costing
   readonly adaAmountContent = this.page.getByTestId("ada-amount-content");
-  readonly adaToUsdConversionRateContent = this.page.getByTestId(
+  readonly usdToAdaConversionRateContent = this.page.getByTestId(
     "usd-to-ada-conversion-rate-content"
-  ); // BUG typo
+  );
   readonly preferredCurrencyContent = this.page.getByTestId(
     "preferred-currency-content"
   );
@@ -271,7 +262,7 @@ export default class BudgetDiscussionSubmissionPage {
     proposalOwnership: BudgetProposalOwnershipProps,
     isNaviagted = true
   ) {
-    await this.companyTypeSelect.click();
+    await this.proposalCommittee.click();
     await this.page
       .getByRole("option", { name: proposalOwnership.companyType })
       .click(); //BUG missing testId
@@ -772,7 +763,7 @@ export default class BudgetDiscussionSubmissionPage {
     await expect(this.adaAmountContent).toHaveText(
       `₳ ${formatWithThousandSeparator(proposalInformations.costing.adaAmount)}`
     );
-    await expect(this.adaToUsdConversionRateContent).toHaveText(
+    await expect(this.usdToAdaConversionRateContent).toHaveText(
       proposalInformations.costing.usdToAdaConversionRate.toString()
     );
 
@@ -836,7 +827,7 @@ export default class BudgetDiscussionSubmissionPage {
     proposalOwnership: BudgetProposalOwnershipProps,
     isValid: boolean = true
   ) {
-    const companyTypeSelectContent = await this.companyTypeSelect.textContent();
+    const companyTypeSelectContent = await this.proposalCommittee.textContent();
 
     if (isValid) {
       if (proposalOwnership.companyType === "Company") {

tests/govtool-frontend/playwright/tests/11-proposal-budget/proposalBudget.dRep.pb.spec.ts

@@ -61,7 +61,9 @@ test.describe("Budget proposal dRep behaviour", () => {
       await budgetDiscussionDetailsPage.voteOnPoll(choice);
       await budgetDiscussionDetailsPage.changePollVote();
 
-      await expect(budgetDiscussionDetailsPage.pollYesBtn).not.toBeVisible({timeout: 60_000});
+      await expect(budgetDiscussionDetailsPage.pollYesBtn).not.toBeVisible({
+        timeout: 60_000,
+      });
       await expect(budgetDiscussionDetailsPage.pollNoBtn).not.toBeVisible();
 
       // vote must be changed

tests/govtool-frontend/playwright/tests/12-proposal-budget-submission/proposalBudgetSubmission.loggedin.pb.spec.ts

@@ -62,15 +62,15 @@ test.describe("Budget proposal 01 wallet", () => {
       test("12D_1. Should verify all field of “proposal ownership” section", async () => {
         // default field
         await expect(
-          budgetProposalSubmissionPage.companyTypeSelect
+          budgetProposalSubmissionPage.proposalCommittee
         ).toBeVisible();
 
         await expect(
           budgetProposalSubmissionPage.contactDetailsInput
         ).toBeVisible();
 
         // company type field
-        await budgetProposalSubmissionPage.companyTypeSelect.click();
+        await budgetProposalSubmissionPage.proposalCommittee.click();
         await budgetProposalSubmissionPage.currentPage
           .getByRole("option", { name: CompanyEnum.Company })
           .click(); //BUG missing testId
@@ -86,7 +86,7 @@ test.describe("Budget proposal 01 wallet", () => {
         ).toBeVisible();
 
         // group type field
-        await budgetProposalSubmissionPage.companyTypeSelect.click();
+        await budgetProposalSubmissionPage.proposalCommittee.click();
         await budgetProposalSubmissionPage.currentPage
           .getByRole("option", { name: CompanyEnum.Group })
           .click(); //BUG missing testId
@@ -436,7 +436,7 @@ test("12C. Should save and view draft proposal", async ({ browser }) => {
 
   await budgetSubmissionPage.viewLastDraft();
 
-  await expect(budgetSubmissionPage.companyTypeSelect).toHaveText(
+  await expect(budgetSubmissionPage.proposalCommittee).toHaveText(
     draftProposalOwnership.companyType
   );