Skip to content

Fixes #629 by expecting NoSuchElementException - #630

Merged
pekkaklarck merged 2 commits into
robotframework:masterfrom
aaltat:fix_empty_except_on_unselect_from_list
Aug 22, 2016
Merged

pekkaklarck merged 2 commits into
robotframework:masterfrom
aaltat:fix_empty_except_on_unselect_from_list

Conversation

@aaltat

@aaltat aaltat commented Aug 8, 2016

Copy link
Copy Markdown
Contributor

Empty expect are not coding pratices. The deselect_by_value
and the deselect_by_visible_text raises NoSuchElementException if it
can not deselect the items.

Also changed the logging from warn to info and formatted the text to
follow pep008.

aaltat added 2 commits August 22, 2016 18:25
Empty expect are not coding pratices. The deselect_by_value
and the deselect_by_visible_text raises NoSuchElementException if it
can not delest the items.
@pekkaklarck
pekkaklarck merged commit 3989a0c into robotframework:master Aug 22, 2016
Gaurang033 pushed a commit to Gaurang033/Selenium2Library that referenced this pull request Oct 17, 2016
…amework#630)

* Fixes robotframework#629 by expecting NoSuchElementException

Empty expect are not coding practices. The deselect_by_value
and the deselect_by_visible_text raises NoSuchElementException if it
can not deselect the items.
VitoAlbano pushed a commit to VitoAlbano/Selenium2Library that referenced this pull request Oct 18, 2016
…amework#630)

* Fixes robotframework#629 by expecting NoSuchElementException

Empty expect are not coding practices. The deselect_by_value
and the deselect_by_visible_text raises NoSuchElementException if it
can not deselect the items.
@aaltat
aaltat deleted the fix_empty_except_on_unselect_from_list branch January 24, 2017 22:13
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.

2 participants