Added more defensive way of checking the @attributes in Item#respond_to?

Closes karmi#944
x0bandeira · Apr 12, 2014 · 1016449 · 1016449
1 parent 58ab10a
commit 1016449
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/lib/tire/results/item.rb b/lib/tire/results/item.rb
@@ -27,7 +27,7 @@ def method_missing(method_name, *arguments)
       end
 
       def respond_to?(method_name, include_private = false)
-        @attributes.has_key?(method_name.to_sym) || super
+        (@attributes || {}).has_key?(method_name.to_sym) || super
       end
 
       def [](key)

diff --git a/test/unit/results_item_test.rb b/test/unit/results_item_test.rb
@@ -82,6 +82,10 @@ class AlmostHash < Hash; end
         assert @document.respond_to?(:title, true)
       end
 
+      should "not raise errors when unserializing" do
+        assert_nothing_raised { YAML.load(YAML.dump(@document)) }
+      end
+
       should "return nil for non-existing keys/methods" do
         assert_nothing_raised { @document.whatever }
         assert_nil @document.whatever