skipped tests should not be reported as failure

Author: MariuszCwikla

acceptance/test_unit_example_test.rb

@@ -14,4 +14,7 @@ class TestUnitExampleTestTwo < Test::Unit::TestCase
   def test_two
     assert true
   end
+  def test_three
+    omit 'omitted'
+  end
 end

acceptance/verification_spec.rb

@@ -36,11 +36,16 @@
 
     it { is_expected.to have(0).errors }
     it { is_expected.to have(0).failures }
-    it { is_expected.to have(1).testcases }
+    it { is_expected.to have(2).testcases }
 
     describe "the assertion count" do
       subject { result.assertions_count }
       it { is_expected.to eql 1 }
+    end    
+
+    describe "the skipped count" do
+      subject { result.skipped_count}
+      it { is_expected.to eql 1 }
     end
 
     it_behaves_like "a report with consistent attribute counts"

lib/ci/reporter/test_unit.rb

@@ -31,6 +31,7 @@ class TestUnitError
       def initialize(fault) @fault = fault end
       def failure?() false end
       def error?() true end
+      def skipped?() false end
       def name() @fault.exception.class.name end
       def message() @fault.exception.message end
       def location() @fault.exception.backtrace.join("\n") end
@@ -41,6 +42,7 @@ class TestUnitFailure
       def initialize(fault) @fault = fault end
       def failure?() true end
       def error?() false end
+      def skipped?() false end
       def name() Test::Unit::AssertionFailedError.name end
       def message() @fault.message end
       def location() @fault.location.join("\n") end
@@ -51,6 +53,7 @@ class TestUnitSkipped
       def initialize(fault) @fault = fault end
       def failure?() false end
       def error?() false end
+      def skipped?() true end
       def name() @fault.class.name end
       def message() @fault.message end
       def location() @fault.location.join("\n") end
@@ -61,6 +64,7 @@ class TestUnitNotification
       def initialize(fault) @fault = fault end
       def failure?() false end
       def error?() false end
+      def skipped?() false end
       def name() @fault.class.name end
       def message() @fault.message end
       def location() @fault.location.join("\n") end
@@ -103,7 +107,9 @@ def test_finished(name)
 
       def fault(fault)
         tc = @current_suite.testcases.last
-        tc.failures << Failure.new(fault)
+        f = Failure.new(fault)
+        tc.failures << f 
+        tc.skipped=true if f.skipped? 
       end
 
       def finished(elapsed_time)