Merge pull request #282 from cllns/add-note-about-dot-load!

Improve NoAdapterError by mentioning `.load!`
hanami · Jan 19, 2016 · 3e14c23 · 3e14c23
2 parents cfd8df3 + 18e3cbb
commit 3e14c23
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/lib/lotus/model/adapters/null_adapter.rb b/lib/lotus/model/adapters/null_adapter.rb
@@ -6,7 +6,9 @@ module Adapters
       # @since 0.2.0
       class NoAdapterError < Lotus::Model::Error
         def initialize(method_name)
-          super("Cannot invoke `#{ method_name }' on repository. Please check if `adapter' and `mapping' are set.")
+          super("Cannot invoke `#{ method_name }' on repository. "\
+                "Please check if `adapter' and `mapping' are set, "\
+                "and that you call `.load!' on the configuration.")
         end
       end
 

diff --git a/test/integration/configuration_test.rb b/test/integration/configuration_test.rb
@@ -42,7 +42,11 @@
   describe "when a repository isn't mapped" do
     it 'raises an error when try to use it' do
       exception = -> { UnmappedRepository.find(1) }.must_raise(Lotus::Model::Adapters::NoAdapterError)
-      exception.message.must_equal("Cannot invoke `find' on repository. Please check if `adapter' and `mapping' are set.")
+      exception.message.must_equal(
+        "Cannot invoke `find' on repository. "\
+        "Please check if `adapter' and `mapping' are set, "\
+        "and that you call `.load!' on the configuration."
+      )
     end
   end