[py] Fix failing test for Edge logging

[cgoldberg]

User description

💥 What does this PR do?

This is a fix to the internal Python test suite.

This PR fixes the test_uses_edgedriver_logging test in py/test/selenium/webdriver/edge/edge_service_tests.py.

Previously, this test was failing because it was attempting to launch 2 browsers using the same Service object. This is not possible and results in an error:

E       selenium.common.exceptions.SessionNotCreatedException: Message: session not created
E       from unknown error: cannot find msedge binary

This PR creates 2 Service objects so each driver instance can use its own.

🔄 Types of changes

• Bug fix (backwards compatible)

---

PR Type

Bug fix

---

Description

• Fixed failing test_uses_edgedriver_logging in Edge service tests.

• Replaced single Service object with two separate instances.

• Ensured proper cleanup of driver instances and log file.

---

Changes walkthrough 📝

	Relevant files
Bug fix	edge_service_tests.py Fix Edge logging test with separate Service instances        py/test/selenium/webdriver/edge/edge_service_tests.py Replaced single Service object with two separate instances. Updated test logic to use distinct services for each driver. Added conditional cleanup for driver1 to avoid potential errors. +13/-4

---

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-code-review]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

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

🧪 PR contains tests

🔒 No security concerns identified

⚡ No major issues detected

[qodo-code-review]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Impact
Possible issue	Use unique log files Using the same log file for both service instances can lead to race conditions and unpredictable test results. Create unique log files for each service to ensure proper isolation. py/test/selenium/webdriver/edge/edge_service_tests.py [33-43] service1 = Service( - log_output=log_file, + log_output="msedgedriver1.log", service_args=service_args, executable_path=driver_executable, ) service2 = Service( - log_output=log_file, + log_output="msedgedriver2.log", service_args=service_args, executable_path=driver_executable, ) Apply this suggestion Suggestion importance[1-10]: 7 __ Why: Using separate log files for each service would prevent potential race conditions and make the test more reliable. This is a valid concern as concurrent writes to the same log file could lead to unpredictable test behavior.	Medium
	Check file exists before removal The test will fail if the log file doesn't exist when attempting to remove it. Add a check to verify the file exists before trying to delete it. py/test/selenium/webdriver/edge/edge_service_tests.py [54-59] finally: if driver1: driver1.quit() if driver2: driver2.quit() - os.remove(log_file) + if os.path.exists(log_file): + os.remove(log_file) Apply this suggestion Suggestion importance[1-10]: 6 __ Why: Adding a check for file existence before removal prevents potential FileNotFoundError exceptions if the log file wasn't created for some reason. This improves error handling and makes the test more robust.	Low
More