flexirest 1.7.9 → 1.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b0a27d77149e8eeba7815235367970828643b622a0925425c0cd2f18a12740dc
-  data.tar.gz: 6c12446837777d3bef7f37d6def7be613eeb98c380406a16f4e4f41e47ae0443
+  metadata.gz: 0646b0d77ed69a83a024542226cd74833092523296edd7c6b48e6524a376c67c
+  data.tar.gz: d645d44a87ff5fabf8a7b9d6306f48bd6b899a7f6c966ded3d37a7456e091edd
 SHA512:
-  metadata.gz: 722724ba195f14caa98a856beb93d527e1e69b8b16b247dc9bf4eabffb80f262ff930ba14bab9c01517bfcaf757e8c53f322f1efa454007ad3819d87214193e5
-  data.tar.gz: 779f448b1ed8c1f2f7244aa672b29e24496949088c3a4248c04a196336c79eeb8733691fd397521527b37ecdacc1368e6ae4345b6dfdeef374d4d56f6d9e58f3
+  metadata.gz: 96a26d232291398f3bc7b13927ecd80175cdf3eea0293fd3d52c317ab55257e8269362ab4b5db0ac87d0f63621777518b6ad3758385bdda84a1782546e892dc3
+  data.tar.gz: c3ee959ee87e8f325d8ff16f1e1a4e2d2746dc6765e159193194fd85aa414ac3c8b5b8491a32d261bca0c982d9feb6c2caafcfa0398b591b9eae5cb2e68c2ef8

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 1.8.0
+
+Features:
+
+- For the weird situations where the remote API expects some weird formatted input, you can now specify a `request_body_type` of plain and pass in to the call `body` (and optionally `content_type`) to pass it untouched to the API. The [docs](docs/body-types.md) have been updated.
+
 ## 1.7.9
 
 Features:

data/docs/body-types.md

