Skip to content

[py] Bugfix: text_to_be_present_in_element_attribute predicate checking#10596

Merged
diemol merged 1 commit intoSeleniumHQ:trunkfrom
symonk:10592-bugfix-attribute-text-present
May 1, 2022
Merged

[py] Bugfix: text_to_be_present_in_element_attribute predicate checking#10596
diemol merged 1 commit intoSeleniumHQ:trunkfrom
symonk:10592-bugfix-attribute-text-present

Conversation

@symonk
Copy link
Copy Markdown
Member

@symonk symonk commented May 1, 2022

minor tweak for text_to_be_present_in_element_attribute which is referencing another expected condition however it is evaluating the result of that call in a boolean context; that function is returning a predicate function and not it's respective boolean return type.

closes #10592

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Merging #10596 (abc3354) into trunk (5b84066) will not change coverage.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##            trunk   #10596   +/-   ##
=======================================
  Coverage   45.02%   45.02%           
=======================================
  Files          85       85           
  Lines        5508     5508           
  Branches      268      268           
=======================================
  Hits         2480     2480           
  Misses       2760     2760           
  Partials      268      268           
Impacted Files Coverage Δ
.../selenium/webdriver/support/expected_conditions.py 33.17% <0.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b84066...abc3354. Read the comment docs.

Copy link
Copy Markdown
Member

@diemol diemol left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, @symonk!

@diemol diemol merged commit d4f5b86 into SeleniumHQ:trunk May 1, 2022
@symonk symonk deleted the 10592-bugfix-attribute-text-present branch May 1, 2022 21:13
elgatov pushed a commit to elgatov/selenium that referenced this pull request Jun 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[🐛 Bug]:

3 participants