prawn 0.11.1 → 0.15.0

data/spec/text_spec.rb

@@ -16,7 +16,7 @@ describe "#height_of" do
     original_y = @pdf.y
     @pdf.text("Foo")
     new_y = @pdf.y
-    @pdf.height_of("Foo").should.be.close(original_y - new_y, 0.0001)
+    @pdf.height_of("Foo").should be_within(0.0001).of(original_y - new_y)
   end
 
   it "should omit the gap below the last descender if :final_gap => false " +
@@ -24,40 +24,35 @@ describe "#height_of" do
     original_y = @pdf.y
     @pdf.text("Foo", :final_gap => false)
     new_y = @pdf.y
-    @pdf.height_of("Foo", :final_gap => false).should.be.close(original_y - new_y, 0.0001)
+    @pdf.height_of("Foo", :final_gap => false).should be_within(0.0001).of(original_y - new_y)
   end
 
-  it "should raise CannotFit if a too-small width is given" do
+  it "should raise_error CannotFit if a too-small width is given" do
     lambda do
       @pdf.height_of("text", :width => 1)
-    end.should.raise(Prawn::Errors::CannotFit)
+    end.should raise_error(Prawn::Errors::CannotFit)
   end
 
-  it "should raise NotImplementedError if :indent_paragraphs option is provided" do
+  it "should raise_error NotImplementedError if :indent_paragraphs option is provided" do
     lambda {
       @pdf.height_of("hai", :width => 300,
                      :indent_paragraphs => 60)
-    }.should.raise(NotImplementedError)
+    }.should raise_error(NotImplementedError)
   end
 
-  it "should not raise Prawn::Errors::UnknownOption if :final_gap option is provided" do
-    lambda {
-      @pdf.height_of("hai", :width => 300,
-                     :final_gap => true)
-    }.should.not.raise(Prawn::Errors::UnknownOption)
+  it "should_not raise_error Prawn::Errors::UnknownOption if :final_gap option is provided" do
+    @pdf.height_of("hai", :width => 300, :final_gap => true)
   end
 end
 
 describe "#text" do
   before(:each) { create_pdf }
 
-  it "should not fail when @output is nil when Prawn::Core::Text::LineWrap#finalize_line is called" do
+  it "should not fail when @output is nil when PDF::Core::Text::LineWrap#finalize_line is called" do
     # need a document with margins for these particulars to produce the
     # condition that was throwing the error
     pdf = Prawn::Document.new
-    lambda {
-      pdf.text "transparency " * 150, :size => 18
-    }.should.not.raise(TypeError)
+    pdf.text "transparency " * 150, :size => 18
   end
 
   it "should allow drawing empty strings to the page" do
@@ -67,6 +62,10 @@ describe "#text" do
     text.strings.each { |str| str.should =~ /\A\s*\z/ }
   end
 
+  it "should ignore call when string is nil" do
+    @pdf.text(nil).should be_false
+  end
+
   it "should correctly render empty paragraphs" do
     @pdf.text "text\n\ntext"
     text = PDF::Inspector::Text.analyze(@pdf.render)
@@ -92,13 +91,13 @@ describe "#text" do
   it "should default to use kerning information" do
     @pdf.text "hello world"
     text = PDF::Inspector::Text.analyze(@pdf.render)
-    text.kerned[0].should.be true
+    text.kerned[0].should be_true
   end
 
   it "should be able to disable kerning with an option" do
     @pdf.text "hello world", :kerning => false
     text = PDF::Inspector::Text.analyze(@pdf.render)
-    text.kerned[0].should.be false
+    text.kerned[0].should be_false
   end
 
   it "should be able to disable kerning document-wide" do
@@ -106,29 +105,29 @@ describe "#text" do
     @pdf.default_kerning = false
     @pdf.text "hello world"
     text = PDF::Inspector::Text.analyze(@pdf.render)
-    text.kerned[0].should.be false
+    text.kerned[0].should be_false
   end
 
   it "option should be able to override document-wide kerning disabling" do
     @pdf.default_kerning = false
     @pdf.text "hello world", :kerning => true
     text = PDF::Inspector::Text.analyze(@pdf.render)
-    text.kerned[0].should.be true
+    text.kerned[0].should be_true
   end
 
-  it "should raise ArgumentError if :at option included" do
-    lambda { @pdf.text("hai", :at => [0, 0]) }.should.raise(ArgumentError)
+  it "should raise_error ArgumentError if :at option included" do
+    lambda { @pdf.text("hai", :at => [0, 0]) }.should raise_error(ArgumentError)
   end
 
   it "should advance down the document based on font_height" do
     position = @pdf.y
     @pdf.text "Foo"
 
-    @pdf.y.should.be.close(position - @pdf.font.height, 0.0001)
+    @pdf.y.should be_within(0.0001).of(position - @pdf.font.height)
 
     position = @pdf.y
     @pdf.text "Foo\nBar\nBaz"
-    @pdf.y.should.be.close(position - 3*@pdf.font.height, 0.0001)
+    @pdf.y.should be_within(0.0001).of(position - 3*@pdf.font.height)
   end
 
   it "should advance down the document based on font_height" +
