From 99790eb2b3dd367b318ecf63474fa13bfe9dfd82 Mon Sep 17 00:00:00 2001
From: Thomas Walpole <twalpole@gmail.com>
Date: Thu, 15 Nov 2018 21:49:46 -0800
Subject: [PATCH] Fix :button selector evaluating :link_or_button expression
 filters

---
 lib/capybara/selector.rb |  8 +++-----
 spec/selector_spec.rb    | 16 ++++++++++++++++
 2 files changed, 19 insertions(+), 5 deletions(-)

diff --git a/lib/capybara/selector.rb b/lib/capybara/selector.rb
index c1bac09a8..713959bde 100644
--- a/lib/capybara/selector.rb
+++ b/lib/capybara/selector.rb
@@ -161,11 +161,9 @@
       image_btn_xpath = image_btn_xpath[alt_matches]
     end
 
-    res_xpath = input_btn_xpath.union(btn_xpath).union(image_btn_xpath)
-
-    res_xpath = expression_filters.keys.inject(res_xpath) { |memo, ef| memo[find_by_attr(ef, options[ef])] }
-
-    res_xpath
+    %i[value title type].inject(input_btn_xpath.union(btn_xpath).union(image_btn_xpath)) do |memo, ef|
+      memo[find_by_attr(ef, options[ef])]
+    end
   end
 
   node_filter(:disabled, :boolean, default: false, skip_if: :all) { |node, value| !(value ^ node.disabled?) }
diff --git a/spec/selector_spec.rb b/spec/selector_spec.rb
index daf9cb9a5..ba76b26d1 100644
--- a/spec/selector_spec.rb
+++ b/spec/selector_spec.rb
@@ -416,6 +416,22 @@
           expect(string.find(:element, 'input', title: XPath.contains_word('button 1'))[:type]).to eq 'button'
         end
       end
+
+      describe ':link_or_button selector' do
+        around(:all) do |example|
+          Capybara.modify_selector(:link_or_button) do
+            expression_filter(:random) { |xpath, _| xpath } # do nothing filter
+          end
+          example.run
+          Capybara::Selector.all[:link_or_button].expression_filters.delete(:random)
+        end
+
+        context 'when modified' do
+          it 'should still work' do
+            expect(string.find(:link_or_button, 'click me', random: 'blah').value).to eq 'click me'
+          end
+        end
+      end
     end
   end
 end