capybara 2.5.0 → 2.18.0

data/lib/capybara.rb

@@ -1,6 +1,9 @@
+# frozen_string_literal: true
 require 'timeout'
 require 'nokogiri'
 require 'xpath'
+require 'forwardable'
+require 'capybara/config'
 
 module Capybara
   class CapybaraError < StandardError; end
@@ -18,13 +21,41 @@ module Capybara
   class WindowError < CapybaraError; end
   class ReadOnlyElementError < CapybaraError; end
 
+
   class << self
-    attr_accessor :asset_host, :app_host, :run_server, :default_host, :always_include_port
-    attr_accessor :server_port, :exact, :match, :exact_options, :visible_text_only
-    attr_accessor :default_selector, :default_max_wait_time, :ignore_hidden_elements
-    attr_accessor :save_and_open_page_path, :wait_on_first_by_default, :automatic_reload, :raise_server_errors, :server_errors
-    attr_writer :default_driver, :current_driver, :javascript_driver, :session_name, :server_host
-    attr_accessor :app
+    extend Forwardable
+
+    # DelegateCapybara global configurations
+    # @!method app
+    #   See {Capybara.configure}
+    # @!method reuse_server
+    #   See {Capybara.configure}
+    # @!method threadsafe
+    #   See {Capybara.configure}
+    # @!method server
+    #   See {Capybara.configure}
+    # @!method default_driver
+    #   See {Capybara.configure}
+    # @!method javascript_driver
+    #   See {Capybara.configure}
+    Config::OPTIONS.each do |method|
+      def_delegators :config, method, "#{method}="
+    end
+
+    # Delegate Capybara global configurations
+    # @!method default_selector
+    #   See {Capybara.configure}
+    # @!method default_max_wait_time
+    #   See {Capybara.configure}
+    # @!method app_host
+    #   See {Capybara.configure}
+    # @!method always_include_port
+    #   See {Capybara.configure}
+    # @!method wait_on_first_by_default
+    #   See {Capybara.configure}
+    SessionConfig::OPTIONS.each do |method|
+      def_delegators :config, method, "#{method}="
+    end
 
     ##
     #
@@ -37,27 +68,33 @@ module Capybara
     #
     # === Configurable options
     #
-    # [app_host = String]                 The default host to use when giving a relative URL to visit  
-    # [always_include_port = Boolean]     Whether the Rack server's port should automatically be inserted into every visited URL (Default: false)  
-    # [asset_host = String]               Where dynamic assets are hosted - will be prepended to relative asset locations if present (Default: nil)  
-    # [run_server = Boolean]              Whether to start a Rack server for the given Rack app (Default: true)  
-    # [raise_server_errors = Boolean]     Should errors raised in the server be raised in the tests? (Default: true)  
-    # [server_errors = Array\<Class\>]    Error classes that should be raised in the tests if they are raised in the server and Capybara.raise_server_errors is true (Default: [StandardError])  
-    # [default_selector = :css/:xpath]    Methods which take a selector use the given type by default (Default: :css)  
-    # [default_max_wait_time = Numeric]   The maximum number of seconds to wait for asynchronous processes to finish (Default: 2)  
-    # [ignore_hidden_elements = Boolean]  Whether to ignore hidden elements on the page (Default: true)  
-    # [automatic_reload = Boolean]        Whether to automatically reload elements as Capybara is waiting (Default: true)  
-    # [save_and_open_page_path = String]  Where to put pages saved through save_and_open_page (Default: Dir.pwd)  
-    # [wait_on_first_by_default = Boolean]   Whether Node#first defaults to Capybara waiting behavior for at least 1 element to match (Default: false) 
+    # [app_host = String/nil]             The default host to use when giving a relative URL to visit, must be a valid URL e.g. http://www.example.com
+    # [always_include_port = Boolean]     Whether the Rack server's port should automatically be inserted into every visited URL unless another port is explicitly specified (Default: false)
+    # [asset_host = String]               Where dynamic assets are hosted - will be prepended to relative asset locations if present (Default: nil)
+    # [run_server = Boolean]              Whether to start a Rack server for the given Rack app (Default: true)
+    # [raise_server_errors = Boolean]     Should errors raised in the server be raised in the tests? (Default: true)
+    # [server_errors = Array\<Class\>]    Error classes that should be raised in the tests if they are raised in the server and Capybara.raise_server_errors is true (Default: [StandardError])
+    # [default_selector = :css/:xpath]    Methods which take a selector use the given type by default (Default: :css)
+    # [default_max_wait_time = Numeric]   The maximum number of seconds to wait for asynchronous processes to finish (Default: 2)
+    # [ignore_hidden_elements = Boolean]  Whether to ignore hidden elements on the page (Default: true)
+    # [automatic_reload = Boolean]        Whether to automatically reload elements as Capybara is waiting (Default: true)
+    # [save_path = String]  Where to put pages saved through save_(page|screenshot), save_and_open_(page|screenshot) (Default: Dir.pwd)
+    # [wait_on_first_by_default = Boolean]   Whether Node#first defaults to Capybara waiting behavior for at least 1 element to match (Default: false)
+    # [automatic_label_click = Boolean]   Whether Node#choose, Node#check, Node#uncheck will attempt to click the associated label element if the checkbox/radio button are non-visible (Default: false)
+    # [enable_aria_label = Boolean]  Whether fields, links, and buttons will match against aria-label attribute (Default: false)
+    # [reuse_server = Boolean]  Reuse the server thread between multiple sessions using the same app object (Default: true)
+    # [threadsafe = Boolean]  Whether sessions can be configured individually (Default: false)
+    # [server = Symbol]  The name of the registered server to use when running the app under test (Default: :webrick)
+    #
     # === DSL Options
     #
     # when using capybara/dsl, the following options are also available:
     #
