search

spyoungtech / behave-webdriver

Selenium webdriver step library for use with the behave BDD testing framework
MIT License
62 stars 21 forks source link

redundant "I pause for " #73

Open mikelietz opened 5 years ago

mikelietz commented 5 years ago

It appears that there are two steps for "I pause for Xms" in actions.py. Can't they (the given and then when) be combined?

https://github.com/spyoungtech/behave-webdriver/blob/615260e06ede1e47b11f56368df1b560b1f888f8/behave_webdriver/steps/actions.py#L12

and https://github.com/spyoungtech/behave-webdriver/blob/615260e06ede1e47b11f56368df1b560b1f888f8/behave_webdriver/steps/actions.py#L171

spyoungtech commented 5 years ago

Yeah, I don't see any reason why they shouldn't be combined.