pdfkit 0.8.2 → 0.8.4.3.1

data/spec/os_spec.rb

@@ -0,0 +1,65 @@
+#encoding: UTF-8
+require 'spec_helper'
+require 'rbconfig'
+
+describe 'OS' do
+  subject { PDFKit::OS }
+
+  describe 'host_is_windows?' do
+    it 'is callable' do
+      expect(subject).to respond_to(:host_is_windows?)
+    end
+
+    def test_is_windows(bool, host_os)
+      allow(RbConfig::CONFIG).to receive(:[]).with('host_os').and_return(host_os)
+
+      expect(subject.host_is_windows?).to be bool
+    end
+
+    it 'returns true if the host_os is set to "mswin"' do
+      test_is_windows(true, 'mswin')
+    end
+
+    it 'returns true if the host_os is set to "msys"' do
+      test_is_windows(true, 'msys')
+    end
+
+    it 'returns false if the host_os is set to "linux-gnu"' do
+      test_is_windows(false, 'linux-gnu')
+    end
+
+    it 'returns false if the host_os is set to "darwin14.1.0"' do
+      test_is_windows(false, 'darwin14.1.0')
+    end
+  end
+
+  describe 'shell_escape_for_os' do
+    it 'is callable' do
+      expect(subject).to respond_to(:shell_escape_for_os)
+    end
+
+    it 'calls shelljoin on linux' do
+      args = double(:shelljoin)
+      allow(RbConfig::CONFIG).to receive(:[]).with('host_os').and_return('linux-gnu')
+
+      expect(args).to receive(:shelljoin)
+      PDFKit::OS.shell_escape_for_os(args)
+    end
+
+    it 'calls shelljoin on darwin14.1.10' do
+      args = double(:shelljoin)
+      allow(RbConfig::CONFIG).to receive(:[]).with('host_os').and_return('darwin14.1.10-gnu')
+
+      expect(args).to receive(:shelljoin)
+      PDFKit::OS.shell_escape_for_os(args)
+    end
+
+    it 'escapes special characters on Windows' do
+      args = ['foo|bar', 'biz(baz)', 'foo<baz>bar', 'hello^world&goodbye']
+      allow(RbConfig::CONFIG).to receive(:[]).with('host_os').and_return('mswin')
+
+      escaped_args = PDFKit::OS.shell_escape_for_os(args)
+      expect(escaped_args).to eq('foo^|bar biz^(baz^) foo^<baz^>bar hello^^world^&goodbye')
+    end
+  end
+end

data/spec/pdfkit_spec.rb

@@ -23,6 +23,13 @@ describe PDFKit do
       expect(pdfkit.source.to_s).to eq(file_path)
     end
 
+    it "accepts a Tempfile as the source" do
+      file_path = File.join(SPEC_ROOT,'fixtures','example.html')
+      pdfkit = PDFKit.new(Tempfile.new(file_path))
+      expect(pdfkit.source).to be_file
+      expect(pdfkit.source.to_s).to match /^#{Dir.tmpdir}/
+    end
+
     # Options
     ## options keys
     it "drops options without values" do
@@ -252,6 +259,12 @@ describe PDFKit do
       expect(pdfkit.command).to match /#{file_path} -$/
     end
 
+    it "specifies the path to the source if it is a tempfile" do
+      file_path = File.join(SPEC_ROOT,'fixtures','example.html')
+      pdfkit = PDFKit.new(Tempfile.new(file_path))
+      expect(pdfkit.command).to match /#{Dir.tmpdir}\S+ -$/
+    end
+
     it "specifies the path for the ouput if a path is given" do
       file_path = "/path/to/output.pdf"
       pdfkit = PDFKit.new("html")
@@ -384,9 +397,27 @@ describe PDFKit do
       end
     end
 
+    it "does not use xvfb-run wrapper by default" do
+      pdfkit = PDFKit.new('html')
+      expect(pdfkit.command).not_to include 'xvfb-run'
+    end
+
+    it "uses xvfb-run wrapper when option of using xvfb is configured" do
+      PDFKit.configure do |config|
+        config.use_xvfb = true
+      end
+
+      pdfkit = PDFKit.new('html')
+      expect(pdfkit.command).to include 'xvfb-run'
+
+      PDFKit.configure do |config|
+        config.use_xvfb = false
+      end
+    end
+
     context "on windows" do
       before do
-        allow_any_instance_of(PDFKit).to receive(:host_is_windows?).and_return(true)
+        allow(PDFKit::OS).to receive(:host_is_windows?).and_return(true)
       end
 
       it "escapes special windows characters" do
@@ -481,7 +512,7 @@ describe PDFKit do
     #NOTICE: This test is failed if use wkhtmltopdf-binary (0.9.9.1)
     it "throws an error if it is unable to connect" do
       pdfkit = PDFKit.new("http://google.com/this-should-not-be-found/404.html")
-      expect { pdfkit.to_pdf }.to raise_error /exitstatus=1/
+      expect { pdfkit.to_pdf }.to raise_error PDFKit::ImproperWkhtmltopdfExitStatus, /exitstatus=1/
     end
 
     it "does not throw an error if it is unable to connect", pending: 'this test works for wkhtmltopdf-binary (0.9.9.1)' do

