Commit 61c79418 authored by Robert Schilling's avatar Robert Schilling

Merge branch 'fix-spinach-unsubscribe-mr-test' into 'master'

Remove timing dependency on unsubscribe merge request test

This test failed for me once. I am now on a mission to remove `sleep` from the spinach tests.

See: http://www.elabs.se/blog/53-why-wait_until-was-removed-from-capybara

See merge request !523
parents ac5af614 48efd65f
...@@ -61,8 +61,7 @@ class Spinach::Features::ProjectMergeRequests < Spinach::FeatureSteps ...@@ -61,8 +61,7 @@ class Spinach::Features::ProjectMergeRequests < Spinach::FeatureSteps
end end
step 'I should see that I am unsubscribed' do step 'I should see that I am unsubscribed' do
sleep 0.2 find(".subscribe-button span").should have_content("Subscribe")
find(".subscribe-button span").text.should == "Subscribe"
end end
step 'I click button "Unsubscribe"' do step 'I click button "Unsubscribe"' do
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment