Commit 96a648de authored by DJ Mountney's avatar DJ Mountney

Fixes the search test that fails whenever faker uses a name with Bar in it....

Fixes the search test that fails whenever faker uses a name with Bar in it. Limit check to being within the search results div.
parent d3a36372
...@@ -13,15 +13,15 @@ Feature: Search ...@@ -13,15 +13,15 @@ Feature: Search
And project has issues And project has issues
When I search for "Foo" When I search for "Foo"
And I click "Issues" link And I click "Issues" link
Then I should see "Foo" link Then I should see "Foo" link in the search results
And I should not see "Bar" link And I should not see "Bar" link in the search results
Scenario: I should see merge requests I am looking for Scenario: I should see merge requests I am looking for
And project has merge requests And project has merge requests
When I search for "Foo" When I search for "Foo"
When I click "Merge requests" link When I click "Merge requests" link
Then I should see "Foo" link Then I should see "Foo" link in the search results
And I should not see "Bar" link And I should not see "Bar" link in the search results
Scenario: I should see project code I am looking for Scenario: I should see project code I am looking for
When I click project "Shop" link When I click project "Shop" link
...@@ -33,14 +33,14 @@ Feature: Search ...@@ -33,14 +33,14 @@ Feature: Search
When I click project "Shop" link When I click project "Shop" link
And I search for "Foo" And I search for "Foo"
And I click "Issues" link And I click "Issues" link
Then I should see "Foo" link Then I should see "Foo" link in the search results
And I should not see "Bar" link And I should not see "Bar" link in the search results
Scenario: I should see project merge requests Scenario: I should see project merge requests
And project has merge requests And project has merge requests
When I click project "Shop" link When I click project "Shop" link
And I search for "Foo" And I search for "Foo"
And I click "Merge requests" link And I click "Merge requests" link
Then I should see "Foo" link Then I should see "Foo" link in the search results
And I should not see "Bar" link And I should not see "Bar" link in the search results
...@@ -59,11 +59,11 @@ class Spinach::Features::Search < Spinach::FeatureSteps ...@@ -59,11 +59,11 @@ class Spinach::Features::Search < Spinach::FeatureSteps
create(:merge_request, :simple, title: "Bar", source_project: project, target_project: project) create(:merge_request, :simple, title: "Bar", source_project: project, target_project: project)
end end
step 'I should see "Foo" link' do step 'I should see "Foo" link in the search results' do
page.should have_link "Foo" find(:css, '.search-results').should have_link 'Foo'
end end
step 'I should not see "Bar" link' do step 'I should not see "Bar" link in the search results' do
page.should_not have_link "Bar" find(:css, '.search-results').should_not have_link 'Bar'
end end
end end
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