rspec-extra-formatters 0.3 → 0.4

data/README.rst

@@ -8,7 +8,7 @@ RSpec Formatters
     ok 1 - TapFormatter should initialize the counter to 0
     ok 2 - TapFormatter example_passed should increment the counter and use the full_description attribute
     ok 3 - TapFormatter example_failed should increment the counter and use the full_description attribute
-    ok 4 - TapFormatter example_pending should do the same as example_failed
+    ok 4 - TapFormatter example_pending should do the same as example_failed with TODO comment
     ok 5 - TapFormatter dump_summary should print the number of tests if there were tests
     ok 6 - TapFormatter dump_summary should print nothing if there were not tests
     ok 7 - JUnitFormatter should initialize the tests with failures and success

data/lib/rspec-extra-formatters/junit_formatter.rb

@@ -35,7 +35,7 @@ class JUnitFormatter < RSpec::Core::Formatters::BaseFormatter
 
   def initialize(output)
     super(output)      
-    @test_results = { :failures => [], :successes => [] }
+    @test_results = { :failures => [], :successes => [], :skipped => [] }
   end
 
   def example_passed(example)
@@ -44,7 +44,8 @@ class JUnitFormatter < RSpec::Core::Formatters::BaseFormatter
   end
 
   def example_pending(example)
-    self.example_failed(example)
+    super(example)
+    @test_results[:skipped].push(example)
   end
 
   def example_failed(example)
@@ -66,7 +67,7 @@ class JUnitFormatter < RSpec::Core::Formatters::BaseFormatter
   def dump_summary(duration, example_count, failure_count, pending_count)
     super(duration, example_count, failure_count, pending_count)
     output.puts("<?xml version=\"1.0\" encoding=\"utf-8\" ?>")
-    output.puts("<testsuite errors=\"0\" failures=\"#{failure_count+pending_count}\" tests=\"#{example_count}\" time=\"#{duration}\" timestamp=\"#{Time.now.iso8601}\">")
+    output.puts("<testsuite errors=\"0\" failures=\"#{failure_count}\" skipped=\"#{pending_count}\" tests=\"#{example_count}\" time=\"#{duration}\" timestamp=\"#{Time.now.iso8601}\">")
     output.puts("  <properties />")
     @test_results[:successes].each do |t|
       md          = t.metadata
@@ -86,6 +87,15 @@ class JUnitFormatter < RSpec::Core::Formatters::BaseFormatter
       output.puts("    </failure>")
       output.puts("  </testcase>")
     end
+    @test_results[:skipped].each do |t|
+      md          = t.metadata
+      description = _xml_escape(md[:full_description])
+      file_path   = _xml_escape(md[:file_path])
+      runtime     = md[:execution_result][:run_time]
+      output.puts("  <testcase classname=\"#{file_path}\" name=\"#{description}\" time=\"#{runtime}\">")
+      output.puts("    <skipped/>")
+      output.puts("  </testcase>")
+    end
     output.puts("</testsuite>")
   end
 

data/lib/rspec-extra-formatters/tap_formatter.rb

@@ -32,6 +32,10 @@ class TapFormatter < RSpec::Core::Formatters::BaseFormatter
 
   attr_reader :total
 
+  OK     = 'ok'
+  NOT_OK = 'not ok'
+  TODO   = '# TODO '
+
   def initialize(output)
     super(output)      
     @total = 0
@@ -39,18 +43,17 @@ class TapFormatter < RSpec::Core::Formatters::BaseFormatter
 
   def example_passed(example)
     super(example)
-    @total += 1
-    output.puts("ok #{@total} - #{example.metadata[:full_description]}")
+    tap_example_output(OK, example)
   end
 
   def example_pending(example)
-    self.example_failed(example)
+    super(example)
+    tap_example_output(NOT_OK, example, TODO)
   end
 
   def example_failed(example)
     super(example)
-    @total += 1
-    output.puts("not ok #{@total} - #{example.metadata[:full_description]}")
+    tap_example_output(NOT_OK, example)
   end
 
   def dump_summary(duration, example_count, failure_count, pending_count)
@@ -60,4 +63,10 @@ class TapFormatter < RSpec::Core::Formatters::BaseFormatter
     end
   end
 
+  private
+  def tap_example_output(ok, example, modifier='')
+    @total += 1
+    output.puts("#{ok} #{@total} - #{modifier}#{example.metadata[:full_description]}")
+  end
+
 end

data/spec/rspec-extra-formatters/junit_formatter_spec.rb

@@ -37,7 +37,7 @@ describe JUnitFormatter do
   end
 
   it "should initialize the tests with failures and success" do
-    JUnitFormatter.new(StringIO.new).test_results.should eql({:failures=>[], :successes=>[]})
+    JUnitFormatter.new(StringIO.new).test_results.should eql({:failures=>[], :successes=>[], :skipped=>[]})
   end
 
   describe "example_passed" do
@@ -62,10 +62,10 @@ describe JUnitFormatter do
 
   describe "example_pending" do
 
-    it "should do the same as example_failed" do
+    it "should push the example obj into the skipped list" do
       f = JUnitFormatter.new(StringIO.new)
       f.example_pending("foobar")
