Skip to content

Commit

Permalink
refactor: proposal info content testIds
Browse files Browse the repository at this point in the history
  • Loading branch information
@kneerose
kneerose committed Dec 10, 2024
1 parent a94ee13 commit c4d77ee
Show file tree
Hide file tree
Showing 2 changed files with 49 additions and 35 deletions.
16 changes: 15 additions & 1 deletion tests/govtool-frontend/playwright/lib/pages/proposalSubmissionPage.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -70,7 +70,21 @@ export default class ProposalSubmissionPage {
readonly closeDraftSuccessModalBtn = this.page.getByTestId("close-button");
readonly linkTextInput = this.page.getByTestId("link-0-text-input");
readonly linkUrlInput = this.page.getByTestId("link-0-url-input");


// content
readonly governanceActionTypeContent = this.page.getByTestId(
"governance-action-type-content"
);
readonly titleContent = this.page.getByTestId("title-content");
readonly abstractContent = this.page.getByTestId("abstract-content");
readonly motivationContent = this.page.getByTestId("motivation-content");
readonly rationaleContent = this.page.getByTestId("rationale-content");
readonly receivingAddressContent = this.page.getByTestId(
"receiving-address-0-content"
);
readonly amountContent = this.page.getByTestId("amount-0-content");
readonly linkTextContent = this.page.getByTestId("link-0-text-content");
readonly linkUrlContent = this.page.getByTestId("link-0-url-content");

constructor(private readonly page: Page) {}

Expand Down
68 changes: 34 additions & 34 deletions ...vtool-frontend/playwright/tests/7-proposal-submission/proposalSubmission.loggedin.spec.ts
View file
Original file line number Diff line number Diff line change
Expand Up @@ -140,22 +140,22 @@ test.describe("Proposal created logged state", () => {
await proposalSubmissionPage.submitBtn.click();

await expect(page.getByTestId("submit-as-GA-button")).toBeVisible();
await expect(page.getByTestId("title-content")).toHaveText(
await expect(proposalSubmissionPage.titleContent).toHaveText(
proposal.prop_name
);
await expect(
page.getByTestId("governance-action-type-content")
proposalSubmissionPage.governanceActionTypeContent
).toHaveText(type);
await expect(page.getByTestId("abstract-content")).toHaveText(
await expect(proposalSubmissionPage.abstractContent).toHaveText(
proposal.prop_abstract
);
await expect(page.getByTestId("motivation-content")).toHaveText(
await expect(proposalSubmissionPage.motivationContent).toHaveText(
proposal.prop_motivation
);
await expect(page.getByTestId("rationale-content")).toHaveText(
await expect(proposalSubmissionPage.rationaleContent).toHaveText(
proposal.prop_rationale
);
await expect(page.getByTestId("link-0-text-content")).toHaveText(
await expect(proposalSubmissionPage.linkTextContent).toHaveText(
proposal.proposal_links[0].prop_link_text
);
});
Expand Down Expand Up @@ -187,28 +187,28 @@ test.describe("Proposal created logged state", () => {
await proposalSubmissionPage.continueBtn.click();

await expect(
page.getByTestId("governance-action-type-content")
proposalSubmissionPage.governanceActionTypeContent
).toHaveText(type);
await expect(page.getByTestId("title-content")).toHaveText(
await expect(proposalSubmissionPage.titleContent).toHaveText(
proposal.prop_name
);
await expect(page.getByTestId("abstract-content")).toHaveText(
await expect(proposalSubmissionPage.abstractContent).toHaveText(
proposal.prop_abstract
);
await expect(page.getByTestId("motivation-content")).toHaveText(
await expect(proposalSubmissionPage.motivationContent).toHaveText(
proposal.prop_motivation
);
await expect(page.getByTestId("rationale-content")).toHaveText(
await expect(proposalSubmissionPage.rationaleContent).toHaveText(
proposal.prop_rationale
);
await expect(page.getByTestId("link-0-text-content")).toHaveText(
await expect(proposalSubmissionPage.linkTextContent).toHaveText(
proposal.proposal_links[0].prop_link_text
);
if (type === ProposalType.treasury) {
await expect(
page.getByTestId("receiving-address-content")
proposalSubmissionPage.receivingAddressContent
).toHaveText(proposal.prop_receiving_address);
await expect(page.getByTestId("amount-content")).toHaveText(
await expect(proposalSubmissionPage.amountContent).toHaveText(
proposal.prop_amount
);
}
Expand Down Expand Up @@ -387,20 +387,20 @@ test.describe("Info Proposal Draft", () => {
await proposalSubmissionPage.titleInput.fill(newTitle);
await proposalSubmissionPage.continueBtn.click();

await expect(page.getByTestId("governance-action-type-content")).toHaveText(
await expect(proposalSubmissionPage.governanceActionTypeContent).toHaveText(
ProposalType.info
);
await expect(page.getByTestId("title-content")).toHaveText(newTitle);
await expect(page.getByTestId("abstract-content")).toHaveText(
await expect(proposalSubmissionPage.titleContent).toHaveText(newTitle);
await expect(proposalSubmissionPage.abstractContent).toHaveText(
proposalFormValue.prop_abstract
);
await expect(page.getByTestId("motivation-content")).toHaveText(
await expect(proposalSubmissionPage.motivationContent).toHaveText(
proposalFormValue.prop_motivation
);
await expect(page.getByTestId("rationale-content")).toHaveText(
await expect(proposalSubmissionPage.rationaleContent).toHaveText(
proposalFormValue.prop_rationale
);
await expect(page.getByTestId("link-0-text-content")).toHaveText(
await expect(proposalSubmissionPage.linkTextContent).toHaveText(
proposalFormValue.proposal_links[0].prop_link_text
);
});
Expand All @@ -421,22 +421,22 @@ test.describe("Info Proposal Draft", () => {
await proposalSubmissionPage.submitBtn.click();

await expect(page.getByTestId("submit-as-GA-button")).toBeVisible();
await expect(page.getByTestId("title-content")).toHaveText(
await expect(proposalSubmissionPage.titleContent).toHaveText(
proposalFormValue.prop_name
);
await expect(page.getByTestId("governance-action-type-content")).toHaveText(
await expect(proposalSubmissionPage.governanceActionTypeContent).toHaveText(
ProposalType.info
);
await expect(page.getByTestId("abstract-content")).toHaveText(
await expect(proposalSubmissionPage.abstractContent).toHaveText(
proposalFormValue.prop_abstract
);
await expect(page.getByTestId("motivation-content")).toHaveText(
await expect(proposalSubmissionPage.motivationContent).toHaveText(
proposalFormValue.prop_motivation
);
await expect(page.getByTestId("rationale-content")).toHaveText(
await expect(proposalSubmissionPage.rationaleContent).toHaveText(
proposalFormValue.prop_rationale
);
await expect(page.getByTestId("link-0-text-content")).toHaveText(
await expect(proposalSubmissionPage.linkTextContent).toHaveText(
proposalFormValue.proposal_links[0].prop_link_text
);
});
Expand All @@ -459,26 +459,26 @@ test.describe("Treasury Proposal Draft", () => {
await proposalSubmissionPage.titleInput.fill(newTitle);
await proposalSubmissionPage.continueBtn.click();

await expect(page.getByTestId("governance-action-type-content")).toHaveText(
await expect(proposalSubmissionPage.governanceActionTypeContent).toHaveText(
ProposalType.treasury
);
await expect(page.getByTestId("title-content")).toHaveText(newTitle);
await expect(page.getByTestId("abstract-content")).toHaveText(
await expect(proposalSubmissionPage.titleContent).toHaveText(newTitle);
await expect(proposalSubmissionPage.abstractContent).toHaveText(
proposalFormValue.prop_abstract
);
await expect(page.getByTestId("motivation-content")).toHaveText(
await expect(proposalSubmissionPage.motivationContent).toHaveText(
proposalFormValue.prop_motivation
);
await expect(page.getByTestId("rationale-content")).toHaveText(
await expect(proposalSubmissionPage.rationaleContent).toHaveText(
proposalFormValue.prop_rationale
);
await expect(page.getByTestId("receiving-address-content")).toHaveText(
await expect(proposalSubmissionPage.receivingAddressContent).toHaveText(
proposalFormValue.prop_receiving_address
);
await expect(page.getByTestId("amount-content")).toHaveText(
await expect(proposalSubmissionPage.amountContent).toHaveText(
proposalFormValue.prop_amount
);
await expect(page.getByTestId("link-0-text-content")).toHaveText(
await expect(proposalSubmissionPage.linkTextContent).toHaveText(
proposalFormValue.proposal_links[0].prop_link_text
);
});
Expand Down

0 comments on commit c4d77ee

Please sign in to comment.