Implement Slop#respond_to_missing?. (#1176)

sparklemotion · Nov 6, 2014 · a655360 · a655360
1 parent e7c4756
commit a655360
Show file tree

Hide file tree

Showing 3 changed files with 18 additions and 2 deletions.
diff --git a/CHANGELOG.rdoc b/CHANGELOG.rdoc
@@ -1,5 +1,9 @@
 === 1.6.5 / unreleased
 
+==== Features
+
+* Implement Slop#respond_to_missing?. (#1176)
+
 ==== Bug fixes
 
 * Fix a bug where CFLAGS passed in are dropped.

diff --git a/lib/nokogiri/decorators/slop.rb b/lib/nokogiri/decorators/slop.rb
@@ -30,6 +30,14 @@ def method_missing name, *args, &block
         super if list.empty?
         list.length == 1 ? list.first : list
       end
+
+      def respond_to_missing? name, include_private = false
+        prefix = implied_xpath_context
+
+        list = xpath("#{prefix}#{name.to_s.sub(/^_/, '')}")
+
+        !list.empty?
+      end
     end
   end
 end
diff --git a/test/decorators/test_slop.rb b/test/decorators/test_slop.rb
@@ -9,8 +9,12 @@ def test_description_tag
           <description>this is the foo thing</description>
         </item>
       eoxml
-      assert doc.item.title
-      assert doc.item._description, 'should have description'
+      assert doc.item.respond_to?(:title)
+      assert_equal 'foo', doc.item.title.text
+      assert doc.item.respond_to?(:_description), 'should have description'
+      assert 'this is the foo thing', doc.item._description.text
+      assert !doc.item.respond_to?(:foo)
+      assert_raise(NoMethodError) { doc.item.foo }
     end
   end
 end