data/spec/source_spec.rb

@@ -12,6 +12,11 @@ describe PDFKit::Source do
       expect(source).not_to be_url
     end
 
+    it "returns false if passed a tempfile" do
+      source = PDFKit::Source.new(::Tempfile.new(__FILE__))
+      expect(source).not_to be_url
+    end
+
     it "returns false if passed HTML" do
       source = PDFKit::Source.new('<blink>Oh Hai!</blink>')
       expect(source).not_to be_url
@@ -29,6 +34,11 @@ describe PDFKit::Source do
       expect(source).to be_file
     end
 
+    it "returns true if passed a tempfile" do
+      source = PDFKit::Source.new(::Tempfile.new(__FILE__))
+      expect(source).to be_file
+    end
+
     it "returns false if passed a url like string" do
       source = PDFKit::Source.new('http://google.com')
       expect(source).not_to be_file
@@ -51,6 +61,11 @@ describe PDFKit::Source do
       expect(source).not_to be_html
     end
 
+    it "returns false if passed a tempfile" do
+      source = PDFKit::Source.new(::Tempfile.new(__FILE__))
+      expect(source).not_to be_html
+    end
+
     it "returns false if passed a url like string" do
       source = PDFKit::Source.new('http://google.com')
       expect(source).not_to be_html
@@ -77,6 +92,11 @@ describe PDFKit::Source do
       source = PDFKit::Source.new(::File.new(__FILE__))
       expect(source.to_input_for_command).to match 'spec/source_spec.rb'
     end
+
+    it "returns the file path for tempfile sources" do
+      source = PDFKit::Source.new(file = ::Tempfile.new(__FILE__))
+      expect(source.to_input_for_command).to match file.path
+    end
   end
 
   describe "#to_s" do
@@ -90,6 +110,11 @@ describe PDFKit::Source do
       expect(source.to_s).to eq(__FILE__)
     end
 
+    it "returns a path if passed a tempfile" do
+      source = PDFKit::Source.new(file = ::Tempfile.new(__FILE__))
+      expect(source.to_s).to eq(file.path)
+    end
+
     it "returns the url if passed a url like string" do
       source = PDFKit::Source.new('http://google.com')
       expect(source.to_s).to eq('http://google.com')

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pdfkit
 version: !ruby/object:Gem::Version
-  version: 0.8.2
+  version: 0.8.4.3.1
 platform: ruby
 authors:
 - Jared Pace
@@ -9,104 +9,90 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-08-26 00:00:00.000000000 Z
+date: 2020-07-06 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.8
+        version: 4.1.11
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
-        version: 3.0.8
+        version: 4.1.11
 - !ruby/object:Gem::Dependency
   name: mocha
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.9.10
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.9.10
 - !ruby/object:Gem::Dependency
   name: rack-test
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.5.6
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: 0.5.6
-- !ruby/object:Gem::Dependency
-  name: i18n
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: 0.6.11
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ~>
-      - !ruby/object:Gem::Version
-        version: 0.6.11
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.9.2
+        version: 12.3.3
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.9.2
+        version: 12.3.3
 - !ruby/object:Gem::Dependency
   name: rdoc
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 4.0.1
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: 4.0.1
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ~>
+    - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
 description: Uses wkhtmltopdf to create PDFs using HTML
@@ -116,12 +102,13 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .document
-- .gitignore
-- .rspec
-- .ruby-gemset
-- .ruby-version
-- .travis.yml
+- ".document"
+- ".github/workflows/stale.yml"
+- ".gitignore"
+- ".rspec"
+- ".ruby-gemset"
+- ".ruby-version"
+- ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - LICENSE
@@ -130,21 +117,27 @@ files:
 - Rakefile
 - lib/pdfkit.rb
 - lib/pdfkit/configuration.rb
+- lib/pdfkit/html_preprocessor.rb
 - lib/pdfkit/middleware.rb
+- lib/pdfkit/os.rb
 - lib/pdfkit/pdfkit.rb
 - lib/pdfkit/source.rb
 - lib/pdfkit/version.rb
+- lib/pdfkit/wkhtmltopdf.rb
 - pdfkit.gemspec
 - spec/configuration_spec.rb
 - spec/fixtures/example.css
 - spec/fixtures/example.html
 - spec/fixtures/example_with_hex_symbol.css
+- spec/html_preprocessor_spec.rb
 - spec/middleware_spec.rb
+- spec/os_spec.rb
 - spec/pdfkit_spec.rb
 - spec/source_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/pdfkit/pdfkit
-licenses: []
+licenses:
+- MIT
 metadata: {}
 post_install_message: 
 rdoc_options: []
@@ -152,17 +145,17 @@ require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
-requirements: []
-rubyforge_project: pdfkit
-rubygems_version: 2.4.6
+requirements:
+- wkhtmltopdf
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: HTML+CSS -> PDF
@@ -171,7 +164,9 @@ test_files:
 - spec/fixtures/example.css
 - spec/fixtures/example.html
 - spec/fixtures/example_with_hex_symbol.css
+- spec/html_preprocessor_spec.rb
 - spec/middleware_spec.rb
+- spec/os_spec.rb
 - spec/pdfkit_spec.rb
 - spec/source_spec.rb
 - spec/spec_helper.rb