[1LP][RFR] Add Button.text and Button.read()

RedHatQE / widgetastic.patternfly

Library of Patternfly widgets for Widgetastic.

Other

13 stars 29 forks source link

[1LP][RFR] Add Button.text and Button.read() #96

Closed mshriver closed 5 years ago

mshriver commented 5 years ago

and unit tests!

Fixes #57

mshriver commented 5 years ago

@izapolsk I thought the same thing, more or less - that the self.browser.text(self) could really be in wt.core. For the moment I decided to proceed with implementing here.

wt.core.Widget read() raises NotImplemented, so changing that to default to returning text property would be major change, which needs to be discussed.

coveralls commented 5 years ago

Coverage increased (+0.1%) to 48.864% when pulling b4b1f4fb62e5a4e03a153573f4e072d369ccfe39 on mshriver:add-button-read into 3758f5fcf836d25609eac83a672866c58d6a227e on RedHatQE:master.