hexapdf 0.32.2 → 0.34.0

data/test/hexapdf/layout/test_list_box.rb

@@ -6,7 +6,9 @@ require 'hexapdf/layout/list_box'
 
 describe HexaPDF::Layout::ListBox do
   before do
-    @frame = HexaPDF::Layout::Frame.new(0, 0, 100, 100)
+    @doc = HexaPDF::Document.new
+    @page = @doc.pages.add
+    @frame = HexaPDF::Layout::Frame.new(0, 0, 100, 100, context: @page)
     inline_box = HexaPDF::Layout::InlineBox.create(width: 10, height: 10) {}
     @text_boxes = 5.times.map do
       HexaPDF::Layout::TextBox.new(items: [inline_box] * 15, style: {position: :default})
@@ -18,13 +20,13 @@ describe HexaPDF::Layout::ListBox do
   end
 
   def check_box(box, width, height, fit_pos = nil)
-    assert(box.fit(@frame.available_width, @frame.available_height, @frame), "box fit?")
+    assert(box.fit(@frame.available_width, @frame.available_height, @frame), "box didn't fit")
     assert_equal(width, box.width, "box width")
     assert_equal(height, box.height, "box height")
     if fit_pos
       results = box.instance_variable_get(:@results)
-      results.each_with_index do |box_fitter, item_index|
-        box_fitter.fit_results.each_with_index do |fit_result, result_index|
+      results.each_with_index do |item_result, item_index|
+        item_result.box_fitter.fit_results.each_with_index do |fit_result, result_index|
           x, y = fit_pos.shift
           assert_equal(x, fit_result.x, "item #{item_index}, result #{result_index}, x")
           assert_equal(y, fit_result.y, "item #{item_index}, result #{result_index}, y")
@@ -47,6 +49,18 @@ describe HexaPDF::Layout::ListBox do
     end
   end
 
+  describe "empty?" do
+    it "is empty if nothing was fit yet" do
+      assert(create_box.empty?)
+    end
+
+    it "is empty if nothing could be fit" do
+      box = create_box(children: [@text_boxes[0]], width: 5)
+      box.fit(@frame.available_width, @frame.available_height, @frame)
+      assert(box.empty?)
+    end
+  end
+
   describe "fit" do
     [:default, :flow].each do |position|
       it "respects the set initial width, position #{position}" do
@@ -78,6 +92,12 @@ describe HexaPDF::Layout::ListBox do
       check_box(box, 100, 90, [[10, 80], [10, 60], [10, 40], [50, 10]])
     end
 
+    it "calculates the correct height if the marker is higher than the content" do
+      box = create_box(children: @text_boxes[0, 1], content_indentation: 20,
+                       style: {font_size: 30})
+      check_box(box, 100, 27, [[20, 80]])
+    end
+
     it "respects the content indentation" do
       box = create_box(children: @text_boxes[0, 1], content_indentation: 30)
       check_box(box, 100, 30, [[30, 70]])
@@ -87,6 +107,11 @@ describe HexaPDF::Layout::ListBox do
       box = create_box(children: @text_boxes[0, 2], item_spacing: 30)
       check_box(box, 100, 70, [[10, 80], [10, 30]])
     end
+
+    it "fails for unknown item types" do
+      box = create_box(children: @text_boxes[0, 1], item_type: :unknown)
+      assert_raises(HexaPDF::Error) { box.fit(100, 100, @frame) }
+    end
   end
 
   describe "split" do
@@ -96,7 +121,7 @@ describe HexaPDF::Layout::ListBox do
       box_a, box_b = box.split(100, 100, @frame)
       assert_same(box, box_a)
       assert_equal(:show_first_marker, box_b.split_box?)
-      assert_equal(1, box_a.instance_variable_get(:@results)[0].fit_results.size)
+      assert_equal(1, box_a.instance_variable_get(:@results)[0].box_fitter.fit_results.size)
       assert_equal(1, box_b.children.size)
       assert_equal(2, box_b.start_number)
     end
@@ -107,7 +132,7 @@ describe HexaPDF::Layout::ListBox do
       box_a, box_b = box.split(100, 100, @frame)
       assert_same(box, box_a)
       assert_equal(:hide_first_marker, box_b.split_box?)
-      assert_equal(1, box_a.instance_variable_get(:@results)[0].fit_results.size)
+      assert_equal(1, box_a.instance_variable_get(:@results)[0].box_fitter.fit_results.size)
       assert_equal(2, box_b.children.size)
       assert_equal(1, box_b.start_number)
     end
@@ -115,20 +140,22 @@ describe HexaPDF::Layout::ListBox do
 
   describe "draw" do
     before do
