flexirest 1.9.12 → 1.9.13

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3320d5c425f13498cf558c361209bed466565f0b173ebcc2bd3f8e6205fd37dd
-  data.tar.gz: dd621beea4b50068147c70fbf079b7753d32e8a625e080a5803f4904cb757eea
+  metadata.gz: d22b9c2c356eec70454c6919ec81ac6fc37a42e500c95c2f40337ba3e40561a6
+  data.tar.gz: eaf44cd2c1e6378fc140beec5b38793880ff34b18ea49b57146e038dfe54cc01
 SHA512:
-  metadata.gz: 8811975e7c20d9b6a69fcb9c11e68883c2950927d2444723872cd302c10cec7802fa732ed50fb7b088a56535beb23d9180d4e717d7884bd7a1901c5fe05295f1
-  data.tar.gz: 9a8db810620535a335d2d94822ba277b5a61fe0963ce19479d53c57c001567afd30c96fb40720d231142ec7fd2dbcc01c8423f0c83d44408f6ea4c231cf6591f
+  metadata.gz: 33fc03579ec1bdd3a2af9542209efced52ab03294f06e36c9b19e0dc526b9f3982befba154a2621c318ecd9d70dda3b31b099b0bd3a13a11748b4678c33d86b1
+  data.tar.gz: aa5c88116eebc320d09b03932660a2299425647457758bcd080d842a2c895ed2584c2c068e1d9de0fcbe474fe76da41371b5631ce36022b8e76cd4a2e6aa6e87

data/CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## 1.9.13
+
+Change:
+
+- Unified the response body to be `body` for all exceptions instead of sometimes being `body` and sometimes being `raw_response`, although both are available as aliases where they were defined before (thanks to Romain Gisiger for the PR).
+- Adjust parsing of attributes to be done in a more unified way and lighten the CPU load (thanks to Romain Gisiger for the PR)
+
 ## 1.9.12
 
 Bugfix:

data/docs/httpparse-error-handling.md

@@ -1,16 +1,29 @@
 # *Flexirest:* HTTP/parse error handling
 
-Sometimes the backend server may respond with a non-200/304 header, in which case the code will raise an `Flexirest::HTTPClientException` for 4xx errors or an `Flexirest::HTTPServerException` for 5xx errors. These both have a `status` accessor and a `result` accessor (for getting access to the parsed body):
+## HTTP Errors
+
+Sometimes the backend server may respond with a non-2xx/3xx header, in which case the code will raise an `Flexirest::HTTPClientException` for 4xx errors or an `Flexirest::HTTPServerException` for 5xx errors. 
+
+These both have a `status` accessor, a `result` accessor (for getting access to the parsed body) and a `body` accessor (for getting access to the original unparsed body). However, you can make use of the `message` method (or the `to_s` method) that will build a nice error message for you (containing the URL and the HTTP method used for the request as well as the HTTP status code and the original body of the response).
 
 ```ruby
 begin
   Person.all
 rescue Flexirest::HTTPClientException, Flexirest::HTTPServerException => e
-  Rails.logger.error("API returned #{e.status} : #{e.result.message}")
+  # Display the HTTP status and parsed body
+  puts "Backend returned HTTP #{e.status} with following parsed body: #{e.result}"
+  
+  # Display the unparsed body
+  puts "Original body is: #{e.body}"
+  
+  # Using the message method, same than calling the to_s method
+  puts e.message
 end
 ```
 
-If the response is unparsable (e.g. not in the desired content type), then it will raise an `Flexirest::ResponseParseException` which has a `status` accessor for the HTTP status code and a `body` accessor for the unparsed response body.
+## Parse Error
+
+If the original response is unparsable (e.g. not in the desired content type), then it will raise an `Flexirest::ResponseParseException` which has a `status` accessor for the HTTP status code and a `body` accessor for the unparsed response body.
 
 -----
 

data/lib/flexirest/attribute_parsing.rb

@@ -3,7 +3,8 @@ module Flexirest
     private
 
     def parse_attribute_value(v)
