Merge pull request #3350 from IntersectMBO/fix/test-8E-8F

Fix: test 8E and 8F

Author: kneerose

tests/govtool-frontend/playwright/lib/_mock/proposalComments.json

@@ -9,6 +9,10 @@
         "comment_text": "Hello",
         "createdAt": "2024-06-14T13:38:35.830Z",
         "updatedAt": "2024-06-14T13:38:35.830Z",
+        "bd_proposal_id": null,
+        "comments_reports": {
+          "data": []
+        },
         "user_govtool_username": "Anonymous",
         "subcommens_number": 0
       }
@@ -22,6 +26,10 @@
         "comment_text": "Nice proposal",
         "createdAt": "2024-06-14T13:38:31.279Z",
         "updatedAt": "2024-06-14T13:38:31.279Z",
+        "bd_proposal_id": null,
+        "comments_reports": {
+          "data": []
+        },
         "user_govtool_username": "Anonymous",
         "subcommens_number": 0
       }
@@ -35,6 +43,10 @@
         "comment_text": "Go Ahead",
         "createdAt": "2024-06-14T13:38:27.286Z",
         "updatedAt": "2024-06-14T13:38:27.286Z",
+        "bd_proposal_id": null,
+        "comments_reports": {
+          "data": []
+        },
         "user_govtool_username": "Anonymous",
         "subcommens_number": 0
       }

tests/govtool-frontend/playwright/tests/7-proposal-submission/proposalSubmission.loggedin.spec.ts

@@ -33,7 +33,7 @@ test.describe("Proposal created logged state", () => {
     await page.goto("/");
     await page.getByTestId("proposal-discussion-link").click();
 
-    await expect(page.getByText(/proposals/i)).toHaveCount(2);
+    await expect(page.getByText("Proposals", { exact: true })).toHaveCount(2);
   });
 
   test.describe("Accept valid data", () => {

tests/govtool-frontend/playwright/tests/8-proposal-discussion/proposalDiscussion.spec.ts

@@ -8,6 +8,7 @@ import { test } from "@fixtures/proposal";
 import { setAllureEpic } from "@helpers/allure";
 import { isBootStrapingPhase, skipIfNotHardFork } from "@helpers/cardano";
 import { injectLogger } from "@helpers/page";
+import { extractProposalIdFromUrl } from "@helpers/string";
 import { functionWaitedAssert } from "@helpers/waitedLoop";
 import ProposalDiscussionDetailsPage from "@pages/proposalDiscussionDetailsPage";
 import ProposalDiscussionPage from "@pages/proposalDiscussionPage";
@@ -195,6 +196,31 @@ test("8S. Should restrict proposal creation on disconnected state", async ({
   await expect(proposalDiscussionPage.proposalCreateBtn).not.toBeVisible();
 });
 
+test("8E. Should share proposed governance action", async ({
+  page,
+  context,
+}) => {
+  await context.grantPermissions(["clipboard-read", "clipboard-write"]);
+  const proposalDiscussionPage = new ProposalDiscussionPage(page);
+  await proposalDiscussionPage.goto();
+
+  await proposalDiscussionPage.viewFirstProposal();
+
+  const currentPageUrl = page.url();
+  const proposalId = extractProposalIdFromUrl(currentPageUrl);
+
+  await page.getByTestId("share-button").click();
+  await page.getByTestId("copy-link").click();
+  await expect(page.getByTestId("copy-link-text")).toBeVisible();
+
+  const copiedTextDRepDirectory = await page.evaluate(() =>
+    navigator.clipboard.readText()
+  );
+  const expectedCopyUrl = `${environments.frontendUrl}/proposal_discussion/${proposalId}`;
+
+  expect(copiedTextDRepDirectory).toEqual(expectedCopyUrl);
+});
+
 test.describe("Mocked proposal", () => {
   let proposalDiscussionDetailsPage: ProposalDiscussionDetailsPage;
 
@@ -221,24 +247,6 @@ test.describe("Mocked proposal", () => {
     await proposalDiscussionDetailsPage.goto(mockProposal.data.id);
   });
 
-  test("8E. Should share proposed governance action", async ({
-    page,
-    context,
-  }) => {
-    await context.grantPermissions(["clipboard-read", "clipboard-write"]);
-
-    await page.getByTestId("share-button").click();
-    await page.getByTestId("copy-link").click();
-    await expect(page.getByTestId("copy-link-text")).toBeVisible();
-
-    const copiedTextDRepDirectory = await page.evaluate(() =>
-      navigator.clipboard.readText()
-    );
-    const expectedCopyUrl = `${environments.frontendUrl}/proposal_discussion/${mockProposal.data.id}`;
-
-    expect(copiedTextDRepDirectory).toEqual(expectedCopyUrl);
-  });
-
   test("8I. Should disable poll voting functionality.", async () => {
     await expect(proposalDiscussionDetailsPage.pollVoteCard).not.toBeVisible();
     await expect(proposalDiscussionDetailsPage.pollYesBtn).not.toBeVisible();