Add ! methods that raise the original exceptions returned by the Vert…

…ex SOAP API

lib/vertex_client.rb

@@ -65,18 +65,34 @@ def quotation(payload)
       Resource::Quotation.new(payload).result
     end
 
+    def quotation!(payload)
+      Resource::Quotation.new(payload).result!
+    end
+
     def invoice(payload)
       Resource::Invoice.new(payload).result
     end
 
+    def invoice!(payload)
+      Resource::Invoice.new(payload).result!
+    end
+
     def distribute_tax(payload)
       Resource::DistributeTax.new(payload).result
     end
 
+    def distribute_tax!(payload)
+      Resource::DistributeTax.new(payload).result!
+    end
+
     def tax_area(payload)
       Resource::TaxArea.new(payload).result
     end
 
+    def tax_area!(payload)
+      Resource::TaxArea.new(payload).result!
+    end
+
     def circuit
       return unless circuit_configured?
 

lib/vertex_client/connection.rb

@@ -11,10 +11,13 @@ def initialize(endpoint, resource_key=nil)
 
     def request(payload)
       call_with_circuit_if_available do
-        client.call(
-          :vertex_envelope,
-          message: shell_with_auth.merge(payload)
-        )
+        raw_request(payload)
+      end
+    end
+
+    def request!(payload)
+      call_with_circuit_if_available(raise_exceptions: true) do
+        raw_request(payload)
       end
     end
 
@@ -32,6 +35,13 @@ def client
 
     private
 
+    def raw_request(payload)
+      client.call(
+        :vertex_envelope,
+        message: shell_with_auth.merge(payload)
+      )
+    end
+
     def config
       @config ||= VertexClient.configuration
     end
@@ -40,14 +50,20 @@ def resource_config
       config.resource_config[@resource_key] || {}
     end
 
-    def call_with_circuit_if_available
+    def call_with_circuit_if_available(raise_exceptions: false)
       if VertexClient.circuit
-        VertexClient.circuit.run(exception: false) { yield }
+        begin
+          VertexClient.circuit.run(exception: raise_exceptions) { yield }
+        rescue Circuitbox::ServiceFailureError => e
+          nil
+          raise e.original if raise_exceptions
+        end
       else
         begin
           yield
-        rescue => _e
+        rescue => e
           nil
+          raise e if raise_exceptions
         end
       end
     end

lib/vertex_client/resources/base.rb

@@ -8,7 +8,11 @@ def initialize(params)
       end
 
       def result
-        @result ||= (response ? formatted_response : fallback_response)
+        @result ||= formatted_response(response) || fallback_response
+      end
+
+      def result!
+        formatted_response(response!)
       end
 
       def config_key
@@ -18,15 +22,19 @@ def config_key
       private
 
       def response
-        @response ||= connection.request(@payload.transform)
+        connection.request(@payload.transform)
+      end
+
+      def response!
+        connection.request!(@payload.transform)
       end
 
       def connection
         @connection ||= Connection.new(self.class::ENDPOINT, config_key)
       end
 
-      def formatted_response
-        response_type.new(response)
+      def formatted_response(raw_response)
+        response_type.new(raw_response) if raw_response
       end
 
       def payload_type

test/connection_test.rb

@@ -62,6 +62,13 @@
       connection.client.expects(:call).with(:vertex_envelope, message: expected_message)
       connection.request({ just_a_test: true })
     end
+
+    it 'raises if config.raise_exceptions? is true' do
+      connection.send(:config).stubs(:raise_exceptions?).returns(true)
+      connection.client.stubs(:call).raises(StandardError)
+
+      assert_raises(StandardError) { connection.request({ just_a_test: true }) }
+    end
   end
 
   describe 'config' do

test/resources/base_test.rb

@@ -51,11 +51,33 @@ class VertexClient::Response::MyTest  < VertexClient::Response::Base; end;
     end
   end
 
+  describe 'result!' do
+    describe 'without exceptions' do
+      before do
+        resource.stubs(:response!).returns({ test: :ok })
+      end
+
+      it 'returns a formatted response' do
+        resource.expects(:formatted_response)
+        resource.result!
+      end
+    end
+
+    describe 'with exceptions' do
+      before do
+        resource.stubs(:response!).raises(StandardError)
+      end
+
+      it 'raises the exception' do
+        assert_raises(StandardError) { resource.result! }
+      end
+    end
+  end
+
   describe 'formatted_response' do
     it 'returns a new response of the proper type' do
-      resource.expects(:response).returns({test: :ok})
       VertexClient::Response::MyTest.expects(:new).with({test: :ok})
-      resource.send(:formatted_response)
+      resource.send(:formatted_response, {test: :ok})
     end
   end
 end

test/vertex_client_test.rb

@@ -23,27 +23,55 @@
     assert VertexClient.quotation(working_quote_params)
   end
 
+  it 'does a quotation!' do
+    VertexClient::Resource::Quotation.expects(:new)
+      .with(working_quote_params)
+      .returns(stub(result!: true))
+    assert VertexClient.quotation!(working_quote_params)
+  end
+
   it 'does invoice' do
     VertexClient::Resource::Invoice.expects(:new)
       .with(working_quote_params)
       .returns(stub(result: true))
     assert VertexClient.invoice(working_quote_params)
   end
 
+  it 'does invoice!' do
+    VertexClient::Resource::Invoice.expects(:new)
+      .with(working_quote_params)
+      .returns(stub(result!: true))
+    assert VertexClient.invoice!(working_quote_params)
+  end
+
   it 'does distribute_tax' do
     VertexClient::Resource::DistributeTax.expects(:new)
       .with(working_quote_params)
       .returns(stub(result: true))
     assert VertexClient.distribute_tax(working_quote_params)
   end
 
+  it 'does distribute_tax!' do
+    VertexClient::Resource::DistributeTax.expects(:new)
+      .with(working_quote_params)
+      .returns(stub(result!: true))
+    assert VertexClient.distribute_tax!(working_quote_params)
+  end
+
   it 'does tax_area' do
     VertexClient::Resource::TaxArea.expects(:new)
       .with(working_quote_params)
       .returns(stub(result: true))
     assert VertexClient.tax_area(working_quote_params)
   end
 
+  it 'does tax_area!' do
+    VertexClient::Resource::TaxArea.expects(:new)
+      .with(working_quote_params)
+      .returns(stub(result!: true))
+    assert VertexClient.tax_area!(working_quote_params)
+  end
+
   describe 'circuit' do
     it 'only exists if circuit_config is provided to configuration' do
       VertexClient.configuration.circuit_config = nil