-    # [default_driver = Symbol]           The name of the driver to use by default. (Default: :rack_test)  
-    # [javascript_driver = Symbol]        The name of a driver to use for JavaScript enabled tests. (Default: :selenium)  
+    # [default_driver = Symbol]           The name of the driver to use by default. (Default: :rack_test)
+    # [javascript_driver = Symbol]        The name of a driver to use for JavaScript enabled tests. (Default: :selenium)
     #
     def configure
-      yield self
+      yield ConfigureDeprecator.new(config)
     end
 
     ##
@@ -77,6 +114,26 @@ module Capybara
       drivers[name] = block
     end
 
+    ##
+    #
+    # Register a new server for Capybara.
+    #
+    #     Capybara.register_server :webrick do |app, port, host|
+    #       require 'rack/handler/webrick'
+    #       Rack::Handler::WEBrick.run(app, ...)
+    #     end
+    #
+    # @param [Symbol] name                    The name of the new driver
+    # @yield [app, port, host]                This block takes a rack app and a port and returns a rack server listening on that port
+    # @yieldparam [<Rack>] app                The rack application that this server will contain.
+    # @yieldparam port                        The port number the server should listen on
+    # @yieldparam host                        The host/ip to bind to
+    # @yieldreturn [Capybara::Driver::Base]   A Capybara driver instance
+    #
+    def register_server(name, &block)
+      servers[name.to_sym] = block
+    end
+
     ##
     #
     # Add a new selector to Capybara. Selectors can be used by various methods in Capybara
@@ -109,29 +166,30 @@ module Capybara
       Capybara::Selector.add(name, &block)
     end
 
-    def drivers
-      @drivers ||= {}
-    end
-
     ##
     #
-    # Register a proc that Capybara will call to run the Rack application.
+    # Modify a selector previously created by {Capybara.add_selector}.
+    # For example, adding a new filter to the :button selector to filter based on
+    # button style (a class) might look like this
     #
-    #     Capybara.server do |app, port|
-    #       require 'rack/handler/mongrel'
-    #       Rack::Handler::Mongrel.run(app, :Port => port)
+    #     Capybara.modify_selector(:button) do
+    #       filter (:style, valid_values: [:primary, :secondary]) { |node, style| node[:class].split.include? "btn-#{style}" }
     #     end
     #
-    # By default, Capybara will try to run webrick.
     #
-    # @yield [app, port]                      This block receives a rack app and port and should run a Rack handler
+    # @param [Symbol] name    The name of the selector to modify
+    # @yield                  A block executed in the context of the existing {Capybara::Selector}
     #
-    def server(&block)
-      if block_given?
-        @server = block
-      else
-        @server
-      end
+    def modify_selector(name, &block)
+      Capybara::Selector.update(name, &block)
+    end
+
+    def drivers
+      @drivers ||= {}
+    end
+
+    def servers
+      @servers ||= {}
     end
 
     ##
