diff options
| author | Robert Schilling <rschilling@student.tugraz.at> | 2015-04-15 06:08:28 +0000 |
|---|---|---|
| committer | Robert Schilling <rschilling@student.tugraz.at> | 2015-04-15 06:08:28 +0000 |
| commit | 61c79418c46105ecedfca3701b7e4ac9ba6ba601 (patch) | |
| tree | 5d3a2832dd6f85d9a44d8aeac94f8c2dd024a6ec | |
| parent | ac5af61499a8612ef6a51b2bebd18c4c1a083583 (diff) | |
| parent | 48efd65ff69a456eaafbd5538e5d69f140ce6b40 (diff) | |
| download | gitlab-ce-61c79418c46105ecedfca3701b7e4ac9ba6ba601.tar.gz | |
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
| -rw-r--r-- | features/steps/project/merge_requests.rb | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/features/steps/project/merge_requests.rb b/features/steps/project/merge_requests.rb index 40c102833a4..bb1f9f129c0 100644 --- a/features/steps/project/merge_requests.rb +++ b/features/steps/project/merge_requests.rb @@ -61,8 +61,7 @@ class Spinach::Features::ProjectMergeRequests < Spinach::FeatureSteps end step 'I should see that I am unsubscribed' do - sleep 0.2 - find(".subscribe-button span").text.should == "Subscribe" + find(".subscribe-button span").should have_content("Subscribe") end step 'I click button "Unsubscribe"' do |