Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

Mark skipped tests as skipped instead of failure #52

Merged
merged 1 commit into from

2 participants

@stopdropandrew

Pretty simple tweak, there weren't any minitest acceptance tests so I just went ahead and made the change.

@nicksieger nicksieger merged commit 30aa3d2 into ci-reporter:master
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 16, 2012
  1. @stopdropandrew
This page is out of date. Refresh to see the latest.
Showing with 5 additions and 1 deletion.
  1. +5 −1 lib/ci/reporter/minitest.rb
View
6 lib/ci/reporter/minitest.rb
@@ -211,7 +211,11 @@ def finish_case
def fault(fault, type = nil, meth = nil)
tc = @current_suite.testcases.last
- tc.failures << Failure.new(fault, type, meth)
+ if :skip == type
+ tc.skipped = true
+ else
+ tc.failures << Failure.new(fault, type, meth)
+ end
end
end
Something went wrong with that request. Please try again.