nokogiri 1.3.1-x86-mingw32 → 1.3.2-x86-mingw32

data/tasks/test.rb

@@ -1,64 +1,3 @@
-# partial-loads-ok and undef-value-errors necessary to ignore
-# spurious (and eminently ignorable) warnings from the ruby
-# interpreter
-VALGRIND_BASIC_OPTS = "--num-callers=50 --error-limit=no --partial-loads-ok=yes --undef-value-errors=no"
-
-class NokogiriTestTask < Rake::TestTask
-  def initialize *args
-    super
-    %w[ ext lib bin test ].each do |dir|
-      self.libs << dir
-    end
-    self.test_files = FileList['test/**/test_*.rb'] +
-      FileList['test/**/*_test.rb']
-    self.verbose = "verbose"
-    self.warning = true
-  end
-end
-
-desc "run test suite under valgrind with basic ruby options"
-NokogiriTestTask.new('test:valgrind').extend(Module.new {
-  def ruby *args
-    run_with_env "valgrind #{VALGRIND_BASIC_OPTS} #{RUBY} #{args.join(' ')} test/test_nokogiri.rb --verbose=verbose"
-  end
-})
-
-desc "run test suite under valgrind with memory-fill ruby options"
-NokogiriTestTask.new('test:valgrind_mem').extend(Module.new {
-  def ruby *args
-    run_with_env "valgrind #{VALGRIND_BASIC_OPTS} --freelist-vol=100000000 --malloc-fill=6D --free-fill=66 #{RUBY} #{args.join(' ')} test/test_nokogiri.rb --verbose=verbose"
-  end
-})
-
-desc "run test suite under valgrind with memory-zero ruby options"
-NokogiriTestTask.new('test:valgrind_mem0').extend(Module.new {
-  def ruby *args
-    run_with_env "valgrind #{VALGRIND_BASIC_OPTS} --freelist-vol=100000000 --malloc-fill=00 --free-fill=00 #{RUBY} #{args.join(' ')} test/test_nokogiri.rb --verbose=verbose"
-  end
-})
-
-desc "run test suite under gdb"
-NokogiriTestTask.new('test:gdb').extend(Module.new {
-  def ruby *args
-    run_with_env "gdb --args #{RUBY} #{args.join(' ')}"
-  end
-})
-
-desc "test coverage"
-NokogiriTestTask.new('test:coverage').extend(Module.new {
-  def ruby *args
-    rm_rf "coverage"
-    run_with_env "rcov -x Library -I lib:ext:test #{args.join(' ')}"
-  end
-})
-
-desc "run test suite with verbose output"
-NokogiriTestTask.new('test:verbose').extend(Module.new {
-  def ruby *args
-    run_with_env "#{RUBY} #{args.join(' ')} test/test_nokogiri.rb --verbose=verbose"
-  end
-})
-
 def run_with_env(cmd)
   cmd = "LD_LIBRARY_PATH='#{ENV['LD_LIBRARY_PATH']}' #{cmd}"
   puts "=> #{cmd}"
@@ -142,7 +81,7 @@ namespace :test do
     directories = ENV['MULTIXML2_DIR'] ? [ENV['MULTIXML2_DIR']] : Dir[File.join(MULTI_XML, 'install', '*')]
     directories.sort.reverse.each do |xml2_version|
       next unless xml2_version =~ /libxml2/
-      extopts = "--with-xml2-include=#{xml2_version}/include/libxml2 --with-xml2-lib=#{xml2_version}/lib --with-xslt-dir=#{libxslt}"
+      extopts = "--with-xml2-include=#{xml2_version}/include/libxml2 --with-xml2-lib=#{xml2_version}/lib --with-xslt-dir=#{libxslt} --with-iconv-dir=/usr"
       cmd = "#{$0} clean test EXTOPTS='#{extopts}' LD_LIBRARY_PATH='#{xml2_version}/lib'"
 
       version = File.basename(xml2_version)

data/test/files/bar/bar.xsd

@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<xsd:schema xmlns:xsd="http://www.w3.org/2001/XMLSchema">
+    <xsd:element name="bar"/>
+</xsd:schema>

data/test/files/foo/foo.xsd

@@ -0,0 +1,4 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<xsd:schema xmlns:xsd="http://www.w3.org/2001/XMLSchema">
+    <xsd:include schemaLocation="../bar/bar.xsd"/>
+</xsd:schema>

data/test/files/snuggles.xml

@@ -0,0 +1,3 @@
+<x xmlns:tenderlove='http://tenderlovemaking.com/'>
+  <tenderlove:foo awesome='true'>snuggles!</tenderlove:foo>
+</x>

data/test/files/valid_bar.xml

