search

Automattic / babble

Multilingual WordPress done right.
https://wordpress.org/plugins/babble/
245 stars 50 forks source link

Implement a Behat test which covers leaving a comment #315

Closed johnbillion closed 9 years ago

johnbillion commented 9 years ago

See #235.

Our existing unit tests cover the need for the home_url filter which is added in Babble_Locale::query_vars().

This new Behat test covers the need for the home_url filter which is added in Babble_Locale::pre_comment_on_post(). The test ensures the redirect after the comment is posted is correct (if it's not, the comment isn't shown and the test fails).

simonwheatley commented 9 years ago

Tests are currently failing: https://travis-ci.org/Automattic/babble/jobs/75423403

Assigning back to you, sorry! :worried:

joehoyle commented 9 years ago

@simonwheatley all tests now passing! Ready for a merge

simonwheatley commented 9 years ago

Thanks @joehoyle and @johnbillion :)