@@ -27,6 +27,16 @@ class Person < Flexirest::Base
 end
 ```
 
+If your API expects some weird formatting on the requests, but you still want to use Flexirest for caching, response parsing, other models, etc you can pass `:plain` as the request body type either at the class level or method mapping level, and this will be passed through to the API. By default `plain` requests are sent with the `Content-Type: text/plain` header, but you can override this with `:content_type` when calling the mapped method.
+
+```ruby
+class Person < Flexirest::Base
+  put :save, '/person/:id/logs', request_body_type: :plain
+end
+
+Person.save(id: 1, body: '["Something here"]', 
+            content_type: "application/json")
+```
 
 -----
 

data/lib/flexirest/active_model_form_support.rb

@@ -0,0 +1,23 @@
+module Flexirest::ActiveModelFormSupport
+  extend ActiveSupport::Concern
+  include ActiveModel::AttributeAssignment
+  include ActiveModel::Validations
+  include ActiveModel::Conversion
+
+  included do
+    extend ActiveModel::Naming
+    extend ActiveModel::Translation
+  end
+
+  def persisted?
+    !dirty?
+  end
+
+  def new_record?
+    id.blank?
+  end
+
+  def errors
+    ActiveSupport::HashWithIndifferentAccess.new([]).merge(self[:errors] || {})
+  end
+end

data/lib/flexirest/request.rb

@@ -430,6 +430,10 @@ module Flexirest
           end
         end
         headers["Content-Type"] ||= "application/json; charset=utf-8"
+      elsif request_body_type == :plain && @post_params[:body].present?
+        @body = @post_params[:body]
+        headers["Content-Type"] ||= "text/plain"
+        headers["Content-Type"] = @post_params[:content_type] if @post_params[:content_type].present?
       end
     end
 

data/lib/flexirest/version.rb

@@ -1,3 +1,3 @@
 module Flexirest
-  VERSION = "1.7.9"
+  VERSION = "1.8.0"
 end

data/spec/lib/request_spec.rb

@@ -27,11 +27,11 @@ describe Flexirest::Request do
         end
       end
 
-      get :all, "/", :has_many => {:expenses => ExampleOtherClient}
-      get :flat, "/", :params_encoder => :flat
+      get :all, "/", has_many: {expenses: ExampleOtherClient}
+      get :flat, "/", params_encoder: :flat
       get :array, "/johnny", array: [:likes, :dislikes]
-      get :babies, "/babies", :has_many => {:children => ExampleOtherClient}
-      get :single_association, "/single", :has_one => {:single => ExampleSingleClient}, :has_many => {:children => ExampleOtherClient}
+      get :babies, "/babies", has_many: {children: ExampleOtherClient}
+      get :single_association, "/single", has_one: {single: ExampleSingleClient}, has_many: {children: ExampleOtherClient}
       get :headers, "/headers"
       get :cancel_callback, "/cancel-callback"
       put :headers_default, "/headers_default"
@@ -54,12 +54,12 @@ describe Flexirest::Request do
       get :requires, "/requires", requires:[:name, :age]
       patch :only_changed_1, "/changed1", only_changed: true
       patch :only_changed_2, "/changed2", only_changed: [:debug1, :debug2]
-      patch :only_changed_3, "/changed3", only_changed: { :debug1 => false, :debug2 => true }
+      patch :only_changed_3, "/changed3", only_changed: { debug1: false, debug2: true }
     end
 
     class ExampleLoadBalancedClient < Flexirest::Base
       base_url ["http://api1.example.com", "http://api2.example.com"]
-      get :all, "/", :has_many => {:expenses => ExampleOtherClient}
+      get :all, "/", has_many: {expenses: ExampleOtherClient}
     end
 
     class AuthenticatedExampleClient < Flexirest::Base
@@ -135,7 +135,7 @@ describe Flexirest::Request do
       base_url "http://www.example.com"
       lazy_load!
       get :fake, "/fake", fake:"{\"result\":true, \"list\":[1,2,3,{\"test\":true}], \"child\":{\"grandchild\":{\"test\":true}}}"
-      get :lazy_test, "/does-not-matter", fake:"{\"people\":[\"http://www.example.com/some/url\"]}", :lazy => [:people]
+      get :lazy_test, "/does-not-matter", fake:"{\"people\":[\"http://www.example.com/some/url\"]}", lazy: [:people]
     end
 
     class VerboseExampleClient < ExampleClient
@@ -169,9 +169,9 @@ describe Flexirest::Request do
         {
           "response": {
             "data": {
-              "object": {            
+              "object": {
                 "title": "Example Multi Level Feed"
-              } 
+              }
             }
           }
         }
@@ -340,12 +340,12 @@ describe Flexirest::Request do
 
   it "should pass through 'array type' get parameters" do
     expect_any_instance_of(Flexirest::Connection).to receive(:get).with("/?include%5B%5D=your&include%5B%5D=friends", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"result\":true}", response_headers:{})))
-    ExampleClient.all :include => [:your,:friends]
+    ExampleClient.all include: [:your,:friends]
   end
 
   it "should pass through 'array type' get parameters using the same parameter name if a flat param_encoder is chosen" do
     expect_any_instance_of(Flexirest::Connection).to receive(:get).with("/?include=your&include=friends", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"result\":true}", response_headers:{})))
-    ExampleClient.flat :include => [:your,:friends]
+    ExampleClient.flat include: [:your,:friends]
   end
 
   it "should encode the body in a form-encoded format by default" do
@@ -359,12 +359,28 @@ describe Flexirest::Request do
     ExampleClient.update id:1234, debug:true, test:'foo'
   end
 
-  it "should encode the body in a JSON format if specified" do
+  it "should encode the body wrapped in a root element in a JSON format if specified" do
     expect_any_instance_of(Flexirest::Connection).to receive(:put).with("/put/1234", %q({"example":{"debug":true,"test":"foo"}}), an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"result\":true}", response_headers:{})))
     ExampleClient.request_body_type :json
     ExampleClient.wrapped id:1234, debug:true, test:'foo'
   end
 
+  it "should pass the body untouched if plain format is specified on the class" do
+    header_expectation = {headers: {"Accept"=>"application/hal+json, application/json;q=0.5", "Content-Type"=>"text/plain"}, api_auth: {api_auth_access_id: "id123", api_auth_secret_key: "secret123", api_auth_options: {}}}
+    expect_any_instance_of(Flexirest::Connection).to receive(:put).with("/put/1234", "debug:true|test:'foo'", header_expectation).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"result\":true}", response_headers:{})))
+    ExampleClient.request_body_type :plain
+    ExampleClient.update id:1234, body: "debug:true|test:'foo'"
+  end
+
+  it "should use the content type specified if plain format is specified on the class" do
+    header_expectation = {headers: {"Accept"=>"application/hal+json, application/json;q=0.5", "Content-Type"=>"application/flexirest"}, api_auth: {api_auth_access_id: "id123", api_auth_secret_key: "secret123", api_auth_options: {}}}
+    expect_any_instance_of(Flexirest::Connection).to receive(:put).
+      with("/put/1234", "debug:true|test:'foo'", header_expectation).
+      and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"result\":true}", response_headers:{})))
+    ExampleClient.request_body_type :plain
+    ExampleClient.update id:1234, body: "debug:true|test:'foo'", content_type: "application/flexirest"
+  end
+
   it "should wrap elements if specified, in form-encoded format" do
     expect_any_instance_of(Flexirest::Connection).to receive(:put).with("/put/1234", %q(example%5Bdebug%5D=true&example%5Btest%5D=foo), an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"result\":true}", response_headers:{})))
     ExampleClient.wrapped id:1234, debug:true, test:'foo'
@@ -897,7 +913,7 @@ describe Flexirest::Request do
   end
 
   it "should raise an exception if you try to pass in an unsupport method" do
-    method = {:method => :wiggle, url:"/"}
+    method = {method: :wiggle, url:"/"}
     class RequestFakeObject < Flexirest::Base
       base_url "http://www.example.com/"
 
@@ -944,7 +960,7 @@ describe Flexirest::Request do
 
   it "should retry if an after_request callback returns :retry" do
     stub_request(:get, "http://www.example.com/do_me_twice").
-      to_return(:status => 200, :body => "", :headers => {})
+      to_return(status: 200, body: "", headers: {})
     RetryingExampleClient.reset_retries
     RetryingExampleClient.do_me_twice
     expect(RetryingExampleClient.retries).to eq(2)
@@ -952,9 +968,9 @@ describe Flexirest::Request do
 
   it "should allow a second call and then retry if an after_request callback returns :retry" do
     stub_request(:get, "http://www.example.com/first_call").
-      to_return(:status => 200, :body => "", :headers => {})
+      to_return(status: 200, body: "", headers: {})
     stub_request(:get, "http://www.example.com/second_call").
-      to_return(:status => 200, :body => "", :headers => {})
+      to_return(status: 200, body: "", headers: {})
     RetryingExampleClient.reset_retries
     RetryingExampleClient.first_call
     expect(RetryingExampleClient.retries).to eq(2)
@@ -1023,7 +1039,7 @@ describe Flexirest::Request do
     end
 
     it "should recognise a HAL response" do
-      method = {:method => :get, url:"/"}
+      method = {method: :get, url:"/"}
       class RequestFakeObject
         def base_url
           "http://www.example.com/"
@@ -1137,7 +1153,7 @@ describe Flexirest::Request do
   it "should ignore a specified root element" do
     expect(IgnoredRootExampleClient.root.title).to eq("Example Feed")
   end
-  
+
   it "should ignore a specified multi-level root element" do
     expect(IgnoredMultiLevelRootExampleClient.multi_level_root.title).to eq("Example Multi Level Feed")
   end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flexirest
 version: !ruby/object:Gem::Version
-  version: 1.7.9
+  version: 1.8.0
 platform: ruby
 authors:
 - Andy Jeffries
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-07-23 00:00:00.000000000 Z
+date: 2019-09-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -275,6 +275,7 @@ files:
 - docs/xml-responses.md
 - flexirest.gemspec
 - lib/flexirest.rb
+- lib/flexirest/active_model_form_support.rb
 - lib/flexirest/associations.rb
 - lib/flexirest/attribute_parsing.rb
 - lib/flexirest/base.rb