Delete Quickfiles a11y tests

Author: DougCorell

components/navbars.py

@@ -19,7 +19,9 @@ class HomeNavbar(BaseElement):
     support_link = Locator(By.ID, 'navbar-support')
     donate_link = Locator(By.ID, 'navbar-donate')
 
-    user_dropdown = Locator(By.CSS_SELECTOR, 'ul.navbar-nav > li:nth-child(6) > a')
+    user_dropdown = Locator(
+        By.CSS_SELECTOR, 'ul.navbar-nav > li.secondary-nav-dropdown'
+    )
     user_dropdown_profile = Locator(
         By.CSS_SELECTOR, 'ul.dropdown-menu-right > li:nth-child(1)'
     )

pages/project.py

@@ -60,6 +60,7 @@ class MyProjectsPage(OSFBasePage):
     first_collection_remove_button = Locator(
         By.CSS_SELECTOR, '[data-target="#removeColl"]', settings.QUICK_TIMEOUT
     )
+    empty_collection_indicator = Locator(By.CLASS_NAME, 'db-non-load-template')
 
     # Components
     create_collection_modal = ComponentLocator(CreateCollectionModal)

pages/quickfiles.py

@@ -1,20 +1,15 @@
 import re
 
-from selenium.webdriver.common.by import By
 from selenium.webdriver.common.keys import Keys
-from selenium.webdriver.support import expected_conditions as EC
-from selenium.webdriver.support.ui import WebDriverWait
 
 import markers
 import settings
-from api import osf_api
 from components.accessibility import ApplyA11yRules as a11y
 from components.email_access import EmailAccess
 from pages.dashboard import DashboardPage
 from pages.landing import LandingPage
 from pages.login import ForgotPasswordPage, ResetPasswordPage
 from pages.project import MyProjectsPage
-from pages.quickfiles import QuickfileDetailPage, QuickfilesPage
 from pages.register import RegisterPage
 from pages.search import SearchPage
 from pages.support import SupportPage
@@ -60,6 +55,7 @@ def test_accessibility(
         my_projects_page = MyProjectsPage(driver)
         my_projects_page.goto()
         assert MyProjectsPage(driver, verify=True)
+        my_projects_page.empty_collection_indicator.present()
         a11y.run_axe(
             driver,
             session,
@@ -69,56 +65,6 @@ def test_accessibility(
         )
 
 
-@markers.ember_page
-class TestMyQuickFilesPage:
-    def test_accessibility(
-        self, driver, session, write_files, exclude_best_practice, must_be_logged_in
-    ):
-        """ The Quick Files Page may or may not have any files listed depending on the
-        current user. Users in the testing environment probably will, but the Production
-        user may not.
-        """
-        quickfiles_page = QuickfilesPage(driver)
-        quickfiles_page.goto()
-        quickfiles_page.loading_indicator.here_then_gone()
-        assert QuickfilesPage(driver, verify=True)
-        a11y.run_axe(
-            driver,
-            session,
-            'quickfiles',
-            write_files=write_files,
-            exclude_best_practice=exclude_best_practice,
-        )
-
-
-@markers.ember_page
-# For this next test we are uploading a text file to the user's Quick Files page so we
-#     don't want to run this in Production.
-@markers.dont_run_on_prod
-class TestMyQuickFileDetailPage:
-    def test_accessibility(
-        self, driver, session, write_files, exclude_best_practice, must_be_logged_in
-    ):
-        osf_api.upload_single_quickfile(session)
-        quickfiles_page = QuickfilesPage(driver)
-        quickfiles_page.goto()
-        quickfiles_page.loading_indicator.here_then_gone()
-        # click on title of file in table to open the Quick File Detail page
-        quickfiles_page.file_titles[0].click()
-        assert QuickfileDetailPage(driver, verify=True)
-        # wait for mfr iframe to be visible before running axe
-        WebDriverWait(driver, 10).until(
-            EC.visibility_of_element_located((By.ID, 'mfrIframeParent'))
-        )
-        a11y.run_axe(
-            driver,
-            session,
-            'qfiledet',
-            write_files=write_files,
-            exclude_best_practice=exclude_best_practice,
-        )
-
-
 @markers.ember_page
 class TestRegisterPage:
     def test_accessibility(self, driver, session, write_files, exclude_best_practice):