@@ -137,11 +136,11 @@ describe "#text" do
     @pdf.text "Foo", :size => 15
 
     @pdf.font_size = 15
-    @pdf.y.should.be.close(position - @pdf.font.height, 0.0001)
+    @pdf.y.should be_within(0.0001).of(position - @pdf.font.height)
 
     position = @pdf.y
     @pdf.text "Foo\nBar\nBaz"
-    @pdf.y.should.be.close(position - 3 * @pdf.font.height, 0.0001)
+    @pdf.y.should be_within(0.0001).of(position - 3 * @pdf.font.height)
   end
 
   it "should advance down the document based on font_height" +
@@ -150,11 +149,11 @@ describe "#text" do
     leading = 2
     @pdf.text "Foo", :leading => leading
 
-    @pdf.y.should.be.close(position - @pdf.font.height - leading, 0.0001)
+    @pdf.y.should be_within(0.0001).of(position - @pdf.font.height - leading)
 
     position = @pdf.y
     @pdf.text "Foo\nBar\nBaz"
-    @pdf.y.should.be.close(position - 3*@pdf.font.height, 0.0001)
+    @pdf.y.should be_within(0.0001).of(position - 3*@pdf.font.height)
   end
 
   it "should advance only to the bottom of the final descender "+
@@ -162,11 +161,11 @@ describe "#text" do
     position = @pdf.y
     @pdf.text "Foo", :final_gap => false
 
-    @pdf.y.should.be.close(position - @pdf.font.ascender - @pdf.font.descender, 0.0001)
+    @pdf.y.should be_within(0.0001).of(position - @pdf.font.ascender - @pdf.font.descender)
 
     position = @pdf.y
     @pdf.text "Foo\nBar\nBaz", :final_gap => false
-    @pdf.y.should.be.close(position - 2*@pdf.font.height - @pdf.font.ascender - @pdf.font.descender, 0.0001)
+    @pdf.y.should be_within(0.0001).of(position - 2*@pdf.font.height - @pdf.font.ascender - @pdf.font.descender)
   end
 
   it "should be able to print text starting at the last line of a page" do
@@ -253,8 +252,12 @@ describe "#text" do
     text.font_settings[1][:name].should == :Helvetica
   end
 
-  it "should raise an exception when an unknown font is used" do
-    lambda { @pdf.font "Pao bu" }.should.raise(Prawn::Errors::UnknownFont)
+  it "should raise_error an exception when an unknown font is used" do
+    lambda { @pdf.font "Pao bu" }.should raise_error(Prawn::Errors::UnknownFont)
+  end
+
+  it "should_not raise_error an exception when providing Pathname instance as font" do
+    @pdf.font Pathname.new("#{Prawn::DATADIR}/fonts/comicsans.ttf")
   end
 
   it "should correctly render a utf-8 string when using a built-in font" do
@@ -268,7 +271,7 @@ describe "#text" do
 
   it "should correctly render a utf-8 string when using a TTF font" do
     str = "©" # copyright symbol
-    @pdf.font "#{Prawn::BASEDIR}/data/fonts/DejaVuSans.ttf"
+    @pdf.font "#{Prawn::DATADIR}/fonts/DejaVuSans.ttf"
     @pdf.text str
 
     # grab the text from the rendered PDF and ensure it matches
@@ -276,6 +279,16 @@ describe "#text" do
     text.strings.first.should == str
   end
 
+  it "subsets mixed low-ASCII and non-ASCII characters when they can be " +
+     "subsetted together" do
+    str = "It’s super effective!"
+    @pdf.font "#{Prawn::DATADIR}/fonts/DejaVuSans.ttf"
+    @pdf.text str
+
+    text = PDF::Inspector::Text.analyze(@pdf.render)
+    text.strings.first.should == str
+  end
+
   it "should correctly render a string with higher bit characters across" +
      " a page break when using a built-in font" do
     str = "©"
@@ -300,33 +313,20 @@ describe "#text" do
     pages[1][:strings].should == [str]
   end
 
