-
Notifications
You must be signed in to change notification settings - Fork 203
Addidng additional examples to step definitons #127
base: master
Are you sure you want to change the base?
Conversation
… additional clarity when running 'behat -di' or 'behat -dl'
… additional clarity when running 'behat -di' or 'behat -dl'
Oops, added hardcoded url possibly causing tests to fail
Seems scrutenizer is failing at the below step which may be unrelated: Installing SSH Keys 00:01 |
Can you split your PR in two: on for the documentation for existing steps, second for new steps. Don’t forget to add tests for the new steps. |
Sure thing, thanks for the feedback. As for tests around the new steps I assume you mean unit tests, which I am still unfamiliar with. Is there any way you can point me to an article or lesson of sorts that will make that learning-curve less intense? I've worked with PHPUnit with a very limited scope in my prior work. |
No, a behat tests. See tests/features/browser.feature |
Sure, thats cake! thanks!!! Also your context work is great and has taught me a lot!!! |
… examples and desc to match requirements set forth by MinkExtension PR
Ok so i added tests in browser.feature. Specifically:
I also did some grammatical changes and follow @stof 's recommendations for examples and methods. Let me know if there is anything else I can do! Thanks again! |
Hmm thats odd, @sanpii I cant reproduce that error, mink allows me to resize my windows for all my tests using your context... I use the same implementation for my instance of Behat.
|
@sanpii LMK if this addresses your initial concerns or if anything needs to be revisited, Thanks! |
added additional documents, added examples to all step definitions for additional clarity when running 'behat -di' or 'behat -dl'