-      return v if v.is_a?(Date) || v.is_a?(DateTime)
+      return v if v.is_a?(Date) || v.is_a?(DateTime) ||
+                  v.kind_of?(NilClass) || v.kind_of?(TrueClass) || v.kind_of?(FalseClass) || v.kind_of?(Numeric)
 
       if v.to_s[(/\A(((19|20)\d\d[- \/.](0[1-9]|1[012]|[1-9])[- \/.](0[1-9]|[12][0-9]|3[01]|[1-9]))|((0[1-9]|1[012]|[1-9])[- \/.](0[1-9]|[12][0-9]|3[01]|[1-9])[- \/.](19|20)\d\d))\Z/)]
         Date.parse(v) rescue v

data/lib/flexirest/request.rb

@@ -656,40 +656,52 @@ module Flexirest
           k = k.to_sym
         end
         overridden_name = select_name(k, overridden_name)
-        if @method[:options][:lazy].include?(k)
-          object._attributes[k] = Flexirest::LazyAssociationLoader.new(overridden_name, v, self, overridden_name:(overridden_name))
-        elsif v.is_a? Hash
-          object._attributes[k] = new_object(v, overridden_name )
-        elsif v.is_a? Array
-          if @method[:options][:array].include?(k)
-            object._attributes[k] = Array.new
-          else
-            object._attributes[k] = Flexirest::ResultIterator.new
-          end
-          v.each do |item|
-            if item.is_a? Hash
-              object._attributes[k] << new_object(item, overridden_name)
-            else
-              object._attributes[k] << item
-            end
-          end
+        set_corresponding_value(v, k, object, overridden_name)
+      end
+      object.clean! unless object_is_class?
+
+      object
+    end
+
+    def set_corresponding_value(value, key = nil, object = nil, overridden_name = nil)
+      optional_args = [key, object, overridden_name]
+      value_from_object = optional_args.all? # trying to parse a JSON Hash value
+      value_from_other_type = optional_args.none? # trying to parse anything else
+      raise Flexirest::InvalidArgumentsException.new("Optional args need all to be filled or none") unless value_from_object || value_from_other_type
+      k = key || :key
+      v = value
+      assignable_hash = value_from_object ? object._attributes : {}
+      if value_from_object && @method[:options][:lazy].include?(k)
+        assignable_hash[k] = Flexirest::LazyAssociationLoader.new(overridden_name, v, self, overridden_name:(overridden_name))
+      elsif v.is_a? Hash
+        assignable_hash[k] = new_object(v, overridden_name )
+      elsif v.is_a? Array
+        if @method[:options][:array].include?(k)
+          assignable_hash[k] = Array.new
         else
-          parse_fields = [ @method[:options][:parse_fields], @object._date_fields ].compact.reduce([], :|)
-          parse_fields = nil if parse_fields.empty?
-          if (parse_fields && parse_fields.include?(k))
-            object._attributes[k] = parse_attribute_value(v)
-          elsif parse_fields
-            object._attributes[k] = v
-          elsif Flexirest::Base.disable_automatic_date_parsing
-            object._attributes[k] = v
+          assignable_hash[k] = Flexirest::ResultIterator.new
+        end
+        v.each do |item|
+          if item.is_a? Hash
+            assignable_hash[k] << new_object(item, overridden_name)
           else
-            object._attributes[k] = parse_attribute_value(v)
+            assignable_hash[k] << set_corresponding_value(item)
           end
         end
+      else
+        parse_fields = [ @method[:options][:parse_fields], @object._date_fields ].compact.reduce([], :|)
+        parse_fields = nil if parse_fields.empty?
+        if (parse_fields && parse_fields.include?(k))
+          assignable_hash[k] = parse_attribute_value(v)
+        elsif parse_fields
+          assignable_hash[k] = v
+        elsif Flexirest::Base.disable_automatic_date_parsing
+          assignable_hash[k] = v
+        else
+          assignable_hash[k] = parse_attribute_value(v)
+        end
       end
-      object.clean! unless object_is_class?
-
-      object
+      value_from_object ? object : assignable_hash[k]
     end
 
     def hal_response?
@@ -822,12 +834,10 @@ module Flexirest
 
     def add_nested_body_to_iterator(result, items)
       items.each do |json_object|
-        if json_object.is_a?(Array)
-          iterator = ResultIterator.new
-          add_nested_body_to_iterator(iterator, json_object)
-          result << iterator
-        else
+        if json_object.is_a? Hash
           result << new_object(json_object, @overridden_name)