-  if "spec".respond_to?(:encode!)
-    # Handle non utf-8 string encodings in a sane way on M17N aware VMs
-    it "should raise an exception when a utf-8 incompatible string is rendered" do
-      str = "Blah \xDD"
-      str.force_encoding("ASCII-8BIT")
-      lambda { @pdf.text str }.should.raise(
-        Prawn::Errors::IncompatibleStringEncoding)
-    end
-    it "should not raise an exception when a shift-jis string is rendered" do
-      datafile = "#{Prawn::BASEDIR}/data/shift_jis_text.txt"
-      sjis_str = File.open(datafile, "r:shift_jis") { |f| f.gets }
-      @pdf.font("#{Prawn::BASEDIR}/data/fonts/gkai00mp.ttf")
-      lambda { @pdf.text sjis_str }.should.not.raise(
-        Prawn::Errors::IncompatibleStringEncoding)
-    end
-  else
-    # Handle non utf-8 string encodings in a sane way on non-M17N aware VMs
-    it "should raise an exception when a corrupt utf-8 string is rendered" do
-      str = "Blah \xDD"
-      lambda { @pdf.text str }.should.raise(
-        Prawn::Errors::IncompatibleStringEncoding)
-    end
-    it "should raise an exception when a shift-jis string is rendered" do
-      sjis_str = File.read("#{Prawn::BASEDIR}/data/shift_jis_text.txt")
-      lambda { @pdf.text sjis_str }.should.raise(
-        Prawn::Errors::IncompatibleStringEncoding)
-    end
+  it "should raise_error an exception when a utf-8 incompatible string is rendered" do
+    str = "Blah \xDD"
+    str.force_encoding(Encoding::ASCII_8BIT)
+    lambda { @pdf.text str }.should raise_error(
+      Prawn::Errors::IncompatibleStringEncoding)
+  end
+
+  it "should_not raise_error an exception when a shift-jis string is rendered" do
+    datafile = "#{Prawn::DATADIR}/shift_jis_text.txt"
+    sjis_str = File.open(datafile, "r:shift_jis") { |f| f.gets }
+    @pdf.font("#{Prawn::DATADIR}/fonts/gkai00mp.ttf")
+
+    # Expect that the call to text will not raise an encoding error
+    @pdf.text(sjis_str)
   end
 
   it "should call move_past_bottom when printing more text than can fit" +
@@ -382,4 +382,56 @@ describe "#text" do
       end
     end
   end
+
+  describe "kerning" do
+    it "should respect text kerning setting (document default)" do
+      create_pdf
+      @pdf.font.expects(:compute_width_of).with do |str, options|
+        str == "VAT" && options[:kerning] == true
+      end.at_least_once.returns(10)
+      @pdf.text "VAT"
+    end
+
+    it "should respect text kerning setting (kerning=true)" do
+      create_pdf
+      @pdf.font.expects(:compute_width_of).with do |str, options|
+        str == "VAT" && options[:kerning] == true
+      end.at_least_once.returns(10)
+      @pdf.text "VAT", :kerning => true
+    end
+
+    it "should respect text kerning setting (kerning=false)" do
+      create_pdf
+      @pdf.font.expects(:compute_width_of).with do |str, options|
+        str == "VAT" && options[:kerning] == false
+      end.at_least_once.returns(10)
+      @pdf.text "VAT", :kerning => false
+    end
+  end
+
+  describe "#shrink_to_fit with special utf-8 text" do
+    it "Should not throw an exception", 
+        :unresolved, :issue => 603 do
+      pages = 0
+      doc = Prawn::Document.new(page_size: 'A4', margin: [2, 2, 2, 2]) do |pdf|
+        add_unicode_fonts(pdf)
+        pdf.bounding_box([1, 1], :width => 90, :height => 50) do
+          broken_text = " Sample Text\nSAMPLE SAMPLE SAMPLEoddělení ZMĚN\nSAMPLE"
+          pdf.text broken_text, :overflow => :shrink_to_fit
+        end
+      end
+    end
+  end
+
+
+  def add_unicode_fonts(pdf)
+    dejavu = "#{::Prawn::BASEDIR}/data/fonts/DejaVuSans.ttf"
+    pdf.font_families.update("dejavu" => {
+      :normal      => dejavu,
+      :italic      => dejavu,
+      :bold        => dejavu,
+      :bold_italic => dejavu
+    })
+    pdf.fallback_fonts = ["dejavu"]
+  end
 end

data/spec/transparency_spec.rb

@@ -1,4 +1,4 @@
-require File.join(File.expand_path(File.dirname(__FILE__)), "spec_helper") 
+require File.join(File.expand_path(File.dirname(__FILE__)), "spec_helper")
 
 module TransparencyHelper
   def make_transparent(opacity, stroke_opacity=opacity)
@@ -17,7 +17,7 @@ describe "Document with transparency" do
     str = @pdf.render
     str[0,8].should == "%PDF-1.4"
   end
-  
+
   it "a new extended graphics state should be created for "+
      "each unique transparency setting" do
     create_pdf
@@ -27,7 +27,7 @@ describe "Document with transparency" do
     extgstates = PDF::Inspector::ExtGState.analyze(@pdf.render).extgstates
     extgstates.length.should == 2
   end
-  
+
   it "a new extended graphics state should not be created for "+
      "each duplicate transparency setting" do
     create_pdf
@@ -70,7 +70,7 @@ describe "Document with transparency" do
     extgstate[:opacity].should == 1.0
     extgstate[:stroke_opacity].should == 1.0
   end
-  
+
   describe "with more than one page" do
     include TransparencyHelper
 
@@ -78,7 +78,7 @@ describe "Document with transparency" do
       create_pdf
       make_transparent(0.5, 0.2)
       @pdf.start_new_page
-      make_transparent(0.5, 0.2) 
+      make_transparent(0.5, 0.2)
       extgstates = PDF::Inspector::ExtGState.analyze(@pdf.render).extgstates
       extgstate = extgstates[0]
       extgstates.length.should == 2