-      f.test_results[:failures].should eql(["foobar"])
+      f.test_results[:skipped].should eql(["foobar"])
     end
 
   end
@@ -129,16 +129,24 @@ describe JUnitFormatter do
                                                                                             , :run_time              => 0.1 \
                                                                                             }
                                                                      })
+
+      example2 = mock("example-2")
+      example2.should_receive(:metadata).and_return({ :full_description => "foobar-pending" \
+                                                   , :file_path        => "lib/foobar-s.rb" \
+                                                   , :execution_result => { :run_time => 0.1 } \
+                                                   })
+
    
       output = StringIO.new
       f = JUnitFormatter.new(output)
       f.example_passed(example0)
       f.example_failed(example1)
-      f.dump_summary("0.1", 2, 1, 0)
+      f.example_pending(example2)      
+      f.dump_summary("0.1", 3, 1, 1)
 
       output.string.should == <<-EOF
 <?xml version="1.0" encoding="utf-8" ?>
-<testsuite errors="0" failures="1" tests="2" time="0.1" timestamp="#{@now.iso8601}">
+<testsuite errors="0" failures="1" skipped="1" tests="3" time="0.1" timestamp="#{@now.iso8601}">
   <properties />
   <testcase classname="lib/foobar-s.rb" name="foobar-success" time="0.1" />
   <testcase classname="lib/foobar-f.rb" name="foobar-failure" time="0.1">
@@ -146,6 +154,9 @@ describe JUnitFormatter do
 <![CDATA[ foobar\nfoo\nbar ]]>
     </failure>
   </testcase>
+  <testcase classname="lib/foobar-s.rb" name="foobar-pending" time="0.1">
+    <skipped/>
+  </testcase>
 </testsuite>
       EOF
     end
@@ -164,7 +175,7 @@ describe JUnitFormatter do
 
       output.string.should == <<-EOF
 <?xml version="1.0" encoding="utf-8" ?>
-<testsuite errors="0" failures="1" tests="2" time="0.1" timestamp="#{@now.iso8601}">
+<testsuite errors="0" failures="1" skipped="0" tests="2" time="0.1" timestamp="#{@now.iso8601}">
   <properties />
   <testcase classname="lib/&gt;foobar-s.rb" name="foobar-success &gt;&gt;&gt; &amp;&quot;&amp; &lt;&lt;&lt;" time="0.1" />
 </testsuite>

data/spec/rspec-extra-formatters/tap_formatter_spec.rb

@@ -68,7 +68,7 @@ describe TapFormatter do
 
   describe "example_pending" do
 
-    it "should do the same as example_failed" do
+    it "should do the same as example_failed with TODO comment" do
       example = mock("example")
       example.should_receive(:metadata).and_return({:full_description => "foobar"})
       
@@ -77,7 +77,7 @@ describe TapFormatter do
       f.example_pending(example)
       
       f.total.should eql(1)
-      output.string.should == "not ok 1 - foobar\n"
+      output.string.should == "not ok 1 - # TODO foobar\n"
     end
 
   end
@@ -100,7 +100,7 @@ describe TapFormatter do
       output.string.should == <<-EOF
 ok 1 - foobar
 not ok 2 - foobar
-not ok 3 - foobar
+not ok 3 - # TODO foobar
 1..3
       EOF
     end

metadata

@@ -1,8 +1,12 @@
 --- !ruby/object:Gem::Specification 
 name: rspec-extra-formatters
 version: !ruby/object:Gem::Version 
+  hash: 3
   prerelease: 
-  version: "0.3"
+  segments: 
+  - 0
+  - 4
+  version: "0.4"
 platform: ruby
 authors: 
 - Diego Souza
@@ -11,8 +15,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-05-26 00:00:00 -03:00
-default_executable: 
+date: 2012-04-24 00:00:00 Z
 dependencies: 
 - !ruby/object:Gem::Dependency 
   name: rspec
@@ -22,6 +25,9 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
         version: "0"
   type: :development
   version_requirements: *id001
@@ -36,16 +42,15 @@ extra_rdoc_files:
 - README.rst
 files: 
 - lib/tap_formatter.rb
-- lib/rspec-extra-formatters/junit_formatter.rb
+- lib/rspec-extra-formatters.rb
 - lib/rspec-extra-formatters/tap_formatter.rb
+- lib/rspec-extra-formatters/junit_formatter.rb
 - lib/j_unit_formatter.rb
-- lib/rspec-extra-formatters.rb
+- LICENSE
+- README.rst
 - spec/rspec-extra-formatters/tap_formatter_spec.rb
 - spec/rspec-extra-formatters/junit_formatter_spec.rb
 - spec/spec_helper.rb
-- LICENSE
-- README.rst
-has_rdoc: true
 homepage: http://dsouza.bitforest.org/2011/01/22/rspec-tap-and-junit-formatters/
 licenses: []
 
@@ -59,17 +64,23 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
       version: "0"
 required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
+      hash: 3
+      segments: 
+      - 0
       version: "0"
 requirements: []
 
 rubyforge_project: 
-rubygems_version: 1.5.0
+rubygems_version: 1.8.6
 signing_key: 
 specification_version: 3
 summary: TAP and JUnit formatters for rspec