+        else
+          result << set_corresponding_value(json_object)
         end
       end
     end
@@ -838,6 +848,7 @@ module Flexirest
   end
 
   class RequestException < StandardError ; end
+  class InvalidArgumentsException < StandardError ; end
 
   class InvalidRequestException < RequestException ; end
   class MissingParametersException < RequestException ; end
@@ -851,18 +862,19 @@ module Flexirest
   end
 
   class HTTPException < RequestException
-    attr_accessor :status, :result, :request_url
+    attr_accessor :status, :result, :request_url, :body, :raw_response
     def initialize(options)
       @status = options[:status]
       @result = options[:result]
       @request_url = options[:url]
-      @raw_response = options[:raw_response]
+      @body = options[:raw_response]
       @method = options[:method]
     end
+    alias_method :body, :raw_response
 
     def message
       method = @method.try(:upcase)
-      "The #{method} to '#{@request_url}' returned a #{@status} status, which raised a #{self.class.to_s} with a body of: #{@raw_response}"
+      "The #{method} to '#{@request_url}' returned a #{@status} status, which raised a #{self.class.to_s} with a body of: #{@body}"
     end
 
     def to_s

data/lib/flexirest/version.rb

@@ -1,3 +1,3 @@
 module Flexirest
-  VERSION = "1.9.12"
+  VERSION = "1.9.13"
 end

data/spec/lib/request_spec.rb

@@ -31,6 +31,8 @@ describe Flexirest::Request do
       get :flat, "/", params_encoder: :flat
       get :array, "/johnny", array: [:likes, :dislikes]
       get :babies, "/babies", has_many: {children: ExampleOtherClient}
+      get :basket, "/basket", array: [:options]
+      get :dates, "/dates", array: [:dates]
       get :single_association, "/single", has_one: {single: ExampleSingleClient}, has_many: {children: ExampleOtherClient}
       get :headers, "/headers"
       get :cancel_callback, "/cancel-callback"
@@ -38,6 +40,7 @@ describe Flexirest::Request do
       put :headers_json, "/headers_json", request_body_type: :json
       get :find, "/:id", required: [:id]
       get :find_cat, "/:id/cat"
+      get :fruits, "/fruits"
       get :change, "/change"
       get :plain, "/plain/:id", plain: true
       post :create, "/create", rubify_names: true
@@ -46,11 +49,14 @@ describe Flexirest::Request do
       put :update, "/put/:id"
       put :wrapped, "/put/:id", wrap_root: "example"
       put :conversion, "/put/:id", parse_fields: [:converted]
+      put :conversion_child, "/put/:id", parse_fields: [:converted_child]
       delete :remove, "/remove/:id"
       delete :remove_body, "/remove/:id", send_delete_body: true
       get :hal, "/hal", fake:"{\"_links\":{\"child\": {\"href\": \"/child/1\"}, \"other\": {\"href\": \"/other/1\"}, \"cars\":[{\"href\": \"/car/1\", \"name\":\"car1\"}, {\"href\": \"/car/2\", \"name\":\"car2\"}, {\"href\": \"/car/not-embed\", \"name\":\"car_not_embed\"} ], \"lazy\": {\"href\": \"/lazy/load\"}, \"invalid\": [{\"href\": \"/invalid/1\"}]}, \"_embedded\":{\"other\":{\"name\":\"Jane\"},\"child\":{\"name\":\"Billy\"}, \"cars\":[{\"_links\": {\"self\": {\"href\": \"/car/1\"} }, \"make\": \"Bugatti\", \"model\": \"Veyron\"}, {\"_links\": {\"self\": {\"href\": \"/car/2\"} }, \"make\": \"Ferrari\", \"model\": \"F458 Italia\"} ], \"invalid\": [{\"present\":true, \"_links\": {} } ] } }", has_many:{other:ExampleOtherClient}
