metanorma-standoc 2.2.2.2 → 2.2.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 99fc70f6e9b8781722b3deed8143fabbadf28dfdda77f88e4ae9f7bfc3b83faf
-  data.tar.gz: 51305f5baa732cbb3b9df0ed6640413df408930365ef3ce8f84c16cad3337288
+  metadata.gz: 91681e7073cc074159e4aab0c10043fd86c8cffeba1c1fef9b1db85d11894a42
+  data.tar.gz: 0b6047784c5a7ba431ddf6ddbdfd6fb6042289b0947eaa15563a5c3d9e731fba
 SHA512:
-  metadata.gz: b757363df3d4c399b067478880534eb0fad2b8cd166e4e0d0e33c138bb599f8f7ac8d017bbbe7500d59ee5f161ff6e43b0205362a6fcb0ba6d0dfa7444dbf7bd
-  data.tar.gz: add6f5d5626ee90828655bdf4feeb47091e362982dfe39372d0c85ff33efa21482b0f79ee4613b19ab438aae260c334bb3e15a3f3bf3eff2f80ed5486e888345
+  metadata.gz: 68dcf12b8ba888e074970b6cea1918ec8223dd9ae5297a3bac0f337bed63731a8bf1579028915dfa07ce0ec7924677c603c81427b3bccb99640b07f2e148da19
+  data.tar.gz: 89458b82d0111581cc81a9cb19ad1c6dcecbdf64916c958e729d4a59424ab61f5c4ab9b7ac73ecbcab242e1e41624e28335bf94edcd7dd50dc80a6f74fd1245e

data/lib/metanorma/standoc/validate.rb

@@ -5,7 +5,6 @@ require "nokogiri"
 require "jing"
 require "iev"
 require "pngcheck"
-require "png"
 
 module Metanorma
   module Standoc
@@ -160,18 +159,46 @@ module Metanorma
       end
 
       def image_validate(doc)
+        image_exists(doc)
+        png_validate(doc)
+      end
+
+      def image_exists(doc)
+        doc.xpath("//image").each do |i|
+          Metanorma::Utils::url?(i["src"]) and next
+          Metanorma::Utils::datauri?(i["src"]) and next
+          expand_path(i["src"]) and next
+          @log.add("Images", i.parent,
+                   "Image not found: #{i['src']}")
+          @fatalerror << "Image not found: #{i['src']}"
+        end
+      end
+
+      def expand_path(loc)
+        relative_path = File.join(@localdir, loc)
+        [loc, relative_path].detect do |p|
+          File.exist?(p) ? p : nil
+        end
+      end
+
+      def png_validate(doc)
         doc.xpath("//image[@mimetype = 'image/png']").each do |i|
-          d = Metanorma::Utils::datauri(i["src"], @localdir)
-          # d.chars.to_a.each_slice(80).to_a.map { |s| s.join }.each { |s| warn s }
-          png_validate1(i, Base64.strict_decode64(d.sub(/^.+?base64,/, "")))
+          Metanorma::Utils::url?(i["src"]) and next
+          decoded = if Metanorma::Utils::datauri?(i["src"])
+                      Metanorma::Utils::decode_datauri(i["src"])[:data]
+                    else
+                      path = expand_path(i["src"]) or next
+                      File.binread(path)
+                    end
+          png_validate1(i, decoded)
         end
       end
 
       def png_validate1(img, buffer)
         PngCheck.check_buffer(buffer)
       rescue PngCheck::CorruptPngError => e
-        @log.add("Images", img.parent, "Corrupt PNG image")
-        @fatalerror << "Exception #{e.message}"
+        @log.add("Images", img.parent,
+                 "Corrupt PNG image detected: #{e.message}")
       end
 
       def validate(doc)

data/lib/metanorma/standoc/version.rb

@@ -19,6 +19,6 @@ module Metanorma
   end
 
   module Standoc
-    VERSION = "2.2.2.2".freeze
+    VERSION = "2.2.3".freeze
   end
 end

