Better error handling of invalid URI's

- fixes #54
- normalizes error handling of invalid command line options
- wraps all Launchy.open exceptions in a Launchy::Errorjj

HISTORY.rdoc

@@ -3,6 +3,7 @@
 
 * Chagne XFCE detection to not depend on grep (copiousfreetime/launchy#52 - thanks bogdan)
 * Suppress forked process output (copiousfreetime/launchy#51)
+* Display help/usage if no url is given (copiousfreetime/launchy#54)
 
 == Version 2.1.2 - 2012-08-06
 

lib/launchy.rb

@@ -25,16 +25,16 @@ class << self
     def open(uri, options = {} )
       begin
         extract_global_options( options )
-        uri = Addressable::URI.parse(  uri )
-        app = Launchy::Application.handling( uri )
-        app.new.open( uri, options )
+        a_uri = Addressable::URI.parse(  uri )
+        raise Launchy::ArgumentError, "Invalid URI given: #{uri.inspect}" unless a_uri
+
+        app = Launchy::Application.handling( a_uri )
+        app.new.open( a_uri, options )
+      rescue Launchy::Error => le
+        raise le
       rescue Exception => e
-        msg = "Failure in opening #{uri} with options #{options.inspect}: #{e}"
-        Launchy.log "#{self.name} : #{msg}"
-        e.backtrace.each do |bt|
-          Launchy.log bt
-        end
-        $stderr.puts msg
+        msg = "Failure in opening uri #{uri.inspect} with options #{options.inspect}: #{e}"
+        raise Launchy::Error, msg
       end
     end
 

lib/launchy/cli.rb

@@ -57,14 +57,10 @@ def parser
     end
 
     def parse( argv, env )
-      begin
-        parser.parse!( argv )
-        return true
-      rescue ::OptionParser::ParseError => pe
-        $stderr.puts "#{parser.program_name}: #{pe}"
-        $stderr.puts "Try `#{parser.program_name} --help for more information."
-        return false
-      end
+      parser.parse!( argv )
+      return true
+    rescue ::OptionParser::ParseError => pe
+      error_output( pe )
     end
 
     def good_run( argv, env )
@@ -74,6 +70,18 @@ def good_run( argv, env )
       else
         return false
       end
+    rescue StandardError => e
+      error_output( e )
+    end
+
+    def error_output( error )
+      $stderr.puts "ERROR: #{error}"
+      Launchy.log "ERROR: #{error}"
+      error.backtrace.each do |bt|
+        Launchy.log bt
+      end
+      $stderr.puts "Try `#{parser.program_name} --help' for more information."
+      return false
     end
 
     def run( argv = ARGV, env = ENV )

lib/launchy/error.rb

@@ -2,4 +2,5 @@ module Launchy
   class Error < ::StandardError; end
   class ApplicationNotFoundError < Error; end
   class CommandNotFoundError < Error; end
+  class ArgumentError < Error; end
 end

spec/launchy_spec.rb

@@ -50,9 +50,8 @@
     Launchy.ruby_engine.must_equal 'myruby'
   end
 
-  it "prints an error on stderr when no scheme is found for the given uri" do
-    Launchy.open( "blah://something/invalid" )
-    $stderr.string.must_match( /Failure in opening/ )
+  it "raises an exception if no scheme is found for the given uri" do
+    lambda { Launchy.open( "blah://something/invalid" ) }.must_raise Launchy::ApplicationNotFoundError
   end
 
 end