Merge pull request #172 from vandrijevik/fix-adapter-load-warning-mes…

…sage

Use puts instead of p for multi-line message

lib/foreigner/adapter.rb

@@ -12,8 +12,8 @@ def load!
         if registered.key?(configured_name)
           require registered[configured_name]
         else
-          p "Database adapter #{configured_name} not supported. Use:\n" +
-            "Foreigner::Adapter.register '#{configured_name}', 'path/to/adapter'"
+          puts "Database adapter #{configured_name} not supported. Use:\n" +
+               "Foreigner::Adapter.register '#{configured_name}', 'path/to/adapter'"
         end
       end
 

test/foreigner/adapter_test.rb

@@ -8,4 +8,22 @@ class Foreigner::AdapterTest < ActiveSupport::TestCase
 
     Foreigner::Adapter.load!
   end
-end
+
+  test "load prints warning message for an unsupported adapter on two lines" do
+    Foreigner::Adapter.stubs(:configured_name).returns('unsupported')
+
+    output = StringIO.new
+    with_stdout(output) { Foreigner::Adapter.load! }
+
+    assert_equal 2, output.string.split("\n").length
+  end
+
+  private
+  def with_stdout(stream)
+    oldstdout = $stdout
+    $stdout = stream
+    yield
+  ensure
+    $stdout = oldstdout
+  end
+end