Call `page.all` instead of `all` in feature steps

There's a naming conflict between Capybara and rspec-matchers which both
define the `all` method.

See https://github.com/jnicklas/capybara/issues/1396

10 files changed, 12 insertions, 12 deletions

diff --git a/features/steps/admin/settings.rb b/features/steps/admin/settings.rb
index 06616bfbd72..1c0b7a4b712 100644
--- a/features/steps/admin/settings.rb
+++ b/features/steps/admin/settings.rb
@@ -45,7 +45,7 @@ class Spinach::Features::AdminSettings < Spinach::FeatureSteps
   end
 
   step 'I should see all checkboxes checked' do
-    all('input[type=checkbox]').each do |checkbox|
+    page.all('input[type=checkbox]').each do |checkbox|
       expect(checkbox).to be_checked
     end
   end
diff --git a/features/steps/dashboard/new_project.rb b/features/steps/dashboard/new_project.rb
index ea89ad215c0..b4ade65ee53 100644
--- a/features/steps/dashboard/new_project.rb
+++ b/features/steps/dashboard/new_project.rb
@@ -22,7 +22,7 @@ class Spinach::Features::NewProject < Spinach::FeatureSteps
     expect(github_modal).to be_visible
     expect(github_modal).to have_content "To enable importing projects from GitHub"
 
-    all('.modal-body').each do |element|
+    page.all('.modal-body').each do |element|
       expect(element).not_to be_visible unless element == github_modal
     end
   end
diff --git a/features/steps/explore/projects.rb b/features/steps/explore/projects.rb
index 17c16c3fb1c..49c2f6a1253 100644
--- a/features/steps/explore/projects.rb
+++ b/features/steps/explore/projects.rb
@@ -22,14 +22,14 @@ class Spinach::Features::ExploreProjects < Spinach::FeatureSteps
 
   step 'I should see empty public project details with http clone info' do
     project = Project.find_by(name: 'Empty Public Project')
-    all(:css, '.git-empty .clone').each do |element|
+    page.all(:css, '.git-empty .clone').each do |element|
       expect(element.text).to include(project.http_url_to_repo)
     end
   end
 
   step 'I should see empty public project details with ssh clone info' do
     project = Project.find_by(name: 'Empty Public Project')
-    all(:css, '.git-empty .clone').each do |element|
+    page.all(:css, '.git-empty .clone').each do |element|
       expect(element.text).to include(project.url_to_repo)
     end
   end
diff --git a/features/steps/project/commits/branches.rb b/features/steps/project/commits/branches.rb
index ff5baf5ddae..338f5e8d3ee 100644
--- a/features/steps/project/commits/branches.rb
+++ b/features/steps/project/commits/branches.rb
@@ -80,6 +80,6 @@ class Spinach::Features::ProjectCommitsBranches < Spinach::FeatureSteps
   end
 
   step "I should not see branch 'improve/awesome'" do
-    expect(all(visible: true)).not_to have_content 'improve/awesome'
+    expect(page.all(visible: true)).not_to have_content 'improve/awesome'
   end
 end
diff --git a/features/steps/project/commits/commits.rb b/features/steps/project/commits/commits.rb
index f0acea1d473..4b19e3beed4 100644
--- a/features/steps/project/commits/commits.rb
+++ b/features/steps/project/commits/commits.rb
@@ -83,7 +83,7 @@ class Spinach::Features::ProjectCommits < Spinach::FeatureSteps
   end
 
   step 'The diff links to both the previous and current image' do
-    links = all('.two-up span div a')
+    links = page.all('.two-up span div a')
     expect(links[0]['href']).to match %r{blob/#{sample_image_commit.old_blob_id}}
     expect(links[1]['href']).to match %r{blob/#{sample_image_commit.new_blob_id}}
   end
diff --git a/features/steps/project/commits/tags.rb b/features/steps/project/commits/tags.rb
index beecfd3a3ce..e6f8faf50fd 100644
--- a/features/steps/project/commits/tags.rb
+++ b/features/steps/project/commits/tags.rb
@@ -61,13 +61,13 @@ class Spinach::Features::ProjectCommitsTags < Spinach::FeatureSteps
 
   step "I should not see tag 'v1.1.0'" do
     page.within '.tags' do
-      expect(all(visible: true)).not_to have_content 'v1.1.0'
+      expect(page.all(visible: true)).not_to have_content 'v1.1.0'
     end
   end
 
   step 'I delete all tags' do
     page.within '.tags' do
-      all('.btn-remove').each do |remove|
+      page.all('.btn-remove').each do |remove|
         remove.click
         sleep 0.05
       end
diff --git a/features/steps/project/issues/issues.rb b/features/steps/project/issues/issues.rb
index 6a462fec269..6873c043e19 100644
--- a/features/steps/project/issues/issues.rb
+++ b/features/steps/project/issues/issues.rb
@@ -190,7 +190,7 @@ class Spinach::Features::ProjectIssues < Spinach::FeatureSteps
 
   step 'I see empty project details with ssh clone info' do
     project = Project.find_by(name: 'Empty Project')
-    all(:css, '.git-empty .clone').each do |element|
+    page.all(:css, '.git-empty .clone').each do |element|
       expect(element.text).to include(project.url_to_repo)
     end
   end
diff --git a/features/steps/project/issues/labels.rb b/features/steps/project/issues/labels.rb
index 2828f02c449..d656acf4220 100644
--- a/features/steps/project/issues/labels.rb
+++ b/features/steps/project/issues/labels.rb
@@ -15,7 +15,7 @@ class Spinach::Features::ProjectIssuesLabels < Spinach::FeatureSteps
 
   step 'I delete all labels' do
     page.within '.labels' do
-      all('.btn-remove').each do |remove|
+      page.all('.btn-remove').each do |remove|
         remove.click
         sleep 0.05
       end
diff --git a/features/steps/project/merge_requests.rb b/features/steps/project/merge_requests.rb
index 6e8c5f5ddd1..62c64e60f6d 100644
--- a/features/steps/project/merge_requests.rb
+++ b/features/steps/project/merge_requests.rb
@@ -121,7 +121,7 @@ class Spinach::Features::ProjectMergeRequests < Spinach::FeatureSteps
   end
 
   step 'I should see the proper Inline and Side-by-side links' do
-    buttons = all('#commit-diff-viewtype')
+    buttons = page.all('#commit-diff-viewtype')
     expect(buttons.count).to eq(2)
 
     buttons.each do |b|
diff --git a/features/steps/project/source/multiselect_blob.rb b/features/steps/project/source/multiselect_blob.rb
index 212537fc333..8e14623b892 100644
--- a/features/steps/project/source/multiselect_blob.rb
+++ b/features/steps/project/source/multiselect_blob.rb
@@ -30,7 +30,7 @@ class Spinach::Features::ProjectSourceMultiselectBlob < Spinach::FeatureSteps
           ids = Array(range).map { |n| "LC#{n}" }
           extra = false
 
-          highlighted = all("#tree-content-holder .highlight .line.hll")
+          highlighted = page.all("#tree-content-holder .highlight .line.hll")
           highlighted.each do |element|
             extra ||= ids.delete(element[:id]).nil?
           end