Merge pull request #12821 from arunagw/aa-removed-missleading-message

clarified missleading message
rails · Nov 9, 2013 · 951348a · 951348a
2 parents 849400c + 539ce8b
commit 951348a
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/activerecord/lib/active_record/core.rb b/activerecord/lib/active_record/core.rb
@@ -109,7 +109,7 @@ def inspect
         elsif abstract_class?
           "#{super}(abstract)"
         elsif !connected?
-          "#{super}(no database connection)"
+          "#{super} (call '#{super}.connection' to establish a connection)"
         elsif table_exists?
           attr_list = columns.map { |c| "#{c.name}: #{c.type}" } * ', '
           "#{super}(#{attr_list})"

diff --git a/activerecord/test/cases/invalid_connection_test.rb b/activerecord/test/cases/invalid_connection_test.rb
@@ -17,6 +17,6 @@ def teardown
   end
 
   test "inspect on Model class does not raise" do
-    assert_equal "#{Bird.name}(no database connection)", Bird.inspect
+    assert_equal "#{Bird.name} (call '#{Bird.name}.connection' to establish a connection)", Bird.inspect
   end
 end