moxml 0.5.145 → 0.5.147

data/spec/moxml/adapter/leptris_parse_lint_spec.rb

@@ -0,0 +1,35 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+# Issue #404: Context#parse_html forwards collect_errors (and mode) to
+# the leptris flags entry (leptris#1771's §13.2.5 parse-lint) — the
+# recorded error tokens surface through parse_diagnostics. Face-probed:
+# bindings predating the kwarg skip.
+RSpec.describe "parse_html collect_errors forwarding" do
+  let(:ctx) { Moxml.new(:leptris) }
+  let(:src) { %(<p class="x>unclosed</p><a href=><p>a<b</p>) }
+
+  before do
+    adapter = ctx.config.adapter
+    supported = adapter.singleton_class.const_get(:COLLECT_ERRORS_NATIVE)
+    skip "binding lacks the collect_errors kwarg (leptris#1771)" unless supported
+  end
+
+  it "surfaces parse-lint errors through parse_diagnostics" do
+    doc = ctx.parse_html(src, collect_errors: true)
+    diags = doc.parse_diagnostics
+    expect(diags).not_to be_empty
+    expect(diags.first).to have_key(:message)
+  end
+
+  it "keeps the default lane silent (no option, no diagnostics)" do
+    doc = ctx.parse_html(src)
+    expect(doc.parse_diagnostics).to be_empty
+  end
+
+  it "forwards through Context#parse_html options" do
+    doc = ctx.parse_html(src, { collect_errors: true })
+    expect(doc.parse_diagnostics).not_to be_empty
+  end
+end

data/spec/moxml/adapter/shared_examples/adapter_contract.rb

@@ -343,13 +343,13 @@ RSpec.shared_examples "xml adapter" do
     it "respects indentation settings" do
       if described_class.name.include?("Oga") &&
           !described_class.indents_output?
-        pending("Oga does not support indentation settings")
+        pending("D-09: Oga does not support indentation settings")
       end
       if described_class.name.include?("Libxml")
-        skip("LibXML serialization does not support indentation (documented limitation)")
+        skip("D-09: LibXML serialization does not support indentation (documented limitation)")
       end
       if described_class.name.include?("Ox")
-        skip("Ox does not support indentation settings (documented limitation)")
+        skip("D-09: Ox does not support indentation settings (documented limitation)")
       end
 
       unindented = described_class.serialize(doc, indent: 0)

data/spec/moxml/c14n/adapter_delegation_spec.rb

@@ -58,9 +58,9 @@ RSpec.describe "Moxml::C14n cross-adapter canonicalization" do
 
   describe "document-level nodes keep document order" do
     # oga surfaces the XML declaration as a <?xml?> PI and keeps the
-    # separator space in PI content; ox/headed_ox drop PI content on
-    # parse. Both are pre-existing parse-layer divergences, out of
-    # scope here.
+    # separator space in PI content (D-04); ox/headed_ox drop PI
+    # content on parse (D-05). Both are pre-existing parse-layer
+    # divergences, out of scope here.
     let(:adapters) { Moxml::Adapter::AVAILABLE_ADAPTERS - %i[oga ox headed_ox] }
 
     it "canonicalizes a leading PI before the root element" do

data/spec/moxml/c14n/comments_pis_spec.rb

@@ -53,13 +53,13 @@ RSpec.describe "C14N comments and PIs" do
     # PIs, and the DataModel.from_document path drops them. Fixing requires
     # adapter-level work and is tracked separately.
     it "renders PIs outside the document element (KNOWN LIMITATION)" do
-      skip "document-level PI rendering not yet implemented"
+      skip "D-08: document-level PI rendering not yet implemented"
     end
   end
 
   describe "comments at document level" do
     it "renders document-level comments when with_comments: true (KNOWN LIMITATION)" do
-      skip "document-level comment rendering not yet implemented"
+      skip "D-08: document-level comment rendering not yet implemented"
     end
   end
 end

data/spec/moxml/c14n/subset_spec.rb

@@ -63,7 +63,7 @@ RSpec.describe "Moxml::C14n subset canonicalization" do
           next if label == "text selection" && adapter_name == :rexml
 
           # rexml's adapter xpath does not return text nodes for
-          # text() node tests (pre-existing capability gap).
+          # text() node tests (D-07, pre-existing capability gap).
 
           it "canonicalizes #{label}" do
             doc = ctx.parse(xml)
@@ -72,9 +72,9 @@ RSpec.describe "Moxml::C14n subset canonicalization" do
         end
 
         # ox/headed_ox strip comment content padding at the parse
-        # layer (pre-existing, same family as their PI content loss);
-        # only the rendered-comment case depends on it. rexml's
-        # comment() selection gap also only affects this case.
+        # layer (D-06, pre-existing, same family as their PI content
+        # loss); only the rendered-comment case depends on it. rexml's
+        # comment() selection gap (D-07) also only affects this case.
         unless COMMENTS_PADDING_LOSS.include?(adapter_name) ||
             COMMENT_SELECTION_GAP.include?(adapter_name)
           it COMMENTS_CASE[0] do

data/spec/moxml/c14n/w3c_examples_spec.rb

@@ -55,7 +55,8 @@ RSpec.describe "Moxml::C14n W3C REC-xml-c14n examples" do
 
       W3C_FULL_FORMS.each_with_index do |(label, xml, expected), idx|
         # ox/headed_ox strip inter-element whitespace, expand 
-        # and pad comments at parse; oga keeps the PI separator space.
+        # and pad comments at parse (D-05, D-06); oga keeps the PI
+        # separator space (D-04).
         skip_shapes = { ox: [0, CR_SHAPE, PI_SHAPE],
                         headed_ox: [0, CR_SHAPE, PI_SHAPE],
                         oga: [PI_SHAPE] }[adapter_name] || []
@@ -72,6 +73,7 @@ RSpec.describe "Moxml::C14n W3C REC-xml-c14n examples" do
 
   describe "W3C §3 document-subset whitespace" do
     Moxml::Adapter::AVAILABLE_ADAPTERS.each do |adapter_name|
+      # ox/headed_ox strip inter-element whitespace at parse (D-06)
       next if WHITESPACE_SHAPES.include?(adapter_name)
 
       it "renders selected nodes only, whitespace runs in place (#{adapter_name})" do

data/spec/moxml/declaration_preservation_spec.rb

@@ -4,9 +4,9 @@ require "spec_helper"
 
 RSpec.describe "XML Declaration Preservation" do
   # Test with all available adapters
-  ADAPTERS = %i[nokogiri oga rexml ox libxml headed_ox].freeze
+  DECLARATION_SPECS_ADAPTERS = %i[nokogiri oga rexml ox libxml headed_ox].freeze
 
-  ADAPTERS.each do |adapter_name|
+  DECLARATION_SPECS_ADAPTERS.each do |adapter_name|
     context "with #{adapter_name} adapter" do
       let(:context) { Moxml.new(adapter_name) }
 

data/spec/moxml/element_add_element_spec.rb

@@ -9,7 +9,7 @@ require "spec_helper"
 # the contract: a missing base default NoMethodErrors on non-face
 # adapters, a broken fallback loses attributes or the attach.
 RSpec.describe Moxml::Element do
-  ADAPTERS = %i[leptris nokogiri ox oga rexml libxml].freeze # rubocop:disable Lint/ConstantDefinitionInBlock, RSpec/LeakyConstantDeclaration
+  ELEMENT_ADD_ELEMENT_ADAPTERS = %i[leptris nokogiri ox oga rexml libxml].freeze # rubocop:disable Lint/ConstantDefinitionInBlock, RSpec/LeakyConstantDeclaration
 
   def with_adapter(name)
     ctx = Moxml.new(name)
@@ -19,7 +19,7 @@ RSpec.describe Moxml::Element do
   end
 
   it "attaches a child with all attributes on every available adapter" do
-    ADAPTERS.each do |name|
+    ELEMENT_ADD_ELEMENT_ADAPTERS.each do |name|
       with_adapter(name) do |ctx|
         doc = ctx.parse("<catalog/>")
         root = doc.root
@@ -35,7 +35,7 @@ RSpec.describe Moxml::Element do
   end
 
   it "nests under a non-root element with empty attrs" do
-    ADAPTERS.each do |name|
+    ELEMENT_ADD_ELEMENT_ADAPTERS.each do |name|
       with_adapter(name) do |ctx|
         doc = ctx.parse("<catalog><record/></catalog>")
         record = doc.root.children.first

data/spec/moxml/element_attribute_pairs_spec.rb

@@ -12,7 +12,7 @@ RSpec.describe Moxml::Element do
     '<catalog><record id="r0" kind="k0"><field name="f0" unit="u0">v</field></record></catalog>'
   end
 
-  ADAPTERS = %i[leptris nokogiri ox oga rexml libxml].freeze # rubocop:disable Lint/ConstantDefinitionInBlock, RSpec/LeakyConstantDeclaration
+  ELEMENT_ATTRIBUTE_PAIRS_ADAPTERS = %i[leptris nokogiri ox oga rexml libxml].freeze # rubocop:disable Lint/ConstantDefinitionInBlock, RSpec/LeakyConstantDeclaration
 
   def with_adapter(name)
     ctx = Moxml.new(name)
@@ -22,7 +22,7 @@ RSpec.describe Moxml::Element do
   end
 
   it "matches the node path on every available adapter" do
-    results = ADAPTERS.filter_map do |name|
+    results = ELEMENT_ATTRIBUTE_PAIRS_ADAPTERS.filter_map do |name|
       with_adapter(name) do |ctx|
         root = ctx.parse(xml).root
         [name, root.attribute_pairs]

data/spec/moxml/node_spec.rb

@@ -157,7 +157,7 @@ RSpec.describe Moxml::Node do
 
         it "locates the node via its path" do
           # Ox translates only a limited XPath subset (no positional predicates)
-          skip "ox xpath translation does not support positional predicates" if adapter_name == :ox
+          skip "D-12: ox xpath translation does not support positional predicates" if adapter_name == :ox
 
           expect(doc.at_xpath(title.path)).to eq(title)
         end

data/spec/moxml/plan_conformance_spec.rb

@@ -27,10 +27,10 @@ RSpec.describe "Moxml::Plan adapter conformance" do
     end
   end
 
-  ADAPTERS = %i[leptris nokogiri ox oga rexml libxml].freeze # rubocop:disable Lint/ConstantDefinitionInBlock, RSpec/LeakyConstantDeclaration
+  PLAN_CONFORMANCE_ADAPTERS = %i[leptris nokogiri ox oga rexml libxml].freeze # rubocop:disable Lint/ConstantDefinitionInBlock, RSpec/LeakyConstantDeclaration
 
   it "executes on every available adapter" do
-    available = ADAPTERS.select do |a|
+    available = PLAN_CONFORMANCE_ADAPTERS.select do |a|
       ctx = Moxml.new(a)
       ctx.parse(xml)
       true
@@ -63,7 +63,7 @@ RSpec.describe "Moxml::Plan adapter conformance" do
       on("field") { |attrs, text, _| [attrs["name"], text] }
     end
 
-    ADAPTERS.each do |a|
+    PLAN_CONFORMANCE_ADAPTERS.each do |a|
       begin
         ctx = Moxml.new(a)
         ctx.parse(xml)

data/spec/moxml/sax_entity_parity_spec.rb

@@ -32,7 +32,7 @@ module SaxEntityParityFixtures
     end
   end
 
-  ADAPTERS = %i[nokogiri ox oga rexml libxml headed_ox].freeze
+  SAX_ENTITY_PARITY_ADAPTERS = %i[nokogiri ox oga rexml libxml headed_ox].freeze
 
   # Each row: [input XML, decoded attribute value, expected attribute
   # value as it appears in the serialized XML]. The serialized form
@@ -85,7 +85,7 @@ module SaxEntityParityFixtures
 end
 
 RSpec.describe "SAX/DOM entity parity" do
-  SaxEntityParityFixtures::ADAPTERS.each do |adapter|
+  SaxEntityParityFixtures::SAX_ENTITY_PARITY_ADAPTERS.each do |adapter|
     context "with #{adapter} adapter" do
       let(:ctx) { Moxml.new(adapter) }
 
@@ -349,7 +349,7 @@ RSpec.describe "SAX/DOM entity parity" do
 
     SaxEntityParityFixtures::ATTRIBUTE_CASES.each do |label, (xml, _expected, expected_serialized)|
       it "every adapter emits the same attribute form: #{label}" do
-        results = SaxEntityParityFixtures::ADAPTERS.to_h { |a| [a, serialize_via(a, xml)] }
+        results = SaxEntityParityFixtures::SAX_ENTITY_PARITY_ADAPTERS.to_h { |a| [a, serialize_via(a, xml)] }
         results.each do |a, out|
           expect(out).to(
             include(%(x="#{expected_serialized}")),

data/spec/moxml/sax_spec.rb

@@ -413,7 +413,7 @@ RSpec.describe Moxml::SAX do
     it "documents CDATA limitation" do
       # Ox SAX does not have separate CDATA events
       # All CDATA content is delivered as text() events
-      skip "Ox SAX does not support separate CDATA events"
+      skip "D-10: Ox SAX does not support separate CDATA events"
 
       xml_with_cdata = "<root><![CDATA[special content]]></root>"
 
@@ -428,7 +428,7 @@ RSpec.describe Moxml::SAX do
 
     it "documents comment limitation" do
       # Ox SAX does not have separate comment events
-      skip "Ox SAX does not support separate comment events"
+      skip "D-10: Ox SAX does not support separate comment events"
 
       xml_with_comment = "<root><!-- comment --><data>content</data></root>"
 
@@ -443,7 +443,7 @@ RSpec.describe Moxml::SAX do
 
     it "documents processing instruction limitation" do
       # Ox SAX does not have separate PI events
-      skip "Ox SAX does not support separate processing instruction events"
+      skip "D-10: Ox SAX does not support separate processing instruction events"
 
       xml_with_pi = '<?xml-stylesheet type="text/xsl" href="style.xsl"?><root/>'
 
@@ -714,7 +714,7 @@ RSpec.describe Moxml::SAX do
     end
 
     it "handles errors gracefully" do
-      skip "Oga SAX parser may be more lenient with malformed XML"
+      skip "D-11: Oga SAX parser may be more lenient with malformed XML"
 
       invalid_xml = "<book><title>Unclosed"
 
@@ -1024,7 +1024,7 @@ RSpec.describe Moxml::SAX do
 
     it "documents CDATA limitation (inherited from Ox)" do
       # HeadedOx inherits Ox's SAX implementation, which does not have separate CDATA events
-      skip "HeadedOx SAX (inherited from Ox) does not support separate CDATA events"
+      skip "D-10: HeadedOx SAX (inherited from Ox) does not support separate CDATA events"
 
       xml_with_cdata = "<root><![CDATA[special content]]></root>"
 
@@ -1038,7 +1038,7 @@ RSpec.describe Moxml::SAX do
 
     it "documents comment limitation (inherited from Ox)" do
       # HeadedOx inherits Ox's SAX implementation, which does not have separate comment events
-      skip "HeadedOx SAX (inherited from Ox) does not support separate comment events"
+      skip "D-10: HeadedOx SAX (inherited from Ox) does not support separate comment events"
 
       xml_with_comment = "<root><!-- comment --><data>content</data></root>"
 
@@ -1052,7 +1052,7 @@ RSpec.describe Moxml::SAX do
 
     it "documents processing instruction limitation (inherited from Ox)" do
       # HeadedOx inherits Ox's SAX implementation, which does not have separate PI events
-      skip "HeadedOx SAX (inherited from Ox) does not support separate processing instruction events"
+      skip "D-10: HeadedOx SAX (inherited from Ox) does not support separate processing instruction events"
 
       xml_with_pi = '<?xml-stylesheet type="text/xsl" href="style.xsl"?><root/>'
 

data/spec/moxml/struct_plan_spec.rb

@@ -28,7 +28,7 @@ RSpec.describe Moxml::StructPlan do
     end
   end
 
-  ADAPTERS = %i[leptris nokogiri ox oga rexml libxml].freeze # rubocop:disable Lint/ConstantDefinitionInBlock, RSpec/LeakyConstantDeclaration
+  STRUCT_PLAN_ADAPTERS = %i[leptris nokogiri ox oga rexml libxml].freeze # rubocop:disable Lint/ConstantDefinitionInBlock, RSpec/LeakyConstantDeclaration
 
   def with_adapter(name)
     yield Moxml.new(name)
@@ -36,7 +36,7 @@ RSpec.describe Moxml::StructPlan do
     skip "#{name} unavailable"
   end
 
-  ADAPTERS.each do |adapter_name|
+  STRUCT_PLAN_ADAPTERS.each do |adapter_name|
     describe "on #{adapter_name}" do
       it "materializes the same structs as Moxml::Plan" do
         with_adapter(adapter_name) do |ctx|

data/spec/moxml/xpath_capabilities_spec.rb

@@ -83,7 +83,7 @@ RSpec.describe "XPath Capabilities" do
           programming = doc.xpath("//book[@category='programming']")
           expect(programming.length).to be >= 0 # May not work on all
         rescue Moxml::XPathError
-          skip "Attribute value predicates not supported on #{adapter_name}"
+          skip "D-12: Attribute value predicates not supported on #{adapter_name}"
         end
       end
 
@@ -98,7 +98,7 @@ RSpec.describe "XPath Capabilities" do
           items = doc.xpath("//book:item", namespaces)
           expect(items.length).to be >= 0
         rescue Moxml::XPathError, NoMethodError
-          skip "Namespace XPath not supported on #{adapter_name}"
+          skip "D-12: Namespace XPath not supported on #{adapter_name}"
         end
 
         it "handles default namespaces" do
@@ -108,7 +108,7 @@ RSpec.describe "XPath Capabilities" do
           library = doc.xpath("//lib:library", namespaces)
           expect(library.length).to be >= 0
         rescue Moxml::XPathError, NoMethodError
-          skip "Default namespace XPath not supported on #{adapter_name}"
+          skip "D-12: Default namespace XPath not supported on #{adapter_name}"
         end
       end
 
@@ -118,7 +118,7 @@ RSpec.describe "XPath Capabilities" do
           first_book = doc.xpath("//book[1]")
           expect(first_book.length).to be >= 0
         rescue Moxml::XPathError
-          skip "Position predicates not supported on #{adapter_name}"
+          skip "D-12: Position predicates not supported on #{adapter_name}"
         end
 
         it "supports position() function" do
@@ -126,7 +126,7 @@ RSpec.describe "XPath Capabilities" do
           first_books = doc.xpath("//book[position()=1]")
           expect(first_books.length).to be >= 0
         rescue Moxml::XPathError
-          skip "position() function not supported on #{adapter_name}"
+          skip "D-12: position() function not supported on #{adapter_name}"
         end
       end
 
@@ -135,11 +135,11 @@ RSpec.describe "XPath Capabilities" do
           doc = Moxml.new.parse(simple_xml)
           result = doc.xpath("count(//book)")
           # REXML's XPath has no function support and yields an empty set
-          skip "count() function not supported on #{adapter_name}" if result.is_a?(Moxml::NodeSet)
+          skip "D-12: count() function not supported on #{adapter_name}" if result.is_a?(Moxml::NodeSet)
 
           expect(result).to eq(2)
         rescue Moxml::XPathError, NoMethodError
-          skip "count() function not supported on #{adapter_name}"
+          skip "D-12: count() function not supported on #{adapter_name}"
         end
 
         it "supports text() node selection" do
@@ -147,7 +147,7 @@ RSpec.describe "XPath Capabilities" do
           texts = doc.xpath("//title/text()")
           expect(texts.length).to be >= 0
         rescue Moxml::XPathError
-          skip "text() not supported on #{adapter_name}"
+          skip "D-12: text() not supported on #{adapter_name}"
         end
       end
 
@@ -157,7 +157,7 @@ RSpec.describe "XPath Capabilities" do
           cheap_books = doc.xpath("//book[@id < 2]")
           expect(cheap_books.length).to be >= 0
         rescue Moxml::XPathError
-          skip "Numeric predicates not supported on #{adapter_name}"
+          skip "D-12: Numeric predicates not supported on #{adapter_name}"
         end
 
         it "supports boolean expressions ([cond1 and cond2])" do
@@ -165,7 +165,7 @@ RSpec.describe "XPath Capabilities" do
           result = doc.xpath("//book[@id and @category]")
           expect(result.length).to be >= 0
         rescue Moxml::XPathError
-          skip "Boolean expressions not supported on #{adapter_name}"
+          skip "D-12: Boolean expressions not supported on #{adapter_name}"
         end
       end
 
@@ -175,7 +175,7 @@ RSpec.describe "XPath Capabilities" do
           children = doc.xpath("/library/child::book")
           expect(children.length).to be >= 0
         rescue Moxml::XPathError
-          skip "child:: axis not supported on #{adapter_name}"
+          skip "D-12: child:: axis not supported on #{adapter_name}"
         end
 
         it "supports descendant axis" do
@@ -183,7 +183,7 @@ RSpec.describe "XPath Capabilities" do
           descendants = doc.xpath("//library/descendant::title")
           expect(descendants.length).to be >= 0
         rescue Moxml::XPathError
-          skip "descendant:: axis not supported on #{adapter_name}"
+          skip "D-12: descendant:: axis not supported on #{adapter_name}"
         end
       end
 
@@ -193,7 +193,7 @@ RSpec.describe "XPath Capabilities" do
           items = doc.xpath("//book | //magazine")
           expect(items.length).to be >= 0
         rescue Moxml::XPathError
-          skip "Union operator not supported on #{adapter_name}"
+          skip "D-12: Union operator not supported on #{adapter_name}"
         end
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: moxml
 version: !ruby/object:Gem::Version
-  version: 0.5.145
+  version: 0.5.147
 platform: ruby
 authors:
 - Ribose Inc.
@@ -73,6 +73,7 @@ files:
 - docs/_pages/compatibility.adoc
 - docs/_pages/configuration.adoc
 - docs/_pages/conversion-apis.adoc
+- docs/_pages/divergences.adoc
 - docs/_pages/error-handling.adoc
 - docs/_pages/headed-ox-limitations.adoc
 - docs/_pages/headed-ox.adoc
@@ -84,6 +85,7 @@ files:
 - docs/_pages/thread-safety.adoc
 - docs/_references/document-api.adoc
 - docs/_references/index.adoc
+- docs/_references/processing-model.adoc
 - docs/_tutorials/basic-usage.adoc
 - docs/_tutorials/builder-pattern.adoc
 - docs/_tutorials/index.adoc
@@ -310,9 +312,14 @@ files:
 - sig/moxml.rbs
 - spec/consistency/README.md
 - spec/consistency/adapter_parity_spec.rb
+- spec/consistency/divergence_register_spec.rb
+- spec/consistency/document_text_spec.rb
 - spec/consistency/eol_normalization_spec.rb
 - spec/consistency/html_doctype_spec.rb
+- spec/consistency/loud_degradation_spec.rb
+- spec/consistency/model_coverage_audit_spec.rb
 - spec/consistency/round_trip_spec.rb
+- spec/consistency/spec_constant_uniqueness_spec.rb
 - spec/consistency/xpath_idioms_spec.rb
 - spec/consistency/xslt_spec.rb
 - spec/examples/README.md
@@ -452,9 +459,12 @@ files:
 - spec/moxml/adapter/leptris_append_gc_spec.rb
 - spec/moxml/adapter/leptris_attribute_snapshot_spec.rb
 - spec/moxml/adapter/leptris_binding_cap_spec.rb
+- spec/moxml/adapter/leptris_contract_reads_spec.rb
+- spec/moxml/adapter/leptris_detached_text_spec.rb
 - spec/moxml/adapter/leptris_doc_parts_spec.rb
 - spec/moxml/adapter/leptris_document_declaration_spec.rb
 - spec/moxml/adapter/leptris_inbody_comment_spec.rb
+- spec/moxml/adapter/leptris_parse_lint_spec.rb
 - spec/moxml/adapter/leptris_parse_skips_spec.rb
 - spec/moxml/adapter/leptris_record_driver_spec.rb
 - spec/moxml/adapter/leptris_sax_dup_attr_spec.rb