Merge pull request #1389 from seleniumbase/prefer-using-js-over-jquery

Prefer JS over jQuery, optimize timings, and refresh example tests

Author: mdmintz

examples/boilerplates/samples/test_page_objects.py

@@ -1,5 +1,4 @@
-""" Example test that uses the Page Object Model """
-
+""" An example test using the Classic Page Object Model """
 from seleniumbase import BaseCase
 
 
@@ -24,8 +23,6 @@ def click_seleniumbase_io_link(self, sb):
 
 class SeleniumBaseIOPage:
     def do_search_and_click(self, sb, search_term):
-        if sb.is_element_visible('[for="__search"] svg'):
-            sb.click('[for="__search"] svg')
         sb.type('form[name="search"] input', search_term)
         sb.click("li.md-search-result__item h1:contains(%s)" % search_term)
 

examples/github_test.py

@@ -11,8 +11,7 @@ def test_github(self):
         self.slow_click('a[href="/seleniumbase/SeleniumBase"]')
         self.click_if_visible('[data-action="click:signup-prompt#dismiss"]')
         self.assert_element("div.repository-content")
-        self.assert_text("SeleniumBase", "h2 strong")
+        self.assert_text("SeleniumBase", "strong a")
         self.slow_click('a[title="seleniumbase"]')
         self.slow_click('a[title="fixtures"]')
-        self.slow_click('a[title="base_case.py"]')
-        self.assert_text("Code", "nav a.selected")
+        self.assert_element('a[title="base_case.py"]')

examples/test_download_files.py

@@ -19,14 +19,8 @@ def test_download_chromedriver_notes(self):
         text = "Switching to nested frame fails with chrome/chromedriver 100"
         self.assert_true(text in notes_data)  # Verify file has expected data
 
-    def test_download_files_from_pypi_with_edge(self):
-        if self.browser != "edge":
-            self.open("data:,")
-            print("\n  This test is only for Microsoft Edge (Chromium)!")
-            print('  (Run this test using "--edge" or "--browser=edge")')
-            self.skip('Use "--edge" or "--browser=edge"')
-
-        self.open("https://pypi.org/project/seleniumbase/#files")
+    def test_download_files_from_pypi(self):
+        self.open("https://pypi.org/project/sbvirtualdisplay/#files")
         pkg_header = self.get_text("h1.package-header__name").strip()
         pkg_name = pkg_header.replace(" ", "-")
         whl_file = pkg_name + "-py2.py3-none-any.whl"

examples/test_usefixtures.py

@@ -12,3 +12,4 @@ def test_usefixtures_on_class(self):
         sb.assert_text("integrations")
         sb.assert_element('a[title="help_docs"]')
         sb.click('a[title="examples"]')
+        sb.assert_exact_text("examples", "strong.final-path")

examples/tour_examples/ReadMe.md

@@ -105,7 +105,7 @@ from seleniumbase import BaseCase
 class MyTourClass(BaseCase):
 
     def test_google_tour(self):
-        self.open('https://google.com')
+        self.open('https://google.com/ncr')
         self.wait_for_element('input[title="Search"]')
 
         self.create_tour(theme="dark")

examples/wordle_archive_test.py

