Rewrote configuration

lib/hijacker.rb

@@ -1,5 +1,6 @@
 require 'drb'
 require 'trollop'
+require 'hijacker/exceptions'
 require 'hijacker/config'
 require 'hijacker/handler'
 

lib/hijacker/config.rb

@@ -11,8 +11,8 @@ def uri(drb)
     def drb_uri
       begin
         @@drb_uri
-      rescue
-        raise "Neither a global nor a local Hijacker server URI is configured. Please refer to the README to find out how to do this."
+      rescue NameError
+        raise UndefinedUriError, "Neither a global nor a local Hijacker server URI is configured. Please refer to the README to find out how to do this."
       end
     end
   end

lib/hijacker/exceptions.rb

@@ -0,0 +1,3 @@
+module Hijacker
+  class UndefinedUriError < StandardError; end;
+end

spec/hijacker/config_spec.rb

@@ -3,11 +3,34 @@
 describe Hijacker, "configuration" do
 
   describe "#configure" do
-    it 'accepts a block with the \'uri\' configuration option' do
-      Hijacker.configure do
-        uri 'druby://localhost:8787'
+    it 'evaluates the passed block' do
+      block = Proc.new {}
+      Hijacker.should_receive(:instance_eval).with(&block).once
+      Hijacker.configure(&block) 
+    end
+  end
+
+  describe "#uri" do
+    it 'assigns the DRb uri as a class variable' do
+      Hijacker.uri 'druby://localhost:8787'
+      Hijacker.class_variable_get(:@@drb_uri).should == 'druby://localhost:8787'
+    end
+  end
+
+  describe "#drb_uri" do
+    context "when the class variable is set" do
+      it 'is an accessor to it' do
+        Hijacker.class_variable_set(:@@drb_uri, 'druby://localhost:8787')
+        Hijacker.drb_uri.should == 'druby://localhost:8787'
+      end
+    end
+    context "otherwise" do
+      it 'raises an error' do
+        Hijacker.remove_class_variable(:@@drb_uri)
+        expect {
+          Hijacker.drb_uri
+        }.to raise_error(Hijacker::UndefinedUriError)
       end
-      Hijacker.drb_uri.should == 'druby://localhost:8787'
     end
   end