Actions
Defect #14598
closedWrong test with logger.info in model mail_handler
Start date:
Due date:
% Done:
0%
Estimated time:
Resolution:
Fixed
Affected version:
Files
Related issues
Updated by Jérôme BATAILLE over 11 years ago
patch made against Redmine V1.2.1
if logger.info --> if logger.info?
Updated by Toshi MARUYAMA over 11 years ago
"logger.info" is always true.
$ RAILS_ENV=test ruby script/rails console Loading test environment (Rails 3.2.13) 1.9.3p429 :001 > logger = Logger.new(STDOUT) => #<Logger:0x000000055b5b80 ... 1.9.3p429 :002 > logger.level = Logger::ERROR => 3 1.9.3p429 :003 > logger.info => true 1.9.3p429 :004 > logger.info? => false 1.9.3p429 :005 > logger.info "test" => true 1.9.3p429 :006 > logger.level = Logger::INFO => 1 1.9.3p429 :007 > logger.info? => true 1.9.3p429 :008 > logger.info nil => true 1.9.3p429 :009 > logger.info "test" test => true
Updated by Toshi MARUYAMA over 11 years ago
- Status changed from New to Closed
Committed in trunk r12073.
Updated by Jérôme BATAILLE about 11 years ago
Any idea of the version in which the revision will be included ?
Updated by Jérôme BATAILLE about 11 years ago
- Status changed from Closed to Reopened
Updated by Jérôme BATAILLE about 11 years ago
I'm sorry to reopen this issue, but because it was closed without a target version, I think it was fallen in the limbo :-)
Updated by Toshi MARUYAMA about 11 years ago
- Status changed from Reopened to Closed
- Target version set to 2.4.0
- Resolution set to Fixed
It is in 2.4.0.
Updated by Go MAEDA almost 9 years ago
- Related to Patch #9102: Dry up MailHandler logging added
Actions