@@ -0,0 +1,107 @@
+""" Solve the Wordle game using SeleniumBase.
+    This test runs on archived versions of Wordle, containing Shadow-DOM. """
+
+import ast
+import random
+import requests
+from seleniumbase import version_info
+from seleniumbase import BaseCase
+
+
+class WordleTests(BaseCase):
+
+    word_list = []
+
+    def initialize_word_list(self):
+        txt_file = "https://seleniumbase.io/cdn/txt/wordle_words.txt"
+        word_string = requests.get(txt_file).text
+        self.word_list = ast.literal_eval(word_string)
+
+    def modify_word_list(self, word, letter_status):
+        new_word_list = []
+        correct_letters = []
+        present_letters = []
+        for i in range(len(word)):
+            if letter_status[i] == "correct":
+                correct_letters.append(word[i])
+                for w in self.word_list:
+                    if w[i] == word[i]:
+                        new_word_list.append(w)
+                self.word_list = new_word_list
+                new_word_list = []
+        for i in range(len(word)):
+            if letter_status[i] == "present":
+                present_letters.append(word[i])
+                for w in self.word_list:
+                    if word[i] in w and word[i] != w[i]:
+                        new_word_list.append(w)
+                self.word_list = new_word_list
+                new_word_list = []
+        for i in range(len(word)):
+            if (
+                letter_status[i] == "absent"
+                and word[i] not in correct_letters
+                and word[i] not in present_letters
+            ):
+                for w in self.word_list:
+                    if word[i] not in w:
+                        new_word_list.append(w)
+                self.word_list = new_word_list
+                new_word_list = []
+
+    def skip_if_incorrect_env(self):
+        if self.headless:
+            message = "This test doesn't run in headless mode!"
+            print(message)
+            self.skip(message)
+        if version_info < [2, 4, 4]:
+            message = "This test requires SeleniumBase 2.4.4 or newer!"
+            print(message)
+            self.skip(message)
+
+    def test_wordle(self):
+        self.skip_if_incorrect_env()
+        random.seed()
+        year = "2022"
+        month = random.randint(3, 5)
+        day = random.randint(1, 30)
+        date = str(year) + "0" + str(month) + str(day)
+        archive = "https://web.archive.org/web/"
+        url = "https://www.nytimes.com/games/wordle/index.html"
+        past_wordle = archive + date + "/" + url
+        print(past_wordle)
+        self.open(past_wordle)
+        self.click("game-app::shadow game-modal::shadow game-icon")
+        self.initialize_word_list()
+        keyboard_base = "game-app::shadow game-keyboard::shadow "
+        word = random.choice(self.word_list)
+        total_attempts = 0
+        success = False
+        for attempt in range(6):
+            total_attempts += 1
+            word = random.choice(self.word_list)
+            letters = []
+            for letter in word:
+                letters.append(letter)
+                button = 'button[data-key="%s"]' % letter
+                self.click(keyboard_base + button)
+            button = "button.one-and-a-half"
+            self.click(keyboard_base + button)
+            row = 'game-app::shadow game-row[letters="%s"]::shadow ' % word
+            tile = row + "game-tile:nth-of-type(%s)"
+            self.wait_for_element(tile % "5" + '::shadow [data-state*="e"]')
+            letter_status = []
+            for i in range(1, 6):
+                letter_eval = self.get_attribute(tile % str(i), "evaluation")
+                letter_status.append(letter_eval)
+            if letter_status.count("correct") == 5:
+                success = True
+                break
+            self.word_list.remove(word)
+            self.modify_word_list(word, letter_status)
+
+        self.save_screenshot_to_logs()
+        print('\nWord: "%s"\nAttempts: %s' % (word.upper(), total_attempts))
+        if not success:
+            self.fail("Unable to solve for the correct word in 6 attempts!")
+        self.sleep(3)

examples/wordle_test.py

@@ -1,9 +1,9 @@
-""" Solve the Wordle game using SeleniumBase. """
+""" Solve the Wordle game using SeleniumBase.
+    The latest version of Wordle no longer uses Shadow-DOM. """
 
 import ast
 import random
 import requests
-from seleniumbase import __version__
 from seleniumbase import BaseCase
 
 
@@ -53,18 +53,12 @@ def skip_if_incorrect_env(self):
             message = "This test doesn't run in headless mode!"
             print(message)
             self.skip(message)
-        version = [int(i) for i in __version__.split(".") if i.isdigit()]
-        if version < [2, 4, 4]:
-            message = "This test requires SeleniumBase 2.4.4 or newer!"
-            print(message)
-            self.skip(message)
 
     def test_wordle(self):
         self.skip_if_incorrect_env()
         self.open("https://www.nytimes.com/games/wordle/index.html")