-      get :fake, "/fake", fake:"{\"result\":true, \"list\":[1,2,3,{\"test\":true}], \"child\":{\"grandchild\":{\"test\":true}}}"
-      get :fake_proc, "/fake", fake:->(request) { "{\"result\":#{request.get_params[:id]}}" }
+      get :fake_object, "/fake", fake:"{\"result\":true, \"list\":[1,2,3,{\"test\":true}], \"child\":{\"grandchild\":{\"test\":true}}}"
+      get :fake_proc_object, "/fake", fake:->(request) { "{\"result\":#{request.get_params[:id]}}" }
+      get :fake_array, "/fake", fake:"[1,2,3,{\"test\":true},null]"
+      get :fake_proc_array, "/fake", fake:->(request) { "[{\"result\":#{request.get_params[:id]}},null]" }
       get :defaults, "/defaults", defaults:{overwrite:"no", persist:"yes"}
       get :requires, "/requires", requires:[:name, :age]
       patch :only_changed_1, "/changed1", only_changed: true
@@ -143,7 +149,8 @@ describe Flexirest::Request do
     class LazyLoadedExampleClient < ExampleClient
       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 :fake_object, "/fake", fake:"{\"result\":true, \"list\":[1,2,3,{\"test\":true}], \"child\":{\"grandchild\":{\"test\":true}}}"
+      get :fake_array, "/fake", fake:"[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]
     end
 
@@ -190,7 +197,8 @@ describe Flexirest::Request do
     class WhitelistedDateClient < Flexirest::Base
       base_url "http://www.example.com"
       put :conversion, "/put/:id"
-      parse_date :converted
+      put :conversion_child, "/put/:id"
+      parse_date :converted, :converted_child
     end
 
     allow_any_instance_of(Flexirest::Request).to receive(:read_cached_response)
@@ -511,33 +519,84 @@ describe Flexirest::Request do
     end
   end
 
-  it "should only convert date times in JSON if specified" do
+  it "should only convert date times in JSON if specified when converted is root property" do
     expect_any_instance_of(Flexirest::Connection).to receive(:put).with("/put/1234", "debug=true", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"converted\":\"2012-03-04T01:02:03Z\", \"not_converted\":\"2012-03-04T01:02:03Z\"}", response_headers:{})))
     object = ExampleClient.conversion id:1234, debug:true
     expect(object.converted).to be_an_instance_of(DateTime)
     expect(object.not_converted).to be_an_instance_of(String)
   end
 
-  it "should convert date times in JSON if whitelisted" do
+  it "should only convert date times in JSON if specified when converted is child property" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:put).with("/put/1234", "debug=true", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"not_converted\":\"2012-03-04T01:02:03Z\", \"child\":{\"converted_child\":\"2012-03-04T01:02:03Z\"}}", response_headers:{})))
+    object = ExampleClient.conversion_child id:1234, debug:true
+    expect(object.child.converted_child).to be_an_instance_of(DateTime)
+    expect(object.not_converted).to be_an_instance_of(String)
+  end
+
+  it "should convert date times in JSON if root property is whitelisted" do
     expect_any_instance_of(Flexirest::Connection).to receive(:put).with("/put/1234", "debug=true", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"converted\":\"2012-03-04T01:02:03Z\", \"not_converted\":\"2012-03-04T01:02:03Z\"}", response_headers:{})))
     object = WhitelistedDateClient.conversion id:1234, debug:true
     expect(object.converted).to be_an_instance_of(DateTime)
     expect(object.not_converted).to be_an_instance_of(String)
   end
 
-  it "should parse JSON and return a nice object for faked responses" do
-    object = ExampleClient.fake id:1234, debug:true
+  it "should convert date times in JSON if child property is whitelisted" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:put).with("/put/1234", "debug=true", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"not_converted\":\"2012-03-04T01:02:03Z\", \"child\":{\"converted_child\":\"2012-03-04T01:02:03Z\"}}", response_headers:{})))
+    object = WhitelistedDateClient.conversion_child id:1234, debug:true
+    expect(object.child.converted_child).to be_an_instance_of(DateTime)
+    expect(object.not_converted).to be_an_instance_of(String)
+  end
+
+  it "should convert date times in JSON from a result iterator response" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:get).with("/dates", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"[\"2012-03-04T01:02:03Z\", \"2012-03-04T01:02:03Z\"]", response_headers:{})))
+    object = ExampleClient.dates
+    expect(object).to be_a(Flexirest::ResultIterator)
+    expect(object.items).to be_a(Array)
+    expect(object.first).to be_an_instance_of(DateTime)
+  end
+
+  it "should convert date times in JSON even in a pure array" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:get).with("/dates", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"dates\":[\"2012-03-04T01:02:03Z\", \"2012-03-04T01:02:03Z\"]}", response_headers:{})))
+    object = ExampleClient.dates
+    expect(object).to be_a(ExampleClient)
+    expect(object.dates).to be_a(Array)
+    expect(object.dates.first).to be_an_instance_of(DateTime)
+  end
+
+  it "should parse JSON object and return a nice object for faked responses" do
+    object = ExampleClient.fake_object id:1234, debug:true
     expect(object.result).to eq(true)
     expect(object.list.first).to eq(1)
     expect(object.list.last.test).to eq(true)
     expect(object.child.grandchild.test).to eq(true)
   end
 