data/lib/metanorma-standoc.rb

@@ -3,6 +3,7 @@ require_relative "isodoc/pdf_convert"
 require_relative "metanorma/standoc/converter"
 require_relative "metanorma/standoc/version"
 require "asciidoctor/extensions"
+require "metanorma"
 
 if defined? Metanorma::Registry
   require_relative "metanorma/standoc"

data/metanorma-standoc.gemspec

@@ -38,7 +38,6 @@ Gem::Specification.new do |spec|
   spec.add_dependency "latexmath"
   spec.add_dependency "mathml2asciimath"
   spec.add_dependency "mn-requirements", "~> 0.1.3"
-  spec.add_dependency "png"
   spec.add_dependency "pngcheck"
   spec.add_dependency "relaton-cli", "~> 1.13.0"
   spec.add_dependency "relaton-iev", "~> 1.1.0"

data/spec/metanorma/validate_spec.rb

@@ -1,5 +1,4 @@
 require "spec_helper"
-require "relaton_iec"
 require "fileutils"
 
 RSpec.describe Metanorma::Standoc do
@@ -783,34 +782,51 @@ RSpec.describe Metanorma::Standoc do
       .not_to include "Symbol reference in `symbol[xyz]` missing:"
   end
 
-  it "warns and aborts if corrupt PNG" do
+  it "warns if corrupt PNG" do
     FileUtils.rm_f "test.xml"
     FileUtils.rm_f "test.err"
-    begin
-      input = <<~INPUT
-        = Document title
-        Author
-        :docfile: test.adoc
-        :no-pdf:
 
-        == Clause
-        image::spec/assets/corrupt.png[]
+    input = <<~INPUT
+      = Document title
+      Author
+      :docfile: test.adoc
+      :no-pdf:
 
-      INPUT
-      expect do
-        Asciidoctor.convert(input, *OPTIONS)
-      end.to raise_error(SystemExit)
-    rescue SystemExit, RuntimeError
-    end
-    warn File.read("test.err")
+      == Clause
+      image::spec/assets/corrupt.png[]
+
+    INPUT
+    Asciidoctor.convert(input, *OPTIONS)
     expect(File.read("test.err"))
       .to include "Corrupt PNG image"
-    expect(File.exist?("test.xml")).to be false
+    expect(File.exist?("test.xml")).to be true
+  end
+
+  it "does not warn if not corrupt PNG" do
+    FileUtils.rm_f "test.xml"
+    FileUtils.rm_f "test.err"
+
+    input = <<~INPUT
+      = Document title
+      Author
+      :docfile: test.adoc
+      :no-pdf:
+
+      == Clause
+      image::spec/assets/correct.png[]
+
+    INPUT
+
+    Asciidoctor.convert(input, *OPTIONS)
+    expect(File.read("test.err"))
+      .not_to include "Corrupt PNG image"
+    expect(File.exist?("test.xml")).to be true
   end
 
-  it "does not warn and abort if not corrupt PNG" do
+  it "warns and aborts if images does not exist" do
     FileUtils.rm_f "test.xml"
     FileUtils.rm_f "test.err"
+
     begin
       input = <<~INPUT
         = Document title
@@ -819,16 +835,19 @@ RSpec.describe Metanorma::Standoc do
         :no-pdf:
 
         == Clause
-        image::spec/assets/correct.png[]
+        image::spec/assets/nonexistent.png[]
 
       INPUT
+
       expect do
         Asciidoctor.convert(input, *OPTIONS)
-      end.not_to raise_error
-    rescue SystemExit, RuntimeError
+      end.to raise_error(SystemExit)
+    rescue SystemExit
     end
+
+    warn File.read("test.err")
     expect(File.read("test.err"))
-      .not_to include "Corrupt PNG image"
-    expect(File.exist?("test.xml")).to be true
+      .to include "Image not found"
+    expect(File.exist?("test.xml")).to be false
   end
 end