-        self.click("game-app::shadow game-modal::shadow game-icon")
+        self.click('svg[data-testid="icon-close"]')
         self.initialize_word_list()
-        keyboard_base = "game-app::shadow game-keyboard::shadow "
         word = random.choice(self.word_list)
         total_attempts = 0
         success = False
@@ -75,15 +69,18 @@ def test_wordle(self):
             for letter in word:
                 letters.append(letter)
                 button = 'button[data-key="%s"]' % letter
-                self.click(keyboard_base + button)
-            button = "button.one-and-a-half"
-            self.click(keyboard_base + button)
-            row = 'game-app::shadow game-row[letters="%s"]::shadow ' % word
-            tile = row + "game-tile:nth-of-type(%s)"
-            self.wait_for_element(tile % "5" + '::shadow [data-state*="e"]')
+                self.click(button)
+            button = 'button[class*="oneAndAHalf"]'
+            self.click(button)
+            row = (
+                'div[class*="lbzlf"] div[class*="Row-module"]:nth-of-type(%s) '
+                % total_attempts
+            )
+            tile = row + 'div:nth-child(%s) div[class*="module_tile__3ayIZ"]'
+            self.wait_for_element(tile % "5" + '[data-state*="e"]')
             letter_status = []
             for i in range(1, 6):
-                letter_eval = self.get_attribute(tile % str(i), "evaluation")
+                letter_eval = self.get_attribute(tile % str(i), "data-state")
                 letter_status.append(letter_eval)
             if letter_status.count("correct") == 5:
                 success = True

seleniumbase/__version__.py

@@ -1,2 +1,2 @@
 # seleniumbase package
-__version__ = "3.3.4"
+__version__ = "3.3.5"

seleniumbase/fixtures/base_case.py

@@ -5036,52 +5036,119 @@ def hide_element(self, selector, by="css selector"):
         """Hide the first element on the page that matches the selector."""
         self.__check_scope()
         selector, by = self.__recalculate_selector(selector, by)
-        selector = self.convert_to_css_selector(selector, by=by)
-        selector = self.__make_css_match_first_element_only(selector)
-        hide_script = """jQuery('%s').hide();""" % selector
-        self.safe_execute_script(hide_script)
+        css_selector = self.convert_to_css_selector(selector, by=by)
+        if ":contains(" in css_selector:
+            selector = self.__make_css_match_first_element_only(css_selector)
+            script = """jQuery('%s').hide();""" % selector
+            self.safe_execute_script(script)
+        else:
+            css_selector = re.escape(css_selector)  # Add "\\" to special chars
+            css_selector = self.__escape_quotes_if_needed(css_selector)
+            script = (
+                'const e = document.querySelector("%s");'
+                'e.style.display="none";e.style.visibility="hidden";'
+                % css_selector)
+            self.execute_script(script)
 
     def hide_elements(self, selector, by="css selector"):
         """Hide all elements on the page that match the selector."""
         self.__check_scope()
         selector, by = self.__recalculate_selector(selector, by)
-        selector = self.convert_to_css_selector(selector, by=by)
-        hide_script = """jQuery('%s').hide();""" % selector
-        self.safe_execute_script(hide_script)
+        css_selector = self.convert_to_css_selector(selector, by=by)
+        if ":contains(" in css_selector:
+            script = """jQuery('%s').hide();""" % css_selector
+            self.safe_execute_script(script)
+        else:
+            css_selector = re.escape(css_selector)  # Add "\\" to special chars
+            css_selector = self.__escape_quotes_if_needed(css_selector)
+            script = (
+                """var $elements = document.querySelectorAll('%s');
+                var index = 0, length = $elements.length;
+                for(; index < length; index++){
+                $elements[index].style.display="none";
+                $elements[index].style.visibility="hidden";}"""
+                % css_selector
+            )
+            self.execute_script(script)
 
     def show_element(self, selector, by="css selector"):
         """Show the first element on the page that matches the selector."""
         self.__check_scope()
         selector, by = self.__recalculate_selector(selector, by)
