prawn-svg 0.34.2 → 0.35.0

data/spec/prawn/svg/document_spec.rb

@@ -1,15 +1,15 @@
-require File.dirname(__FILE__) + '/../../spec_helper'
+require "#{File.dirname(__FILE__)}/../../spec_helper"
 
 describe Prawn::SVG::Document do
   let(:bounds) { [100, 100] }
   let(:options) { {} }
 
-  describe "#initialize" do
-    context "with a well-formed document" do
-      let(:svg) { "<svg></svg>" }
-      let(:options) { {color_mode: :cmyk} }
+  describe '#initialize' do
+    context 'with a well-formed document' do
+      let(:svg) { '<svg></svg>' }
+      let(:options) { { color_mode: :cmyk } }
 
-      it "parses the XML and yields itself to its block" do
+      it 'parses the XML and yields itself to its block' do
         yielded = nil
 
         document = Prawn::SVG::Document.new(svg, bounds, options) do |doc|
@@ -22,23 +22,24 @@ describe Prawn::SVG::Document do
       end
     end
 
-    context "when unparsable XML is provided" do
+    context 'when unparsable XML is provided' do
       let(:svg) { "this isn't SVG data" }
 
-      it "raises an exception" do
-        expect {
+      it 'raises an exception' do
+        expect do
           Prawn::SVG::Document.new(svg, bounds, options)
-        }.to raise_error Prawn::SVG::Document::InvalidSVGData, "The data supplied is not a valid SVG document."
+        end.to raise_error Prawn::SVG::Document::InvalidSVGData, 'The data supplied is not a valid SVG document.'
       end
     end
 
-    context "when the user passes in a filename instead of SVG data" do
-      let(:svg) { "some_file.svg" }
+    context 'when the user passes in a filename instead of SVG data' do
+      let(:svg) { 'some_file.svg' }
 
       it "raises an exception letting them know what they've done" do
-        expect {
+        expect do
           Prawn::SVG::Document.new(svg, bounds, options)
-        }.to raise_error Prawn::SVG::Document::InvalidSVGData, "The data supplied is not a valid SVG document.  It looks like you've supplied a filename instead; use IO.read(filename) to get the data before you pass it to prawn-svg."
+        end.to raise_error Prawn::SVG::Document::InvalidSVGData,
+          "The data supplied is not a valid SVG document.  It looks like you've supplied a filename instead; use IO.read(filename) to get the data before you pass it to prawn-svg."
       end
     end
   end

data/spec/prawn/svg/elements/base_spec.rb

@@ -1,104 +1,107 @@
 require 'spec_helper'
 
 describe Prawn::SVG::Elements::Base do
-  let(:svg) { "<svg></svg>" }
-  let(:document) { Prawn::SVG::Document.new(svg, [800, 600], {}, font_registry: Prawn::SVG::FontRegistry.new("Helvetica" => {:normal => nil})) }
+  let(:svg) { '<svg></svg>' }
+  let(:document) do
+    Prawn::SVG::Document.new(svg, [800, 600], {},
+      font_registry: Prawn::SVG::FontRegistry.new('Helvetica' => { normal: nil }))
+  end
   let(:parent_calls) { [] }
   let(:element) { Prawn::SVG::Elements::Base.new(document, document.root, parent_calls, fake_state) }
 
-  describe "#initialize" do
+  describe '#initialize' do
     let(:svg) { '<something id="hello"/>' }
 
-    it "adds itself to the elements_by_id hash if an id attribute is supplied" do
+    it 'adds itself to the elements_by_id hash if an id attribute is supplied' do
       element
-      expect(document.elements_by_id["hello"]).to eq element
+      expect(document.elements_by_id['hello']).to eq element
     end
   end
 
-  describe "#process" do
-    it "calls #parse and #apply so subclasses can parse the element" do
+  describe '#process' do
+    it 'calls #parse and #apply so subclasses can parse the element' do
       expect(element).to receive(:parse).ordered
       expect(element).to receive(:apply).ordered
       element.process
     end
 