-  it "should parse JSON from a fake response generated by a proc" do
-    object = ExampleClient.fake_proc id:1234
+  it "should parse JSON object from a fake response generated by a proc" do
+    object = ExampleClient.fake_proc_object id:1234
     expect(object.result).to eq(1234)
   end
 
+  it "should parse JSON array and return a nice result iterator for faked responses" do
+    object = ExampleClient.fake_array debug:true
+    expect(object).to be_instance_of(Flexirest::ResultIterator)
+    expect(object.size).to eq(5)
+    expect(object.first).to eq(1)
+    expect(object.last).to eq(nil)
+    expect(object[3]).to be_instance_of(ExampleClient)
+    expect(object[3].test).to eq(true)
+    expect(object._status).to eq(200)
+  end
+
+  it "should parse JSON array from a fake response generated by a proc" do
+    object = ExampleClient.fake_proc_array id:1234
+    expect(object).to be_instance_of(Flexirest::ResultIterator)
+    expect(object.size).to eq(2)
+    expect(object.first).to be_instance_of(ExampleClient)
+    expect(object.first.result).to eq(1234)
+    expect(object.last).to eq(nil)
+    expect(object._status).to eq(200)
+  end
+
   it "should not parse JSON from a plain request" do
     response_body = "This is another non-JSON string"
     expect_any_instance_of(Flexirest::Connection).to receive(:get).with(any_args).and_return(::FaradayResponseMock.new(OpenStruct.new(status:200, response_headers:{}, body:response_body)))
@@ -570,19 +629,36 @@ describe Flexirest::Request do
     expect(ExampleClient.all).to be_truthy
   end
 
-  it "should return a lazy loader object if lazy loading is enabled" do
-    object = LazyLoadedExampleClient.fake id:1234, debug:true
+  it "should return a lazy loader object if lazy loading is enabled for JSON object" do
+    object = LazyLoadedExampleClient.fake_object id:1234, debug:true
     expect(object).to be_an_instance_of(Flexirest::LazyLoader)
   end
 
-  it "should proxy through nice object for lazy loaded responses" do
-    object = LazyLoadedExampleClient.fake id:1234, debug:true
-    expect(object.result).to eq(true)
+  it "should proxy through nice object for lazy loaded responses from JSON object" do
+    object = LazyLoadedExampleClient.fake_object id:1234, debug:true
+    expect(object.instance_variable_get(:@result)).to be(nil)
+    expect(object.result).to eq(true) # method call the attribute received in response and never the instance attribute of the LazyLoader class
+    expect(object.instance_variable_get(:@result)).to be_a(LazyLoadedExampleClient)
     expect(object.list.first).to eq(1)
     expect(object.list.last.test).to eq(true)
     expect(object.child.grandchild.test).to eq(true)
   end
 
+  it "should return a lazy loader object if lazy loading is enabled for JSON array" do
+    object = LazyLoadedExampleClient.fake_array debug:true
+    expect(object).to be_an_instance_of(Flexirest::LazyLoader)
+  end
+
+  it "should proxy through nice result iterator for lazy loaded responses from JSON array" do
+    object = LazyLoadedExampleClient.fake_array debug:true
+    expect(object.instance_variable_get(:@result)).to be(nil)
+    expect(object.items).to be_a(Array)
+    expect(object.instance_variable_get(:@result)).to be_a(Flexirest::ResultIterator)
+    expect(object.first).to eq(1)
+    expect(object[3].test).to eq(true)
+    expect(object.last.child.grandchild.test).to eq(true)
+  end
+
   it "should return a LazyAssociationLoader for lazy loaded properties" do
     object = LazyLoadedExampleClient.lazy_test
     expect(object.people.size).to eq(1)
