Adds spec for custom logger

lib/savon/options.rb

@@ -137,17 +137,15 @@ def raise_errors(raise_errors)
     def log(log)
       if log
         HTTPI.log = true
-        target = $stdout
       else
         HTTPI.log = false
         windows = RUBY_PLATFORM =~ /(mingw|bccwin|wince|mswin32)/i
         target = windows ? "NUL:" : "/dev/null"
+        @options[:logger] = Logger.new(target)
       end
-
-      @options[:logger] = Logger.new(target)
     end
 
-    # The logger to use. Defaults to a Savon::Logger instance.
+    # The logger to use. Defaults to a Logger instance.
     def logger(logger)
       @options[:logger] = logger
     end

spec/savon/options_spec.rb

@@ -1,5 +1,6 @@
 require "spec_helper"
 require "integration/support/server"
+require "logger"
 
 describe "Options" do
 
@@ -213,6 +214,14 @@
       logger = new_client.globals[:logger]
       expect(logger).to be_a(Logger)
     end
+
+    it "allows a custom logger to be set" do
+      custom_logger = Logger.new($stdout)
+      c = new_client(logger: custom_logger, log: true)
+      logger = c.globals[:logger]
+
+      expect(logger).to be_equal(custom_logger)
+    end
   end
 
   context "global :log_level" do