[py] Fix Selenium Manager tests on Windows

Author: cgoldberg

.github/workflows/ci-python.yml

@@ -99,10 +99,6 @@ jobs:
             os: ubuntu
           - browser: firefox
             os: ubuntu
-          - browser: chrome
-            os: windows
-          - browser: edge
-            os: windows
     with:
       name: Integration Tests (${{ matrix.browser }}, ${{ matrix.os }})
       browser: ${{ matrix.browser }}

py/test/selenium/webdriver/common/selenium_manager_tests.py

@@ -14,6 +14,7 @@
 # KIND, either express or implied.  See the License for the
 # specific language governing permissions and limitations
 # under the License.
+
 import json
 import platform
 import sys
@@ -43,12 +44,13 @@ def test_gets_results(monkeypatch):
 
 
 def test_uses_environment_variable(monkeypatch):
-    monkeypatch.setenv("SE_MANAGER_PATH", "/path/to/manager")
+    sm_path =  r"\path\to\manager" if sys.platform.startswith("win") else "path/to/manager"
+    monkeypatch.setenv("SE_MANAGER_PATH", sm_path)
     monkeypatch.setattr(Path, "is_file", lambda _: True)
 
     binary = SeleniumManager()._get_binary()
 
-    assert str(binary) == "/path/to/manager"
+    assert str(binary) == sm_path
 
 
 def test_uses_windows(monkeypatch):
@@ -59,16 +61,22 @@ def test_uses_windows(monkeypatch):
     assert binary == project_root.joinpath("selenium/webdriver/common/windows/selenium-manager.exe")
 
 
+
 def test_uses_linux(monkeypatch):
     monkeypatch.setattr(sys, "platform", "linux")
+    monkeypatch.setattr("platform.machine", lambda: "x86_64")
+
+    binary = SeleniumManager()._get_binary()
+    project_root = Path(selenium.__file__).parent.parent
+    assert binary == project_root.joinpath("selenium/webdriver/common/linux/selenium-manager")
 
-    if platform.machine() == "arm64":
-        with pytest.raises(WebDriverException, match="Unsupported platform/architecture combination: linux/arm64"):
-            SeleniumManager()._get_binary()
-    else:
-        binary = SeleniumManager()._get_binary()
-        project_root = Path(selenium.__file__).parent.parent
-        assert binary == project_root.joinpath("selenium/webdriver/common/linux/selenium-manager")
+
+def test_uses_linux_arm64(monkeypatch):
+    monkeypatch.setattr(sys, "platform", "linux")
+    monkeypatch.setattr("platform.machine", lambda: "arm64")
+
+    with pytest.raises(WebDriverException, match="Unsupported platform/architecture combination: linux/arm64"):
+        SeleniumManager()._get_binary()
 
 
 def test_uses_mac(monkeypatch):
@@ -97,11 +105,12 @@ def test_errors_if_invalid_os(monkeypatch):
 
 
 def test_error_if_invalid_env_path(monkeypatch):
-    monkeypatch.setenv("SE_MANAGER_PATH", "/path/to/manager")
+    sm_path =  r"\path\to\manager" if sys.platform.startswith("win") else "path/to/manager"
+    monkeypatch.setenv("SE_MANAGER_PATH", sm_path)
 
     with pytest.raises(WebDriverException) as excinfo:
         SeleniumManager()._get_binary()
-    assert "Unable to obtain working Selenium Manager binary; /path/to/manager" in str(excinfo.value)
+    assert f"Unable to obtain working Selenium Manager binary; {sm_path}" in str(excinfo.value)
 
 
 def test_run_successful():