change def axe.execute() to axe.run() to mirror axe-core naming

mozilla-services / axe-selenium-python

aXe Selenium Integration python package

https://pypi.python.org/pypi/axe-selenium-python/

Mozilla Public License 2.0

58 stars 50 forks source link

change def axe.execute() to axe.run() to mirror axe-core naming #162

Closed Michael-Overall closed 5 years ago

Michael-Overall commented 5 years ago

Renamed execute() in axe.py to run() to mirror axe-core API's naming of the function.

Also updated README to use new axe.run() function name in documentation, and the function call in the _perform_axe_run() test so the test suite didn't think the code had burst into flame.

fixes #160

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 404

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals
Change from base Build 403:	0.0%
Covered Lines:
Relevant Lines:	0

💛 - Coveralls

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 404

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals
Change from base Build 403:	0.0%
Covered Lines:
Relevant Lines:	0

💛 - Coveralls

coveralls commented 5 years ago

Pull Request Test Coverage Report for Build 404

0 of 0 changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at ?%

Totals
Change from base Build 403:	0.0%
Covered Lines:
Relevant Lines:	0

💛 - Coveralls

kimberlythegeek commented 5 years ago

Thank you, @Michael-Overall !

kimberlythegeek commented 5 years ago

@Michael-Overall If you're interested, there is another python package that depends on this one, and very similar changes need to be made: https://github.com/mozilla-services/pytest-axe/issues/52