Add a YAML config file for specifying browser paths on Windows.

Run `rake test:run` once and it’ll copy the sample config file to
`test/unit/browsers.yml`.

Author: savetheclocktower

.gitignore

@@ -1,6 +1,7 @@
 .DS_Store
 pkg
 test/unit/tmp/*
+test/unit/browsers.yml
 doc
 tmp
 *.pdoc.yaml

test/unit/browsers.sample.yml

@@ -0,0 +1,9 @@
+#
+# If the test runner doesn't know where your browsers are, you can define
+# them here.
+#
+
+browsers:
+  firefox: C:\Program Files\Mozilla Firefox\firefox.exe
+  opera: C:\Program Files\Opera\launcher.exe
+  chrome: C:\Program Files\Google\Chrome\Application\chrome.exe

test/unit/runner.rb

@@ -3,6 +3,7 @@
 require 'cgi'
 require 'json'
 require 'rbconfig'
+require 'yaml'
 
 # A barebones runner for testing across multiple browsers quickly.
 #
@@ -57,6 +58,26 @@ module Runner
   host = RbConfig::CONFIG['host']
   IS_WINDOWS = host.include?('mswin') || host.include?('mingw32')
 
+  class << self
+    PWD = Pathname.new(File.dirname(__FILE__))
+    CONFIG_FILE = PWD.join('browsers.yml')
+
+    unless CONFIG_FILE.exist?
+      # Copy the sample config file to an actual config file.
+      sample = PWD.join('browsers.sample.yml')
+      File.open(CONFIG_FILE, 'w') do |file|
+        file.write( File.read(sample) )
+      end
+    end
+
+    CONFIG = YAML::load_file(CONFIG_FILE)
+
+    def config
+      CONFIG
+    end
+  end
+
+
   module Browsers
 
     class Abstract
@@ -87,6 +108,15 @@ def linux?
         host.include?('linux')
       end
 
+      def configured_path
+        browsers = Runner::config['browsers']
+        browsers[short_name.to_s]
+      end
+
+      def default_path
+        nil
+      end
+
       def visit(url)
         if windows?
           system(%Q["#{path}" "#{url}"])
@@ -106,6 +136,10 @@ def name
         linux? ? n.downcase : n
       end
 
+      def short_name
+        nil
+      end
+
       def escaped_name
         name.gsub(' ', '\ ')
       end
@@ -114,15 +148,19 @@ def path
         if macos?
           File.expand_path("/Applications/#{name}.app")
         else
-          @path || nil
+          configured_path || default_path || nil
         end
       end
     end
 
     class Firefox < Abstract
 
-      def initialize(path=File.join(ENV['ProgramFiles'] || 'c:\Program Files', '\Mozilla Firefox\firefox.exe'))
-        @path = path
+      def short_name
+        :firefox
+      end
+
+      def default_path
+        'C:\Program Files\Mozilla Firefox\firefox.exe'
       end
 
       def supported?
@@ -133,6 +171,10 @@ def supported?
 
     class IE < Abstract
 
+      def short_name
+        :ie
+      end
+
       def setup
         require 'win32ole' if windows?
       end
@@ -155,6 +197,10 @@ def visit(url)
 
     class Safari < Abstract
 
+      def short_name
+        :safari
+      end
+
       def supported?
         macos?
       end
@@ -163,8 +209,12 @@ def supported?
 
     class Chrome < Abstract
 
-      def initialize(path=nil)
-        @path = path || 'C:\Program Files\Google\Chrome\Application\chrome.exe'
+      def short_name
+        :chrome
+      end
+
+      def default_path
+        'C:\Program Files\Google\Chrome\Application\chrome.exe'
       end
 
       def name
@@ -175,8 +225,12 @@ def name
 
     class Opera < Abstract
 
-      def initialize(path='C:\Program Files\Opera\Opera.exe')
-        @path = path
+      def short_name
+        :opera
+      end
+
+      def default_path
+        'C:\Program Files\Opera\launcher.exe'
       end
 
     end
@@ -284,6 +338,7 @@ def run(browsers=nil, tests=nil, grep=nil)
         end
         if !browser.installed?
           puts "Skipping #{browser.name} (not installed on this OS)"
+          puts "  (edit test/unit/browsers.yml if this is in error)"
           next
         end
         print "Running in #{browser.name}... "