[py] Added Deprecation of WebElement.get_attribute() per #13334

[shbenzer]

User description

Description

added deprecation warning to WebElement.get_attribute()

Motivation and Context

#13334

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist

• I have read the contributing document.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have added tests to cover my changes.
• All new and existing tests passed.

---

PR Type

enhancement, documentation

---

Description

• Added a deprecation warning to the WebElement.get_attribute() method in the Selenium WebDriver Python library.
• Users are advised to use the get_dom_attribute() method instead.
• This change addresses issue [🚀 Feature]: deprecate WebElement.getAttribute #13334 and aims to guide users towards more modern practices.

---

PRDescriptionHeader.CHANGES_WALKTHROUGH

	Relevant files
Enhancement	webelement.py Deprecate `WebElement.get_attribute()` method with warning py/selenium/webdriver/remote/webelement.py Added a deprecation warning for WebElement.get_attribute(). Suggested using get_dom_attribute() instead. +7/-0

---

💡 PR-Agent usage: Comment /help "your question" on any pull request to receive relevant information

[qodo-merge-pro]

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

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

🧪 No relevant tests

🔒 No security concerns identified

⚡ Recommended focus areas for review Deprecation Warning The deprecation warning message is missing a comma after the warning text, which may cause a syntax error.

[qodo-merge-pro]

PR Code Suggestions ✨

Explore these optional code suggestions:

Category	Suggestion	Score
Possible bug	✅ Fix syntax error in warning message by adding a missing comma Suggestion Impact: The suggestion was implemented by adding the missing comma after the warning message string in the warnings.warn() call. code diff: - "using WebElement.get_attribute() has been deprecated. Please use get_dom_attribute() instead." + "using WebElement.get_attribute() has been deprecated. Please use get_dom_attribute() instead.", Add the missing comma after the warning message string in the warnings.warn() call to fix the syntax error. py/selenium/webdriver/remote/webelement.py [177-181] warnings.warn( - "using WebElement.get_attribute() has been deprecated. Please use get_dom_attribute() instead." + "using WebElement.get_attribute() has been deprecated. Please use get_dom_attribute() instead.", DeprecationWarning, stacklevel=2, ) Apply this suggestion Suggestion importance[1-10]: 10 Why: The suggestion correctly identifies a syntax error due to a missing comma in the warnings.warn() call, which is crucial for the code to function properly. Adding the comma is necessary to avoid runtime errors, making this a high-impact fix.	10

💡 Need additional feedback ? start a PR chat

[shbenzer]

@VietND96 Failing test is unrelated

[shbenzer]

Just pushed a fix for that flaky test actually #14701

[shbenzer]

@VietND96 now that the bidi_tests.py fix has been merged, could you run the ci workflow again?