Updated create_project function and related tests.

When a project is created we now check that we can change the privacy property, the name and the short description.
Also added tests to check that we can change name and short description when on editor.

Author: konstantina

codebender_testing/utils.py

@@ -857,17 +857,24 @@ def execute_script(self, script, *deps):
             )
         return self.driver.execute_script(script)
 
-    def create_sketch(self, name):
+    def create_sketch(self, privacy, name, description):
         """Creates a sketch with a given name"""
         createSketchBtn = self.driver.find_element_by_id('create_sketch_btn')
         createSketchBtn.click()
-        createBtn = self.get_element(By.ID, 'create-sketch-modal-action-button')
-        createBtn.click()
         WebDriverWait(self.driver, VERIFY_TIMEOUT).until(
                 expected_conditions.visibility_of_element_located(
-                    (By.CSS_SELECTOR, "#editor-loading-screen")
+                    (By.CSS_SELECTOR, "#create-sketch-modal")
                 )
             )
+
+        self.change_privacy(privacy)
+
+        self.change_name(name)
+
+        self.change_short_description(description)
+
+        createBtn = self.get_element(By.ID, 'create-sketch-modal-action-button')
+        createBtn.click()
         WebDriverWait(self.driver, VERIFY_TIMEOUT).until(
                 expected_conditions.invisibility_of_element_located(
                     (By.CSS_SELECTOR, "#editor-loading-screen")

tests/common/sketch/test_sketch.py

@@ -8,6 +8,7 @@
 from codebender_testing.utils import SeleniumTestCase
 from codebender_testing.utils import SELECT_BOARD_SCRIPT
 from codebender_testing.utils import throttle_compile
+from selenium.webdriver.common.keys import Keys
 
 
 # How long to wait before we give up on trying to assess the result of commands
@@ -25,7 +26,13 @@ class TestSketch(SeleniumTestCase):
     def create_test_project(self, tester_login):
         """Makes sure we are logged in and have a project open before
         performing any of these tests."""
-        self.create_sketch(TEST_PROJECT_NAME)
+        self.create_sketch('private' , 'project', 'short description')
+
+    def test_rename_project(self):
+        self.change_name_editor(TEST_PROJECT_NAME)
+
+    def test_change_short_description(self):
+        self.change_short_description_editor('decription')
 
     def test_verify_code(self):
         """Ensures that we can compile code and see the success message."""
@@ -102,7 +109,7 @@ def test_clone_project(self):
         clone_link.click()
         project_name = self.get_element(By.ID, 'editor_heading_project_name')
         # Here, I use `startswith` in case the user has a bunch of
-        # projects like "test_project copy copy copy" ...
+        # projects like "test_project copy copy copy"
         assert project_name.text.startswith("%s copy" % TEST_PROJECT_NAME)
 
         # Cleanup: delete the project we just created.

tests/noplugin/alerts/test_alerts.py

@@ -14,7 +14,7 @@ class TestAlerts(SeleniumTestCase):
     def create_test_project(self, tester_login):
         """Makes sure we are logged in and have a project open before
         performing any of these tests."""
-        self.create_sketch(TEST_PROJECT_NAME)
+        self.create_sketch('private' , TEST_PROJECT_NAME, 'short description')
 
     @pytest.mark.does_not_require_extension
     def test_alert(self):