Skip to content
Browse files

Merge pull request #73 from seanwalbran/fix-const-defined-false-positive

do not search ancestors when checking Test::Unit.const_defined?
  • Loading branch information...
2 parents 63ea29a + 6535342 commit d20a3409d721811b4b4112a7d2d50a222aab22ec @nicksieger nicksieger committed
Showing with 2 additions and 2 deletions.
  1. +2 −2 lib/ci/reporter/test_unit.rb
View
4 lib/ci/reporter/test_unit.rb
@@ -13,8 +13,8 @@ module Reporter
class Failure
def self.new(fault)
return TestUnitFailure.new(fault) if fault.kind_of?(Test::Unit::Failure)
- return TestUnitSkipped.new(fault) if Test::Unit.const_defined?(:Omission) && (fault.kind_of?(Test::Unit::Omission) || fault.kind_of?(Test::Unit::Pending))
- return TestUnitNotification.new(fault) if Test::Unit.const_defined?(:Notification) && fault.kind_of?(Test::Unit::Notification)
+ return TestUnitSkipped.new(fault) if Test::Unit.const_defined?(:Omission, false) && (fault.kind_of?(Test::Unit::Omission) || fault.kind_of?(Test::Unit::Pending))
+ return TestUnitNotification.new(fault) if Test::Unit.const_defined?(:Notification, false) && fault.kind_of?(Test::Unit::Notification)
TestUnitError.new(fault)
end
end

0 comments on commit d20a340

Please sign in to comment.
Something went wrong with that request. Please try again.