Merge pull request #7673 from emilghittasv/playwright-new-tests

Playwright: Expand test coverage to Moderation /flagged page

Author: emilghittasv

.github/workflows/playwright.yml

@@ -46,6 +46,7 @@ on:
                     - communityForums
                     - adminAnnouncementBanners
                     - userAdminPages
+                    - flaggedModerationPage
                     - smokeTest
 
 env:
@@ -102,7 +103,7 @@ jobs:
         run: |
             source ../.venv/bin/activate
             declare dispatch_test_suite="${{inputs.TestSuite}}"
-            declare all_test_suites=("homePageTests" "topNavbarTests" "footerSectionTests" "contributePagesTests" "messagingSystem" "userContributionTests" "userProfile" "userSettings" "editUserProfileTests" "userQuestions" "contactSupportPage" "productSolutionsPage" "productSupportPage" "productTopicsPage" "aaqPage" "postedQuestions" "kbProductsPage" "kbArticleCreationAndAccess" "beforeThreadTests" "articleThreads" "afterThreadTests" "kbArticleShowHistory" "recentRevisionsDashboard" "kbDashboard" "kbRestrictedVisibility" "kbArticleTranslation" "exploreByTopics" "searchTests" "contributorForumSearch" "userGroupsTests" "antiSpamTests" "contributorDiscussions" "contributorDiscussionsThreads" "userDeletion" "communityForums" "adminAnnouncementBanners" "userAdminPages" "mediaGalleryTests" "kbTemplatesCategoryPage")
+            declare all_test_suites=("homePageTests" "topNavbarTests" "footerSectionTests" "contributePagesTests" "messagingSystem" "userContributionTests" "userProfile" "userSettings" "editUserProfileTests" "userQuestions" "contactSupportPage" "productSolutionsPage" "productSupportPage" "productTopicsPage" "aaqPage" "postedQuestions" "kbProductsPage" "kbArticleCreationAndAccess" "beforeThreadTests" "articleThreads" "afterThreadTests" "kbArticleShowHistory" "recentRevisionsDashboard" "kbDashboard" "kbRestrictedVisibility" "kbArticleTranslation" "exploreByTopics" "searchTests" "contributorForumSearch" "userGroupsTests" "antiSpamTests" "contributorDiscussions" "contributorDiscussionsThreads" "userDeletion" "communityForums" "adminAnnouncementBanners" "userAdminPages" "mediaGalleryTests" "kbTemplatesCategoryPage" "flaggedModerationPage")
             if [ "$dispatch_test_suite" == "All" ] ; then
                 for test in "${all_test_suites[@]}"; do
                     if ! pytest -m ${test} --numprocesses 6 --browser ${BROWSER} --reruns 2 -v; then

playwright_tests/flows/ask_a_question_flows/aaq_flows/aaq_flow.py