-        selector = self.convert_to_css_selector(selector, by=by)
-        selector = self.__make_css_match_first_element_only(selector)
-        show_script = """jQuery('%s').show(0);""" % selector
-        self.safe_execute_script(show_script)
+        css_selector = self.convert_to_css_selector(selector, by=by)
+        if ":contains(" in css_selector:
+            selector = self.__make_css_match_first_element_only(css_selector)
+            script = """jQuery('%s').show(0);""" % selector
+            self.safe_execute_script(script)
+        else:
+            css_selector = re.escape(css_selector)  # Add "\\" to special chars
+            css_selector = self.__escape_quotes_if_needed(css_selector)
+            script = (
+                'const e = document.querySelector("%s");'
+                'e.style.display="";e.style.visibility="visible";'
+                % css_selector
+            )
+            self.execute_script(script)
 
     def show_elements(self, selector, by="css selector"):
         """Show all elements on the page that match the selector."""
         self.__check_scope()
         selector, by = self.__recalculate_selector(selector, by)
-        selector = self.convert_to_css_selector(selector, by=by)
-        show_script = """jQuery('%s').show(0);""" % selector
-        self.safe_execute_script(show_script)
+        css_selector = self.convert_to_css_selector(selector, by=by)
+        if ":contains(" in css_selector:
+            script = """jQuery('%s').show(0);""" % css_selector
+            self.safe_execute_script(script)
+        else:
+            css_selector = re.escape(css_selector)  # Add "\\" to special chars
+            css_selector = self.__escape_quotes_if_needed(css_selector)
+            script = (
+                """var $elements = document.querySelectorAll('%s');
+                var index = 0, length = $elements.length;
+                for(; index < length; index++){
+                $elements[index].style.display="";
+                $elements[index].style.visibility="visible";}"""
+                % css_selector
+            )
+            self.execute_script(script)
 
     def remove_element(self, selector, by="css selector"):
         """Remove the first element on the page that matches the selector."""
         self.__check_scope()
         selector, by = self.__recalculate_selector(selector, by)
-        selector = self.convert_to_css_selector(selector, by=by)
-        selector = self.__make_css_match_first_element_only(selector)
-        remove_script = """jQuery('%s').remove();""" % selector
-        self.safe_execute_script(remove_script)
+        css_selector = self.convert_to_css_selector(selector, by=by)
+        if ":contains(" in css_selector:
+            selector = self.__make_css_match_first_element_only(css_selector)
+            script = """jQuery('%s').remove();""" % selector
+            self.safe_execute_script(script)
+        else:
+            css_selector = re.escape(css_selector)  # Add "\\" to special chars
+            css_selector = self.__escape_quotes_if_needed(css_selector)
+            script = (
+                'const e = document.querySelector("%s");'
+                'e.parentElement.removeChild(e);'
+                % css_selector
+            )
+            self.execute_script(script)
 
     def remove_elements(self, selector, by="css selector"):
         """Remove all elements on the page that match the selector."""
         self.__check_scope()
         selector, by = self.__recalculate_selector(selector, by)
-        selector = self.convert_to_css_selector(selector, by=by)
-        remove_script = """jQuery('%s').remove();""" % selector
-        self.safe_execute_script(remove_script)
+        css_selector = self.convert_to_css_selector(selector, by=by)
+        if ":contains(" in css_selector:
+            script = """jQuery('%s').remove();""" % css_selector
+            self.safe_execute_script(script)
+        else:
+            css_selector = re.escape(css_selector)  # Add "\\" to special chars
+            css_selector = self.__escape_quotes_if_needed(css_selector)
+            script = (
+                """var $elements = document.querySelectorAll('%s');
+                var index = 0, length = $elements.length;
+                for(; index < length; index++){
+                $elements[index].remove();}"""
+                % css_selector
+            )
+            self.execute_script(script)
 
     def ad_block(self):
         """Block ads that appear on the current web page."""