moxml 0.1.3 → 0.1.6

data/spec/support/shared_examples/node_set.rb

@@ -25,31 +25,47 @@ RSpec.shared_examples "Moxml::NodeSet" do
 
     describe "enumeration" do
       it "iterates over nodes" do
+        pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+
         texts = []
         nodes.each { |node| texts << node.text }
         expect(texts).to eq(%w[First Second Third])
       end
 
       it "maps nodes" do
+        pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+
         texts = nodes.map(&:text)
         expect(texts).to eq(%w[First Second Third])
       end
 
       it "selects nodes" do
+        pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+
         selected = nodes.select { |node| node.text.include?("i") }
         expect(selected.size).to eq(2)
         expect(selected.map(&:text)).to eq(%w[First Third])
       end
+
+      it "compares nodes" do
+        pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+
+        expect(doc.xpath("//child")).to eq(doc.root.children)
+      end
     end
 
     describe "access methods" do
       it "accesses by index" do
+        pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+
         expect(nodes[0].text).to eq("First")
         expect(nodes[1].text).to eq("Second")
         expect(nodes[-1].text).to eq("Third")
       end
 
       it "accesses by range" do
+        pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+
         subset = nodes[0..1]
         expect(subset).to be_a(described_class)
         expect(subset.size).to eq(2)
@@ -57,6 +73,8 @@ RSpec.shared_examples "Moxml::NodeSet" do
       end
 
       it "provides first and last" do
+        pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+
         expect(nodes.first.text).to eq("First")
         expect(nodes.last.text).to eq("Third")
       end
@@ -77,6 +95,8 @@ RSpec.shared_examples "Moxml::NodeSet" do
 
     describe "concatenation" do
       it "combines node sets" do
+        pending "Ox doesn't have a native XPath" if context.config.adapter_name == :ox
+
         other_doc = context.parse("<root><item>Fourth</item></root>")
         other_nodes = other_doc.xpath("//item")
         combined = nodes + other_nodes

data/spec/support/shared_examples/processing_instruction.rb

@@ -43,7 +43,7 @@ RSpec.shared_examples "Moxml::ProcessingInstruction" do
 
   describe "serialization" do
     it "formats processing instruction" do
-      expect(pi.to_xml).to eq('<?xml-stylesheet href="style.xsl" type="text/xsl"?>')
+      expect(pi.to_xml.strip).to end_with('<?xml-stylesheet href="style.xsl" type="text/xsl"?>')
     end
 
     it "handles special characters" do

data/spec/support/shared_examples/text.rb

@@ -33,7 +33,8 @@ RSpec.shared_examples "Moxml::Text" do
   describe "special characters" do
     it "encodes basic XML entities" do
       text.content = "< > & \" '"
-      expect(text.to_xml).to eq("&lt; &gt; &amp; \" '")
+      doc.add_child(text)
+      expect(doc.to_xml.strip).to end_with("&lt; &gt; &amp; \" '")
     end
 
     it "preserves whitespace" do

data/spec/support/shared_examples/xml_adapter.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+# This test works directly with native nodes which looks quite strange
+# A better way is to run it through Moxml wrappers
 RSpec.shared_examples "xml adapter" do
   let(:xml) do
     <<~XML
@@ -80,7 +82,7 @@ RSpec.shared_examples "xml adapter" do
       expect(children.length).to eq(3)
     end
 
-    it "gets siblings", skip: "Oga includes text nodes into siblings" do
+    it "gets siblings" do
       children = described_class.children(root)
       first = children[0]
       second = children[1]
@@ -97,13 +99,16 @@ RSpec.shared_examples "xml adapter" do
 
     it "adds text child" do
       described_class.add_child(root, "text")
-      expect(described_class.children(root).last.text).to eq("text")
+      # a workaround for Rexml until we convert tests to work with Moxml wrappers
+      last_child = described_class.children(root).last
+      last_text = last_child.respond_to?(:text) ? last_child.text : last_child.to_s
+      expect(last_text).to eq("text")
     end
   end
 
   describe "attributes" do
-    let(:doc) { described_class.parse(xml) }
-    let(:element) { described_class.children(described_class.root(doc.native)).first }
+    let(:doc) { described_class.parse(xml).native }
+    let(:element) { described_class.children(described_class.root(doc)).first }
 
     it "gets attributes" do
       attrs = described_class.attributes(element)
@@ -123,7 +128,9 @@ RSpec.shared_examples "xml adapter" do
 
     it "handles special characters in attributes" do
       described_class.set_attribute(element, "special", '< > & " \'')
