changed spec tests to Test Unit Assertions and added more tests

Rakefile

@@ -1,9 +1,23 @@
-require 'rubygems'
-require 'spec/rake/spectask'
+require 'rake'
+require 'rake/testtask'
+require 'rake/rdoctask'
 
-Spec::Rake::SpecTask.new do |t|
-  t.warning = true
-  t.rcov = true
+desc 'Default: run unit tests.'
+task :default => :test
+
+desc 'Test the wicked_pdf plugin.'
+Rake::TestTask.new(:test) do |t|
+  t.libs << 'lib'
+  t.libs << 'test'
+  t.pattern = 'test/**/*_test.rb'
+  t.verbose = true
 end
 
-task :default => :spec
+desc 'Generate documentation for the wicked_pdf plugin.'
+Rake::RDocTask.new(:rdoc) do |rdoc|
+  rdoc.rdoc_dir = 'rdoc'
+  rdoc.title    = 'WickedPdf'
+  rdoc.options << '--line-numbers' << '--inline-source'
+  rdoc.rdoc_files.include('README')
+  rdoc.rdoc_files.include('lib/**/*.rb')
+end

lib/pdf_helper.rb

@@ -35,12 +35,6 @@ def make_and_send_pdf(pdf_name, options = {})
       end
     end
 
-    def parse_options opts
-      s = "#{parse_header_footer(:header => opts.delete(:header), :footer => opts.delete(:footer), :layout => opts[:layout])} "
-      s += "#{parse_toc(opts.delete(:toc))} #{parse_outline(opts.delete(:outline))} "
-      s + "#{parse_margins(opts.delete(:margin))} #{parse_others(opts)} "
-    end
-
     def make_option name, value, type=:string
       "--#{name.gsub('_', '-')} " + case type
         when :boolean: ""
@@ -53,6 +47,12 @@ def make_options opts, names, prefix="", type=:string
       names.collect {|o| make_option("#{prefix.blank? ? "" : prefix + "-"}#{o.to_s}", opts[o], type) unless opts[o].blank?}.join
     end
 
+    def parse_options opts
+      "#{parse_header_footer(:header => opts.delete(:header), :footer => opts.delete(:footer), :layout => opts[:layout])} " + \
+      "#{parse_toc(opts.delete(:toc))} #{parse_outline(opts.delete(:outline))} " + \
+      "#{parse_margins(opts.delete(:margin))} #{parse_others(opts)} "
+    end
+
     def parse_header_footer opts
       r=""
       [:header, :footer].collect do |hf| 

lib/wicked_pdf.rb

@@ -9,9 +9,11 @@ class WickedPdf
 
   def initialize(wkhtmltopdf_binary_path = nil)
     @exe_path = wkhtmltopdf_binary_path 
-    @exe_path ||= WICKED_PDF[:exe_path] unless WICKED_PDF.blank?
+    @exe_path ||= WICKED_PDF[:exe_path] unless WICKED_PDF.empty?
     @exe_path ||= `which wkhtmltopdf`.chomp
-    raise "Location of wkhtmltopdf unknown" if @exe_path.blank?
+    raise "Location of wkhtmltopdf unknown" if @exe_path.empty? 
+    raise "Bad wkhtmltopdf's path" unless File.exists?(@exe_path)
+    raise "Wkhtmltopdf is not executable" unless File.executable?(@exe_path)
   end
 
   def pdf_from_string(string, options=nil)