@@ -140,7 +198,16 @@ module Capybara
     # in a {Capybara::Node::Simple} which exposes all {Capybara::Node::Matchers},
     # {Capybara::Node::Finders} and {Capybara::Node::DocumentMatchers}. This allows you to query
     # any string containing HTML in the exact same way you would query the current document in a Capybara
-    # session. For example:
+    # session.
+    #
+    # Example: A single element
+    #
+    #     node = Capybara.string('<a href="foo">bar</a>')
+    #     anchor = node.first('a')
+    #     anchor[:href] #=> 'foo'
+    #     anchor.text #=> 'bar'
+    #
+    # Example: Multiple elements
     #
     #     node = Capybara.string <<-HTML
     #       <ul>
@@ -150,7 +217,7 @@ module Capybara
     #     HTML
     #
     #     node.find('#projects').text # => 'Projects'
-    #     node.has_selector?('li#home', :text => 'Home')
+    #     node.has_selector?('li#home', text: 'Home')
     #     node.has_selector?('#projects')
     #     node.find('ul').find('li:first-child').text # => 'Home'
     #
@@ -168,19 +235,10 @@ module Capybara
     # manually.
     #
     # @param [Rack Application] app    The rack application to run
-    # @param [Fixnum] port              The port to run the application on
+    # @param [Integer] port              The port to run the application on
     #
     def run_default_server(app, port)
-      require 'rack/handler/webrick'
-      Rack::Handler::WEBrick.run(app, :Host => server_host, :Port => port, :AccessLog => [], :Logger => WEBrick::Log::new(nil, 0))
-    end
-
-    ##
-    #
-    # @return [Symbol]    The name of the driver to use by default
-    #
-    def default_driver
-      @default_driver || :rack_test
+      servers[:webrick].call(app, port, server_host)
     end
 
     ##
@@ -188,16 +246,20 @@ module Capybara
     # @return [Symbol]    The name of the driver currently in use
     #
     def current_driver
-      @current_driver || default_driver
+      if threadsafe
+        Thread.current['capybara_current_driver']
+      else
+        @current_driver
+      end || default_driver
     end
     alias_method :mode, :current_driver
 
-    ##
-    #
-    # @return [Symbol]    The name of the driver used when JavaScript is needed
-    #
-    def javascript_driver
-      @javascript_driver || :selenium
+    def current_driver=(name)
+      if threadsafe
+        Thread.current['capybara_current_driver'] = name
+      else
+        @current_driver = name
+      end
     end
 
     ##
@@ -205,7 +267,7 @@ module Capybara
     # Use the default driver as the current driver
     #
     def use_default_driver
-      @current_driver = nil
+      self.current_driver = nil
     end
 
     ##
@@ -217,15 +279,7 @@ module Capybara
       Capybara.current_driver = driver
       yield
     ensure
-      @current_driver = previous_driver
-    end
-
-    ##
-    #
-    # @return [String]    The IP address bound by default server
-    #
-    def server_host
-      @server_host || '127.0.0.1'
+      self.current_driver = previous_driver
     end
 
     ##
@@ -256,7 +310,8 @@ module Capybara
     # as cookies.
     #
     def reset_sessions!
-      session_pool.each { |mode, session| session.reset! }
+      #reset in reverse so sessions that started servers are reset last
+      session_pool.reverse_each { |_mode, session| session.reset! }
     end
     alias_method :reset!, :reset_sessions!
 
@@ -267,7 +322,19 @@ module Capybara
     # @return [Symbol]    The name of the currently used session.
     #
     def session_name
-      @session_name ||= :default
+      if threadsafe
+        Thread.current['capybara_session_name'] ||= :default
+      else
+        @session_name ||= :default
+      end
+    end
+
+    def session_name=(name)
+      if threadsafe
+        Thread.current['capybara_session_name'] = name
+      else
+        @session_name = name
+      end
     end
 
     ##
@@ -275,11 +342,19 @@ module Capybara
     # Yield a block using a specific session name.
     #
     def using_session(name)
-      previous_session_name = self.session_name
+      previous_session_info = {
+        session_name: session_name,
+        current_driver: current_driver,
+        app: app
+      }
       self.session_name = name
       yield
     ensure
