Skip to content

Commit

Permalink
Remove two unnecessary and now invalid tests
Browse files Browse the repository at this point in the history
We're far more user friendly than we were before so let's not assert
that we can misconfigure wkhtmltopdf and pdfkit.
  • Loading branch information
Amy Unger and Ian Cordasco committed Oct 10, 2015
1 parent e700b45 commit 7be8e09
Showing 1 changed file with 0 additions and 11 deletions.
11 changes: 0 additions & 11 deletions spec/configuration_spec.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,17 +3,6 @@
describe PDFKit::Configuration do
subject { PDFKit::Configuration.new }
describe "#wkhtmltopdf" do
it "can be configured" do
subject.wkhtmltopdf = '/my/bin/wkhtmltopdf'
expect(subject.wkhtmltopdf).to eql '/my/bin/wkhtmltopdf'
end

# This test documents existing functionality. Feel free to fix.
it "can be poorly configured" do
subject.wkhtmltopdf = 1234
expect(subject.wkhtmltopdf).to eql 1234
end

context "when not explicitly configured" do
it "detects the existance of bundler" do
# Test assumes bundler is installed in your test environment
Expand Down

0 comments on commit 7be8e09

Please sign in to comment.