-    describe "applying calls from the standard attributes" do
+    describe 'applying calls from the standard attributes' do
       let(:svg) do
         <<-SVG
           <something transform="scale(2)" fill-opacity="0.5" fill="red" stroke="blue" stroke-width="5"/>
         SVG
       end
 
-      it "appends the relevant calls" do
+      it 'appends the relevant calls' do
         element.process
         expect(element.base_calls).to eq [
-          ["transformation_matrix", [2, 0, 0, 2, 0, 0], {}, [
-            ["transparent", [0.5, 1], {}, [
-              ["fill_color", ["ff0000"], {}, []],
-              ["stroke_color", ["0000ff"], {}, []],
-              ["line_width", [5.0], {}, []],
-              ["fill_and_stroke", [], {}, []]
+          ['transformation_matrix', [2, 0, 0, 2, 0, 0], {}, [
+            ['transparent', [0.5, 1], {}, [
+              ['fill_color', ['ff0000'], {}, []],
+              ['stroke_color', ['0000ff'], {}, []],
+              ['line_width', [5.0], {}, []],
+              ['fill_and_stroke', [], {}, []]
             ]]
           ]]
         ]
       end
     end
 
-    describe "fills and strokes" do
+    describe 'fills and strokes' do
       before { element.process }
       subject { element.base_calls.last }
 
-      context "with neither fill nor stroke" do
+      context 'with neither fill nor stroke' do
         let(:svg) { '<rect style="fill: none;"></rect>' }
         it        { is_expected.to eq ['end_path', [], {}, []] }
       end
 
-      context "with a fill only" do
+      context 'with a fill only' do
         let(:svg) { '<rect style="fill: black;"></rect>' }
         it        { is_expected.to eq ['fill', [], {}, []] }
       end
 
-      context "with a stroke only" do
+      context 'with a stroke only' do
         let(:svg) { '<rect style="fill: none; stroke: black;"></rect>' }
         it        { is_expected.to eq ['stroke', [], {}, []] }
       end
 
-      context "with fill and stroke" do
+      context 'with fill and stroke' do
         let(:svg) { '<rect style="fill: black; stroke: black;"></rect>' }
         it        { is_expected.to eq ['fill_and_stroke', [], {}, []] }
       end
 
-      context "with fill with evenodd fill rule" do
+      context 'with fill with evenodd fill rule' do
         let(:svg) { '<rect style="fill: black; fill-rule: evenodd;"></rect>' }
-        it        { is_expected.to eq ['fill', [], {fill_rule: :even_odd}, []] }
+        it        { is_expected.to eq ['fill', [], { fill_rule: :even_odd }, []] }
       end
     end
 
-    it "appends calls to the parent element" do
+    it 'appends calls to the parent element' do
       expect(element).to receive(:apply) do
-        element.send :add_call, "test", "argument", kw: 'argument'
+        element.send :add_call, 'test', 'argument', kw: 'argument'
       end
 
       element.process
-      expect(element.parent_calls).to eq [["fill", [], {}, [["test", ["argument"], {kw: 'argument'}, []]]]]
+      expect(element.parent_calls).to eq [['fill', [], {}, [['test', ['argument'], { kw: 'argument' }, []]]]]
     end
 
-    it "quietly absorbs a SkipElementQuietly exception" do
+    it 'quietly absorbs a SkipElementQuietly exception' do
       expect(element).to receive(:parse).and_raise(Prawn::SVG::Elements::Base::SkipElementQuietly)
       expect(element).to_not receive(:apply)
       element.process
       expect(document.warnings).to be_empty
     end
 
-    it "absorbs a SkipElementError exception, logging a warning" do
-      expect(element).to receive(:parse).and_raise(Prawn::SVG::Elements::Base::SkipElementError, "hello")
+    it 'absorbs a SkipElementError exception, logging a warning' do
+      expect(element).to receive(:parse).and_raise(Prawn::SVG::Elements::Base::SkipElementError, 'hello')
       expect(element).to_not receive(:apply)
       element.process
-      expect(document.warnings).to eq ["hello"]
+      expect(document.warnings).to eq ['hello']
     end
   end
 
-  describe "#apply_colors" do
+  describe '#apply_colors' do
     before do
       element.send(:extract_attributes_and_properties)
     end
@@ -141,8 +144,10 @@ describe Prawn::SVG::Elements::Base do
       subject
     end
 
-    context "with a color attribute defined on a parent element" do
-      let(:svg) { '<svg style="color: green;"><g style="color: red;"><rect width="10" height="10" style="fill: currentColor;"></rect></g></svg>' }
+    context 'with a color attribute defined on a parent element' do
+      let(:svg) do
+        '<svg style="color: green;"><g style="color: red;"><rect width="10" height="10" style="fill: currentColor;"></rect></g></svg>'
+      end
       let(:element) { Prawn::SVG::Elements::Root.new(document, document.root, parent_calls) }
       let(:flattened_calls) { flatten_calls(element.base_calls) }
 
@@ -162,7 +167,7 @@ describe Prawn::SVG::Elements::Base do
     end
   end
 
-  describe "stylesheets" do
+  describe 'stylesheets' do
     let(:svg) { <<-SVG }
       <svg>
         <style>
@@ -177,7 +182,7 @@ describe Prawn::SVG::Elements::Base do
       </svg>
     SVG
 
-    it "applies stylesheet styling but style attributes take precedence" do
+    it 'applies stylesheet styling but style attributes take precedence' do
       document = Prawn::SVG::Document.new(svg, [100, 100], {})
       calls = []
       element = Prawn::SVG::Elements::Root.new(document, document.root, calls)

data/spec/prawn/svg/elements/gradient_spec.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Prawn::SVG::Elements::Gradient do
-  let(:document) { Prawn::SVG::Document.new(svg, [800, 600], {width: 800, height: 600}) }
+  let(:document) { Prawn::SVG::Document.new(svg, [800, 600], { width: 800, height: 600 }) }
   let(:element)  { Prawn::SVG::Elements::Gradient.new(document, document.root, [], fake_state) }
 
   before do
@@ -9,7 +9,7 @@ describe Prawn::SVG::Elements::Gradient do
     element.process
   end
 
-  describe "object bounding box with linear gradient" do
+  describe 'object bounding box with linear gradient' do
     let(:svg) do
       <<-SVG
         <linearGradient id="flag" x1="0" x2="0.2" y1="0" y2="1">
@@ -20,17 +20,17 @@ describe Prawn::SVG::Elements::Gradient do
       SVG
     end
 
-    it "is stored in the document gradients table" do
-      expect(document.gradients["flag"]).to eq element
+    it 'is stored in the document gradients table' do
+      expect(document.gradients['flag']).to eq element
     end
 
-    it "returns correct gradient arguments for an element" do
+    it 'returns correct gradient arguments for an element' do
       arguments = element.gradient_arguments(double(bounding_box: [100, 100, 200, 0]))
       expect(arguments).to eq(
-        from: [100.0, 100.0],
-        to:   [120.0, 0.0],
-        stops: [[0, "ff0000"], [0.25, "ff0000"], [0.5, "ffffff"], [0.75, "0000ff"], [1, "0000ff"]],
-        apply_transformations: true,
+        from:                  [100.0, 100.0],
+        to:                    [120.0, 0.0],
+        stops:                 [[0, 'ff0000'], [0.25, 'ff0000'], [0.5, 'ffffff'], [0.75, '0000ff'], [1, '0000ff']],
+        apply_transformations: true
       )
     end
 
@@ -40,7 +40,7 @@ describe Prawn::SVG::Elements::Gradient do
     end
   end
 
-  describe "object bounding box with radial gradient" do
+  describe 'object bounding box with radial gradient' do
     let(:svg) do
       <<-SVG
         <radialGradient id="flag" cx="0" cy="0.2" fx="0.5" r="0.8">
@@ -51,24 +51,24 @@ describe Prawn::SVG::Elements::Gradient do
       SVG
     end
 
-    it "is stored in the document gradients table" do
-      expect(document.gradients["flag"]).to eq element
+    it 'is stored in the document gradients table' do
+      expect(document.gradients['flag']).to eq element
     end
 
-    it "returns correct gradient arguments for an element" do
+    it 'returns correct gradient arguments for an element' do
       arguments = element.gradient_arguments(double(bounding_box: [100, 100, 200, 0]))
       expect(arguments).to eq(
-        from: [150, 80],
-        to:   [100, 80],
-        r1:   0,
-        r2:   Math.sqrt((0.8 * 100) ** 2 + (0.8 * 100) ** 2),
-        stops: [[0, "ff0000"], [0.25, "ff0000"], [0.5, "ffffff"], [0.75, "0000ff"], [1, "0000ff"]],
-        apply_transformations: true,
+        from:                  [150, 80],
+        to:                    [100, 80],
+        r1:                    0,
+        r2:                    Math.sqrt(((0.8 * 100)**2) + ((0.8 * 100)**2)),
+        stops:                 [[0, 'ff0000'], [0.25, 'ff0000'], [0.5, 'ffffff'], [0.75, '0000ff'], [1, '0000ff']],
+        apply_transformations: true
       )
     end
   end
 
-  describe "user space on use with linear gradient" do
+  describe 'user space on use with linear gradient' do
     let(:svg) do
       <<-SVG
         <linearGradient id="flag" gradientUnits="userSpaceOnUse" x1="100" y1="500" x2="200" y2="600">
@@ -78,18 +78,18 @@ describe Prawn::SVG::Elements::Gradient do
       SVG
     end
 
-    it "returns correct gradient arguments for an element" do
+    it 'returns correct gradient arguments for an element' do
       arguments = element.gradient_arguments(double)
       expect(arguments).to eq(
-        from: [100.0, 100.0],
-        to:   [200.0, 0.0],
-        stops: [[0, "ff0000"], [1, "0000ff"]],
-        apply_transformations: true,
+        from:                  [100.0, 100.0],
+        to:                    [200.0, 0.0],
+        stops:                 [[0, 'ff0000'], [1, '0000ff']],
+        apply_transformations: true
       )
     end
   end
 
-  describe "user space on use with radial gradient" do
+  describe 'user space on use with radial gradient' do
     let(:svg) do
       <<-SVG
         <radialGradient id="flag" gradientUnits="userSpaceOnUse" fx="100" fy="500" cx="200" cy="600" r="150">
@@ -99,20 +99,20 @@ describe Prawn::SVG::Elements::Gradient do
       SVG
     end
 
-    it "returns correct gradient arguments for an element" do
+    it 'returns correct gradient arguments for an element' do
       arguments = element.gradient_arguments(double)
       expect(arguments).to eq(
-        from: [100.0, 100.0],
-        to:   [200.0, 0.0],
-        r1:   0,
-        r2:   150,
-        stops: [[0, "ff0000"], [1, "0000ff"]],
-        apply_transformations: true,
+        from:                  [100.0, 100.0],
+        to:                    [200.0, 0.0],
+        r1:                    0,
+        r2:                    150,
+        stops:                 [[0, 'ff0000'], [1, '0000ff']],
+        apply_transformations: true
       )
     end
   end
 
-  context "when gradientTransform is specified" do
+  context 'when gradientTransform is specified' do
     let(:svg) do
       <<-SVG
         <linearGradient id="flag" gradientTransform="translateX(10) scale(2)" x1="0" y1="0" x2="10" y2="10">
@@ -122,14 +122,14 @@ describe Prawn::SVG::Elements::Gradient do
       SVG
     end
 
-    it "passes in the transform via the apply_transformations option" do
+    it 'passes in the transform via the apply_transformations option' do
       arguments = element.gradient_arguments(double(bounding_box: [0, 0, 10, 10]))
 
       expect(arguments).to eq(
-        from: [0, 0],
-        to:   [10, 10],
-        stops: [[0, "ff0000"], [1, "0000ff"]],
-        apply_transformations: [2, 0, 0, 2, 10, 0],
+        from:                  [0, 0],
+        to:                    [10, 10],
+        stops:                 [[0, 'ff0000'], [1, '0000ff']],
+        apply_transformations: [2, 0, 0, 2, 10, 0]
       )
     end
   end

data/spec/prawn/svg/elements/line_spec.rb

@@ -1,54 +1,54 @@
 require 'spec_helper'
 
 RSpec.describe Prawn::SVG::Elements::Line do
-  let(:document) { Prawn::SVG::Document.new(svg, [800, 600], {width: 800, height: 600}) }
+  let(:document) { Prawn::SVG::Document.new(svg, [800, 600], { width: 800, height: 600 }) }
 
   subject do
     Prawn::SVG::Elements::Line.new(document, document.root, [], fake_state)
   end
 
-  context "with attributes specified" do
+  context 'with attributes specified' do
     let(:svg) { '<line x1="5" y1="10" x2="15" y2="20" stroke="black" />' }
 
-    it "renders the line" do
+    it 'renders the line' do
       subject.process
       expect(subject.base_calls).to eq [
-        ["stroke_color", ["000000"], {}, []],
-        ["stroke", [], {}, [
-          ["move_to", [[5.0, 590.0]], {}, []],
-          ["line_to", [[15.0, 580.0]], {}, []]]
-        ]
+        ['stroke_color', ['000000'], {}, []],
+        ['stroke', [], {}, [
+          ['move_to', [[5.0, 590.0]], {}, []],
+          ['line_to', [[15.0, 580.0]], {}, []]
+        ]]
       ]
     end
   end
 
-  context "with no attributes nor stroke specified" do
+  context 'with no attributes nor stroke specified' do
     let(:svg) { '<line />' }
 
-    it "outlines a path from 0,0 to 0,0" do
+    it 'outlines a path from 0,0 to 0,0' do
       subject.process
       expect(subject.base_calls).to eq [
-        ["end_path", [], {}, [
-          ["move_to", [[0, 600]], {}, []],
-          ["line_to", [[0, 600]], {}, []]]
-        ]
+        ['end_path', [], {}, [
+          ['move_to', [[0, 600]], {}, []],
+          ['line_to', [[0, 600]], {}, []]
+        ]]
       ]
     end
   end
 
-  context "with a fill specified" do
+  context 'with a fill specified' do
     let(:svg) { '<line x1="0" y1="0" x2="15" y2="20" style="stroke: red; fill: blue;" />' }
 
-    it "ignores the fill" do
+    it 'ignores the fill' do
       subject.process
 
       expect(subject.base_calls).to eq [
-        ["fill_color", ["0000ff"], {}, []],
-        ["stroke_color", ["ff0000"], {}, []],
-        ["stroke", [], {}, [
-          ["move_to", [[0, 600]], {}, []],
-          ["line_to", [[15.0, 580.0]], {}, []]]
-        ]
+        ['fill_color', ['0000ff'], {}, []],
+        ['stroke_color', ['ff0000'], {}, []],
+        ['stroke', [], {}, [
+          ['move_to', [[0, 600]], {}, []],
+          ['line_to', [[15.0, 580.0]], {}, []]
+        ]]
       ]
     end
   end

data/spec/prawn/svg/elements/marker_spec.rb

@@ -17,7 +17,7 @@ RSpec.describe Prawn::SVG::Elements::Marker do
     SVG
   end
 
-  let(:document) { Prawn::SVG::Document.new(svg, [800, 600], {width: 800, height: 600}) }
+  let(:document) { Prawn::SVG::Document.new(svg, [800, 600], { width: 800, height: 600 }) }
 
   def new_state
     state = Prawn::SVG::State.new
@@ -33,15 +33,15 @@ RSpec.describe Prawn::SVG::Elements::Marker do
     Prawn::SVG::Elements::Marker.new(document, document.root.elements[1], [], new_state)
   end
 
-  describe "#parse" do
-    it "forces display none" do
+  describe '#parse' do
+    it 'forces display none' do
       subject.parse
       expect(subject.properties.display).to eq 'none'
     end
   end
 
-  describe "#apply_marker" do
-    it "adds the line and its marker to the call stack" do
+  describe '#apply_marker' do
+    it 'adds the line and its marker to the call stack' do
       subject.process
       line_element.process
 
@@ -55,41 +55,37 @@ RSpec.describe Prawn::SVG::Elements::Marker do
       # in section 11.6.3.
 
       expect(line_element.base_calls).to eq [
-        ["stroke_color", ["000000"], {}, []],
-        ["line_width", [100.0], {}, []],
-        ["stroke", [], {}, [
-            ["move_to", [[0.0, 600.0]], {}, []],
-            ["line_to", [[10.0, 590.0]], {}, []]
-          ]
-        ],
-        ["save", [], {}, []],
-        ["transformation_matrix", [1, 0, 0, 1, 10, -10], {}, []],
-        ["rotate", [-45], {origin: [0, 600.0]}, [
-            ["transformation_matrix", [100.0, 0, 0, 100.0, 0, 0], {}, []],
-            ["transformation_matrix", [1, 0, 0, 1, -0.0, 1.5], {}, []],
-            ["rectangle", [[-0.5, 600.0], 4.0, 3.0], {}, []],
-            ["clip", [], {}, []],
-            ["transformation_matrix", [0.3, 0, 0, 0.3, 0, 0], {}, []],
-            ["transparent", [1.0, 1.0], {}, [
-                ["fill_color", ["000000"], {}, []],
-                ["line_width", [1.0], {}, []],
-                ["cap_style", [:butt], {}, []],
-                ["join_style", [:miter], {}, []],
-                ["undash", [], {}, []],
-                ["save", [], {}, []],
-                ["fill", [], {}, [
-                    ["move_to", [[0.0, 600.0]], {}, []],
-                    ["line_to", [[10.0, 595.0]], {}, []],
-                    ["line_to", [[0.0, 590.0]], {}, []],
-                    ["close_path", [], {}, []]
-                  ]
-                ],
-                ["restore", [], {}, []],
-              ]
-            ]
-          ]
-        ],
-        ["restore", [], {}, []]
+        ['stroke_color', ['000000'], {}, []],
+        ['line_width', [100.0], {}, []],
+        ['stroke', [], {}, [
+          ['move_to', [[0.0, 600.0]], {}, []],
+          ['line_to', [[10.0, 590.0]], {}, []]
+        ]],
+        ['save', [], {}, []],
+        ['transformation_matrix', [1, 0, 0, 1, 10, -10], {}, []],
+        ['rotate', [-45], { origin: [0, 600.0] }, [
+          ['transformation_matrix', [100.0, 0, 0, 100.0, 0, 0], {}, []],
+          ['transformation_matrix', [1, 0, 0, 1, -0.0, 1.5], {}, []],
+          ['rectangle', [[-0.5, 600.0], 4.0, 3.0], {}, []],
+          ['clip', [], {}, []],
+          ['transformation_matrix', [0.3, 0, 0, 0.3, 0, 0], {}, []],
+          ['transparent', [1.0, 1.0], {}, [
+            ['fill_color', ['000000'], {}, []],
+            ['line_width', [1.0], {}, []],
+            ['cap_style', [:butt], {}, []],
+            ['join_style', [:miter], {}, []],
+            ['undash', [], {}, []],
+            ['save', [], {}, []],
+            ['fill', [], {}, [
+              ['move_to', [[0.0, 600.0]], {}, []],
+              ['line_to', [[10.0, 595.0]], {}, []],
+              ['line_to', [[0.0, 590.0]], {}, []],
+              ['close_path', [], {}, []]
+            ]],
+            ['restore', [], {}, []]
+          ]]
+        ]],
+        ['restore', [], {}, []]
       ]
     end
   end