[rust] Reuse constant for --version

[bonigarcia]

🔗 Related Issues

💥 What does this PR do?

🔧 Implementation Notes

🤖 AI assistance

• No substantial AI assistance used
• AI assisted (complete below)
  • Tool(s):
  • What was generated:
  • I reviewed all AI output and can explain the change

💡 Additional Considerations

🔄 Types of changes

• Cleanup (formatting, renaming)
• Bug fix (backwards compatible)
• New feature (non-breaking change which adds functionality and tests!)
• Breaking change (fix or feature that would cause existing functionality to change)

[qodo-code-review]

Review Summary by Qodo

Reuse DASH_DASH_VERSION constant for driver version checks

✨ Enhancement

Walkthroughs

Description

• Reuse DASH_DASH_VERSION constant instead of hardcoded strings
• Replace hardcoded "--version" with constant in driver version command
• Update test utilities to use the shared constant
• Improve code maintainability and consistency

Diagram

flowchart LR
  A["Hardcoded --version strings"] -->|Replace with constant| B["DASH_DASH_VERSION constant"]
  B -->|Used in| C["lib.rs find_driver_in_path"]
  B -->|Used in| D["tests/common.rs exec_driver"]

Loading

File Changes

1. rust/src/lib.rs ✨ Enhancement +4/-2

Use constant for driver version command

• Replace hardcoded "--version" string with DASH_DASH_VERSION constant
• Reformat Command::new() call for better readability
• Maintain same functionality while improving code reusability

rust/src/lib.rs

---

2. rust/tests/common.rs ✨ Enhancement +3/-1

Import and use version constant in tests

• Import DASH_DASH_VERSION constant from selenium_manager
• Replace hardcoded "--version" string with constant in exec_driver function
• Align test code with main library constant usage

rust/tests/common.rs

---

[qodo-code-review]

Code Review by Qodo

🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)

Great, no issues found!

Qodo reviewed your code and found no material issues that require review