flexirest 1.10.10 → 1.10.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 55d28434a66d8b6b7b6421c2778bb23c0f548c2f80d1b37bcab0dc6b95188bc6
-  data.tar.gz: 3659a7ab74c41da1846372ebdd406cb232579acedbe31c26b304bebbacbb0003
+  metadata.gz: 60510339b01a0ade8d9cd94686ecec5f431bcb58570c66b48fdc76075faf4d4f
+  data.tar.gz: 203e88d288ba8fa7f46f757a9aa24177908df116a272b754dc9bd685b19302db
 SHA512:
-  metadata.gz: 2ad7882b187e5906b49acbca900fbd8a954e30533ef280d8cb61d82b9a4a9a1ef3307e355fbc293a050672b72d176ef32c3ce11101974c5116681fc9b1ce1015
-  data.tar.gz: c3f789601f482846bbd495a89796d907dc6c8e06d11af81170f0176e700164594892fa9ea4f842be8e98997a06719292a7ebfa0f53527f4845149fb1f13251fc
+  metadata.gz: f2d51bcbce5d390baa8140cc222cbe69bfb873cb6db399a3e955358b36b39aea6241df15abd8458b79e03d87e3e4643551d2d809dabd0a4ae92c9131e9277da1
+  data.tar.gz: 73445e4f9be17c311354112a41e7a9290695da228d853e1a67ea063d8aae8a1370f7bf4991fa43bd70a264b608c94416ad005929aa055863686ca919c6fe19ce

data/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 1.10.11
+
+Bugfix:
+
+- HTTPClientException's instance body was return nil instead of the response. (thanks to @pinifloyd for the issue and PR)
+
 ## 1.10.10
 
 Bugfix:

data/docs/body-types.md

@@ -45,10 +45,10 @@ class Person < Flexirest::Base
   put :save, '/person/:id/logs', request_body_type: :plain
 end
 
-Person.save(id: 1, body: '["Something here"]', 
+Person.save(id: 1, body: '["Something here"]',
             content_type: "application/json")
 ```
 
 -----
 
-[< Authentication](authentication.md) | [Parallel requests >](parallel-requests.md)
+[< Authentication](authentication.md) | [Empty body handling >](empty-body-handling.md)

data/docs/empty-body-handling.md

@@ -20,3 +20,7 @@ class Person < Flexirest::Base
   put :save, "/people/:id", ignore_empty_response: true
 end
 ```
+
+-----
+
+[< Body types](body-types.md) | [Parallel requests >](parallel-requests.md)

data/docs/parallel-requests.md

@@ -25,4 +25,4 @@ puts @employers.size #=> 7
 
 -----
 
-[< Body types](body-types.md) | [Faking calls >](faking-calls.md)
+[< Empty body handling](empty-body-handling.md) | [Faking calls >](faking-calls.md)

data/docs/using-callbacks.md

@@ -105,6 +105,7 @@ class Animal < Flexirest::Base
       Flexirest::Logger.info("  \033[1;4;32m#{Flexirest.name}\033[0m Invalidating cache for #{self.class.name} #{request.url}")
       Rails.cache.delete("#{self.class.name}:#{request.url}")
     end
+    nil
   end
 
   def cache(name, response)

data/lib/flexirest/base_without_validation.rb

@@ -66,7 +66,7 @@ module Flexirest
       @dirty_attributes.keys
     end
 
-    # Returns hash of old and new vaules for each changed field
+    # Returns hash of old and new values for each changed field
     def changes
       @dirty_attributes
     end

data/lib/flexirest/request.rb

@@ -404,7 +404,7 @@ module Flexirest
           end
         end
         if missing.any?
-          raise Flexirest::MissingParametersException.new("The following parameters weren't specifed: #{missing.join(", ")}")
+          raise Flexirest::MissingParametersException.new("The following parameters weren't specified: #{missing.join(", ")}")
         end
       end
     end
@@ -434,7 +434,7 @@ module Flexirest
       end
 
       if missing.present?
-        raise Flexirest::MissingParametersException.new("The following parameters weren't specifed: #{missing.join(", ")}")
+        raise Flexirest::MissingParametersException.new("The following parameters weren't specified: #{missing.join(", ")}")
       end
     end
 
@@ -934,7 +934,7 @@ module Flexirest
   end
 
   class HTTPException < RequestException
-    attr_accessor :status, :result, :request_url, :body, :raw_response
+    attr_accessor :status, :result, :request_url, :body
     def initialize(options)
       @status = options[:status]
       @result = options[:result]
@@ -942,7 +942,7 @@ module Flexirest
       @body = options[:raw_response]
       @method = options[:method]
     end
-    alias_method :body, :raw_response
+    alias_method :raw_response, :body
 
     def message
       method = @method.try(:upcase)

data/lib/flexirest/version.rb

@@ -1,3 +1,3 @@
 module Flexirest
-  VERSION = "1.10.10"
+  VERSION = "1.10.11"
 end

data/spec/lib/base_without_validation_spec.rb

@@ -114,12 +114,12 @@ describe Flexirest::BaseWithoutValidation do
     expect(client["test"]).to be_an_instance_of(DateTime)
   end
 
-  it "should allow strings of 4 digits and not intepret them as dates" do
+  it "should allow strings of 4 digits and not interpret them as dates" do
     client = EmptyExample.new(test: "2015")
     expect(client["test"]).to be_an_instance_of(String)
   end
 
-  it "should allow strings of 8 digits and not intepret them as dates" do
+  it "should allow strings of 8 digits and not interpret them as dates" do
     client = EmptyExample.new(test: "1266129")
     expect(client["test"]).to be_an_instance_of(String)
   end

data/spec/lib/configuration_spec.rb

@@ -209,7 +209,7 @@ describe Flexirest::Configuration do
     end
   end
 
-  it "should default to non-verbose loggingg" do
+  it "should default to non-verbose logging" do
     class VerboseConfigurationExample1
       include Flexirest::Configuration
     end

data/spec/lib/connection_spec.rb

@@ -10,7 +10,7 @@ describe Flexirest::Connection do
     @connection.reconnect
   end
 
-  it "should contain a Farday connection" do
+  it "should contain a Faraday connection" do
     expect(@connection.session).to be_a_kind_of(Faraday::Connection)
   end
 

data/spec/lib/request_spec.rb

@@ -532,7 +532,7 @@ describe Flexirest::Request do
     expect(ret).to be_truthy
   end
 
-  it "should handle a 204 response and not erase the instance's attributes" do
+  it "should handle a 200 response with an empty body and not erase the instance's attributes" do
     expect_any_instance_of(Flexirest::Connection).to receive(:put).and_return(::FaradayResponseMock.new(OpenStruct.new(body: "", response_headers: {}, status: 200)))
     client = ExampleClient.new
     client.id = "1234"

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flexirest
 version: !ruby/object:Gem::Version
-  version: 1.10.10
+  version: 1.10.11
 platform: ruby
 authors:
 - Andy Jeffries
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-01-26 00:00:00.000000000 Z
+date: 2022-07-22 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -370,7 +370,7 @@ licenses:
 - MIT
 metadata:
   source_code_uri: https://github.com/flexirest/flexirest
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -385,8 +385,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.2
-signing_key: 
+rubygems_version: 3.3.7
+signing_key:
 specification_version: 4
 summary: This gem is for accessing REST services in a flexible way.  ActiveResource
   already exists for this, but it doesn't work where the resource naming doesn't follow