@@ -215,19 +215,24 @@ def post_question_reply_flow(self, repliant_username: str, reply='', submit_repl
         if submit_reply:
             return self.question_page.click_on_post_reply_button(repliant_username, fetch_id)
 
-    def report_question_abuse(self, answer_id="", text=''):
+    def report_question_abuse(self, answer_id="", text='', report_reason=""):
         """
         Flow for reporting question as abusive.
         Args:
             answer_id (str): The ID of the question reply.
             text: Text to be added inside the report.
+            report_reason (str): The reason radio value to select (e.g. 'spam', 'language',
+                'abuse', 'other'). Defaults to the preselected 'spam' reason when not provided.
         """
 
         if answer_id:
             self.question_page.click_on_report_abuse_for_a_certain_reply(answer_id)
         else:
             self.question_page.click_on_question_report_abuse_option()
 
+        if report_reason:
+            self.question_page.click_on_report_abuse_reason_option(report_reason)
+
         if text:
             self.question_page.add_text_to_report_abuse_textarea(text)
 

playwright_tests/messages/ask_a_question_messages/AAQ_messages/question_page_messages.py

@@ -3,6 +3,8 @@ class QuestionPageMessages:
     EDIT_REPLY_URL = "/edit/"
     DELETE_QUESTION_URL_ENDPOINT = '/delete'
     DELETE_QUESTION_REPLY_URL = '/delete/'
+    DELETE_QUESTION_CONFIRMATION_HEADING = "Are you sure you want to delete this question?"
+    DELETE_ANSWER_CONFIRMATION_HEADING = "Are you sure you want to delete this answer?"
     LOCKED_THREAD_BANNER = "This thread was closed. Please ask a new question if you need help."
     ARCHIVED_THREAD_BANNER = ("This thread was archived. Please ask a new question if you need "
                               "help.")

playwright_tests/messages/common_elements_messages.py

@@ -2,3 +2,4 @@ class CommonElementsMessages:
     AVOID_SCAM_BANNER_TEXT = ("Avoid support scams. We will never ask you to call or text a phone "
                               "number or share personal information. Please report suspicious "
                               "activity using the “Report Abuse” option.")
+    CONTRIBUTOR_TOOLS_SIDEBAR_HEADING = "Contributor tools"

playwright_tests/messages/contribute_messages/con_tools/moderate_forum_messages.py

@@ -1,3 +1,10 @@
+from playwright_tests.messages.homepage_messages import HomepageMessages
+
+
 class ModerateForumContentPageMessages:
+    PAGE_URL = HomepageMessages.STAGE_HOMEPAGE_URL_EN_US + "flagged"
+    DEACTIVATED_USERS_PAGE_URL = HomepageMessages.STAGE_HOMEPAGE_URL_EN_US + (
+        "users/deactivation_log")
     UPDATE_STATUS_FIRST_VALUE = "1"
     UPDATE_STATUS_SECOND_VALUE = "2"
+    SIDEBAR_OPTION_NAME = "Moderate forum content"

playwright_tests/pages/ask_a_question/posted_question_pages/questions_page.py

@@ -223,11 +223,19 @@ def __init__(self, page: Page):
         """Locators belonging to the delete question section."""
         self.delete_question_delete_button = page.locator("input[value='Delete']")
         self.delete_question_cancel_button = page.get_by_role("link").filter(has_text="Cancel")
+        self.delete_confirmation_section = page.locator("div[class='to-delete highlight-box']")
+        self.delete_confirmation_heading = page.locator("div[class='to-delete highlight-box'] h1")
 
         """Locators belonging to the report abuse section."""
         self.report_abuse_submit_button = page.locator(
             "div[class='mzp-c-modal-inner'] button[type='submit']")
         self.report_abuse_textarea = page.locator("div[class='mzp-c-modal-inner'] textarea")
+        # The reason radio options ('spam', 'language', 'abuse', 'other') inside the currently
+        # open report-abuse modal. 'spam' is preselected by default. The radio <input>s are
+        # visually hidden by the styled-radio CSS, so the (clickable) <label> is targeted
+        # instead - matched by its 'for' attribute suffix, since the id prefix is dynamic.
+        self.report_abuse_reason_option = lambda reason_value: page.locator(
+            "div[class='mzp-c-modal-inner']").locator(f"label[for$='_{reason_value}']")
         self.report_abuse_flagged_this_content_message = page.locator(
             "div[class='mzp-c-modal-inner'] span[class='message']")
         self.report_abuse_modal_close_button = page.locator(
@@ -256,6 +264,9 @@ def click_abuse_modal_close_button(self):
     def add_text_to_report_abuse_textarea(self, text: str):
         self._fill(self.report_abuse_textarea, text)
 
+    def click_on_report_abuse_reason_option(self, reason_value: str):
+        self._click(self.report_abuse_reason_option(reason_value))
+
     def click_on_report_abuse_submit_button(self):
         self._click(self.report_abuse_submit_button)
 

playwright_tests/pages/contribute/contributor_tools_pages/moderate_forum_content.py

@@ -7,7 +7,11 @@ def __init__(self, page: Page):
         super().__init__(page)
 
         """Locators belonging to the View All deactivated users page."""
-        self.view_all_deactivated_users_button = page.locator("div[class='sumo-button-wrap'] a")
+        self.view_all_deactivated_users_button = page.get_by_role(
+            "link", name="View all deactivated users", exact=True)
+        self.deactivation_log_deactivated_user = lambda username: page.locator(
+            f"//table[@id='deactivations-log']/tbody/tr/td[2]/a[normalize-space(text())="
+            f"'{username}']")
 
         """Locators belonging to the flagged questions page."""
         self.flagged_question = lambda question_info: page.locator("p").get_by_text(
@@ -18,13 +22,13 @@ def __init__(self, page: Page):
             question_title, exact=True).locator("> div[class='content'] p")
         self.created_by_link_text = lambda question_info: page.locator(
             f"//p[normalize-space(text())='{question_info}']/ancestor::div"
-            f"[@class='flagged-item-content']//h3[text()='Created:']/following-sibling::p/a")
+            f"[@class='flagged-item-content']//h3[text()='Created:']/following-sibling::p[1]/a")
         self.flagged_by_link_text = lambda question_info: page.locator(
             f"//p[normalize-space(text())='{question_info}']/ancestor::div"
-            f"[@class='flagged-item-content']//h3[text()='Flagged:']/following-sibling::p/a")
-        self.take_action_view_option = lambda question_info: page.get_by_role(
-            "paragraph", name=question_info, exact=True).locator("+ div").get_by_role(
-            "link", name="View", exact=True)
+            f"[@class='flagged-item-content']//h3[text()='Flagged:']/following-sibling::p[1]/a")
+        self.take_action_view_option = lambda flagged_info: page.locator(
+            f"//li[.//*[normalize-space(text())='{flagged_info}']]"
+            f"//a[normalize-space(text())='View']")
         self.take_action_edit_option = lambda question_info: page.locator(
             f"//p[normalize-space(text())='{question_info}']/ancestor::"
             f"div[@class='flagged-item-content']//a[text()='Edit']")