-      value = described_class.get_attribute(element, "special")&.to_xml
+      # a workaround for Rexml until we convert tests to work with Moxml wrappers
+      attr = described_class.get_attribute(element, "special")
+      value = attr.respond_to?(:to_xml) ? attr&.to_xml : attr.to_s
       expect(value).to match(/&lt; &gt; &amp; (&quot;|") ('|&apos;)/)
     end
   end
@@ -156,7 +163,10 @@ RSpec.shared_examples "xml adapter" do
       expect(result).to include("</root>")
     end
 
-    it "respects indentation settings", skip: "Indent cannot be negative, and zero indent doesn't remove newlines" do
+    it "respects indentation settings" do
+      pending("Oga does not support indentation settings") if described_class.name.include?("Oga")
+      pending("Postponed for Rexml till better times") if described_class.name.include?("Rexml")
+
       unindented = described_class.serialize(doc, indent: 0)
       indented = described_class.serialize(doc, indent: 2)
 
@@ -176,7 +186,7 @@ RSpec.shared_examples "xml adapter" do
   end
 
   describe "xpath" do
-    let(:doc) { described_class.parse(xml) }
+    let(:doc) { described_class.parse(xml).native }
 
     it "finds nodes by xpath" do
       nodes = described_class.xpath(doc, "//xmlns:child")
@@ -188,4 +198,156 @@ RSpec.shared_examples "xml adapter" do
       expect(described_class.get_attribute_value(node, "id")).to eq("1")
     end
   end
+
+  describe "namespace handling" do
+    context "case 1" do
+      let(:xml) do
+        <<~XML
+          <?xml version="1.0" encoding="UTF-8"?>
+          <svg xmlns="http://www.w3.org/2000/svg"
+               xmlns:xlink="http://www.w3.org/1999/xlink"
+               width="100" height="100">
+            <defs>
+              <circle id="myCircle" cx="50" cy="50" r="40"/>
+            </defs>
+            <use xlink:href="#myCircle" fill="red"/>
+            <text x="50" y="50" text-anchor="middle">SVG</text>
+          </svg>
+        XML
+      end
+
+      it "preserves and correctly handles multiple namespaces" do
+        pending("Rexml does not respect ZPath namespaces") if described_class.name.include?("Rexml")
+        # Parse original XML
+        doc = described_class.parse(xml).native
+
+        # Test namespace preservation in serialization
+        result = described_class.serialize(doc)
+        expect(result).to include('xmlns="http://www.w3.org/2000/svg"')
+        expect(result).to include('xmlns:xlink="http://www.w3.org/1999/xlink"')
+
+        # Test xpath with namespaces
+        namespaces = {
+          "svg" => "http://www.w3.org/2000/svg",
+          "xlink" => "http://www.w3.org/1999/xlink"
+        }
+
+        # Find use element and verify xlink:href attribute
+        use_elem = described_class.at_xpath(doc, "//svg:use", namespaces)
+        expect(use_elem).not_to be_nil
+        expect(described_class.get_attribute_value(use_elem, "xlink:href")).to eq("#myCircle")
+
+        # Verify circle element exists in defs
+        circle = described_class.at_xpath(doc, "//svg:defs/svg:circle", namespaces)
+        expect(circle).not_to be_nil
+        expect(described_class.get_attribute_value(circle, "id")).to eq("myCircle")
+
+        # Test default SVG namespace
+        text = described_class.at_xpath(doc, "//svg:text", namespaces)
+        expect(described_class.text_content(text)).to eq("SVG")
+      end
+    end
+
+    context "case 2" do
+      let(:xml) do
+        <<~XML
+          <?xml version="1.0" encoding="UTF-8"?>
+          <rss version="2.0"#{" "}
+            xmlns:atom="http://www.w3.org/2005/Atom"
+            xmlns:dc="http://purl.org/dc/elements/1.1/"
+            xmlns:content="http://purl.org/rss/1.0/modules/content/">
+            <channel>
+              <title>Example RSS Feed</title>
+              <atom:link href="http://example.com/feed" rel="self"/>
+              <item>
+                <title>Example Post</title>
+                <dc:creator>John Doe</dc:creator>
+                <content:encoded><![CDATA[<p>Post content</p>]]></content:encoded>
+              </item>
+            </channel>
+          </rss>
+        XML
+      end
+
+      it "preserves and correctly handles multiple namespaces" do
+        # Parse original XML
+        doc = described_class.parse(xml).native
+
+        # Test namespace preservation in serialization
+        result = described_class.serialize(doc)
+        expect(result).to include('xmlns:atom="http://www.w3.org/2005/Atom"')
+        expect(result).to include('xmlns:dc="http://purl.org/dc/elements/1.1/"')
+        expect(result).to include('xmlns:content="http://purl.org/rss/1.0/modules/content/"')
+
+        # Test xpath with namespaces
+        namespaces = {
+          "atom" => "http://www.w3.org/2005/Atom",
+          "dc" => "http://purl.org/dc/elements/1.1/",
+          "content" => "http://purl.org/rss/1.0/modules/content/"
+        }
+
+        # Find creator using namespaced xpath
+        creator = described_class.at_xpath(doc, "//dc:creator", namespaces)
+        expect(described_class.text_content(creator)).to eq("John Doe")
+
+        # Verify atom:link exists
+        link = described_class.at_xpath(doc, "//atom:link", namespaces)
+        expect(link).not_to be_nil
+        expect(described_class.get_attribute_value(link, "href")).to eq("http://example.com/feed")
+
+        # Verify CDATA in content:encoded
+        content = described_class.at_xpath(doc, "//content:encoded", namespaces)
+        expect(described_class.text_content(content)).to eq("<p>Post content</p>")
+      end
+    end
+
+    context "case 3" do
+      let(:xml) do
+        <<~XML
+          <?xml version="1.0" encoding="UTF-8"?>
+          <soap:Envelope#{" "}
+            xmlns:soap="http://www.w3.org/2003/05/soap-envelope"
+            xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+            xmlns:ns="urn:example:namespace">
+            <soap:Header>
+              <ns:SessionId>12345</ns:SessionId>
+            </soap:Header>
+            <soap:Body>
+              <ns:GetUserRequest>
+                <ns:UserId xsi:type="xsi:string">user123</ns:UserId>
+              </ns:GetUserRequest>
+            </soap:Body>
+          </soap:Envelope>
+        XML
+      end
+
+      it "preserves and correctly handles multiple namespaces" do
+        # Parse original XML
+        doc = described_class.parse(xml).native
+
+        # Test namespace preservation in serialization
+        result = described_class.serialize(doc)
+        expect(result).to include('xmlns:soap="http://www.w3.org/2003/05/soap-envelope"')
+        expect(result).to include('xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"')
+        expect(result).to include('xmlns:ns="urn:example:namespace"')
+
+        # Test xpath with namespaces
+        namespaces = {
+          "soap" => "http://www.w3.org/2003/05/soap-envelope",
+          "ns" => "urn:example:namespace"
+        }
+
+        # Find user ID using namespaced xpath
+        user_id = described_class.at_xpath(doc, "//ns:UserId", namespaces)
+        expect(described_class.text_content(user_id)).to eq("user123")
+
+        # Verify soap:Body exists
+        body = described_class.at_xpath(doc, "//soap:Body", namespaces)
+        expect(body).not_to be_nil
+
+        # Verify attribute with namespace
+        expect(described_class.get_attribute_value(user_id, "xsi:type")).to eq("xsi:string")
+      end
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: moxml
 version: !ruby/object:Gem::Version
-  version: 0.1.3
+  version: 0.1.6
 platform: ruby
 authors:
 - Ribose Inc.
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-02-26 00:00:00.000000000 Z
+date: 2025-07-29 00:00:00.000000000 Z
 dependencies: []
 description: 'Moxml is a unified XML manipulation library that provides a common API.
 
@@ -27,6 +27,7 @@ files:
 - ".rubocop_todo.yml"
 - ".ruby-version"
 - Gemfile
+- LICENSE.md
 - README.adoc
 - Rakefile
 - bin/console
@@ -36,9 +37,14 @@ files:
 - lib/moxml/adapter/base.rb
 - lib/moxml/adapter/customized_oga/xml_declaration.rb
 - lib/moxml/adapter/customized_oga/xml_generator.rb
+- lib/moxml/adapter/customized_ox/attribute.rb
+- lib/moxml/adapter/customized_ox/namespace.rb
+- lib/moxml/adapter/customized_ox/text.rb
+- lib/moxml/adapter/customized_rexml/formatter.rb
 - lib/moxml/adapter/nokogiri.rb
 - lib/moxml/adapter/oga.rb
 - lib/moxml/adapter/ox.rb
+- lib/moxml/adapter/rexml.rb
 - lib/moxml/attribute.rb
 - lib/moxml/builder.rb
 - lib/moxml/cdata.rb
@@ -59,11 +65,14 @@ files:
 - lib/moxml/version.rb
 - lib/moxml/xml_utils.rb
 - lib/moxml/xml_utils/encoder.rb
+- lib/ox/node.rb
 - moxml.gemspec
 - sig/moxml.rbs
+- spec/fixtures/small.xml
 - spec/moxml/adapter/nokogiri_spec.rb
 - spec/moxml/adapter/oga_spec.rb
 - spec/moxml/adapter/ox_spec.rb
+- spec/moxml/adapter/rexml_spec.rb
 - spec/moxml/all_with_adapters_spec.rb
 - spec/moxml/config_spec.rb
 - spec/moxml/error_spec.rb
@@ -83,6 +92,7 @@ files:
 - spec/support/shared_examples/element.rb
 - spec/support/shared_examples/examples/attribute.rb
 - spec/support/shared_examples/examples/basic_usage.rb
+- spec/support/shared_examples/examples/benchmark_spec.rb
 - spec/support/shared_examples/examples/memory.rb
 - spec/support/shared_examples/examples/namespace.rb
 - spec/support/shared_examples/examples/readme_examples.rb
@@ -116,7 +126,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.27
+rubygems_version: 3.5.22
 signing_key:
 specification_version: 4
 summary: Unified XML manipulation library