-      self.session_name = previous_session_name
+      self.session_name = previous_session_info[:session_name]
+      if threadsafe
+        self.current_driver = previous_session_info[:current_driver]
+        self.app = previous_session_info[:app]
+      end
     end
 
     ##
@@ -292,21 +367,13 @@ module Capybara
     def HTML(html)
       Nokogiri::HTML(html).tap do |document|
         document.xpath('//textarea').each do |textarea|
-          textarea.content=textarea.content.sub(/\A\n/,'')
+          textarea['_capybara_raw_value'] = textarea.content.sub(/\A\n/,'')
         end
       end
     end
-    
-    # @deprecated Use default_max_wait_time instead
-    def default_wait_time
-      deprecate('default_wait_time', 'default_max_wait_time', true)
-      default_max_wait_time
-    end
-    
-    # @deprecated Use default_max_wait_time= instead
-    def default_wait_time=(t)
-      deprecate('default_wait_time=', 'default_max_wait_time=')
-      self.default_max_wait_time = t
+
+    def session_options
+      config.session_options
     end
 
     def included(base)
@@ -314,13 +381,10 @@ module Capybara
       warn "`include Capybara` is deprecated. Please use `include Capybara::DSL` instead."
     end
 
-    def deprecate(method, alternate_method, once=false)
-      @deprecation_notified ||= {}
-      warn "DEPRECATED: ##{method} is deprecated, please use ##{alternate_method} instead" unless once and @deprecation_notified[method]
-      @deprecation_notified[method]=true
-    end
-
   private
+    def config
+      @config ||= Capybara::Config.new
+    end
 
     def session_pool
       @session_pool ||= {}
@@ -337,7 +401,6 @@ module Capybara
 
   require 'capybara/helpers'
   require 'capybara/session'
-  require 'capybara/dsl'
   require 'capybara/window'
   require 'capybara/server'
   require 'capybara/selector'
@@ -345,11 +408,15 @@ module Capybara
   require 'capybara/version'
 
   require 'capybara/queries/base_query'
-  require 'capybara/query'
+  require 'capybara/queries/selector_query'
   require 'capybara/queries/text_query'
   require 'capybara/queries/title_query'
   require 'capybara/queries/current_path_query'
-  
+  require 'capybara/queries/match_query'
+  require 'capybara/queries/ancestor_query'
+  require 'capybara/queries/sibling_query'
+  require 'capybara/query'
+
   require 'capybara/node/finders'
   require 'capybara/node/matchers'
   require 'capybara/node/actions'
@@ -372,10 +439,24 @@ module Capybara
   require 'capybara/selenium/driver'
 end
 
+Capybara.register_server :default do |app, port, _host|
+  Capybara.run_default_server(app, port)
+end
+
+Capybara.register_server :webrick do |app, port, host, options={}|
+  require 'rack/handler/webrick'
+  Rack::Handler::WEBrick.run(app, {Host: host, Port: port, AccessLog: [], Logger: WEBrick::Log::new(nil, 0)}.merge(options))
+end
+
+Capybara.register_server :puma do |app, port, host, options={}|
+  require 'rack/handler/puma'
+  Rack::Handler::Puma.run(app, {Host: host, Port: port, Threads: "0:4", workers: 0, daemon: false}.merge(options))
+end
+
 Capybara.configure do |config|
   config.always_include_port = false
   config.run_server = true
-  config.server {|app, port| Capybara.run_default_server(app, port)}
+  config.server = :default
   config.default_selector = :css
   config.default_max_wait_time = 2
   config.ignore_hidden_elements = true
@@ -383,10 +464,14 @@ Capybara.configure do |config|
   config.automatic_reload = true
   config.match = :smart
   config.exact = false
+  config.exact_text = false
   config.raise_server_errors = true
   config.server_errors = [StandardError]
   config.visible_text_only = false
   config.wait_on_first_by_default = false
+  config.automatic_label_click = false
+  config.enable_aria_label = false
+  config.reuse_server = true
 end
 
 Capybara.register_driver :rack_test do |app|
@@ -396,3 +481,14 @@ end
 Capybara.register_driver :selenium do |app|
   Capybara::Selenium::Driver.new(app)
 end