-      @canvas = HexaPDF::Document.new.pages.add.canvas
+      @canvas = @page.canvas
       draw_block = lambda {|canvas, box| }
       @fixed_size_boxes = 5.times.map { HexaPDF::Layout::Box.new(width: 20, height: 10, &draw_block) }
     end
 
     it "draws the result" do
-      box = create_box(children: @fixed_size_boxes[0, 2])
+      box = create_box(children: @fixed_size_boxes[0, 2],
+                       style: {font_size: 11, fill_color: 0.5})
       box.fit(100, 100, @frame)
-      box.draw(@canvas, 0, 0)
+      box.draw(@canvas, 0, 100 - box.height)
       operators = [
         [:save_graphics_state],
-        [:set_font_and_size, [:F1, 10]],
+        [:set_font_and_size, [:F1, 11]],
+        [:set_device_gray_non_stroking_color, [0.5]],
         [:begin_text],
-        [:set_text_matrix, [1, 0, 0, 1, 1.5, 93.17]],
+        [:set_text_matrix, [1, 0, 0, 1, 1.15, 92.487]],
         [:show_text, ["\x95".b]],
         [:end_text],
         [:restore_graphics_state],
@@ -137,9 +164,10 @@ describe HexaPDF::Layout::ListBox do
         [:restore_graphics_state],
 
         [:save_graphics_state],
-        [:set_font_and_size, [:F1, 10]],
+        [:set_font_and_size, [:F1, 11]],
+        [:set_device_gray_non_stroking_color, [0.5]],
         [:begin_text],
-        [:set_text_matrix, [1, 0, 0, 1, 1.5, 83.17]],
+        [:set_text_matrix, [1, 0, 0, 1, 1.15, 82.487]],
         [:show_text, ["\x95".b]],
         [:end_text],
         [:restore_graphics_state],
@@ -150,16 +178,18 @@ describe HexaPDF::Layout::ListBox do
       assert_operators(@canvas.contents, operators)
     end
 
-    it "draws a cicle as marker" do
-      box = create_box(children: @fixed_size_boxes[0, 1], item_type: :circle)
+    it "draws a circle as marker" do
+      box = create_box(children: @fixed_size_boxes[0, 1], item_type: :circle,
+                       style: {font_size: 11, fill_color: 0.5})
       box.fit(100, 100, @frame)
-      box.draw(@canvas, 0, 0)
+      box.draw(@canvas, 0, 100 - box.height)
       operators = [
         [:save_graphics_state],
-        [:set_font_and_size, [:F1, 5]],
-        [:set_text_rise, [-5.555556]],
+        [:set_font_and_size, [:F1, 5.5]],
+        [:set_text_rise, [-6.111111]],
+        [:set_device_gray_non_stroking_color, [0.5]],
         [:begin_text],
-        [:set_text_matrix, [1, 0, 0, 1, 0.635, 100]],
+        [:set_text_matrix, [1, 0, 0, 1, 0.1985, 100]],
         [:show_text, ["m".b]],
         [:end_text],
         [:restore_graphics_state],
@@ -171,15 +201,17 @@ describe HexaPDF::Layout::ListBox do
     end
 
     it "draws a square as marker" do
-      box = create_box(children: @fixed_size_boxes[0, 1], item_type: :square)
+      box = create_box(children: @fixed_size_boxes[0, 1], item_type: :square,
+                       style: {font_size: 11, fill_color: 0.5})
       box.fit(100, 100, @frame)
-      box.draw(@canvas, 0, 0)
+      box.draw(@canvas, 0, 100 - box.height)
       operators = [
         [:save_graphics_state],
-        [:set_font_and_size, [:F1, 5]],
-        [:set_text_rise, [-5.555556]],
+        [:set_font_and_size, [:F1, 5.5]],
+        [:set_text_rise, [-6.111111]],
+        [:set_device_gray_non_stroking_color, [0.5]],
         [:begin_text],
-        [:set_text_matrix, [1, 0, 0, 1, 1.195, 100]],
+        [:set_text_matrix, [1, 0, 0, 1, 0.8145, 100]],
         [:show_text, ["n".b]],
         [:end_text],
         [:restore_graphics_state],
@@ -192,14 +224,16 @@ describe HexaPDF::Layout::ListBox do
 
     it "draws decimal numbers as marker" do
       box = create_box(children: @fixed_size_boxes[0, 2], item_type: :decimal,
+                       style: {font_size: 11, fill_color: 0.5},
                        content_indentation: 20)
       box.fit(100, 100, @frame)
-      box.draw(@canvas, 0, 0)
+      box.draw(@canvas, 0, 100 - box.height)
       operators = [
         [:save_graphics_state],
-        [:set_font_and_size, [:F1, 10]],
+        [:set_font_and_size, [:F1, 11]],
+        [:set_device_gray_non_stroking_color, [0.5]],
         [:begin_text],
-        [:set_text_matrix, [1, 0, 0, 1, 7.5, 93.17]],
+        [:set_text_matrix, [1, 0, 0, 1, 6.75, 92.487]],
         [:show_text, ["1.".b]],
         [:end_text],
         [:restore_graphics_state],
@@ -208,9 +242,10 @@ describe HexaPDF::Layout::ListBox do
         [:restore_graphics_state],
 
         [:save_graphics_state],
-        [:set_font_and_size, [:F1, 10]],
+        [:set_font_and_size, [:F1, 11]],
+        [:set_device_gray_non_stroking_color, [0.5]],
         [:begin_text],
-        [:set_text_matrix, [1, 0, 0, 1, 7.5, 83.17]],
+        [:set_text_matrix, [1, 0, 0, 1, 6.75, 82.487]],
         [:show_text, ["2.".b]],
         [:end_text],
         [:restore_graphics_state],
@@ -227,7 +262,7 @@ describe HexaPDF::Layout::ListBox do
       end
       box = create_box(children: @fixed_size_boxes[0, 1], item_type: marker)
       box.fit(100, 100, @frame)
-      box.draw(@canvas, 0, 0)
+      box.draw(@canvas, 0, 100 - box.height)
       operators = [
         [:save_graphics_state],
         [:concatenate_matrix, [1, 0, 0, 1, 0, 90]],
@@ -239,10 +274,26 @@ describe HexaPDF::Layout::ListBox do
       assert_operators(@canvas.contents, operators)
     end
 
-    it "fails for unknown item types" do
-      box = create_box(children: @fixed_size_boxes[0, 1], item_type: :unknown)
+    it "takes a different final location into account" do
+      box = create_box(children: @fixed_size_boxes[0, 1])
       box.fit(100, 100, @frame)
-      assert_raises(HexaPDF::Error) { box.draw(@canvas, 0, 0) }
+      box.draw(@canvas, 20, 10)
+      operators = [
+        [:save_graphics_state],
+        [:concatenate_matrix, [1, 0, 0, 1, 20, -80]],
+        [:save_graphics_state],
+        [:set_font_and_size, [:F1, 10]],
+        [:begin_text],
+        [:set_text_matrix, [1, 0, 0, 1, 1.5, 93.17]],
+        [:show_text, ["\x95".b]],
+        [:end_text],
+        [:restore_graphics_state],
+        [:save_graphics_state],
+        [:concatenate_matrix, [1, 0, 0, 1, 10, 90]],
+        [:restore_graphics_state],
+        [:restore_graphics_state],
+      ]
+      assert_operators(@canvas.contents, operators)
     end
   end
 end

data/test/hexapdf/layout/test_page_style.rb

@@ -5,11 +5,12 @@ require 'hexapdf/layout/page_style'
 require 'hexapdf/document'
 
 describe HexaPDF::Layout::PageStyle do
-  it "allows assigning the page size, orientation and template on initialization" do
+  it "allows assigning the page size, orientation, next style and template on initialization" do
     block = lambda {}
-    style = HexaPDF::Layout::PageStyle.new(page_size: :A3, orientation: :landscape, &block)
+    style = HexaPDF::Layout::PageStyle.new(page_size: :A3, orientation: :landscape, next_style: :a, &block)
     assert_equal(:A3, style.page_size)
     assert_equal(:landscape, style.orientation)
+    assert_equal(:a, style.next_style)
     assert_same(block, style.template)
   end
 

data/test/hexapdf/layout/test_style.rb

@@ -156,15 +156,24 @@ describe HexaPDF::Layout::Style::Border do
     assert_kind_of(HexaPDF::Layout::Style::Quad, border.style)
   end
 
+  it "has an accessor for the draw-on-bounds option" do
+    border = create_border
+    refute(border.draw_on_bounds)
+    border.draw_on_bounds = true
+    assert(border.draw_on_bounds)
+  end
+
   it "can be duplicated" do
     border = create_border
     copy = border.dup
     border.width.top = 10
     border.color.top = :red
     border.style.top = :dotted
+    border.draw_on_bounds = true
     assert_equal(0, copy.width.top)
     assert_equal(0, copy.color.top)
     assert_equal(:solid, copy.style.top)
+    refute(copy.draw_on_bounds)
   end
 
   it "can be asked whether a border is defined" do
@@ -259,6 +268,19 @@ describe HexaPDF::Layout::Style::Border do
                [:restore_graphics_state]]
         assert_operators(@canvas.contents, ops)
       end
+
+      it "works if the border is drawn on the bounds" do
+        border = create_border(width: 10, color: 0.5, style: :solid, draw_on_bounds: true)
+        border.draw(@canvas, 0, 0, 100, 100)
+        assert_operators(@canvas.contents, [[:save_graphics_state],
+                                            [:set_device_gray_stroking_color, [0.5]],
+                                            [:set_line_width, [10]],
+                                            [:append_rectangle, [-5, -5, 110, 110]],
+                                            [:clip_path_non_zero], [:end_path],
+                                            [:append_rectangle, [0, 0, 100, 100]],
+                                            [:stroke_path],
+                                            [:restore_graphics_state]])
+      end
     end
 
     describe "complex borders where edges have different width/color/style values" do
@@ -369,6 +391,34 @@ describe HexaPDF::Layout::Style::Border do
                [:restore_graphics_state], [:restore_graphics_state]]
         assert_operators(@canvas.contents, ops)
       end
+
+      it "works if the border is drawn on the bounds" do
+        border = create_border(width: [20, 10, 40, 30], draw_on_bounds: true)
+        border.draw(@canvas, 0, 0, 100, 200)
+        ops = [[:save_graphics_state],
+               [:save_graphics_state],
+               [:move_to, [-15, 210]], [:line_to, [105, 210]],
+               [:line_to, [95, 190]], [:line_to, [15, 190]], [:clip_path_non_zero], [:end_path],
+               [:set_line_width, [20]],
+               [:move_to, [-15, 200]], [:line_to, [105, 200]], [:stroke_path],
+               [:restore_graphics_state], [:save_graphics_state],
+               [:move_to, [105, 210]], [:line_to, [105, -20]],
+               [:line_to, [95, 20]], [:line_to, [95, 190]], [:clip_path_non_zero], [:end_path],
+               [:set_line_width, [10]],
+               [:move_to, [100, 210]], [:line_to, [100, -20]], [:stroke_path],
+               [:restore_graphics_state], [:save_graphics_state],
+               [:move_to, [105, -20]], [:line_to, [-15, -20]],
+               [:line_to, [15, 20]], [:line_to, [95, 20]], [:clip_path_non_zero], [:end_path],
+               [:set_line_width, [40]],
+               [:move_to, [105, 0]], [:line_to, [-15, 0]], [:stroke_path],
+               [:restore_graphics_state], [:save_graphics_state],
+               [:move_to, [-15, -20]], [:line_to, [-15, 210]],
+               [:line_to, [15, 190]], [:line_to, [15, 20]], [:clip_path_non_zero], [:end_path],
+               [:set_line_width, [30]],
+               [:move_to, [0, -20]], [:line_to, [0, 210]], [:stroke_path],
+               [:restore_graphics_state], [:restore_graphics_state]]
+        assert_operators(@canvas.contents, ops)
+      end
     end
 
     describe "border width greater than edge length" do
@@ -530,7 +580,10 @@ describe HexaPDF::Layout::Style::LinkLayer do
     it "fails if more than one possible target is chosen" do
       assert_raises(ArgumentError) { HexaPDF::Layout::Style::LinkLayer.new(dest: true, uri: true) }
       assert_raises(ArgumentError) { HexaPDF::Layout::Style::LinkLayer.new(dest: true, file: true) }
+      assert_raises(ArgumentError) { HexaPDF::Layout::Style::LinkLayer.new(dest: true, action: true) }
       assert_raises(ArgumentError) { HexaPDF::Layout::Style::LinkLayer.new(uri: true, file: true) }
+      assert_raises(ArgumentError) { HexaPDF::Layout::Style::LinkLayer.new(uri: true, action: true) }
+      assert_raises(ArgumentError) { HexaPDF::Layout::Style::LinkLayer.new(file: true, action: true) }
     end
 
     it "fails if an invalid border is provided" do
@@ -606,6 +659,12 @@ describe HexaPDF::Layout::Style::LinkLayer do
       assert_nil(annot[:Dest])
     end
 
+    it "works for actions" do
+      annot = call_link(action: {Type: :Action, S: :SetOCGState})
+      assert_equal({Type: :Action, S: :SetOCGState}, annot[:A].value)
+      assert_nil(annot[:Dest])
+    end
+
     it "works for destinations set via the 'link' custom box property" do
       @box.properties['link'] = [@canvas.context, :FitH]
       annot = call_link({})
@@ -723,6 +782,7 @@ describe HexaPDF::Layout::Style do
     refute(@style.subscript)
     refute(@style.superscript)
     refute(@style.last_line_gap)
+    refute(@style.fill_horizontal)
     assert_kind_of(HexaPDF::Layout::Style::Layers, @style.underlays)
     assert_kind_of(HexaPDF::Layout::Style::Layers, @style.overlays)
   end