From dbe37d2f6b588b399b8be95c696c770199df5cbd Mon Sep 17 00:00:00 2001 From: Stan Hu Date: Mon, 12 Sep 2016 12:35:08 -0700 Subject: [PATCH] Add link to article on how to write reliable, asynchronous tests with Capybara [ci skip] --- CONTRIBUTING.md | 2 ++ 1 file changed, 2 insertions(+) diff --git a/CONTRIBUTING.md b/CONTRIBUTING.md index c77dcd96a7d..7a2ee38ad8f 100644 --- a/CONTRIBUTING.md +++ b/CONTRIBUTING.md @@ -289,6 +289,8 @@ request is as follows: 1. For more complex migrations, write tests. 1. Merge requests **must** adhere to the [merge request performance guidelines](doc/development/merge_request_performance_guidelines.md). +1. For tests that use Capybara or PhantomJS, see this [article on how + to write reliable asynchronous tests](https://robots.thoughtbot.com/write-reliable-asynchronous-integration-tests-with-capybara). The **official merge window** is in the beginning of the month from the 1st to the 7th day of the month. This is the best time to submit an MR and get