Use const_defined? for checking a constant is defined #58

Merged
merged 1 commit into from May 11, 2012

2 participants

@kou

#51 uses !(Test::Unit.constants & [:Omission, "Omission"]).empty? for
checking a constant is defined. But it's better that Module#const_defined?
is used for that propose.

@kou kou Use const_defined? for checking a constant is defined
#51 uses !(Test::Unit.constants & [:Omission, "Omission"]).empty? for
checking a constant is defined. But it's better that Module#const_defined?
is used for that propose.
e7a0ff8
@nicksieger nicksieger merged commit 31f0dd7 into ci-reporter:master May 11, 2012
@nicksieger
ci-reporter member

I agree, much better. Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment