capybara-screenshot 0.3.1 → 0.3.2

data/README.md

@@ -63,7 +63,7 @@ The gem supports the default rendering method for Capybara to generate the scree
 
 	page.driver.render(path)
 
-There are also some specific driver configurations for Selenium, Webkit, and Poltergeist. See [https://github.com/mattheworiordan/capybara-screenshot/blob/master/lib/capybara-screenshot.rb](the definitions here). The Rack::Test driver, Rails' default, does not allow
+There are also some specific driver configurations for Selenium, Webkit, and Poltergeist. See [the definitions here](https://github.com/mattheworiordan/capybara-screenshot/blob/master/lib/capybara-screenshot.rb). The Rack::Test driver, Rails' default, does not allow
 rendering, so it has a driver definition as a noop.
 
 If a driver is not found the default rendering will be used. If this doesn't work with your driver, then you can
@@ -86,6 +86,14 @@ you can override how the basename is generated for the file like so
 	  end
 	end
 
+Screenshot path in RSpec metadata
+---------------------------------
+
+By default capybara-screenshot will append the screenshot path to a failing spec's full_description which effectively
+changes the name of the spec and breaks trend reporting using tools like CI Reporter.  You can turn off the default behavior
+with the following invocation
+
+    Capybara::Screenshot.append_screenshot_path = false
 
 Example application
 -------------------

data/lib/capybara-screenshot.rb

@@ -4,11 +4,13 @@ module Capybara
       attr_accessor :autosave_on_failure
       attr_accessor :registered_drivers
       attr_accessor :filename_prefix_formatters
+      attr_accessor :append_screenshot_path
     end
 
     self.autosave_on_failure = true
     self.registered_drivers = {}
     self.filename_prefix_formatters = {}
+    self.append_screenshot_path = true
 
     def self.screenshot_and_save_page
       saver = Saver.new(Capybara, Capybara.page)
@@ -82,7 +84,11 @@ Capybara::Screenshot.class_eval do
   end
 
   register_driver(:webkit) do |driver, path|
-    driver.render(path)
+    if driver.respond_to?(:save_screenshot)
+      driver.save_screenshot(path)
+    else
+      driver.render(path)
+    end
   end
 
   register_driver(:webkit_debug) do |driver, path|

data/lib/capybara-screenshot/rspec.rb

@@ -8,8 +8,10 @@ RSpec.configure do |config|
         saver = Capybara::Screenshot::Saver.new(Capybara, Capybara.page, true, filename_prefix)
         saver.save
 
-        example.metadata[:full_description] += "\n     Screenshot: #{saver.screenshot_path}"
+        if Capybara::Screenshot.append_screenshot_path
+          example.metadata[:full_description] += "\n     Screenshot: #{saver.screenshot_path}"
+        end
       end
     end
   end
-end
+end

data/lib/capybara-screenshot/version.rb

@@ -1,5 +1,5 @@
 module Capybara
   module Screenshot
-    VERSION = "0.3.1"
+    VERSION = "0.3.2"
   end
 end

data/spec/capybara-screenshot/saver_spec.rb

@@ -138,10 +138,28 @@ describe Capybara::Screenshot::Saver do
       capybara_mock.stub(:current_driver).and_return(:webkit)
     end
 
-    it 'should save driver render' do
-      driver_mock.should_receive(:render).with(screenshot_path)
+    context 'has render method' do
+      before do
+        driver_mock.stub(:respond_to?).with(:'save_screenshot').and_return(false)
+      end
 
-      saver.save
+      it 'should save driver render' do
+        driver_mock.should_receive(:render).with(screenshot_path)
+
+        saver.save
+      end
+    end
+
+    context 'has save_screenshot method' do
+      before do
+        driver_mock.stub(:respond_to?).with(:'save_screenshot').and_return(true)
+      end
+
+      it 'should save driver render' do
+        driver_mock.should_receive(:save_screenshot).with(screenshot_path)
+
+        saver.save
+      end
     end
   end
 
@@ -176,4 +194,4 @@ describe Capybara::Screenshot::Saver do
       saver.save
     end
   end
-end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: capybara-screenshot
 version: !ruby/object:Gem::Version
-  version: 0.3.1
+  version: 0.3.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-26 00:00:00.000000000 Z
+date: 2012-12-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: capybara
@@ -106,7 +106,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -502489258301273906
+      hash: -1096024297736661067
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -115,7 +115,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -502489258301273906
+      hash: -1096024297736661067
 requirements: []
 rubyforge_project: capybara-screenshot
 rubygems_version: 1.8.24