@@ -0,0 +1,2 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<bar />

data/test/helper.rb

@@ -23,6 +23,7 @@ module Nokogiri
     PO_SCHEMA_FILE  = File.join(ASSETS_DIR, 'po.xsd')
     ADDRESS_SCHEMA_FILE = File.join(ASSETS_DIR, 'address_book.rlx')
     ADDRESS_XML_FILE = File.join(ASSETS_DIR, 'address_book.xml')
+    SNUGGLES_FILE   = File.join(ASSETS_DIR, 'snuggles.xml')
 
     unless RUBY_VERSION >= '1.9'
       undef :default_test
@@ -55,10 +56,10 @@ module Nokogiri
   module SAX
     class TestCase < Nokogiri::TestCase
       class Doc < XML::SAX::Document
-        attr_reader :start_elements, :start_elements_ns, :start_document_called
-        attr_reader :end_elements, :end_elements_ns, :end_document_called
-        attr_reader :data, :comments, :cdata_blocks
-        attr_reader :errors, :warnings
+        attr_reader :start_elements, :start_document_called
+        attr_reader :end_elements, :end_document_called
+        attr_reader :data, :comments, :cdata_blocks, :start_elements_namespace
+        attr_reader :errors, :warnings, :end_elements_namespace
 
         def start_document
           @start_document_called = true
@@ -85,8 +86,8 @@ module Nokogiri
           super
         end
 
-        def start_element_ns *args
-          (@start_elements_ns ||= []) << args
+        def start_element_namespace *args
+          (@start_elements_namespace ||= []) << args
           super
         end
 
@@ -95,8 +96,8 @@ module Nokogiri
           super
         end
 
-        def end_element_ns *args
-          (@end_elements_ns ||= []) << args
+        def end_element_namespace *args
+          (@end_elements_namespace ||= []) << args
           super
         end
 

data/test/html/test_document_fragment.rb

@@ -92,6 +92,20 @@ module Nokogiri
         assert_equal "<span>foo<br/></span><span>bar</span>", fragment.to_xml
       end
 
+      def test_fragment_script_tag_with_cdata
+        doc = HTML::Document.new
+        fragment = doc.fragment("<script>var foo = 'bar';</script>")
+        assert_equal("<script>var foo = 'bar';</script>",
+          fragment.to_s)
+      end
+
+      def test_fragment_with_comment
+        doc = HTML::Document.new
+        fragment = doc.fragment("<p>hello<!-- your ad here --></p>")
+        assert_equal("<p>hello<!-- your ad here --></p>",
+          fragment.to_s)
+      end
+
     end
   end
 end

data/test/test_reader.rb

@@ -3,11 +3,7 @@ require File.expand_path(File.join(File.dirname(__FILE__), "helper"))
 
 class TestReader < Nokogiri::TestCase
   def test_from_io_sets_io_as_source
-    io = StringIO.new(<<-eoxml)
-    <x xmlns:tenderlove='http://tenderlovemaking.com/'>
-      <tenderlove:foo awesome='true'>snuggles!</tenderlove:foo>
-    </x>
-    eoxml
+    io = File.open SNUGGLES_FILE
     reader = Nokogiri::XML::Reader.from_io(io)
     assert_equal io, reader.source
   end
@@ -33,11 +29,7 @@ class TestReader < Nokogiri::TestCase
   end
 
   def test_from_io
-    io = StringIO.new(<<-eoxml)
-    <x xmlns:tenderlove='http://tenderlovemaking.com/'>
-      <tenderlove:foo awesome='true'>snuggles!</tenderlove:foo>
-    </x>
-    eoxml
+    io = File.open SNUGGLES_FILE
     reader = Nokogiri::XML::Reader.from_io(io)
     assert_equal false, reader.default?
     assert_equal [false, false, false, false, false, false, false],
@@ -45,6 +37,14 @@ class TestReader < Nokogiri::TestCase
   end
 
   def test_io
+    io = File.open SNUGGLES_FILE
+    reader = Nokogiri::XML::Reader(io)
+    assert_equal false, reader.default?
+    assert_equal [false, false, false, false, false, false, false],
+      reader.map { |x| x.default? }
+  end
+
+  def test_string_io
     io = StringIO.new(<<-eoxml)
     <x xmlns:tenderlove='http://tenderlovemaking.com/'>
       <tenderlove:foo awesome='true'>snuggles!</tenderlove:foo>

data/test/xml/sax/test_parser.rb

@@ -9,12 +9,89 @@ module Nokogiri
           @parser = XML::SAX::Parser.new(Doc.new)
         end
 
