http_stub 0.9.4 → 0.9.5

data/lib/http_stub/configurer/command_processor.rb

@@ -8,8 +8,26 @@ module HttpStub
       end
 
       def process(command)
-        response = Net::HTTP.start(@configurer.get_host, @configurer.get_port) { |http| http.request(command.request) }
-        raise "Error occurred #{command.description}: #{response.message}" unless response.code == "200"
+        begin
+          response = Net::HTTP.start(host, port) { |http| http.request(command.request) }
+          raise "#{error_message_prefix(command)}: #{response.code} #{response.message}" unless response.code == "200"
+        rescue Exception => exc
+          raise "#{error_message_prefix(command)}: #{exc}"
+        end
+      end
+
+      private
+
+      def host
+        @configurer.get_host
+      end
+
+      def port
+        @configurer.get_port
+      end
+
+      def error_message_prefix(command)
+        "Error occurred #{command.description} whilst configuring #{@configurer.get_base_uri}: "
       end
 
     end

data/lib/http_stub/version.rb

@@ -1,3 +1,3 @@
 module HttpStub
-  VERSION = "0.9.4"
+  VERSION = "0.9.5"
 end

data/spec/lib/http_stub/configurer/command_processor_integration_spec.rb

@@ -1,26 +1,56 @@
-describe HttpStub::Configurer::CommandProcessor, "when the server is running" do
-  include_context "server integration"
+describe HttpStub::Configurer::CommandProcessor do
 
-  let(:configurer) { double(HttpStub::Configurer, get_host: "localhost", get_port: 8001) }
   let(:command) { HttpStub::Configurer::Command.new(request: request, description: "performing an operation") }
+  let(:server_base_uri) { "http://localhost:8001" }
+  let(:configurer) do
+    double(HttpStub::Configurer, get_base_uri: server_base_uri, get_host: "localhost", get_port: 8001)
+  end
 
   let(:command_processor) { HttpStub::Configurer::CommandProcessor.new(configurer) }
 
-  describe "#process" do
+  context "when the server is running" do
+
+    include_context "server integration"
+
+    describe "#process" do
+
+      describe "when the server responds with a 200 response" do
 
-    describe "when the server responds with a 200 response" do
+        let(:request) { Net::HTTP::Get.new("/stubs") }
 
-      let(:request) { Net::HTTP::Get.new("/stubs") }
+        it "should execute without error" do
+          lambda { command_processor.process(command) }.should_not raise_error
+        end
+
+      end
+
+      describe "when the server responds with a non-200 response" do
+
+        let(:request) { Net::HTTP::Get.new("/causes_error") }
+
+        it "should raise an exception that includes the server base URI" do
+          lambda { command_processor.process(command) }.should raise_error(/#{server_base_uri}/)
+        end
+
+        it "should raise an exception that includes the commands description" do
+          lambda { command_processor.process(command) }.should raise_error(/performing an operation/)
+        end
 
-      it "should execute without error" do
-        lambda { command_processor.process(command) }.should_not raise_error
       end
 
     end
 
-    describe "when the server responds with a non-200 response" do
+  end
+
+  context "when the server is unavailable" do
 
-      let(:request) { Net::HTTP::Get.new("/causes_error") }
+    let(:request) { Net::HTTP::Get.new("/does/not/exist") }
+
+    describe "#process" do
+
+      it "should raise an exception that includes the server base URI" do
+        lambda { command_processor.process(command) }.should raise_error(/#{server_base_uri}/)
+      end
 
       it "should raise an exception that includes the commands description" do
         lambda { command_processor.process(command) }.should raise_error(/performing an operation/)

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: http_stub
 version: !ruby/object:Gem::Version
-  version: 0.9.4
+  version: 0.9.5
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-15 00:00:00.000000000 Z
+date: 2013-10-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sinatra
@@ -115,7 +115,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '4.1'
+        version: '4.2'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -123,7 +123,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '4.1'
+        version: '4.2'
 - !ruby/object:Gem::Dependency
   name: travis-lint
   requirement: !ruby/object:Gem::Requirement
@@ -195,7 +195,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '0.11'
+        version: '0.12'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -203,7 +203,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '0.11'
+        version: '0.12'
 - !ruby/object:Gem::Dependency
   name: wait_until
   requirement: !ruby/object:Gem::Requirement
@@ -325,7 +325,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 2243155845271337400
+      hash: 3958008164617711389
 requirements: []
 rubyforge_project: http_stub
 rubygems_version: 1.8.25