Vote #74207
完了Wrong test with logger.info in model mail_handler
0%
journals
patch made against Redmine V1.2.1
if logger.info --> if logger.info?
--------------------------------------------------------------------------------
"logger.info" is always true.
<pre>
$ 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
</pre>
--------------------------------------------------------------------------------
Committed in trunk r12073.
--------------------------------------------------------------------------------
Any idea of the version in which the revision will be included ?
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
I'm sorry to reopen this issue, but because it was closed without a target version, I think it was fallen in the limbo :-)
--------------------------------------------------------------------------------
It is in version:2.4.0.
--------------------------------------------------------------------------------
Thanks a lot !
--------------------------------------------------------------------------------
--------------------------------------------------------------------------------
related_issues
relates,Closed,9102,Dry up MailHandler logging