svg_conform 0.1.0

data/spec/fixtures/svgcheck/repair/viewBox-height.svg.file

@@ -0,0 +1,4 @@
+<?xml version='1.0' encoding='utf-8'?>
+<svg xmlns="http://www.w3.org/2000/svg" height="100" datatype="foobar">
+  <circle cx="50" cy="50" r="40" stroke="black" stroke-width="3" fill="black"/>
+</svg>

data/spec/fixtures/svgcheck/repair/viewBox-height.svg.out

@@ -0,0 +1,3 @@
+Tests/viewBox-height.svg:1: The attribute viewBox is required on the root svg element
+Tests/viewBox-height.svg:2: The attribute 'fill' does not allow the value 'red', replaced with 'black'
+ERROR: File does not conform to SVG requirements

data/spec/fixtures/svgcheck/repair/viewBox-none.svg.code

@@ -0,0 +1 @@
+1

data/spec/fixtures/svgcheck/repair/viewBox-none.svg.file

@@ -0,0 +1,4 @@
+<?xml version='1.0' encoding='utf-8'?>
+<svg xmlns="http://www.w3.org/2000/svg" datatype="foobar">
+  <circle cx="50" cy="50" r="40" stroke="black" stroke-width="3" fill="black"/>
+</svg>

data/spec/fixtures/svgcheck/repair/viewBox-none.svg.out

@@ -0,0 +1,3 @@
+Tests/viewBox-none.svg:1: The attribute viewBox is required on the root svg element
+Tests/viewBox-none.svg:2: The attribute 'fill' does not allow the value 'red', replaced with 'black'
+ERROR: File does not conform to SVG requirements

data/spec/fixtures/svgcheck/repair/viewBox-width.svg.code

@@ -0,0 +1 @@
+1

data/spec/fixtures/svgcheck/repair/viewBox-width.svg.file

@@ -0,0 +1,4 @@
+<?xml version='1.0' encoding='utf-8'?>
+<svg xmlns="http://www.w3.org/2000/svg" width="100" datatype="foobar">
+  <circle cx="50" cy="50" r="40" stroke="black" stroke-width="3" fill="black"/>
+</svg>

data/spec/fixtures/svgcheck/repair/viewBox-width.svg.out

@@ -0,0 +1,3 @@
+Tests/viewBox-width.svg:1: The attribute viewBox is required on the root svg element
+Tests/viewBox-width.svg:2: The attribute 'fill' does not allow the value 'red', replaced with 'black'
+ERROR: File does not conform to SVG requirements

data/spec/fixtures/viewbox_required/inputs/missing_viewbox.svg

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<svg xmlns="http://www.w3.org/2000/svg" version="1.2" baseProfile="tiny"
+     width="100" height="80">
+  <title>ViewBox Required Test</title>
+  <desc>Testing viewBox requirement - root SVG missing viewBox</desc>
+
+  <rect x="10" y="10" width="80" height="60" fill="black" stroke="none"/>
+  <circle cx="50" cy="40" r="20" fill="white"/>
+  <text x="50" y="45" text-anchor="middle" font-family="serif">No ViewBox</text>
+</svg>

data/spec/fixtures/viewbox_required/repair/missing_viewbox.svg

@@ -0,0 +1,10 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<svg xmlns="http://www.w3.org/2000/svg" version="1.2" baseProfile="tiny"
+     width="100" height="80" viewBox="0 0 100 80">
+  <title>ViewBox Required Test</title>
+  <desc>Testing viewBox requirement - root SVG missing viewBox</desc>
+
+  <rect x="10" y="10" width="80" height="60" fill="black" stroke="none"/>
+  <circle cx="50" cy="40" r="20" fill="white"/>
+  <text x="50" y="45" text-anchor="middle" font-family="serif">No ViewBox</text>
+</svg>

data/spec/spec_helper.rb

@@ -0,0 +1,16 @@
+# frozen_string_literal: true
+
+require "svg_conform"
+require "canon"
+
+RSpec.configure do |config|
+  # Enable flags like --only-failures and --next-failure
+  config.example_status_persistence_file_path = ".rspec_status"
+
+  # Disable RSpec exposing methods globally on `Module` and `main`
+  config.disable_monkey_patching!
+
+  config.expect_with :rspec do |c|
+    c.syntax = :expect
+  end
+end

data/spec/svg_conform/batch_report_spec.rb

