Merge pull request #50 from rspec/move-textmate-formatter

Author: jperkins

Support/lib/rspec/mate.rb

@@ -7,6 +7,7 @@
 require 'rspec/mate/runner'
 require 'rspec/mate/options'
 require 'rspec/mate/switch_command'
+require 'rspec/mate/text_mate_formatter'
 
 rspec_lib = nil
 

Support/lib/rspec/mate/runner.rb

@@ -35,12 +35,12 @@ def run_focussed(stdout, options={})
       end
 
       def run(stdout, options)
-        formatter = ENV['TM_RSPEC_FORMATTER'] || 'textmate'
-
+        formatter  = ENV['TM_RSPEC_FORMATTER'] || 'RSpec::Mate::Formatters::TextMateFormatter'
         stderr     = StringIO.new
         old_stderr = $stderr
         $stderr    = stderr
 
+
         argv = options[:files].dup
         argv << '--format' << formatter
 

Support/lib/rspec/mate/text_mate_formatter.rb

@@ -0,0 +1,34 @@
+require 'cgi'
+require 'rspec/core/formatters/html_formatter'
+
+module RSpec
+  module Mate
+    module Formatters
+      # Formats backtraces so they're clickable by TextMate
+      class TextMateFormatter < RSpec::Core::Formatters::HtmlFormatter
+        def backtrace_line(line, skip_textmate_conversion=false)
+          if skip_textmate_conversion
+            super(line)
+          else
+            format_backtrace_line_for_textmate(super(line))
+          end
+        end
+
+        def format_backtrace_line_for_textmate(line)
+          return nil unless line
+          CGI.escapeHTML(line).sub(/([^:]*\.e?rb):(\d*)/) do
+            "<a href=\"txmt://open?url=file://#{File.expand_path($1)}&line=#{$2}\">#{$1}:#{$2}</a> "
+          end
+        end
+
+        def extra_failure_content(exception)
+          require 'rspec/core/formatters/snippet_extractor'
+          backtrace = exception.backtrace.map {|line| backtrace_line(line, :skip_textmate_conversion)}
+          backtrace.compact!
+          @snippet_extractor ||= RSpec::Core::Formatters::SnippetExtractor.new
+          "    <pre class=\"ruby\"><code>#{@snippet_extractor.snippet(backtrace)}</code></pre>"
+        end
+      end
+    end
+  end
+end