Add json_parse_options config

Author: ursm

README.md

@@ -129,6 +129,18 @@ Fetch.configure do |config|
 end
 ```
 
+### Parsing JSON
+
+By default, `Fetch::Response#json` calls `JSON.parse` without options. Additional options can be specified in `Fetch.config.json_parse_options`.
+
+``` ruby
+Fetch.configure do |config|
+  config.json_parse_options = {
+    symbolize_names: true
+  }
+end
+```
+
 ## Development
 
 After checking out the repo, run `bin/setup` to install dependencies. Then, run `rake spec` to run the tests. You can also run `bin/console` for an interactive prompt that will allow you to experiment.

lib/fetch.rb

@@ -12,5 +12,6 @@ def self.configure(&block)
   configure do |config|
     config.connection_max_idle_time = 30
     config.on_connection_create     = -> (*) {}
+    config.json_parse_options       = {}
   end
 end

lib/fetch/config.rb

@@ -1,3 +1,7 @@
 module Fetch
-  Config = Struct.new(:connection_max_idle_time, :on_connection_create)
+  Config = Struct.new(
+    :connection_max_idle_time,
+    :on_connection_create,
+    :json_parse_options
+  )
 end

lib/fetch/response.rb

@@ -21,10 +21,10 @@ def status_text
       Rack::Utils::HTTP_STATUS_CODES[status]
     end
 
-    def json(...)
+    def json(**json_parse_options)
       return nil unless body
 
-      JSON.parse(body, ...)
+      JSON.parse(body, **Fetch.config.json_parse_options, **json_parse_options)
     end
   end
 end

sig/fetch/config.rbs

@@ -2,5 +2,6 @@ module Fetch
   class Config
     attr_accessor connection_max_idle_time: Integer
     attr_accessor on_connection_create: ^(Net::HTTP, URI::HTTP) -> void
+    attr_accessor json_parse_options: json_options
   end
 end

sig/fetch/response.rbs

@@ -9,6 +9,6 @@ module Fetch
     def initialize: (url: String, status: Integer, headers: Headers, body: String?, redirected: bool) -> void
     def ok: () -> bool
     def status_text: () -> String
-    def json: (json_options) -> untyped
+    def json: (**json_options) -> untyped
   end
 end

spec/fetch/response_spec.rb

@@ -1,5 +1,7 @@
 require 'spec_helper'
 
+require 'active_support/core_ext/object/with'
+
 RSpec.describe Fetch::Response do
   def create_response(url: 'http://example.com', status: 200, headers: [], body: nil, redirected: false)
     Fetch::Response.new(url:, status:, headers:, body:, redirected:)
@@ -24,4 +26,12 @@ def create_response(url: 'http://example.com', status: 200, headers: [], body: n
 
     expect(res.json).to eq('foo' => 'bar')
   end
+
+  example '#json with config' do
+    Fetch::config.with json_parse_options: {symbolize_names: true} do
+      res = create_response(body: '{"foo":"bar"}')
+
+      expect(res.json).to eq(foo: 'bar')
+    end
+  end
 end