+        def test_namespace_declaration_order_is_saved
+          @parser.parse <<-eoxml
+<root xmlns:foo='http://foo.example.com/' xmlns='http://example.com/'>
+  <a foo:bar='hello' />
+</root>
+          eoxml
+          assert_equal 2, @parser.document.start_elements_namespace.length
+          el = @parser.document.start_elements_namespace.first
+          namespaces = el.last
+          assert_equal ['foo', 'http://foo.example.com/'], namespaces.first
+          assert_equal [nil, 'http://example.com/'], namespaces.last
+        end
+
         def test_bad_document_calls_error_handler
           @parser.parse('<foo><bar></foo>')
           assert @parser.document.errors
           assert @parser.document.errors.length > 0
         end
 
+        def test_namespace_are_super_fun_to_parse
+          @parser.parse <<-eoxml
+<root xmlns:foo='http://foo.example.com/'>
+  <a foo:bar='hello' />
+  <b xmlns:foo='http://bar.example.com/'>
+    <a foo:bar='hello' />
+  </b>
+  <foo:bar>hello world</foo:bar>
+</root>
+          eoxml
+          assert @parser.document.start_elements_namespace.length > 0
+          el = @parser.document.start_elements_namespace[1]
+          assert_equal 'a', el.first
+          assert_equal 1, el[1].length
+
+          attribute = el[1].first
+          assert_equal 'bar', attribute.localname
+          assert_equal 'foo', attribute.prefix
+          assert_equal 'hello', attribute.value
+          assert_equal 'http://foo.example.com/', attribute.uri
+        end
+
+        def test_sax_v1_namespace_attribute_declarations
+          @parser.parse <<-eoxml
+<root xmlns:foo='http://foo.example.com/' xmlns='http://example.com/'>
+  <a foo:bar='hello' />
+  <b xmlns:foo='http://bar.example.com/'>
+    <a foo:bar='hello' />
+  </b>
+  <foo:bar>hello world</foo:bar>
+</root>
+          eoxml
+          assert @parser.document.start_elements.length > 0
+          elm = @parser.document.start_elements.first
+          assert_equal 'root', elm.first
+          assert elm[1].include?(['xmlns:foo', 'http://foo.example.com/'])
+          assert elm[1].include?(['xmlns', 'http://example.com/'])
+        end
+
+        def test_sax_v1_namespace_nodes
+          @parser.parse <<-eoxml
+<root xmlns:foo='http://foo.example.com/' xmlns='http://example.com/'>
+  <a foo:bar='hello' />
+  <b xmlns:foo='http://bar.example.com/'>
+    <a foo:bar='hello' />
+  </b>
+  <foo:bar>hello world</foo:bar>
+</root>
+          eoxml
+          assert_equal 5, @parser.document.start_elements.length
+          assert @parser.document.start_elements.map { |se|
+            se.first
+          }.include?('foo:bar')
+          assert @parser.document.end_elements.map { |se|
+            se.first
+          }.include?('foo:bar')
+        end
+
+        def test_start_is_called_without_namespace
+          @parser.parse('<foo:f><bar></foo:f>')
+          assert_equal ['foo:f', 'bar'],
+            @parser.document.start_elements.map { |x| x.first }
+        end
+
         def test_parser_sets_encoding
           parser = XML::SAX::Parser.new(Doc.new, 'UTF-8')
           assert_equal 'UTF-8', parser.encoding

data/test/xml/sax/test_push_parser.rb

@@ -43,12 +43,16 @@ module Nokogiri
             <stream:stream xmlns='jabber:client' xmlns:stream='http://etherx.jabber.org/streams' version='1.0' size='large'></stream:stream>
           eoxml
 
-          assert_equal [[ 'stream',
-                          {'version' => '1.0', 'size' => 'large'},
-                          'stream',
-                          'http://etherx.jabber.org/streams',
-                          {nil => 'jabber:client', 'stream' => 'http://etherx.jabber.org/streams'}]],
-            @parser.document.start_elements_ns
+          assert_equal 1, @parser.document.start_elements_namespace.length
+          el = @parser.document.start_elements_namespace.first
+
+          assert_equal 'stream', el.first
+          assert_equal 2, el[1].length
+          assert_equal [['version', '1.0'], ['size', 'large']],
+            el[1].map { |x| [x.localname, x.value] }
+
+          assert_equal 'stream', el[2]
+          assert_equal 'http://etherx.jabber.org/streams', el[3]
           @parser.finish
         end
 
@@ -58,7 +62,7 @@ module Nokogiri
           eoxml
 
           assert_equal [['stream', 'stream', 'http://etherx.jabber.org/streams']],
-            @parser.document.end_elements_ns
+            @parser.document.end_elements_namespace
           @parser.finish
         end
 

data/test/xml/test_document.rb

@@ -10,6 +10,31 @@ module Nokogiri
         @xml = Nokogiri::XML.parse(File.read(XML_FILE), XML_FILE)
       end
 
+      def test_validate
+        assert_equal 44, @xml.validate.length
+      end
+
+      def test_validate_no_internal_subset
+        doc = Nokogiri::XML('<test/>')
+        assert_nil doc.validate
+      end
+
+      def test_clone
+        assert @xml.clone
+      end
+
+      def test_document_should_not_have_default_ns
+        doc = Nokogiri::XML::Document.new
+
+        assert_raises NoMethodError do
+          doc.default_namespace = 'http://innernet.com/'
+        end
+
+        assert_raises NoMethodError do
+          doc.add_namespace_definition('foo', 'bar')
+        end
+      end
+
       def test_parse_takes_block
         options = nil
         Nokogiri::XML.parse(File.read(XML_FILE), XML_FILE) do |cfg|

data/test/xml/test_dtd.rb

@@ -1,7 +1,7 @@
 require File.expand_path(File.join(File.dirname(__FILE__), '..', "helper"))
 
 module Nokogiri
-  module HTML
+  module XML
     class TestDTD < Nokogiri::TestCase
       def setup
         super
@@ -9,6 +9,11 @@ module Nokogiri
         assert @dtd = @xml.internal_subset
       end
 
+      def test_validate
+        list = @xml.internal_subset.validate @xml
+        assert_equal 44, list.length
+      end
+
       def test_external_subsets
         assert subset = @xml.internal_subset
         assert_equal 'staff', subset.name

data/test/xml/test_node.rb

@@ -157,6 +157,13 @@ module Nokogiri
         assert_equal 'http://tenderlovemaking.com', node.namespaces['xmlns:foo']
       end
 
+      def test_add_namespace_twice
+        node = @xml.at('address')
+        ns = node.add_namespace('foo', 'http://tenderlovemaking.com')
+        ns2 = node.add_namespace('foo', 'http://tenderlovemaking.com')
+        assert_equal ns, ns2
+      end
+
       def test_add_default_ns
         node = @xml.at('address')
         node.add_namespace(nil, 'http://tenderlovemaking.com')

data/test/xml/test_node_set.rb

@@ -8,6 +8,25 @@ module Nokogiri
         @xml = Nokogiri::XML(File.read(XML_FILE), XML_FILE)
       end
 
+      def test_search_empty_node_set
+        set = Nokogiri::XML::NodeSet.new(Nokogiri::XML::Document.new)
+        assert_equal 0, set.css('foo').length
+        assert_equal 0, set.xpath('.//foo').length
+        assert_equal 0, set.search('foo').length
+      end
+
+      def test_css_searches_match_self
+        html = Nokogiri::HTML("<html><body><div class='a'></div></body></html>")
+        set = html.xpath("/html/body/div")
+        assert_equal set.first, set.css(".a").first
+      end
+
+      def test_search_with_css_matches_self
+        html = Nokogiri::HTML("<html><body><div class='a'></div></body></html>")
+        set = html.xpath("/html/body/div")
+        assert_equal set.first, set.search(".a").first
+      end
+
       def test_double_equal
         assert node_set_one = @xml.xpath('//employee')
         assert node_set_two = @xml.xpath('//employee')

data/test/xml/test_schema.rb

@@ -7,6 +7,30 @@ module Nokogiri
         assert @xsd = Nokogiri::XML::Schema(File.read(PO_SCHEMA_FILE))
       end
 
+      def test_schema_from_document
+        doc = Nokogiri::XML(File.open(PO_SCHEMA_FILE))
+        assert doc
+        xsd = Nokogiri::XML::Schema.from_document doc
+        assert_instance_of Nokogiri::XML::Schema, xsd
+      end
+
+      def test_schema_from_document_node
+        doc = Nokogiri::XML(File.open(PO_SCHEMA_FILE))
+        assert doc
+        xsd = Nokogiri::XML::Schema.from_document doc.root
+        assert_instance_of Nokogiri::XML::Schema, xsd
+      end
+
+      def test_schema_validates_with_relative_paths
+        xsd = File.join(ASSETS_DIR, 'foo', 'foo.xsd')
+        xml = File.join(ASSETS_DIR, 'valid_bar.xml')
+        doc = Nokogiri::XML(File.open(xsd))
+        xsd = Nokogiri::XML::Schema.from_document doc
+
+        doc = Nokogiri::XML(File.open(xml))
+        assert xsd.valid?(doc)
+      end
+
       def test_parse_with_memory
         assert_instance_of Nokogiri::XML::Schema, @xsd
         assert_equal 0, @xsd.errors.length