Merge pull request #104 from edx/tasawer/story/ecom-4512-credentials-acceptance-tests

Credentials acceptance tests updated.

Author: tasawernawaz

acceptance_tests/pages.py

@@ -61,7 +61,6 @@ def login(self, username, password):
 class LMSDashboardPage(LMSPage):
     def __init__(self, browser):
         super(LMSDashboardPage, self).__init__(browser)
-        self.credential_css_selector = '.wrapper-xseries-certificates'
 
     @property
     def url(self):
@@ -70,11 +69,27 @@ def url(self):
     def is_browser_on_page(self):
         return self.browser.title.startswith('Dashboard')
 
+    def go_to_programs_tab(self):
+        self.q(css='a[href^="/dashboard/programs/"]').click()
+
+
+class LMSProgramListingPage(LMSPage):
+    def __init__(self, browser):
+        super(LMSProgramListingPage, self).__init__(browser)
+        self.credential_css_selector = '.certificate-container'
+
+    @property
+    def url(self):
+        return self._build_url('dashboard/programs')
+
+    def is_browser_on_page(self):
+        return self.browser.title.startswith('Programs')
+
     def are_credential_links_present(self):
         return self.q(css=self.credential_css_selector).present
 
     def click_credential_link(self):
-        self.q(css=self.credential_css_selector + ' ul li a').click()
+        self.q(css=self.credential_css_selector + ' a').click()
 
     def get_credential_link(self):
-        return self.q(css=self.credential_css_selector + ' ul li a').attrs('href')[0]
+        return self.q(css=self.credential_css_selector + ' a').attrs('href')[0]

acceptance_tests/test_rendering.py

@@ -4,7 +4,7 @@
 from selenium.webdriver.support.ui import WebDriverWait
 
 from acceptance_tests.mixins import LoginMixin, CredentialsApiMixin
-from acceptance_tests.pages import LMSDashboardPage
+from acceptance_tests.pages import LMSDashboardPage, LMSProgramListingPage
 
 
 class RenderCredentialTests(LoginMixin, WebAppTest, CredentialsApiMixin):
@@ -20,11 +20,15 @@ def test_student_dashboard_with_certificate(self):
         student_dashboard = LMSDashboardPage(self.browser).wait_for_page()
         student_dashboard.is_browser_on_page()
 
-        self.assertTrue(student_dashboard.are_credential_links_present())
+        student_dashboard.go_to_programs_tab()
+        program_listing_page = LMSProgramListingPage(self.browser).wait_for_page()
+        program_listing_page.is_browser_on_page()
 
-        credential_link = student_dashboard.get_credential_link()
+        self.assertTrue(program_listing_page.are_credential_links_present())
 
-        student_dashboard.click_credential_link()
+        credential_link = program_listing_page.get_credential_link()
+
+        program_listing_page.click_credential_link()
         WebDriverWait(self.browser, 10).until(EC.presence_of_element_located((By.ID, 'action-print-view')))
 
         # check the credential hash-id matches with rendered credential page.
@@ -41,4 +45,8 @@ def test_student_dashboard_with_revoked_certificate(self):
         student_dashboard = LMSDashboardPage(self.browser).wait_for_page()
         student_dashboard.is_browser_on_page()
 
-        self.assertFalse(student_dashboard.are_credential_links_present())
+        student_dashboard.go_to_programs_tab()
+        program_listing_page = LMSProgramListingPage(self.browser).wait_for_page()
+        program_listing_page.is_browser_on_page()
+
+        self.assertFalse(program_listing_page.are_credential_links_present())