search

MattiSG / Watai

Integration testing for the web of components
62 stars 7 forks source link

Ignore features on the fly #111

Closed debona closed 10 years ago

debona commented 10 years ago

Allow to ignore features of a suite test on the fly by providing { "without" : [1, 4] } in config.

MattiSG commented 10 years ago

Thank you very much ;)

I will try to fix #112 before merging this. If it's too long, I'll merge as it is.

MattiSG commented 10 years ago

112 has been fixed (in branch v0.6.2). I will uncomment the tests you provided, unless you prefer to do it yourself.

MattiSG commented 10 years ago

Even though I really like without as a CLI switch, I'm wondering if ignore would not be a more proper switch in a config file. What do you think?

MattiSG commented 10 years ago

Added some commits in branch ignore-features, simply about uniforming error handling and improving messages.

Waiting for your feedback on the switch name before merging and publishing v0.6.2 :)

debona commented 10 years ago

--ignore switch sounds a little bit more GNUish and it would be clearer in a config file. Even if I like --without too. ^^

debona commented 10 years ago

You just have to git revert 8beb67d to enable the integration tests. I let you handle merges to deliver that in v0.6.2.

MattiSG commented 10 years ago

Merged in 51faa9c8593bf7e47cc62a2a79b75d0e46ec8d49, published since 0.6.2-alpha.2. Thanks! :)

MattiSG commented 10 years ago

Released and acknowledged, thanks again! :)