End-to-end bok choy test for cohorted courseware.

common/test/acceptance/pages/lms/courseware.py

@@ -33,12 +33,19 @@ def num_subsections(self):
         """
         return len(self.q(css=self.subsection_selector))
 
+    @property
+    def xblock_components(self):
+        """
+        Return the xblock components within the unit on the page.
+        """
+        return self.q(css=self.xblock_component_selector)
+
     @property
     def num_xblock_components(self):
         """
         Return the number of rendered xblocks within the unit on the page
         """
-        return len(self.q(css=self.xblock_component_selector))
+        return len(self.xblock_components)
 
     def xblock_component_type(self, index=0):
         """

common/test/acceptance/pages/lms/instructor_dashboard.py

@@ -133,6 +133,10 @@ def get_selected_cohort(self):
         """
         Returns the name of the selected cohort.
         """
+        EmptyPromise(
+            lambda: len(self._get_cohort_options().results) > 0,
+            "Waiting for cohort selector to populate"
+        ).fulfill()
         return self._cohort_name(
             self._get_cohort_options().filter(lambda el: el.is_selected()).first.text[0]
         )
@@ -163,7 +167,10 @@ def add_cohort(self, cohort_name, content_group=None):
         Adds a new manual cohort with the specified name.
         If a content group should also be associated, the name of the content group should be specified.
         """
-        self.q(css=self._bounded_selector("div.cohort-management-nav .action-create")).first.click()
+        create_buttons = self.q(css=self._bounded_selector(".action-create"))
+        # There are 2 create buttons on the page. The second one is only present when no cohort yet exists
+        # (in which case the first is not visible). Click on the last present create button.
+        create_buttons.results[len(create_buttons.results) - 1].click()
         textinput = self.q(css=self._bounded_selector("#cohort-name")).results[0]
         textinput.send_keys(cohort_name)
         if content_group:

common/test/acceptance/pages/studio/auto_auth.py

@@ -15,7 +15,7 @@ class AutoAuthPage(PageObject):
     this url will create a user and log them in.
     """
 
-    def __init__(self, browser, username=None, email=None, password=None, staff=None, course_id=None, roles=None):
+    def __init__(self, browser, username=None, email=None, password=None, staff=None, course_id=None, roles=None, no_login=None):
         """
         Auto-auth is an end-point for HTTP GET requests.
         By default, it will create accounts with random user credentials,
@@ -51,6 +51,9 @@ def __init__(self, browser, username=None, email=None, password=None, staff=None
         if roles is not None:
             self._params['roles'] = roles
 
+        if no_login:
+            self._params['no_login'] = True
+
     @property
     def url(self):
         """
@@ -66,7 +69,7 @@ def url(self):
 
     def is_browser_on_page(self):
         message = self.q(css='BODY').text[0]
-        match = re.search(r'Logged in user ([^$]+) with password ([^$]+) and user_id ([^$]+)$', message)
+        match = re.search(r'(Logged in|Created) user ([^$]+) with password ([^$]+) and user_id ([^$]+)$', message)
         return True if match else False
 
     def get_user_id(self):

common/test/acceptance/tests/lms/test_lms_user_preview.py

@@ -266,28 +266,26 @@ def populate_course_fixture(self, course_fixture):
             </problem>
         """)
 
+        self.alpha_text = "VISIBLE TO ALPHA"
+        self.beta_text = "VISIBLE TO BETA"
+        self.everyone_text = "VISIBLE TO EVERYONE"
+
         course_fixture.add_children(
             XBlockFixtureDesc('chapter', 'Test Section').add_children(
                 XBlockFixtureDesc('sequential', 'Test Subsection').add_children(
-                    XBlockFixtureDesc(
-                        'problem', 'Visible to alpha', data=problem_data, metadata={"group_access": {0: [0]}}
-                    ),
-                    XBlockFixtureDesc(
-                        'problem', 'Visible to beta', data=problem_data, metadata={"group_access": {0: [1]}}
-                    ),
-                    XBlockFixtureDesc('problem', 'Visible to everyone', data=problem_data)
+                    XBlockFixtureDesc('vertical', 'Test Unit').add_children(
+                        XBlockFixtureDesc(
+                            'problem', self.alpha_text, data=problem_data, metadata={"group_access": {0: [0]}}
+                        ),
+                        XBlockFixtureDesc(
+                            'problem', self.beta_text, data=problem_data, metadata={"group_access": {0: [1]}}
+                        ),
+                        XBlockFixtureDesc('problem', self.everyone_text, data=problem_data)
+                    )
                 )
             )
         )
 
-    def _verify_visible_problems(self, expected_items):
-        """
-        Verify that the expected problems are visible.
-        """
-        course_nav = CourseNavPage(self.browser)
-        actual_items = course_nav.sequence_items
-        self.assertItemsEqual(expected_items, actual_items)
-
     def test_staff_sees_all_problems(self):
         """
         Scenario: Staff see all problems
