svg_conform 0.1.2 → 0.1.3

data/lib/svg_conform/validation_context.rb

@@ -26,7 +26,7 @@ module SvgConform
     # Also marks all descendants as invalid since they'll be removed with the parent
     def mark_node_structurally_invalid(node)
       node_id = generate_node_id(node)
-      return if node_id.nil?  # Safety check
+      return if node_id.nil? # Safety check
 
       @structurally_invalid_node_ids.add(node_id)
 
@@ -36,11 +36,13 @@ module SvgConform
 
     # Mark all descendants of a node as structurally invalid
     def mark_descendants_invalid(node)
-      return unless node.respond_to?(:children)
+      # In SAX mode, ElementProxy doesn't have children yet
+      # Children will be validated individually and will check parent validity
+      return unless node.respond_to?(:children) && node.children
 
       node.children.each do |child|
         child_id = generate_node_id(child)
-        return if child_id.nil?  # Safety check
+        next if child_id.nil? # Skip if can't generate ID
 
         @structurally_invalid_node_ids.add(child_id)
         # Recursively mark descendants
@@ -51,7 +53,7 @@ module SvgConform
     # Check if a node is structurally invalid
     def node_structurally_invalid?(node)
       node_id = generate_node_id(node)
-      return false if node_id.nil?  # Safety check
+      return false if node_id.nil? # Safety check
 
       @structurally_invalid_node_ids.include?(node_id)
     end
@@ -152,7 +154,7 @@ requirement_id: nil, severity: nil, fix: nil, data: {})
     # Populate cache for all nodes using document.traverse with parent tracking
     def populate_node_id_cache
       parent_stack = []
-      counter_stack = [{}]  # Stack of {element_name => count} hashes
+      counter_stack = [{}] # Stack of {element_name => count} hashes
 
       @document.traverse do |node|
         next unless node.respond_to?(:name) && node.name
@@ -161,7 +163,7 @@ requirement_id: nil, severity: nil, fix: nil, data: {})
         current_parent = node.respond_to?(:parent) ? node.parent : nil
 
         # Adjust stack based on actual parent
-        while parent_stack.size > 0 && !parent_stack.last.equal?(current_parent)
+        while parent_stack.size.positive? && !parent_stack.last.equal?(current_parent)
           parent_stack.pop
           counter_stack.pop
         end
@@ -187,7 +189,7 @@ requirement_id: nil, severity: nil, fix: nil, data: {})
       return unless node.respond_to?(:name) && node.name
 
       # Increment counter for this node name at current level
-     sibling_counters[node.name] ||= 0
+      sibling_counters[node.name] ||= 0
       sibling_counters[node.name] += 1
       position = sibling_counters[node.name]
 

data/lib/svg_conform/validator.rb

@@ -9,7 +9,7 @@ module SvgConform
       @options = {
         fix: false,
         strict: false,
-        mode: :auto  # :auto, :dom, or :sax
+        mode: :sax, # Testing SAX mode
       }.merge(options)
     end
 
@@ -152,7 +152,7 @@ module SvgConform
         warnings: [],
         file_path: file_path,
         error?: true,
-        to_s: -> { "Error processing #{file_path}: #{error.message}" }
+        to_s: -> { "Error processing #{file_path}: #{error.message}" },
       )
     end
   end

data/lib/svg_conform/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module SvgConform
-  VERSION = "0.1.2"
+  VERSION = "0.1.3"
 end

data/spec/spec_helper.rb

@@ -3,6 +3,9 @@
 require "svg_conform"
 require "canon"
 
+# Load shared examples
+Dir[File.expand_path("support/**/*.rb", __dir__)].each { |f| require f }
+
 RSpec.configure do |config|
   # Enable flags like --only-failures and --next-failure
   config.example_status_persistence_file_path = ".rspec_status"

data/spec/support/shared_examples_for_validation_modes.rb

@@ -0,0 +1,71 @@
+# frozen_string_literal: true
+
+# Shared examples for testing both DOM and SAX validation modes
+RSpec.shared_examples "validates in both DOM and SAX modes" do |file_path, profile_name|
+  let(:file_path) { file_path }
+  let(:profile) { profile_name }
+
+  it "produces identical results in DOM and SAX modes" do
+    dom_result = SvgConform::Validator.new(mode: :dom).validate_file(file_path,
+                                                                     profile: profile)
+    sax_result = SvgConform::Validator.new(mode: :sax).validate_file(file_path,
+                                                                     profile: profile)
+
+    expect(sax_result.errors.size).to eq(dom_result.errors.size),
+                                      "SAX mode should produce same error count as DOM mode (DOM: #{dom_result.errors.size}, SAX: #{sax_result.errors.size})"
+
+    expect(sax_result.warnings.size).to eq(dom_result.warnings.size),
+                                        "SAX mode should produce same warning count as DOM mode"
+
+    expect(sax_result.valid?).to eq(dom_result.valid?),
+                                 "SAX mode should have same validity as DOM mode"
+  end
+
+  it "DOM mode validates correctly" do
+    result = SvgConform::Validator.new(mode: :dom).validate_file(file_path,
+                                                                 profile: profile)
+    expect(result).to be_a(SvgConform::ValidationResult)
+  end
+
+  it "SAX mode validates correctly" do
+    result = SvgConform::Validator.new(mode: :sax).validate_file(file_path,
+                                                                 profile: profile)
+    expect(result).to be_a(SvgConform::ValidationResult)
+  end
+
+  describe "performance comparison" do
+    it "SAX mode is faster than or equal to DOM mode" do
+      skip "Performance test - run manually for large files"
+
+      dom_time = Benchmark.realtime do
+        SvgConform::Validator.new(mode: :dom).validate_file(file_path,
+                                                            profile: profile)
+      end
+
+      sax_time = Benchmark.realtime do
+        SvgConform::Validator.new(mode: :sax).validate_file(file_path,
+                                                            profile: profile)
+      end
+
+      expect(sax_time).to be <= dom_time
+    end
+  end
+end
+
+RSpec.shared_examples "validates content in both modes" do |svg_content, profile_name|
+  let(:content) { svg_content }
+  let(:profile) { profile_name }
+
+  it "produces identical results for content validation" do
+    dom_result = SvgConform::Validator.new(mode: :dom).validate(content,
+                                                                profile: profile)
+    sax_result = SvgConform::Validator.new(mode: :sax).validate(content,
+                                                                profile: profile)
+
+    expect(sax_result.errors.size).to eq(dom_result.errors.size),
+                                      "SAX mode should produce same error count as DOM mode (DOM: #{dom_result.errors.size}, SAX: #{sax_result.errors.size})"
+
+    expect(sax_result.valid?).to eq(dom_result.valid?),
+                                 "SAX mode should have same validity as DOM mode"
+  end
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: svg_conform
 version: !ruby/object:Gem::Version
-  version: 0.1.2
+  version: 0.1.3
 platform: ruby
 authors:
 - Ribose
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-11-18 00:00:00.000000000 Z
+date: 2025-11-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: lutaml-model
@@ -372,6 +372,7 @@ files:
 - spec/fixtures/viewbox_required/inputs/missing_viewbox.svg
 - spec/fixtures/viewbox_required/repair/missing_viewbox.svg
 - spec/spec_helper.rb
+- spec/support/shared_examples_for_validation_modes.rb
 - spec/svg_conform/batch_report_spec.rb
 - spec/svg_conform/commands/check_command_spec.rb
 - spec/svg_conform/commands/profiles_command_spec.rb