Allow users to customize how the WebDriver is built

- Allows the user to customize the profile to do things like modify
  script timeouts

[finish #37287185]

Author: Gregg Van Hove

README.md

@@ -94,6 +94,28 @@ Create a jasmine_selenium_runner.yml in spec/javascripts/support/ with the follo
     selenium_server: <full url to selenium server>
     browser: <%= ENV['JASMINE_BROWSER'] %>
 
+### Customizing the browser profile
+
+Make a class that extends `JasmineSeleniumRunner::ConfigureJasmine` and override the `selenium_options` method
+
+    class MyConfigurer < JasmineSeleniumRunner::ConfigureJasmine
+      def selenium_options
+        options = super
+        if browser =~ /^firefox/
+          options = super
+          options[:profile] ||= Selenium::WebDriver::Firefox::Profile.new
+          options[:profile]['dom.max_chrome_script_run_time'] = 20
+          options[:profile]['dom.max_script_run_time'] = 20
+        end
+        options
+      end
+    end
+
+Create a jasmine_selenium_runner.yml in spec/javascripts/support/ with the following content:
+
+    ---
+    configuration_class: MyConfigurer
+
 ## Contributing
 
 1. Fork it

lib/jasmine_selenium_runner/configure_jasmine.rb

@@ -10,7 +10,12 @@ def self.install_selenium_runner
         config.port = 5555 if runner_config['use_sauce'] #Sauce only proxies certain ports
 
         config.runner = lambda { |formatter, jasmine_server_url|
-          new(formatter, jasmine_server_url, runner_config).make_runner
+          configuration_class = if runner_config['configuration_class']
+                                  const_get(runner_config['configuration_class'])
+                                else
+                                  self
+                                end
+          configuration_class.new(formatter, jasmine_server_url, runner_config).make_runner
         }
       end
     end
@@ -76,14 +81,18 @@ def remote_webdriver(server_url)
     end
 
     def local_webdriver
-      selenium_options = {}
+      Selenium::WebDriver.for(browser.to_sym, selenium_options)
+    end
+
+    def selenium_options
       if browser == 'firefox-firebug'
         require File.join(File.dirname(__FILE__), 'firebug/firebug')
-        (profile = Selenium::WebDriver::Firefox::Profile.new)
+        profile = Selenium::WebDriver::Firefox::Profile.new
         profile.enable_firebug
-        selenium_options[:profile] = profile
+        { :profile => profile }
+      else
+        {}
       end
-      Selenium::WebDriver.for(browser.to_sym, selenium_options)
     end
 
     protected

spec/jasmine_selenium_runner/configure_jasmine_spec.rb

@@ -25,4 +25,48 @@
       configurer.make_runner
     end
   end
+
+  context "specifying a custom configurer" do
+    class FakeConfig
+      attr_accessor :port, :runner
+    end
+
+    def configure
+      Dir.stub(:pwd).and_return(working_dir)
+      Jasmine.stub(:configure).and_yield(fake_config)
+      JasmineSeleniumRunner::ConfigureJasmine.install_selenium_runner
+    end
+
+    def stub_config_file(config_obj)
+      config_path = File.join(working_dir, 'spec', 'javascripts', 'support', 'jasmine_selenium_runner.yml')
+      File.stub(:exist?).and_call_original
+      File.stub(:exist?).with(config_path).and_return(true)
+      File.stub(:read).and_call_original
+      File.stub(:read).with(config_path).and_return(YAML.dump(config_obj))
+    end
+
+    let(:working_dir) { 'hi' }
+    let(:fake_config) { FakeConfig.new }
+
+    module Foo
+      class Bar
+        def initialize(formatter, jasmine_server_url, config)
+        end
+
+        def make_runner
+        end
+      end
+    end
+
+    before do
+      stub_config_file 'configuration_class' => 'Foo::Bar'
+      configure
+    end
+
+    it "should use the custom class" do
+      Selenium::WebDriver.should_not_receive(:for)
+      Foo::Bar.any_instance.should_receive(:make_runner)
+      fake_config.runner.call(nil, nil)
+    end
+  end
 end