@@ -296,8 +294,8 @@ def test_staff_sees_all_problems(self):
         When I view the courseware in the LMS with staff access
         Then I see all the problems, regardless of their group_access property
         """
-        self._goto_staff_page()
-        self._verify_visible_problems(['Visible to alpha', 'Visible to beta', 'Visible to everyone'])
+        course_page = self._goto_staff_page()
+        verify_expected_problem_visibility(self, course_page, [self.alpha_text, self.beta_text, self.everyone_text])
 
     def test_student_not_in_content_group(self):
         """
@@ -310,7 +308,7 @@ def test_student_not_in_content_group(self):
         """
         course_page = self._goto_staff_page()
         course_page.set_staff_view_mode('Student')
-        self._verify_visible_problems(['Visible to everyone'])
+        verify_expected_problem_visibility(self, course_page, [self.everyone_text])
 
     def test_as_student_in_alpha(self):
         """
@@ -323,7 +321,7 @@ def test_as_student_in_alpha(self):
         """
         course_page = self._goto_staff_page()
         course_page.set_staff_view_mode('Student in alpha')
-        self._verify_visible_problems(['Visible to alpha', 'Visible to everyone'])
+        verify_expected_problem_visibility(self, course_page, [self.alpha_text, self.everyone_text])
 
     def test_as_student_in_beta(self):
         """
@@ -336,4 +334,15 @@ def test_as_student_in_beta(self):
         """
         course_page = self._goto_staff_page()
         course_page.set_staff_view_mode('Student in beta')
-        self._verify_visible_problems(['Visible to beta', 'Visible to everyone'])
+        verify_expected_problem_visibility(self, course_page, [self.beta_text, self.everyone_text])
+
+
+def verify_expected_problem_visibility(test, courseware_page, expected_problems):
+    """
+    Helper method that checks that the expected problems are visible on the current page.
+    """
+    test.assertEqual(
+        len(expected_problems), courseware_page.num_xblock_components, "Incorrect number of visible problems"
+    )
+    for index, expected_problem in enumerate(expected_problems):
+        test.assertIn(expected_problem, courseware_page.xblock_components[index].text)

common/test/acceptance/tests/studio/base_studio_test.py

@@ -53,12 +53,12 @@ class ContainerBase(StudioCourseTest):
     Base class for tests that do operations on the container page.
     """
 
-    def setUp(self):
+    def setUp(self, is_staff=False):
         """
         Create a unique identifier for the course used in this test.
         """
         # Ensure that the superclass sets up
-        super(ContainerBase, self).setUp()
+        super(ContainerBase, self).setUp(is_staff=is_staff)
 
         self.outline = CourseOutlinePage(
             self.browser,

common/test/acceptance/tests/studio/test_studio_split_test.py

@@ -14,7 +14,6 @@
 from ...fixtures.course import XBlockFixtureDesc
 from ...pages.studio.component_editor import ComponentEditorView
 from ...pages.studio.overview import CourseOutlinePage, CourseOutlineUnit
-from ...pages.studio.settings_advanced import AdvancedSettingsPage
 from ...pages.studio.container import ContainerPage
 from ...pages.studio.settings_group_configurations import GroupConfigurationsPage
 from ...pages.studio.utils import add_advanced_component