Merge pull request ci-reporter#58 from kou/test-unit-use-const-defined

Use const_defined? for checking a constant is defined
spraints · May 11, 2012 · 31f0dd7 · 31f0dd7
2 parents eac828b + e7a0ff8
commit 31f0dd7
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/lib/ci/reporter/test_unit.rb b/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.constants & [:Omission, "Omission"]).empty? && (fault.kind_of?(Test::Unit::Omission) || fault.kind_of?(Test::Unit::Pending))
-        return TestUnitNotification.new(fault) if !(Test::Unit.constants & [:Notification, "Notification"]).empty? && fault.kind_of?(Test::Unit::Notification)
+        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)
         TestUnitError.new(fault)
       end
     end