Merge pull request #116 from Watson1978/pathname

Accept the object that has #to_path in Logger::LogDevice.new

Author: hsbt

lib/logger/log_device.rb

@@ -99,6 +99,7 @@ def fixup_mode(dev, filename)
     else
       def fixup_mode(dev, filename)
         return dev if @binmode
+        filename = filename.respond_to?(:to_path) ? filename.to_path : filename
         dev.autoclose = false
         old_dev = dev
         dev = File.new(dev.fileno, mode: MODE, path: filename)

test/logger/test_logdevice.rb

@@ -3,6 +3,7 @@
 require 'logger'
 require 'tempfile'
 require 'tmpdir'
+require 'pathname'
 
 class TestLogDevice < Test::Unit::TestCase
   class LogExcnRaiser
@@ -79,6 +80,20 @@ def test_initialize
       File.unlink(tempfile)
       tempfile.close(true)
     end
+    # logfile object with Pathname object
+    tempfile = Tempfile.new("logger")
+    pathname = Pathname.new(tempfile.path)
+    logdev = d(pathname)
+    begin
+      logdev.write('world')
+      logfile = File.read(pathname)
+      assert_equal(1, logfile.split(/\n/).size)
+      assert_match(/^world$/, logfile)
+      assert_equal(pathname, logdev.filename)
+    ensure
+      logdev.close
+      tempfile.close(true)
+    end
   end
 
   def test_write