Project

General

Profile

Actions

Defect #14598

closed

Wrong test with logger.info in model mail_handler

Added by Jérôme BATAILLE over 11 years ago. Updated about 11 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
Email receiving
Target version:
Start date:
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Files

RM_mail_handler_bug_fix.diff (2.95 KB) RM_mail_handler_bug_fix.diff Jérôme BATAILLE, 2013-08-01 15:33

Related issues

Related to Redmine - Patch #9102: Dry up MailHandler loggingClosed2011-08-22

Actions
Actions #1

Updated by Jérôme BATAILLE over 11 years ago

patch made against Redmine V1.2.1

if logger.info --> if logger.info?

Actions #2

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 
Actions #3

Updated by Toshi MARUYAMA over 11 years ago

  • Status changed from New to Closed

Committed in trunk r12073.

Actions #4

Updated by Jérôme BATAILLE about 11 years ago

Any idea of the version in which the revision will be included ?

Actions #5

Updated by Jérôme BATAILLE about 11 years ago

  • Status changed from Closed to Reopened
Actions #6

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 :-)

Actions #7

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.

Actions #8

Updated by Jérôme BATAILLE about 11 years ago

Thanks a lot !

Actions #9

Updated by Go MAEDA almost 9 years ago

  • Related to Patch #9102: Dry up MailHandler logging added
Actions

Also available in: Atom PDF