+
+Capybara.register_driver :selenium_chrome do |app|
+  Capybara::Selenium::Driver.new(app, :browser => :chrome)
+end
+
+Capybara.register_driver :selenium_chrome_headless do |app|
+  browser_options = ::Selenium::WebDriver::Chrome::Options.new()
+  browser_options.args << '--headless'
+  browser_options.args << '--disable-gpu'
+  Capybara::Selenium::Driver.new(app, browser: :chrome, options: browser_options)
+end

data/spec/basic_node_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Capybara do
@@ -51,7 +52,7 @@ RSpec.describe Capybara do
 
     it "allows using custom matchers" do
       Capybara.add_selector :lifeform do
-        xpath { |name| "//option[contains(.,'#{name}')]" }
+        xpath { |name| ".//option[contains(.,'#{name}')]" }
       end
       expect(string).to have_selector(:id, "page")
       expect(string).not_to have_selector(:id, 'does-not-exist')
@@ -68,13 +69,13 @@ RSpec.describe Capybara do
     end
 
     it "allows using matchers with text option" do
-      expect(string).to have_css('h1', :text => 'Totally awesome')
-      expect(string).not_to have_css('h1', :text => 'Not so awesome')
+      expect(string).to have_css('h1', text: 'Totally awesome')
+      expect(string).not_to have_css('h1', text: 'Not so awesome')
     end
 
     it "allows finding only visible nodes" do
-      expect(string.all(:css, '#secret', :visible => true)).to be_empty
-      expect(string.all(:css, '#secret', :visible => false).size).to eq(1)
+      expect(string.all(:css, '#secret', visible: true)).to be_empty
+      expect(string.all(:css, '#secret', visible: false).size).to eq(1)
     end
 
     it "allows finding elements and extracting text from them" do
@@ -100,7 +101,7 @@ RSpec.describe Capybara do
 
     it "allows finding elements and checking if they are visible" do
       expect(string.find('//h1')).to be_visible
-      expect(string.find(:css, "#secret", :visible => false)).not_to be_visible
+      expect(string.find(:css, "#secret", visible: false)).not_to be_visible
     end
 
     it "allows finding elements and checking if they are disabled" do

data/spec/capybara_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 
 RSpec.describe Capybara do
@@ -13,8 +14,8 @@ RSpec.describe Capybara do
     end
 
     it "should be accesible as the deprecated default_wait_time" do
-      expect_any_instance_of(Kernel).to receive(:warn).once.with('DEPRECATED: #default_wait_time= is deprecated, please use #default_max_wait_time= instead')
-      expect_any_instance_of(Kernel).to receive(:warn).once.with('DEPRECATED: #default_wait_time is deprecated, please use #default_max_wait_time instead')
+      expect(Capybara.send(:config)).to receive(:warn).ordered.with('DEPRECATED: #default_wait_time= is deprecated, please use #default_max_wait_time= instead')
+      expect(Capybara.send(:config)).to receive(:warn).ordered.with('DEPRECATED: #default_wait_time is deprecated, please use #default_max_wait_time instead')
       @previous_default_time = Capybara.default_max_wait_time
       Capybara.default_wait_time = 5
       expect(Capybara.default_wait_time).to eq(5)
@@ -33,9 +34,24 @@ RSpec.describe Capybara do
     end
   end
 
+  describe '.register_server' do
+    it "should add a new server" do
+      handler = double("handler")
+      Capybara.register_server :blob do |app, port, host|
+        handler.run
+      end
+
+      expect(Capybara.servers).to have_key(:blob)
+    end
+  end
+
   describe ".server" do
+    before do
+      @old_server = Capybara.server
+    end
+
     after do
-      Capybara.server {|app, port| Capybara.run_default_server(app, port)}
+      Capybara.server(&@old_server)
     end
 
     it "should default to a proc that calls run_default_server" do
@@ -49,11 +65,81 @@ RSpec.describe Capybara do
       Capybara.server(&server)
       expect(Capybara.server).to eq(server)
     end
