mno-enterprise-core 2.0.3 → 2.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4ea3b96e6b15c4fda3cdc009b995ddfe93bb9fb4
-  data.tar.gz: 6aa3d2e3497f8ecb676d6b124133902672e97408
+  metadata.gz: 2db1615a4972df4d796099d38b3e492585349bb6
+  data.tar.gz: cd2d825dc1e24d86fea3b5cde5af1be440341c53
 SHA512:
-  metadata.gz: f7aeb4bf0ad920fa897d4d5376ce1611bafe6ef74df40a7ebdb2855c973d184561cb58ddb347c5546c444e625aa823cbe28399f3dcbf10197817b255773d6ced
-  data.tar.gz: dd9375dda66ff69af79361cb4ade7456ba7998afc3cf882aea7712b038abc326eb2a0dcf7861c86e6209fdbf466e3b1073f78d09bd4460db202367d645b17534
+  metadata.gz: 2b7605252044117c2c99749f54cf4c8d198063b371ec010a78a45d32b5d89483dca8e540ccb4dadb0f2690e754448ef11c5b9a41e520f16e76b10cfd6db158d3
+  data.tar.gz: aabb872ecfe84d47f813ae1f363cfaa8c1009336268fa028e61068bdb59f6a6abd4fbc80af932caf68828e0377a1a686b3cca4abeba9bbc589b79a79b9f88184

data/lib/her_extension/middleware/mnoe_raise_error.rb

@@ -0,0 +1,21 @@
+module Her
+  module Middleware
+    # This middleware will raise errors based on the response status
+    # Same as Faraday::Response::RaiseError, except it will catch specific
+    # errors codes rather than everything in 400..600
+    class MnoeRaiseError < Faraday::Response::RaiseError
+      def on_complete(env)
+        case env[:status]
+        when 407
+          # mimic the behavior that we get with proxy requests with HTTPS
+          raise Faraday::Error::ConnectionFailed,
+                %(407 "Proxy Authentication Required ")
+        when 502..504
+          raise Faraday::Error::ConnectionFailed, response_values(env)
+        when 401, 500
+          raise Faraday::Error::ClientError, response_values(env)
+        end
+      end
+    end
+  end
+end

data/lib/her_extension/model/relation.rb

@@ -26,6 +26,9 @@ module Her
       # Use filter instead of raw parameters
       def where(params = {})
         return self if !params || params.empty?
+        # If a value is an empty array, it'll be excluded when calling params.to_query, so convert it to nil instead
+        params.each { |k, v| params[k] = v.presence if v.is_a?(Array) }
+
         self.params[:filter] ||= {}
         self.params[:filter].merge!(params)
         self

data/lib/mno_enterprise/core.rb

@@ -18,6 +18,7 @@ require "her_extension/model/associations/association"
 require "her_extension/model/associations/association_proxy"
 require "her_extension/model/associations/has_many_association"
 require "her_extension/middleware/mnoe_api_v1_parse_json"
+require "her_extension/middleware/mnoe_raise_error"
 require "faraday_middleware"
 require "mno_enterprise/engine"
 
@@ -266,6 +267,9 @@ module MnoEnterprise
 
         # Adapter
         c.use Faraday::Adapter::NetHttpNoProxy
+
+        # Error Handling
+        c.use Her::Middleware::MnoeRaiseError
       end
     end
 end

data/lib/mno_enterprise/testing_support/mno_enterprise_api_test_helper.rb

@@ -124,7 +124,8 @@ module MnoEnterpriseApiTestHelper
 
         # Response
         c.use Her::Middleware::MnoeApiV1ParseJson
-      
+        c.use Her::Middleware::MnoeRaiseError
+
         # Add stubs on the test adapter
         c.use MnoeFaradayTestAdapter do |receiver|
           @_stub_list.each do |key,stub|

data/lib/mno_enterprise/version.rb

@@ -1,3 +1,3 @@
 module MnoEnterprise
-  VERSION = '2.0.3'
+  VERSION = '2.0.4'
 end

data/spec/lib/her_extension/model/relation_spec.rb

@@ -3,5 +3,19 @@ require 'rails_helper'
 module MnoEnterprise
   RSpec.describe Her::Model::Relation do
     pending "add specs for Her::Model::Relation monkey patch: #{__FILE__}"
+
+    let(:dummy_class) { Class.new { include Her::Model } }
+
+    describe '.where' do
+      it 'adds the filter to params[:filter]' do
+        rel = Her::Model::Relation.new(dummy_class).where('uid.in' => [1, 2], 'foo' => 'bar')
+        expect(rel.params[:filter]).to eq('uid.in' => [1, 2], 'foo' => 'bar')
+      end
+
+      it 'replaces empty array values with nil' do
+        rel = Her::Model::Relation.new(dummy_class).where('id.in' => [])
+        expect(rel.params[:filter]).to eq('id.in' => nil)
+      end
+    end
   end
 end

data/spec/models/mno_enterprise/base_resource_spec.rb

@@ -2,6 +2,28 @@ require 'rails_helper'
 
 module MnoEnterprise
   RSpec.describe BaseResource, type: :model do
+    describe 'Error Handling' do
+      class Test < BaseResource; end
+
+      context 'Connection Errors' do
+        ((502..504).to_a<<407).each do |code|
+          it "handles #{code} error" do
+            api_stub_for(get: "/tests/1", code: code)
+            expect { Test.find(1) }.to raise_error(Faraday::ConnectionFailed)
+          end
+        end
+      end
+
+      context 'Server Errors' do
+        [401, 500].each do |code|
+          it "handles #{code} error" do
+            api_stub_for(get: "/tests/1", code: code)
+            expect { Test.find(1) }.to raise_error(Faraday::Error::ClientError)
+          end
+        end
+      end
+    end
+
     describe '#cache_key' do
       context 'for existing record' do
         let(:user) { build(:user) }

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: mno-enterprise-core
 version: !ruby/object:Gem::Version
-  version: 2.0.3
+  version: 2.0.4
 platform: ruby
 authors:
 - Arnaud Lachaume
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-07-12 00:00:00.000000000 Z
+date: 2016-08-29 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -313,6 +313,7 @@ files:
 - lib/generators/mno_enterprise/templates/scripts/upstart/app.conf
 - lib/her_extension/her_orm_adapter.rb
 - lib/her_extension/middleware/mnoe_api_v1_parse_json.rb
+- lib/her_extension/middleware/mnoe_raise_error.rb
 - lib/her_extension/model/associations/association.rb
 - lib/her_extension/model/associations/association_proxy.rb
 - lib/her_extension/model/associations/has_many_association.rb