fix: CDATA.new parameter type checks

(cherry picked from commit 547c27c)

ext/java/nokogiri/XmlCdata.java

@@ -46,6 +46,10 @@ public class XmlCdata extends XmlText
     IRubyObject rbDocument = args[0];
     content = args[1];
 
+    if (!(rbDocument instanceof XmlNode)) {
+      String msg = "expected first parameter to be a Nokogiri::XML::Document, received " + rbDocument.getMetaClass();
+      throw context.runtime.newTypeError(msg);
+    }
     if (!(rbDocument instanceof XmlDocument)) {
       // TODO: deprecate allowing Node
       context.runtime.getWarnings().warn("Passing a Node as the first parameter to CDATA.new is deprecated. Please pass a Document instead. This will become an error in a future release of Nokogiri.");

ext/nokogiri/xml_cdata.c

@@ -25,6 +25,12 @@ rb_xml_cdata_s_new(int argc, VALUE *argv, VALUE klass)
 
   rb_scan_args(argc, argv, "2*", &rb_document, &rb_content, &rb_rest);
 
+  if (!rb_obj_is_kind_of(rb_document, cNokogiriXmlNode)) {
+    rb_raise(rb_eTypeError,
+             "expected first parameter to be a Nokogiri::XML::Document, received %"PRIsVALUE,
+             rb_obj_class(rb_document));
+  }
+
   if (!rb_obj_is_kind_of(rb_document, cNokogiriXmlDocument)) {
     xmlNodePtr deprecated_node_type_arg;
     // TODO: deprecate allowing Node

test/xml/test_cdata.rb

@@ -40,6 +40,13 @@
       assert_raises(TypeError) { Nokogiri::XML::CDATA.new(doc, 1.234) }
       assert_raises(TypeError) { Nokogiri::XML::CDATA.new(doc, {}) }
     end
+
+    it "does not accept anything other than Node or Document" do
+      assert_raises(TypeError) { Nokogiri::XML::CDATA.new(1234, "hello world") }
+      assert_raises(TypeError) { Nokogiri::XML::CDATA.new("asdf", "hello world") }.inspect
+      assert_raises(TypeError) { Nokogiri::XML::CDATA.new({}, "hello world") }
+      assert_raises(TypeError) { Nokogiri::XML::CDATA.new(nil, "hello world") }
+    end
   end
 
   it "supports #content and #content=" do