@@ -592,7 +668,7 @@ describe Flexirest::Request do
   it "should log faked responses" do
     allow(Flexirest::Logger).to receive(:debug)
     expect(Flexirest::Logger).to receive(:debug).with(/Faked response found/)
-    ExampleClient.fake id:1234, debug:true
+    ExampleClient.fake_object id:1234, debug:true
   end
 
   it "should parse an array within JSON to be a result iterator" do
@@ -628,6 +704,51 @@ describe Flexirest::Request do
     #TODO
   end
 
+  it "should parse an attribute to be an array if attribute included in a nested array" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:get).with("/johnny", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"first_name\":\"Johnny\", \"nested_array\":[[{\"likes\":[\"donuts\", \"bacon\"], \"dislikes\":[\"politicians\", \"lawyers\", \"taxes\"]}]]}", status:200, response_headers:{})))
+    object = ExampleClient.array
+    nested_array = object.nested_array[0][0]
+    expect(nested_array.likes).to be_instance_of(Array)
+    expect(nested_array.likes.size).to eq(2)
+    expect(nested_array.likes[0]).to eq("donuts")
+    expect(nested_array.likes[1]).to eq("bacon")
+    expect(nested_array.dislikes).to be_instance_of(Array)
+    expect(nested_array.dislikes.size).to eq(3)
+    expect(nested_array.dislikes[0]).to eq("politicians")
+    expect(nested_array.dislikes[1]).to eq("lawyers")
+    expect(nested_array.dislikes[2]).to eq("taxes")
+    #TODO - Pasted from the Test above
+  end
+
+  it "should parse an attribute to be either a result iterator or an array and containing simple values like integers" do
+      expect_any_instance_of(Flexirest::Connection).to receive(:get).with("/basket", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"{\"products\":[101, 55, 37], \"options\":[854, 225, 772]}", status:200, response_headers:{})))
+      object = ExampleClient.basket
+      expect(object).to be_instance_of(ExampleClient)
+      expect(object.products).to be_a(Flexirest::ResultIterator)
+      expect(object.products.size).to eq(3)
+      expect(object.products.first).to eq(101)
+      expect(object.products[1]).to eq(55)
+      expect(object.products.last).to eq(37)
+      expect(object.options).to be_a(Array)
+      expect(object.options.size).to eq(3)
+      expect(object.options.first).to eq(854)
+      expect(object.options[1]).to eq(225)
+      expect(object.options.last).to eq(772)
+      expect(object._status).to eq(200)
+      #TODO - Pasted from the Test above
+    end
+
+  it "should parse the response to be a result iterator and containing simple values like strings" do
+    expect_any_instance_of(Flexirest::Connection).to receive(:get).with("/fruits", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"[\"apple\", \"banana\", \"pear\", \"watermelon\"]", status:200, response_headers:{})))
+    object = ExampleClient.fruits
+    expect(object).to be_instance_of(Flexirest::ResultIterator)
+    expect(object.first).to eq('apple')
+    expect(object[1]).to eq('banana')
+    expect(object.last).to eq('watermelon')
+    expect(object._status).to eq(200)
+    #TODO - Pasted from the Test above
+  end
+
   it "should only send changed attributes if only_changed:true" do
     expect_any_instance_of(Flexirest::Connection).to receive(:patch).with("/changed1", "debug=true", an_instance_of(Hash)).and_return(::FaradayResponseMock.new(OpenStruct.new(body:"[{\"first_name\":\"Johnny\"}, {\"first_name\":\"Billy\"}, {\"debug\":\"true\"}]", status:200, response_headers:{})))
     object = ExampleClient.new

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: flexirest
 version: !ruby/object:Gem::Version
-  version: 1.9.12
+  version: 1.9.13
 platform: ruby
 authors:
 - Andy Jeffries
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-02-14 00:00:00.000000000 Z
+date: 2020-04-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler