moxml 0.1.6 → 0.1.8

data/spec/{support/shared_examples/examples/attribute.rb → examples/attribute_examples_spec.rb}

@@ -27,7 +27,7 @@ RSpec.shared_examples "Attribute Examples" do
       element["special"] = '< > & " \''
 
       expect(element.to_xml).to include(
-        'special="&lt; &gt; &amp; &quot; \''
+        'special="&lt; &gt; &amp; &quot; \'',
       )
     end
 

data/spec/{support/shared_examples/examples/basic_usage.rb → examples/basic_usage_spec.rb}

@@ -20,7 +20,7 @@ RSpec.shared_examples "Basic Usage Examples" do
 
       expect(doc.to_xml).to include(
         '<?xml version="1.0" encoding="UTF-8"?>',
-        "<book></book>"
+        "<book></book>",
       )
     end
 
@@ -33,7 +33,7 @@ RSpec.shared_examples "Basic Usage Examples" do
 
       expect(doc.to_xml).to include(
         '<?xml-stylesheet type="text/xsl" href="style.xsl"?>',
-        "<root></root>"
+        "<root></root>",
       )
     end
   end

data/spec/{support/shared_examples/examples/namespace.rb → examples/namespace_examples_spec.rb}

@@ -25,7 +25,7 @@ RSpec.shared_examples "Namespace Examples" do
 
       expect(doc.to_xml).to include(
         'xmlns:dc="http://purl.org/dc/elements/1.1/"',
-        "<dc:title>Test</dc:title>"
+        "<dc:title>Test</dc:title>",
       )
     end
 
@@ -42,7 +42,7 @@ RSpec.shared_examples "Namespace Examples" do
 
       expect(doc.to_xml).to include(
         "<ns:child>",
-        "<ns:grandchild>"
+        "<ns:grandchild>",
       )
     end
 
@@ -58,7 +58,7 @@ RSpec.shared_examples "Namespace Examples" do
 
       expect(doc.to_xml).to include(
         'xmlns:ns="http://example.org/1"',
-        'xmlns:ns="http://example.org/2"'
+        'xmlns:ns="http://example.org/2"',
       )
     end
   end

data/spec/{support/shared_examples/examples/readme_examples.rb → examples/readme_examples_spec.rb}

@@ -22,7 +22,7 @@ RSpec.shared_examples "README Examples" do
         '<?xml version="1.0" encoding="UTF-8"?>',
         '<book xmlns:dc="http://purl.org/dc/elements/1.1/">',
         "<dc:title>XML Processing with Ruby</dc:title>",
-        "</book>"
+        "</book>",
       )
     end
   end
@@ -57,7 +57,7 @@ RSpec.shared_examples "README Examples" do
         "<!--Publication details-->",
         '<published year="2024"></published>',
         "<![CDATA[<custom>metadata</custom>]]>",
-        "</book>"
+        "</book>",
       )
     end
   end
@@ -102,7 +102,7 @@ RSpec.shared_examples "README Examples" do
         "<dc:title>Ruby</dc:title>",
         "<![CDATA[About Ruby...]]>",
         "<dc:title>XML</dc:title>",
-        "<![CDATA[About XML...]]>"
+        "<![CDATA[About XML...]]>",
       )
     end
   end
@@ -110,7 +110,9 @@ RSpec.shared_examples "README Examples" do
   describe "Error handling example" do
     it "handles errors as shown in README" do
       context = Moxml.new
-      pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+      if context.config.adapter_name == :ox
+        pending "Ox doesn't have a native XPath"
+      end
 
       expect do
         context.parse("<invalid>")

data/spec/{support/shared_examples/examples/xpath.rb → examples/xpath_examples_spec.rb}

@@ -18,15 +18,15 @@ RSpec.shared_examples "XPath Examples" do
     end
 
     it "finds nodes by XPath" do
-      pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
-
       books = doc.xpath("//book")
       expect(books.size).to eq(2)
       expect(books.map { _1["id"] }).to eq(%w[1 2])
     end
 
     it "finds nodes with namespaces" do
-      pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+      if context.config.adapter_name == :ox
+        pending "Ox doesn't have a native XPath"
+      end
 
       titles = doc.xpath("//dc:title",
                          "dc" => "http://purl.org/dc/elements/1.1/")
@@ -34,7 +34,9 @@ RSpec.shared_examples "XPath Examples" do
     end
 
     it "finds nodes by attributes" do
-      pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+      if %i[ox headed_ox].include?(context.config.adapter_name)
+        pending "Ox doesn't have native XPath, HeadedOx parser doesn't support .// syntax yet"
+      end
 
       book = doc.at_xpath('//book[@id="2"]')
       expect(book).not_to be_nil
@@ -44,7 +46,9 @@ RSpec.shared_examples "XPath Examples" do
     end
 
     it "finds nested attributes efficiently" do
-      pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+      if %i[ox headed_ox].include?(context.config.adapter_name)
+        pending "Ox doesn't have native XPath, HeadedOx parser doesn't support .// syntax yet"
+      end
       # More efficient - specific path
       titles1 = doc.xpath("//book/dc:title")
 
@@ -59,7 +63,7 @@ RSpec.shared_examples "XPath Examples" do
         # Each book is a mapped Moxml::Element
         book.at_xpath(".//dc:title", "dc" => "http://purl.org/dc/elements/1.1/").native
       end
-      titles4 = ::Moxml::NodeSet.new(nodes, doc.context)
+      titles4 = Moxml::NodeSet.new(nodes, doc.context)
 
       expect(titles1.count).to eq(2)
       expect(titles1).to eq(titles2)

data/spec/integration/README.md

@@ -0,0 +1,71 @@
+# Integration Tests
+
+## Purpose
+
+This directory contains integration tests that verify Moxml wrapper classes work correctly with all adapters. These tests use shared examples to ensure consistent behavior across different adapter implementations.
+
+## What Should Be Placed Here
+
+- ✅ Tests for wrapper classes working with all adapters
+- ✅ Tests for complex workflows involving multiple classes
+- ✅ Tests for edge cases that involve adapter interaction
+- ✅ Shared examples that run for each adapter
+- ✅ Cross-adapter behavioral consistency tests
+
+## What Should NOT Be Placed Here
+
+- ❌ Pure unit tests (use unit/ instead)
+- ❌ Adapter-specific implementation tests (use adapter/ instead)
+- ❌ Performance benchmarks (use performance/ instead)
+- ❌ Documentation examples (use examples/ instead)
+
+## How to Run
+
+```bash
+# Run all integration tests
+bundle exec rake spec:integration
+
+# Run specific integration test
+bundle exec rspec spec/integration/all_adapters_spec.rb
+```
+
+## Directory Structure
+
+```
+integration/
+├── shared_examples/
+│   ├── node_wrappers/           # Node wrapper behavior tests
+│   │   ├── attribute_behavior.rb
+│   │   ├── cdata_behavior.rb
+│   │   ├── comment_behavior.rb
+│   │   ├── declaration_behavior.rb
+│   │   ├── doctype_behavior.rb
+│   │   ├── document_behavior.rb
+│   │   ├── element_behavior.rb
+│   │   ├── namespace_behavior.rb
+│   │   ├── node_behavior.rb
+│   │   ├── node_set_behavior.rb
+│   │   ├── processing_instruction_behavior.rb
+│   │   └── text_behavior.rb
+│   ├── high_level/              # High-level pattern tests
+│   │   ├── builder_behavior.rb
+│   │   ├── context_behavior.rb
+│   │   └── document_builder_behavior.rb
+│   ├── edge_cases.rb            # Edge case scenarios
+│   └── integration_workflows.rb # Complete workflows
+└── all_adapters_spec.rb         # Runs all shared examples for all adapters
+```
+
+## Writing Integration Tests
+
+Integration tests should be written as shared examples that can run with any adapter:
+
+```ruby
+RSpec.shared_examples "element wrapper behavior" do
+  it "creates elements" do
+    # Test that works with any adapter
+  end
+end
+```
+
+The `all_adapters_spec.rb` file runs these shared examples for each adapter.

data/spec/{moxml/all_with_adapters_spec.rb → integration/all_adapters_spec.rb}

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
-RSpec.describe "Test all shared examples" do
+RSpec.describe "Cross-adapter integration" do
+  # Integration shared examples - use the names as defined in the files
   all_shared_examples = [
     "Moxml::Node",
     "Moxml::Namespace",
@@ -26,7 +27,7 @@ RSpec.describe "Test all shared examples" do
     "XPath Examples",
     "Memory Usage Examples",
     "Thread Safety Examples",
-    "Performance Examples"
+    "Performance Examples",
   ]
 
   Moxml::Adapter::AVALIABLE_ADAPTERS.each do |adapter_name|

data/spec/integration/headed_ox_integration_spec.rb

@@ -0,0 +1,326 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe "HeadedOx Integration" do
+  let(:xml) do
+    <<~XML
+      <library>
+        <book id="1" price="15.99">
+          <title>Ruby Programming</title>
+          <author>Alice</author>
+          <year>2020</year>
+        </book>
+        <book id="2" price="25.99">
+          <title>Python Programming</title>
+          <author>Bob</author>
+          <year>2021</year>
+        </book>
+        <book id="3" price="12.99">
+          <title>JavaScript Programming</title>
+          <author>Charlie</author>
+          <year>2022</year>
+        </book>
+      </library>
+    XML
+  end
+
+  let(:context) { Moxml.new(:headed_ox) }
+  let(:doc) { context.parse(xml) }
+
+  describe "parsing and querying XML end-to-end" do
+    it "parses XML document" do
+      expect(doc).to be_a(Moxml::Document)
+      expect(doc.root.name).to eq("library")
+    end
+
+    it "executes simple XPath queries" do
+      books = doc.xpath("//book")
+
+      expect(books.size).to eq(3)
+      expect(books).to all(be_a(Moxml::Element))
+    end
+
+    it "executes XPath with numeric predicates" do
+      cheap_books = doc.xpath("//book[@price < 20]")
+
+      expect(cheap_books.size).to eq(2)
+      expect(cheap_books.map { |b| b["id"] }).to contain_exactly("1", "3")
+    end
+
+    it "executes XPath with function calls" do
+      count = doc.xpath("count(//book)")
+
+      expect(count).to eq(3.0)
+    end
+
+    it "executes complex XPath expressions" do
+      titles = doc.xpath("//book[@price < 20]/title")
+
+      expect(titles.size).to eq(2)
+      expect(titles.map(&:text)).to contain_exactly(
+        "Ruby Programming",
+        "JavaScript Programming",
+      )
+    end
+  end
+
+  describe "XPath function categories" do
+    context "string functions" do
+      it "retrieves text content with string()" do
+        title = doc.xpath("string(//book[1]/title)")
+
+        expect(title).to eq("Ruby Programming")
+      end
+
+      it "searches with contains()" do
+        ruby_books = doc.xpath("//book[contains(title, 'Ruby')]")
+
+        expect(ruby_books.size).to eq(1)
+        expect(ruby_books.first["id"]).to eq("1")
+      end
+
+      it "filters with starts-with()" do
+        prog_books = doc.xpath("//book[starts-with(title, 'Python')]")
+
+        expect(prog_books.size).to eq(1)
+        expect(prog_books.first["id"]).to eq("2")
+      end
+
+      it "concatenates strings with concat()" do
+        result = doc.xpath("concat('Book: ', //book[1]/title)")
+
+        expect(result).to eq("Book: Ruby Programming")
+      end
+
+      it "normalizes whitespace with normalize-space()" do
+        result = doc.xpath("normalize-space('  hello  world  ')")
+
+        expect(result).to eq("hello world")
+      end
+    end
+
+    context "numeric functions" do
+      it "converts to number with number()" do
+        price = doc.xpath("number(//book[1]/@price)")
+
+        expect(price).to eq(15.99)
+      end
+
+      it "sums values with sum()" do
+        total = doc.xpath("sum(//book/@price)")
+
+        expect(total).to be_within(0.01).of(54.97)
+      end
+
+      it "counts nodes with count()" do
+        count = doc.xpath("count(//book)")
+
+        expect(count).to eq(3.0)
+      end
+
+      it "applies floor() function" do
+        result = doc.xpath("floor(15.99)")
+
+        expect(result).to eq(15.0)
+      end
+
+      it "applies ceiling() function" do
+        result = doc.xpath("ceiling(12.01)")
+
+        expect(result).to eq(13.0)
+      end
+
+      it "applies round() function" do
+        result = doc.xpath("round(15.5)")
+
+        expect(result).to eq(16.0)
+      end
+    end
+
+    context "boolean functions" do
+      it "converts to boolean with boolean()" do
+        has_books = doc.xpath("boolean(//book)")
+
+        expect(has_books).to be true
+      end
+
+      it "negates with not()" do
+        result = doc.xpath("not(false())")
+
+        expect(result).to be true
+      end
+
+      it "returns true with true()" do
+        result = doc.xpath("true()")
+
+        expect(result).to be true
+      end
+
+      it "returns false with false()" do
+        result = doc.xpath("false()")
+
+        expect(result).to be false
+      end
+    end
+
+    context "node functions" do
+      it "gets node name with name()" do
+        name = doc.xpath("name(//book[1])")
+
+        expect(name).to eq("book")
+      end
+
+      it "gets local name with local-name()" do
+        name = doc.xpath("local-name(//book[1])")
+
+        expect(name).to eq("book")
+      end
+    end
+
+    context "position functions" do
+      it "filters by position with position()" do
+        second_book = doc.xpath("//book[position() = 2]")
+
+        expect(second_book.size).to eq(1)
+        expect(second_book.first["id"]).to eq("2")
+      end
+
+      it "selects last node with last()" do
+        skip "HeadedOx limitation: last() in predicate context needs temporary nodeset. See docs/HEADED_OX_LIMITATIONS.md"
+        last_book = doc.xpath("//book[position() = last()]")
+
+        expect(last_book.size).to eq(1)
+        expect(last_book.first["id"]).to eq("3")
+      end
+    end
+  end
+
+  describe "real-world use cases" do
+    it "finds books by author and price range" do
+      skip "HeadedOx limitation: Text content access from nested elements needs investigation. See docs/HEADED_OX_LIMITATIONS.md"
+      results = doc.xpath(
+        "//book[contains(author, 'Alice') and @price < 20]",
+      )
+
+      expect(results.size).to eq(1)
+      expect(results.first.xpath("title").first.text).to eq("Ruby Programming")
+    end
+
+    it "calculates statistics" do
+      avg_price = doc.xpath("sum(//book/@price) div count(//book)")
+
+      expect(avg_price).to be_within(0.01).of(18.32)
+    end
+
+    it "combines multiple axes and predicates" do
+      # Books with price < 20, get their authors
+      authors = doc.xpath("//book[@price < 20]/author")
+
+      expect(authors.size).to eq(2)
+      expect(authors.map(&:text)).to contain_exactly("Alice", "Charlie")
+    end
+
+    it "uses parent axis for traversal" do
+      # Find title "Ruby Programming", then get parent book's author
+      author = doc.xpath(
+        "//title[contains(., 'Ruby')]/parent::book/author",
+      )
+
+      expect(author.size).to eq(1)
+      expect(author.first.text).to eq("Alice")
+    end
+  end
+
+  describe "comparison with standard Ox adapter" do
+    let(:ox_context) { Moxml.new(:ox) }
+    let(:ox_doc) { ox_context.parse(xml) }
+
+    it "handles predicates that Ox cannot" do
+      # HeadedOx handles complex predicates
+      headed_result = doc.xpath("//book[@price < 20]")
+      expect(headed_result.size).to eq(2)
+
+      # Standard Ox would have issues with this predicate
+      # (it would need translation to locate() syntax)
+    end
+
+    it "provides same parsing speed as Ox" do
+      # Both use Ox for parsing, so speed should be similar
+      start = Time.now
+      context.parse(xml)
+      headed_time = Time.now - start
+
+      start = Time.now
+      ox_context.parse(xml)
+      ox_time = Time.now - start
+
+      # Should be within same magnitude (both very fast)
+      expect(headed_time).to be < 0.1
+      expect(ox_time).to be < 0.1
+    end
+  end
+
+  describe "edge cases" do
+    it "handles empty results" do
+      result = doc.xpath("//nonexistent")
+
+      expect(result).to be_a(Moxml::NodeSet)
+      expect(result).to be_empty
+    end
+
+    it "handles nested predicates" do
+      result = doc.xpath(
+        "//book[author[contains(., 'Alice')]]",
+      )
+
+      expect(result.size).to eq(1)
+    end
+
+    it "handles union expressions" do
+      result = doc.xpath("//title | //author")
+
+      expect(result.size).to eq(6) # 3 titles + 3 authors
+    end
+
+    it "handles attribute selection" do
+      prices = doc.xpath("//book/@price")
+
+      expect(prices.size).to eq(3)
+      expect(prices).to all(be_a(Moxml::Attribute))
+    end
+  end
+
+  describe "namespace support" do
+    let(:ns_xml) do
+      <<~XML
+        <library xmlns:bk="http://example.com/books">
+          <bk:book>
+            <bk:title>Test Book</bk:title>
+          </bk:book>
+        </library>
+      XML
+    end
+
+    let(:ns_doc) { context.parse(ns_xml) }
+
+    it "queries with namespace prefixes" do
+      result = ns_doc.xpath(
+        "//bk:book",
+        { "bk" => "http://example.com/books" },
+      )
+
+      expect(result.size).to eq(1)
+    end
+
+    it "queries namespace elements with functions" do
+      result = ns_doc.xpath(
+        "//bk:title",
+        { "bk" => "http://example.com/books" },
+      )
+
+      expect(result.size).to eq(1)
+      expect(result.first.text).to eq("Test Book")
+    end
+  end
+end

data/spec/{support → integration}/shared_examples/edge_cases.rb

@@ -4,7 +4,8 @@ RSpec.shared_examples "Moxml Edge Cases" do
   let(:context) { Moxml.new }
 
   describe "special characters handling" do
-    it "handles all kinds of whitespace", skip: "carriege returns are troublesome" do
+    it "handles all kinds of whitespace",
+       skip: "carriege returns are troublesome" do
       # Nokogiri can't handle carriege returns properly
       # https://github.com/sparklemotion/nokogiri/issues/1356
       xml = "<root>\u0020\u0009 \u000D\u000A \u000D</root>"
@@ -31,12 +32,17 @@ RSpec.shared_examples "Moxml Edge Cases" do
 
   describe "malformed content handling" do
     it "handles CDATA with nested markers" do
-      pending "Ox doesn't escape the end token" if context.config.adapter_name == :ox
+      if context.config.adapter_name == :ox
+        pending "Ox doesn't escape the end token"
+      end
+      if context.config.adapter_name == :headed_ox
+        skip "HeadedOx limitation: Ox doesn't escape CDATA end markers. See docs/HEADED_OX_LIMITATIONS.md"
+      end
       cdata_text = "]]>]]>]]>"
       doc = context.create_document
       cdata = doc.create_cdata(cdata_text)
       expect(cdata.to_xml).to include(
-        "]]]]><![CDATA[>]]]]><![CDATA[>]]]]><![CDATA[>"
+        "]]]]><![CDATA[>]]]]><![CDATA[>]]]]><![CDATA[>",
       )
     end
 
@@ -51,21 +57,24 @@ RSpec.shared_examples "Moxml Edge Cases" do
       doc = context.create_document
       expect do
         doc.create_comment("-- test -- comment --")
-      end.to raise_error(Moxml::ValidationError, "XML comment cannot start or end with a hyphen")
+      end.to raise_error(Moxml::ValidationError,
+                         "XML comment cannot start or end with a hyphen")
     end
 
     it "rejects comments starting with hyphen" do
       doc = context.create_document
       expect do
         doc.create_comment("-starting with hyphen")
-      end.to raise_error(Moxml::ValidationError, "XML comment cannot start or end with a hyphen")
+      end.to raise_error(Moxml::ValidationError,
+                         "XML comment cannot start or end with a hyphen")
     end
 
     it "rejects comments ending with hyphen" do
       doc = context.create_document
       expect do
         doc.create_comment("ending with hyphen-")
-      end.to raise_error(Moxml::ValidationError, "XML comment cannot start or end with a hyphen")
+      end.to raise_error(Moxml::ValidationError,
+                         "XML comment cannot start or end with a hyphen")
     end
 
     it "accepts valid comments" do
@@ -77,7 +86,15 @@ RSpec.shared_examples "Moxml Edge Cases" do
 
   describe "namespace edge cases" do
     it "handles default namespace changes" do
-      pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+      if context.config.adapter_name == :ox
+        pending "Ox doesn't have a native XPath"
+      end
+      if context.config.adapter_name == :headed_ox
+        skip "HeadedOx limitation: Namespace methods not implemented in adapter. Requires Ox namespace API enhancement. See docs/HEADED_OX_LIMITATIONS.md"
+      end
+      if context.config.adapter_name == :libxml
+        skip "LibXML cannot query empty default namespace with XPath (documented limitation)"
+      end
       xml = <<~XML
         <root xmlns="http://default1.org">
           <child xmlns="http://default2.org">
@@ -92,7 +109,12 @@ RSpec.shared_examples "Moxml Edge Cases" do
     end
 
     it "handles recursive namespace definitions" do
-      pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+      if context.config.adapter_name == :ox
+        pending "Ox doesn't have a native XPath"
+      end
+      if context.config.adapter_name == :headed_ox
+        skip "HeadedOx limitation: Namespace methods not implemented in adapter. Requires Ox namespace API enhancement. See docs/HEADED_OX_LIMITATIONS.md"
+      end
 
       xml = <<~XML
         <root xmlns:a="http://a.org">
@@ -110,7 +132,12 @@ RSpec.shared_examples "Moxml Edge Cases" do
 
   describe "attribute edge cases" do
     it "handles attributes with same local name but different namespaces" do
-      pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+      if context.config.adapter_name == :headed_ox
+        skip "HeadedOx limitation: Namespace-prefixed attribute access needs Ox namespace API enhancement. See docs/HEADED_OX_LIMITATIONS.md"
+      end
+      if context.config.adapter_name == :ox
+        skip "Ox doesn't have a native XPath"
+      end
 
       xml = <<~XML
         <root xmlns:a="http://a.org" xmlns:b="http://b.org">
@@ -133,7 +160,7 @@ RSpec.shared_examples "Moxml Edge Cases" do
         "space" => " ",
         "tabs_newlines" => "\t\n",
         "unicode" => "⚡",
-        "entities" => "<&>'\""
+        "entities" => "<&>'\"",
       }
 
       special_values.each do |name, value|

data/spec/{support/shared_examples/context.rb → integration/shared_examples/high_level/context_behavior.rb}

@@ -16,7 +16,8 @@ RSpec.shared_examples "Moxml::Context" do
       expect(doc.root.children.first.children.first).to be_a(Moxml::Text)
     end
 
-    it "maintains document structure", skip: "Nokogiri doesn't consider the declaration as a child node" do
+    it "maintains document structure",
+       skip: "Nokogiri doesn't consider the declaration as a child node" do
       xml = <<~XML
         <?xml version="1.0"?>
         <!-- comment -->