+
+    it "should have :webrick registered" do
+      require 'rack/handler/webrick'
+      mock_app = double('app')
+      Capybara.server = :webrick
+      expect(Rack::Handler::WEBrick).to receive(:run)
+      Capybara.server.call(mock_app, 8000)
+    end
+
+    it "should have :puma registered" do
+      require 'rack/handler/puma'
+      mock_app = double('app')
+      Capybara.server = :puma
+      expect(Rack::Handler::Puma).to receive(:run).with(mock_app, hash_including(Host: nil, Port: 8000))
+      Capybara.server.call(mock_app, 8000)
+    end
+
+    it "should pass options to server" do
+      require 'rack/handler/puma'
+      mock_app = double('app')
+      Capybara.server = :puma, { Silent: true }
+      expect(Rack::Handler::Puma).to receive(:run).with(mock_app, hash_including(Host: nil, Port: 9000, Silent: true))
+      Capybara.server.call(mock_app, 9000)
+    end
+  end
+
+  describe 'app_host' do
+    after do
+      Capybara.app_host = nil
+    end
+
+    it "should warn if not a valid URL" do
+      expect { Capybara.app_host = "www.example.com" }.to raise_error(ArgumentError, /Capybara\.app_host should be set to a url/)
+    end
+
+    it "should not warn if a valid URL" do
+      expect { Capybara.app_host = "http://www.example.com" }.not_to raise_error
+    end
+
+    it "should not warn if nil" do
+      expect { Capybara.app_host = nil }.not_to raise_error
+    end
+  end
+
+  describe 'default_host' do
+    around do |test|
+      old_default = Capybara.default_host
+      test.run
+      Capybara.default_host = old_default
+    end
+
+    it "should warn if not a valid URL" do
+      expect { Capybara.default_host = "www.example.com" }.to raise_error(ArgumentError, /Capybara\.default_host should be set to a url/)
+    end
+
+    it "should not warn if a valid URL" do
+      expect { Capybara.default_host = "http://www.example.com" }.not_to raise_error
+    end
+  end
+
+  describe "configure" do
+    it 'deprecates calling non configuration option methods in configure' do
+      expect_any_instance_of(Kernel).to receive(:warn).
+        with('Calling register_driver from Capybara.configure is deprecated - please call it on Capybara directly ( Capybara.register_driver(...) )')
+      Capybara.configure do |config|
+        config.register_driver(:random_name) do
+          #just a random block
+        end
+      end
+    end
   end
 end
 
 RSpec.describe Capybara::Session do
-  context 'with non-existant driver' do
+  context 'with nonexistent driver' do
     it "should raise an error" do
       expect {
         Capybara::Session.new(:quox, TestApp).driver

data/spec/dsl_spec.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
 require 'spec_helper'
 require 'capybara/dsl'
 
@@ -5,7 +6,7 @@ class TestClass
   include Capybara::DSL
 end
 
-Capybara::SpecHelper.run_specs TestClass.new, "DSL", :capybara_skip => [
+Capybara::SpecHelper.run_specs TestClass.new, "DSL", capybara_skip: [
   :js,
   :modals,
   :screenshot,
@@ -15,6 +16,7 @@ Capybara::SpecHelper.run_specs TestClass.new, "DSL", :capybara_skip => [
   :server,
   :hover,
   :about_scheme,
+  :psc
 ]
 
 RSpec.describe Capybara::DSL do

data/spec/filter_set_spec.rb

@@ -0,0 +1,28 @@
+# frozen_string_literal: true
+require 'spec_helper'
+
+RSpec.describe Capybara::Selector::FilterSet do
+  after do
+    Capybara::Selector::FilterSet.remove(:test)
+  end
+
+  it "allows node filters" do
+    fs = Capybara::Selector::FilterSet.add(:test) do
+      filter(:node_test, :boolean) { |node, value| true }
+      expression_filter(:expression_test, :boolean) { |expr, value| true }
+    end
+
+    expect(fs.node_filters.keys).to include(:node_test)
+    expect(fs.node_filters.keys).not_to include(:expression_test)
+  end
+
+  it "allows expression filters" do
+    fs = Capybara::Selector::FilterSet.add(:test) do
+      filter(:node_test, :boolean) { |node, value| true }
+      expression_filter(:expression_test, :boolean) { |expr, value| true }
+    end
+
+    expect(fs.expression_filters.keys).to include(:expression_test)
+    expect(fs.expression_filters.keys).not_to include(:node_test)
+  end
+end

data/spec/fixtures/capybara.csv

@@ -0,0 +1 @@
+test, mime-type, file