Merge pull request #105 from jhollinger/explain-missing-attributes

Specify which attribute is missing on which Entity

Author: dblock

.rubocop_todo.yml

@@ -12,7 +12,7 @@ Metrics/AbcSize:
 # Offense count: 1
 # Configuration parameters: CountComments.
 Metrics/ClassLength:
-  Max: 292
+  Max: 296
 
 # Offense count: 4
 Metrics/CyclomaticComplexity:

CHANGELOG.md

@@ -4,6 +4,7 @@ Next Release
 * Your contribution here.
 * [#98](https://github.com/intridea/grape-entity/pull/98): Add nested conditionals - [@zbelzer](https://github.com/zbelzer).
 * [#91](https://github.com/intridea/grape-entity/pull/91): Fix OpenStruct serializing - [@etehtsea](https://github.com/etehtsea).
+* [#105](https://github.com/intridea/grape-entity/pull/105): Specify which attribute is missing in which Entity - [@jhollinger](https://github.com/jhollinger).
 
 0.4.4 (2014-08-17)
 ==================

lib/grape_entity/entity.rb

@@ -558,7 +558,11 @@ def delegate_attribute(attribute)
         elsif object.respond_to?(:fetch, true)
           object.fetch(name)
         else
-          object.send(name)
+          begin
+            object.send(name)
+          rescue NoMethodError
+            raise NoMethodError, "#{self.class.name} missing attribute `#{name}' on #{object}"
+          end
         end
       end
     end

spec/grape_entity/entity_spec.rb

@@ -462,7 +462,7 @@ class Parent < Person
         subject.expose(:awesome)
         expect do
           subject.represent(Object.new, serializable: true)
-        end.to raise_error(NoMethodError)
+        end.to raise_error(NoMethodError, /missing attribute `awesome'/)
       end
     end