@@ -0,0 +1,99 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe SvgConform::BatchReport do
+  describe "initialization" do
+    it "creates a batch report with default values" do
+      report = described_class.new
+      expect(report.files).to eq([])
+      expect(report.manifest).to eq({})
+      expect(report.timestamp).not_to be_nil
+    end
+  end
+
+  describe "calculate_statistics" do
+    it "calculates statistics correctly" do
+      report = described_class.new
+      report.profile = "metanorma"
+
+      # Add some file results
+      3.times do |i|
+        file_result = SvgConform::FileResult.new
+        file_result.filename = "file#{i}.svg"
+        file_result.valid_before = false
+        file_result.valid_after = i < 2 # 2 out of 3 valid after
+        file_result.status = i < 2 ? "remediated" : "failed"
+        report.files << file_result
+      end
+
+      report.calculate_statistics
+
+      expect(report.total_files).to eq(3)
+      expect(report.valid_before).to eq(0)
+      expect(report.valid_after).to eq(2)
+      expect(report.remediated).to eq(2)
+      expect(report.failed).to eq(1)
+      expect(report.success_rate).to eq(66.7)
+    end
+  end
+
+  describe "serialization" do
+    it "serializes to JSON" do
+      report = described_class.new
+      report.profile = "metanorma"
+      report.total_files = 10
+
+      json = report.to_json
+      expect(json).to be_a(String)
+      expect(json).to include('"profile":"metanorma"')
+      expect(json).to include('"total_files":10')
+    end
+
+    it "serializes to YAML" do
+      report = described_class.new
+      report.profile = "metanorma"
+      report.total_files = 10
+
+      yaml = report.to_yaml
+      expect(yaml).to be_a(String)
+      expect(yaml).to include("profile: metanorma")
+      expect(yaml).to include("total_files: 10")
+    end
+  end
+end
+
+RSpec.describe SvgConform::FileResult do
+  describe "initialization" do
+    it "creates a file result" do
+      result = described_class.new
+      result.filename = "test.svg"
+      result.status = "valid"
+
+      expect(result.filename).to eq("test.svg")
+      expect(result.status).to eq("valid")
+    end
+  end
+
+  describe "serialization" do
+    it "serializes to JSON" do
+      result = described_class.new
+      result.filename = "test.svg"
+      result.status = "remediated"
+
+      json = result.to_json
+      expect(json).to include('"filename":"test.svg"')
+      expect(json).to include('"status":"remediated"')
+    end
+
+    it "serializes to YAML" do
+      result = described_class.new
+      result.filename = "test.svg"
+      result.status = "remediated"
+
+      yaml = result.to_yaml
+      expect(yaml).to include("filename: test.svg")
+      expect(yaml).to include("status: remediated")
+    end
+  end
+end

data/spec/svg_conform/commands/check_command_spec.rb

