[bazel + rust] Set the path to selenium-manager in tests based on the OS

[shs96c]

PR Type

Bug fix, Enhancement

---

Description

• Adjusted CARGO_BIN_EXE_selenium-manager path for OS compatibility.

• Used Bazel select to handle OS-specific paths.

---

Changes walkthrough 📝

	Relevant files
Enhancement	BUILD.bazel Adjust `selenium-manager` path based on OS in Bazel            rust/tests/BUILD.bazel Replaced hardcoded CARGO_BIN_EXE_selenium-manager path with OS-specific paths. Used Bazel select to differentiate between Windows and default conditions. +4/-3

---

Need help?

Type /help how to ... in the comments thread for any questions about Qodo Merge usage.

Check out the documentation for more information.

[qodo-merge-pro]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪

🧪 No relevant tests

🔒 No security concerns identified

⚡ No major issues detected

[qodo-merge-pro]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Fix Windows path escaping Use double backslashes for Windows paths to properly escape the backslash character in string literals. Single backslashes in strings are treated as escape characters. rust/tests/BUILD.bazel [16-19] rustc_env = select({ - "//common:windows": {"CARGO_BIN_EXE_selenium-manager": "rust\\selenium-manager"}, + "//common:windows": {"CARGO_BIN_EXE_selenium-manager": "rust\\\\selenium-manager"}, "//conditions:default": {"CARGO_BIN_EXE_selenium-manager": "rust/selenium-manager"}, }), Apply this suggestion Suggestion importance[1-10]: 9 __ Why: The suggestion correctly identifies a critical issue with Windows path escaping. In the existing code, a single backslash is used which would be interpreted as an escape character, potentially causing runtime errors. The improved code properly escapes the backslash by using four backslashes (which become two in the string literal).	High
More