[ci][py] Enabled browser tests

Signed-off-by: Viet Nguyen Duc <nguyenducviet4496@gmail.com>
SeleniumHQ · Nov 14, 2024 · 6819795 · 6819795
1 parent 1492a3a
commit 6819795
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 12 deletions.
diff --git a/.github/workflows/ci-python.yml b/.github/workflows/ci-python.yml
@@ -103,7 +103,11 @@ jobs:
           - browser: safari
             os: macos
           - browser: chrome
-            os: macos
+            os: ubuntu
+          - browser: edge
+            os: ubuntu
+          - browser: firefox
+            os: ubuntu
     with:
       name: Integration Tests (${{ matrix.browser }}, ${{ matrix.os }})
       browser: ${{ matrix.browser }}

diff --git a/py/test/selenium/webdriver/marionette/mn_options_tests.py b/py/test/selenium/webdriver/marionette/mn_options_tests.py
@@ -41,20 +41,20 @@ def test_we_can_pass_options(self, driver, pages):
 class TestUnit:
     def test_ctor(self):
         opts = Options()
-        assert opts._binary is None
-        assert not opts._preferences
+        assert opts.binary_location == ""
+        assert opts._preferences == {"remote.active-protocols": 3}
         assert opts._profile is None
         assert not opts._arguments
         assert isinstance(opts.log, Log)
 
     def test_binary(self):
         opts = Options()
-        assert opts.binary is None
+        assert opts.binary_location == ""
 
         other_binary = FirefoxBinary()
-        assert other_binary != opts.binary
+        assert other_binary != opts.binary_location
         opts.binary = other_binary
-        assert other_binary == opts.binary
+        assert other_binary._start_cmd == opts.binary_location
 
         path = "/path/to/binary"
         opts.binary = path
@@ -63,16 +63,16 @@ def test_binary(self):
 
     def test_prefs(self):
         opts = Options()
-        assert len(opts.preferences) == 0
+        assert len(opts.preferences) == 1
         assert isinstance(opts.preferences, dict)
 
         opts.set_preference("spam", "ham")
-        assert len(opts.preferences) == 1
-        opts.set_preference("eggs", True)
         assert len(opts.preferences) == 2
+        opts.set_preference("eggs", True)
+        assert len(opts.preferences) == 3
         opts.set_preference("spam", "spam")
-        assert len(opts.preferences) == 2
-        assert opts.preferences == {"spam": "spam", "eggs": True}
+        assert len(opts.preferences) == 3
+        assert opts.preferences == {"eggs": True, "remote.active-protocols": 3, "spam": "spam"}
 
     def test_profile(self, tmpdir_factory):
         opts = Options()
@@ -99,7 +99,12 @@ def test_arguments(self):
     def test_to_capabilities(self):
         opts = Options()
         firefox_caps = DesiredCapabilities.FIREFOX.copy()
-        firefox_caps.update({"pageLoadStrategy": PageLoadStrategy.normal})
+        firefox_caps.update(
+            {
+                "pageLoadStrategy": PageLoadStrategy.normal,
+                "moz:firefoxOptions": {"prefs": {"remote.active-protocols": 3}},
+            }
+        )
         assert opts.to_capabilities() == firefox_caps
 
         profile = FirefoxProfile()