@@ -0,0 +1,90 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+require "tempfile"
+
+RSpec.describe SvgConform::Commands::Check do
+  let(:valid_svg) do
+    <<~SVG
+      <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">
+        <rect x="10" y="10" width="50" height="50" fill="red"/>
+      </svg>
+    SVG
+  end
+
+  let(:invalid_svg) do
+    <<~SVG
+      <svg width="100" height="100">
+        <rect x="10" y="10" width="50" height="50" fill="red"/>
+      </svg>
+    SVG
+  end
+
+  describe "#execute" do
+    it "validates a file and returns status" do
+      tempfile = Tempfile.new(["test", ".svg"])
+      tempfile.write(valid_svg)
+      tempfile.close
+
+      options = { profile: "svg_1_2_rfc", format: "table" }
+      command = described_class.new(tempfile.path, options)
+
+      expect { command.execute }.to output(/SVG Validation Report/).to_stdout
+        .and raise_error(SystemExit)
+
+      tempfile.unlink
+    end
+
+    it "reports errors for invalid files" do
+      tempfile = Tempfile.new(["test", ".svg"])
+      tempfile.write(invalid_svg)
+      tempfile.close
+
+      options = { profile: "svg_1_2_rfc", format: "table" }
+      command = described_class.new(tempfile.path, options)
+
+      expect { command.execute }.to output(/Validation Errors/).to_stdout
+        .and raise_error(SystemExit)
+
+      tempfile.unlink
+    end
+
+    it "supports YAML output format" do
+      tempfile = Tempfile.new(["test", ".svg"])
+      tempfile.write(valid_svg)
+      tempfile.close
+
+      options = { profile: "svg_1_2_rfc", format: "yaml" }
+      command = described_class.new(tempfile.path, options)
+
+      expect { command.execute }.to output(/---/).to_stdout
+        .and raise_error(SystemExit)
+
+      tempfile.unlink
+    end
+
+    it "supports JSON output format" do
+      tempfile = Tempfile.new(["test", ".svg"])
+      tempfile.write(valid_svg)
+      tempfile.close
+
+      options = { profile: "svg_1_2_rfc", format: "json" }
+      command = described_class.new(tempfile.path, options)
+
+      expect { command.execute }.to output(/\{/).to_stdout
+        .and raise_error(SystemExit)
+
+      tempfile.unlink
+    end
+
+    it "handles missing files gracefully" do
+      options = { profile: "svg_1_2_rfc", format: "table" }
+      command = described_class.new("nonexistent.svg", options)
+
+      expect do
+        command.execute
+      end.to output(/Error: File 'nonexistent.svg' not found/).to_stdout
+        .and raise_error(SystemExit)
+    end
+  end
+end

data/spec/svg_conform/commands/profiles_command_spec.rb

@@ -0,0 +1,20 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe SvgConform::Commands::Profiles do
+  describe "#execute" do
+    it "initializes with options" do
+      options = {}
+      command = described_class.new(options)
+      expect(command).to be_a(described_class)
+    end
+
+    it "lists available profiles" do
+      options = {}
+      command = described_class.new(options)
+
+      expect { command.execute }.to output(/Available SVG Profiles/).to_stdout
+    end
+  end
+end

data/spec/svg_conform/commands/svgcheck_compare_command_spec.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe SvgConform::Commands::SvgcheckCompare do
+  describe "#execute" do
+    it "initializes with file and options" do
+      options = { profile: "svg_1_2_rfc" }
+      command = described_class.new("test.svg", options)
+      expect(command).to be_a(described_class)
+    end
+  end
+end

data/spec/svg_conform/commands/svgcheck_compatibility_command_spec.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe SvgConform::Commands::SvgcheckCompatibility do
+  describe "#execute" do
+    it "initializes with options" do
+      options = {}
+      command = described_class.new(options)
+      expect(command).to be_a(described_class)
+    end
+  end
+end

data/spec/svg_conform/commands/svgcheck_generate_command_spec.rb

@@ -0,0 +1,14 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe SvgConform::Commands::SvgcheckGenerate do
+  describe "#execute" do
+    it "initializes with svgcheck_repo_path and options" do
+      svgcheck_repo_path = "svgcheck-reference"
+      options = { single_file: "test.svg" }
+      command = described_class.new(svgcheck_repo_path, options)
+      expect(command).to be_a(described_class)
+    end
+  end
+end

data/spec/svg_conform/profiles/base_profile_spec.rb

@@ -0,0 +1,42 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe "Base Profile" do
+  let(:base_profile_path) { "config/profiles/base.yml" }
+
+  describe "profile loading" do
+    it "loads the base profile successfully" do
+      profile = SvgConform::Profile.load_from_file(base_profile_path)
+      expect(profile).not_to be_nil
+      expect(profile.name).to eq("base")
+    end
+
+    it "has minimal requirements" do
+      profile = SvgConform::Profile.load_from_file(base_profile_path)
+      expect(profile.requirements).to be_an(Array)
+    end
+
+    it "serves as a foundation for other profiles" do
+      profile = SvgConform::Profile.load_from_file(base_profile_path)
+      expect(profile.description).to be_a(String)
+    end
+  end
+
+  describe "validation" do
+    it "can validate a simple SVG document" do
+      svg_content = <<~SVG
+        <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">
+          <rect x="10" y="10" width="50" height="50" fill="red"/>
+        </svg>
+      SVG
+
+      document = SvgConform::Document.from_content(svg_content)
+      profile = SvgConform::Profile.load_from_file(base_profile_path)
+
+      result = profile.validate(document)
+      expect(result).to respond_to(:valid?)
+      expect(result).to respond_to(:errors)
+    end
+  end
+end

data/spec/svg_conform/profiles/lucid_fix_profile_spec.rb

@@ -0,0 +1,46 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe "Lucid Fix Profile" do
+  let(:lucid_fix_profile_path) { "config/profiles/lucid_fix.yml" }
+
+  describe "profile loading" do
+    it "loads the lucid fix profile successfully" do
+      profile = SvgConform::Profile.load_from_file(lucid_fix_profile_path)
+      expect(profile).not_to be_nil
+      expect(profile.name).to eq("lucid_fix")
+    end
+
+    it "includes requirements specific to Lucid diagrams" do
+      profile = SvgConform::Profile.load_from_file(lucid_fix_profile_path)
+      expect(profile.requirements).to be_an(Array)
+      expect(profile.requirements).not_to be_empty
+    end
+
+    it "has description indicating Lucid-specific fixes" do
+      profile = SvgConform::Profile.load_from_file(lucid_fix_profile_path)
+      expect(profile.description).to be_a(String)
+    end
+  end
+
+  describe "validation with lucid fixtures" do
+    let(:lucid_fixtures_dir) { "spec/fixtures/lucid" }
+
+    it "handles lucid-specific SVG patterns" do
+      skip "Lucid fixtures not found" unless Dir.exist?(lucid_fixtures_dir)
+
+      input_files = Dir.glob(File.join(lucid_fixtures_dir, "inputs/*.svg"))
+      skip "No lucid input files found" if input_files.empty?
+
+      profile = SvgConform::Profile.load_from_file(lucid_fix_profile_path)
+      input_file = input_files.first
+
+      document = SvgConform::Document.from_file(input_file)
+      result = profile.validate(document)
+
+      expect(result).to respond_to(:valid?)
+      expect(result).to respond_to(:errors)
+    end
+  end
+end

data/spec/svg_conform/profiles/lucid_profile_spec.rb

@@ -0,0 +1,84 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+require "canon"
+require "tempfile"
+
+RSpec.describe "Lucid Profile" do
+  describe "namespace attribute remediation" do
+    let(:lucid_profile_path) { "config/profiles/lucid_fix.yml" }
+    let(:fixtures_dir) { "spec/fixtures/lucid" }
+
+    # Get all fixture files that have both input and expected repair versions
+    fixture_files = Dir.glob("spec/fixtures/lucid/inputs/*.svg").select do |input_file|
+      basename = File.basename(input_file)
+      File.exist?("spec/fixtures/lucid/repair/#{basename}")
+    end.map do |f|
+      File.basename(f, ".svg")
+    end
+
+    fixture_files.each do |fixture_name|
+      describe "fixture: #{fixture_name}" do
+        let(:input_file) { "spec/fixtures/lucid/inputs/#{fixture_name}.svg" }
+        let(:expected_output_file) do
+          "spec/fixtures/lucid/repair/#{fixture_name}.svg"
+        end
+
+        it "produces canonically equivalent output using profile remediation" do
+          skip "Expected repair file not found" unless File.exist?(expected_output_file)
+
+          # Load the document and apply profile remediations
+          document = SvgConform::Document.from_file(input_file)
+          profile = SvgConform::Profile.load_from_file(lucid_profile_path)
+
+          changes = profile.apply_remediations(document)
+
+          # Verify namespace remediation changes were made
+          namespace_changes = changes.select do |c|
+            c[:description]&.include?("namespace attribute")
+          end
+          expect(namespace_changes).not_to be_empty
+
+          # Get the remediated XML
+          actual_xml = document.to_xml
+          expected_xml = File.read(expected_output_file)
+
+          # Use canon matcher for XML equivalence
+          expect(actual_xml).to be_analogous_with(expected_xml)
+        end
+
+        it "removes lucid namespace attributes correctly" do
+          # Load the document and profile
+          document = SvgConform::Document.from_file(input_file)
+          profile = SvgConform::Profile.load_from_file(lucid_profile_path)
+
+          # Check that the document contains lucid attributes before remediation
+          original_xml = document.to_xml
+          expect(original_xml).to include("lucid:page-tab-id")
+          expect(original_xml).to include('xmlns:lucid="lucid"')
+
+          # Apply profile remediations
+          changes = profile.apply_remediations(document)
+
+          # Verify changes were made
+          expect(changes).not_to be_empty
+
+          # Find namespace attribute changes
+          attr_changes = changes.select do |c|
+            c[:description]&.include?("attribute")
+          end
+          expect(attr_changes).not_to be_empty
+
+          # Verify the result doesn't contain the lucid page-tab-id attribute
+          actual_xml = document.to_xml
+          expect(actual_xml).not_to include("lucid:page-tab-id")
+
+          # The namespace declaration removal is the main functionality we want to test
+          # For now, let's verify the attribute removal is working
+          puts "Namespace attribute remediation is working - removed: #{attr_changes.size} attributes"
+          puts "xmlns:lucid still present: #{actual_xml.include?('xmlns:lucid=\"lucid\"')}"
+        end
+      end
+    end
+  end
+end

data/spec/svg_conform/profiles/metanorma_profile_spec.rb

@@ -0,0 +1,62 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe "Metanorma Profile" do
+  let(:metanorma_profile_path) { "config/profiles/metanorma.yml" }
+
+  describe "profile loading" do
+    it "loads the Metanorma profile successfully" do
+      profile = SvgConform::Profile.load_from_file(metanorma_profile_path)
+      expect(profile).not_to be_nil
+      expect(profile.name).to eq("metanorma")
+    end
+
+    it "includes external resource requirements" do
+      profile = SvgConform::Profile.load_from_file(metanorma_profile_path)
+      requirement_ids = profile.requirements.map(&:id)
+      expect(requirement_ids).to include("no_external_css")
+      expect(requirement_ids).to include("no_external_fonts")
+      expect(requirement_ids).to include("no_external_images")
+    end
+
+    it "includes comprehensive element support" do
+      profile = SvgConform::Profile.load_from_file(metanorma_profile_path)
+      requirement_ids = profile.requirements.map(&:id)
+      expect(requirement_ids).to include("allowed_elements")
+    end
+  end
+
+  describe "validation" do
+    it "allows flexible colors unlike RFC 7996" do
+      svg_with_colors = <<~SVG
+        <svg xmlns="http://www.w3.org/2000/svg" version="1.2" baseProfile="tiny" viewBox="0 0 100 100">
+          <rect x="10" y="10" width="50" height="50" fill="#ff0000"/>
+        </svg>
+      SVG
+
+      document = SvgConform::Document.from_content(svg_with_colors)
+      profile = SvgConform::Profile.load_from_file(metanorma_profile_path)
+
+      result = profile.validate(document)
+      color_errors = result.errors.select do |e|
+        e.requirement_id == "color_restrictions"
+      end
+      expect(color_errors).to be_empty
+    end
+
+    it "supports image elements" do
+      svg_with_image = <<~SVG
+        <svg xmlns="http://www.w3.org/2000/svg" version="1.2" baseProfile="tiny" viewBox="0 0 100 100">
+          <image href="data:image/png;base64,abc" width="50" height="50"/>
+        </svg>
+      SVG
+
+      document = SvgConform::Document.from_content(svg_with_image)
+      profile = SvgConform::Profile.load_from_file(metanorma_profile_path)
+
+      result = profile.validate(document)
+      expect(result).to respond_to(:valid?)
+    end
+  end
+end

data/spec/svg_conform/profiles/no_external_css_profile_spec.rb

@@ -0,0 +1,66 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe "No External CSS Profile" do
+  let(:no_external_css_profile_path) { "config/profiles/no_external_css.yml" }
+
+  describe "profile loading" do
+    it "loads the no external css profile successfully" do
+      profile = SvgConform::Profile.load_from_file(no_external_css_profile_path)
+      expect(profile).not_to be_nil
+      expect(profile.name).to eq("no_external_css")
+    end
+
+    it "includes no_external_css requirement" do
+      profile = SvgConform::Profile.load_from_file(no_external_css_profile_path)
+      requirement_ids = profile.requirements.map(&:id)
+      expect(requirement_ids).to include("no_external_css")
+    end
+
+    it "has description indicating external CSS restriction" do
+      profile = SvgConform::Profile.load_from_file(no_external_css_profile_path)
+      expect(profile.description).to be_a(String)
+      expect(profile.description.downcase).to match(/external.*css/)
+    end
+  end
+
+  describe "validation" do
+    it "rejects SVG with external CSS references" do
+      svg_with_external_css = <<~SVG
+        <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">
+          <defs>
+            <style>
+              @import url("external.css");
+            </style>
+          </defs>
+          <rect x="10" y="10" width="50" height="50"/>
+        </svg>
+      SVG
+
+      document = SvgConform::Document.from_content(svg_with_external_css)
+      profile = SvgConform::Profile.load_from_file(no_external_css_profile_path)
+
+      result = profile.validate(document)
+      expect(result.valid?).to be false
+      expect(result.errors.map(&:requirement_id)).to include("no_external_css")
+    end
+
+    it "accepts SVG with inline styles" do
+      svg_with_inline_styles = <<~SVG
+        <svg xmlns="http://www.w3.org/2000/svg" viewBox="0 0 100 100">
+          <rect x="10" y="10" width="50" height="50" style="fill:red"/>
+        </svg>
+      SVG
+
+      document = SvgConform::Document.from_content(svg_with_inline_styles)
+      profile = SvgConform::Profile.load_from_file(no_external_css_profile_path)
+
+      result = profile.validate(document)
+      external_css_errors = result.errors.select do |e|
+        e.requirement_id == "no_external_css"
+      end
+      expect(external_css_errors).to be_empty
+    end
+  end
+end