@@ -37,9 +41,23 @@ def __init__(self, page: Page):
         self.update_status_button = lambda question_info: page.locator(
             f"//p[normalize-space(text())='{question_info}']/ancestor::div"
             f"[@class='flagged-item-content']//following-sibling::form/input[@value='Update']")
+        # Profile tickets render the flagged username inside an <h2> (rather than a <p>), so the
+        # profile update-status controls are anchored on that heading instead.
+        self.update_profile_status_option = lambda username: page.locator(
+            f"//h2[@class='sumo-page-subheading' and text()='{username}']/ancestor::div"
+            f"[@class='flagged-item-content']//following-sibling::form/select")
+        self.update_profile_status_button = lambda username: page.locator(
+            f"//h2[@class='sumo-page-subheading' and text()='{username}']/ancestor::div"
+            f"[@class='flagged-item-content']//following-sibling::form/input[@value='Update']")
         self.paginator_section = page.locator("//ol[@class='pagination cf']")
         self.last_paginator_option = page.locator("//ol[@class='pagination cf']/li/a").last
 
+        """Locators belonging to the 'Filter by reason' and 'Filter by type' dropdowns."""
+        self.filter_by_reason_dropdown = page.locator("select#flagit-reason-filter")
+        self.filter_by_type_dropdown = page.locator("select#flagit-content-type-filter")
+        self.filter_by_type_option = lambda type_name: page.locator(
+            f"//select[@id='flagit-content-type-filter']/option[normalize-space()='{type_name}']")
+
         """Locators belonging to the flagged profile tickets"""
         self.profile_flagged_ticket = lambda username: page.locator(
             f"//h2[@class='sumo-page-subheading' and text()='{username}']")
@@ -57,8 +75,8 @@ def click_created_by_link(self, question_info: str):
     def click_flagged_by_link(self, question_info: str):
         self._click(self.flagged_by_link_text(question_info))
 
-    def click_take_action_view_option(self, question_info: str):
-        self._click(self.take_action_view_option(question_info))
+    def click_take_action_view_option(self, flagged_info: str):
+        self._click(self.take_action_view_option(flagged_info))
 
     def click_take_action_edit_option(self, question_info: str):
         self._click(self.take_action_edit_option(question_info))
@@ -72,9 +90,22 @@ def select_update_status_option(self, question_info: str, select_value: str):
     def click_on_the_update_button(self, question_info: str):
         self._click(self.update_status_button(question_info))
 
+    def select_update_profile_status_option(self, username: str, select_value: str):
+        self._select_option_by_value(self.update_profile_status_option(username), select_value)
+
+    def click_on_the_update_profile_button(self, username: str):
+        self._click(self.update_profile_status_button(username))
+
     def click_view_all_deactivated_users_button(self):
         self._click(self.view_all_deactivated_users_button)
 
+    def filter_flagged_content_by_reason(self, reason_value: str):
+        self._select_option_by_value(self.filter_by_reason_dropdown, reason_value)
+
+    def filter_flagged_content_by_type(self, type_name: str):
+        value = self._get_element_attribute_value(self.filter_by_type_option(type_name), "value")
+        self._select_option_by_value(self.filter_by_type_dropdown, value)
+
     def is_paginator_visible(self) -> bool:
         self.wait_for_dom_to_load()
         return self._is_element_visible(self.paginator_section)

playwright_tests/pytest.ini

@@ -39,4 +39,5 @@ markers =
     communityForums: Tests belonging to the community forums suite.
     adminAnnouncementBanners: Tests belonging to announcement banners.
     userAdminPages: Tests belonging to the admin user pages.
+    flaggedModerationPage: Tests belonging to the flagged content moderation page.
 addopts = --alluredir=./reports/allure_reports --video retain-on-failure --tb=no