faraday_hal_middleware 0.1.1 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a7e094397fe655ac925713ff3747b751f468b1c7581b42b08784caafe1f58790
-  data.tar.gz: 1cc42cd8fe10bb81be514aa2a8a1a3cafad1ed7ced10a2ad0b774e721968cf2f
+  metadata.gz: 129ae7965c9e0d8c0e7b4e83386f4555a1b694d704b559f18e7f62e16b815851
+  data.tar.gz: 639ae15211c5ec76fe2e29492a7c0fc2791127eddfd65360fb9442453cd7d825
 SHA512:
-  metadata.gz: 69362407c7e92bd6f24bd174dfc8dc3fa79a43e67f5b978618615c993497606a6d77d72de8ca9a2fa224b834dd6f5bfc8782b30b6243a641bbd147a724ffa7fc
-  data.tar.gz: 1d50efa0d668beb116a9a891442911a2d6b132bcbe06a6140ee16d2d2604ff9375458e0debc6c55d2cac1721cc51d8828e9d638423556066e342b43ff222be9a
+  metadata.gz: cd2271ad2df1acec1af6c6eeaba8ba187869a4196e0bb7d3a0c1b7b44f9fd699fe73d2affc7286d136f4e933c672673b236ef5aee11d9c40910f4cd7013442d5
+  data.tar.gz: 85c9cac01aa86c83994a19336db03754e9530b0dbe047c2a7fa11c51c6f996ea4921fdde98c51cf2c5c44360be6be2c428c227b87ce55c4fcd01014c460ac523

data/.github/workflows/lint.yml

@@ -0,0 +1,16 @@
+name: lint
+on: [push, pull_request]
+jobs:
+  rubocop:
+    name: RuboCop
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v3
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: 2.7
+          bundler-cache: true
+      - run: bundle exec rubocop
+

data/.github/workflows/test.yml

@@ -0,0 +1,18 @@
+---
+name: test
+on: [push, pull_request]
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      fail-fast: false
+      matrix:
+        ruby: [ '2.7', '3.0', '3.1', '3.2' ]
+    name: test (${{ matrix.ruby }})
+    steps:
+      - uses: actions/checkout@v3
+      - uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+      - run: bundle install --jobs=3 --retry=3 --path=vendor/bundle
+      - run: bundle exec rake spec

data/.rubocop.yml

@@ -1,5 +1,10 @@
 AllCops:
+  NewCops: enable
   Exclude:
     - vendor/**/*
 
+require:
+  - rubocop-rake
+  - rubocop-rspec
+
 inherit_from: .rubocop_todo.yml

data/.rubocop_todo.yml

@@ -1,36 +1,83 @@
 # This configuration was generated by
 # `rubocop --auto-gen-config`
-# on 2018-01-10 10:44:07 -0500 using RuboCop version 0.52.1.
+# on 2024-02-01 13:35:51 UTC using RuboCop version 1.60.2.
 # The point is for the user to remove these configuration records
 # one by one as the offenses are removed from the code base.
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
-# Offense count: 2
-# Configuration parameters: CountComments, ExcludedMethods.
-Metrics/BlockLength:
-  Max: 72
+# Offense count: 1
+# Configuration parameters: Severity, Include.
+# Include: **/*.gemspec
+Gemspec/RequiredRubyVersion:
+  Exclude:
+    - 'faraday_hal_middleware.gemspec'
 
 # Offense count: 1
-# Configuration parameters: NamePrefix, NamePrefixBlacklist, NameWhitelist, MethodDefinitionMacros.
-# NamePrefix: is_, has_, have_
-# NamePrefixBlacklist: is_, has_, have_
-# NameWhitelist: is_a?
-# MethodDefinitionMacros: define_method, define_singleton_method
-Naming/PredicateName:
+# Configuration parameters: AllowedMethods, AllowedPatterns, CountRepeatedAttributes.
+Metrics/AbcSize:
+  Max: 21
+
+# Offense count: 10
+# Configuration parameters: Prefixes, AllowedPatterns.
+# Prefixes: when, with, without
+RSpec/ContextWording:
   Exclude:
-    - 'spec/**/*'
-    - 'lib/faraday_middleware/request/encode_hal_json.rb'
+    - 'spec/unit/request_spec.rb'
+    - 'spec/unit/response_spec.rb'
 
 # Offense count: 2
-Style/Documentation:
+# Configuration parameters: Include, CustomTransform, IgnoreMethods, SpecSuffixOnly.
+# Include: **/*_spec*rb*, **/spec/**/*
+RSpec/FilePath:
   Exclude:
-    - 'spec/**/*'
-    - 'test/**/*'
-    - 'lib/faraday_hal_middleware.rb'
+    - 'spec/unit/request_spec.rb'
+    - 'spec/unit/response_spec.rb'
+
+# Offense count: 4
+# Configuration parameters: .
+# SupportedStyles: have_received, receive
+RSpec/MessageSpies:
+  EnforcedStyle: receive
+
+# Offense count: 7
+RSpec/MultipleExpectations:
+  Max: 2
+
+# Offense count: 4
+# Configuration parameters: AllowSubject.
+RSpec/MultipleMemoizedHelpers:
+  Max: 6
+
+# Offense count: 2
+RSpec/RepeatedExample:
+  Exclude:
+    - 'spec/unit/response_spec.rb'
+
+# Offense count: 2
+# Configuration parameters: Include, CustomTransform, IgnoreMethods, IgnoreMetadata.
+# Include: **/*_spec.rb
+RSpec/SpecFilePathFormat:
+  Exclude:
+    - '**/spec/routing/**/*'
+    - 'spec/unit/request_spec.rb'
+    - 'spec/unit/response_spec.rb'
+
+# Offense count: 4
+RSpec/StubbedMock:
+  Exclude:
+    - 'spec/unit/response_spec.rb'
 
 # Offense count: 1
-# Configuration parameters: MinBodyLength.
-Style/GuardClause:
+# Configuration parameters: IgnoreNameless, IgnoreSymbolicNames.
+RSpec/VerifiedDoubles:
   Exclude:
-    - 'lib/faraday_middleware/request/encode_hal_json.rb'
+    - 'spec/unit/response_spec.rb'
+
+# Offense count: 1
+# Configuration parameters: AllowedConstants.
+Style/Documentation:
+  Exclude:
+    - 'spec/**/*'
+    - 'test/**/*'
+    - 'lib/faraday/hal_json.rb'

data/CHANGELOG.md

@@ -1,3 +1,8 @@
+### 0.2.0 (2024/02/01)
+
+* [#11](https://github.com/fetch/faraday_hal_middleware/pull/11): Refactor into Faraday::HalJson::Request and Response - [@dblock](https://github.com/dblock).
+* [#9](https://github.com/fetch/faraday_hal_middleware/pull/9): Added support for Faraday 2, removed dependency on deprecated `faraday_hal_middleware` - [@dblock](https://github.com/dblock).
+
 ### 0.1.1 (2020/05/10)
 
 * [#6](https://github.com/fetch/faraday_hal_middleware/pull/6): Added CHANGELOG, CONTRIBUTING and RELEASING - [@dblock](https://github.com/dblock).

data/Gemfile

@@ -1,10 +1,14 @@
+# frozen_string_literal: true
+
 source 'https://rubygems.org'
 
 gem 'json'
 gem 'rake', '>= 0.9'
 
 group :development do
-  gem 'rubocop', '0.52.1'
+  gem 'rubocop', '1.60.2'
+  gem 'rubocop-rake'
+  gem 'rubocop-rspec'
 end
 
 group :test do

data/README.md

@@ -2,7 +2,7 @@
 
 [Faraday](https://github.com/lostisland/faraday) Middleware for JSON HAL requests and responses with `application/hal+json` content-type.
 
-[![Build Status](https://travis-ci.org/fetch/faraday_hal_middleware.svg?branch=master)](https://travis-ci.org/fetch/faraday_hal_middleware)
+[![test](https://github.com/fetch/faraday_hal_middleware/actions/workflows/test.yml/badge.svg)](https://github.com/fetch/faraday_hal_middleware/actions/workflows/test.yml)
 
 ## Installation
 
@@ -12,9 +12,11 @@ Add this line to your application's Gemfile:
 gem 'faraday_hal_middleware'
 ```
 
+Use version 0.1.1 for Faraday 1.x, and >= 0.2.0 for newer versions of Faraday.
+
 ## Usage
 
-Use `faraday_hal_middleware` in a similar way to [other Faraday middleware](https://github.com/lostisland/faraday_middleware).
+Use `faraday_hal_middleware`.
 
 ```ruby
 require 'faraday_hal_middleware'

data/Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'bundler/gem_tasks'
 
 require 'rspec/core/rake_task'

data/faraday_hal_middleware.gemspec

@@ -1,11 +1,13 @@
+# frozen_string_literal: true
 
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path('lib', __dir__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'faraday_hal_middleware/version'
+
+require 'faraday/hal_json/version'
 
 Gem::Specification.new do |spec|
   spec.name          = 'faraday_hal_middleware'
-  spec.version       = FaradayHalMiddleware::VERSION
+  spec.version       = Faraday::HalJson::VERSION
   spec.authors       = ['Koen Punt']
   spec.email         = ['koen@fetch.nl']
   spec.summary       = 'Faraday Middleware for JSON HAL requests and responses.'
@@ -15,8 +17,8 @@ Gem::Specification.new do |spec|
 
   spec.files         = `git ls-files -z`.split("\x0")
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
-  spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.add_dependency 'faraday_middleware', '>= 0.9'
+  spec.add_dependency 'faraday', '~> 2.0'
+  spec.metadata['rubygems_mfa_required'] = 'true'
 end

data/lib/faraday/hal_json/request.rb

@@ -0,0 +1,68 @@
+# frozen_string_literal: true
+
+require 'faraday'
+
+module Faraday
+  module HalJson
+    # Request middleware that encodes the body as JSON.
+    #
+    # Processes only requests with matching Content-type or those without a type.
+    # If a request doesn't have a type but has a body, it sets the Content-type
+    # to JSON MIME-type.
+    #
+    # Doesn't try to encode bodies that already are in string form.
+    class Request < Faraday::Middleware
+      CONTENT_TYPE = 'Content-Type'
+      MIME_TYPE    = 'application/hal+json'
+
+      def on_request(env)
+        match_content_type(env) do |data|
+          env[:body] = encode(data)
+        end
+      end
+
+      private
+
+      def encode(data)
+        if options[:encoder].is_a?(Array) && options[:encoder].size >= 2
+          options[:encoder][0].public_send(options[:encoder][1], data)
+        elsif options[:encoder].respond_to?(:dump)
+          options[:encoder].dump(data)
+        else
+          ::JSON.generate(data)
+        end
+      end
+
+      def match_content_type(env)
+        return unless process_request?(env)
+
+        env[:request_headers][CONTENT_TYPE] ||= MIME_TYPE
+        yield env[:body] unless env[:body].respond_to?(:to_str)
+      end
+
+      def process_request?(env)
+        type = request_type(env)
+        body?(env) && (type.empty? || type == MIME_TYPE)
+      end
+
+      def body?(env)
+        body = env[:body]
+        case body
+        when true, false
+          true
+        when nil
+          # NOTE: nil can be converted to `"null"`, but this middleware doesn't process `nil` for the compatibility.
+          false
+        else
+          !(body.respond_to?(:to_str) && body.empty?)
+        end
+      end
+
+      def request_type(env)
+        type = env[:request_headers][CONTENT_TYPE].to_s
+        type = type.split(';', 2).first if type.index(';')
+        type
+      end
+    end
+  end
+end

data/lib/faraday/hal_json/response.rb

@@ -0,0 +1,8 @@
+# frozen_string_literal: true
+
+module Faraday
+  module HalJson
+    # JSON+HAL is just JSON
+    class Response < ::Faraday::Response::Json; end
+  end
+end

data/lib/faraday/hal_json/version.rb

@@ -0,0 +1,7 @@
+# frozen_string_literal: true
+
+module Faraday
+  module HalJson
+    VERSION = '0.2.0'
+  end
+end

data/lib/faraday/hal_json.rb

@@ -0,0 +1,12 @@
+# frozen_string_literal: true
+
+require_relative 'hal_json/version'
+require_relative 'hal_json/request'
+require_relative 'hal_json/response'
+
+module Faraday
+  module HalJson
+    ::Faraday::Request.register_middleware(hal_json: ::Faraday::HalJson::Request)
+    ::Faraday::Response.register_middleware(hal_json: ::Faraday::HalJson::Response)
+  end
+end

data/lib/faraday_hal_middleware.rb

@@ -1,19 +1,4 @@
-require 'faraday_hal_middleware/version'
-require 'faraday'
-require 'faraday_middleware'
-
-module FaradayHalMiddleware
-end
-
-module FaradayMiddleware
-  autoload :EncodeHalJson,     'faraday_middleware/request/encode_hal_json'
-  autoload :ParseHalJson,      'faraday_middleware/response/parse_hal_json'
+# frozen_string_literal: true
 
-  if Faraday::Middleware.respond_to? :register_middleware
-    Faraday::Request.register_middleware \
-      hal_json: -> { EncodeHalJson }
-
-    Faraday::Response.register_middleware \
-      hal_json: -> { ParseHalJson }
-  end
-end
+require 'faraday'
+require_relative 'faraday/hal_json'

data/spec/spec_helper.rb

@@ -1,43 +1,11 @@
+# frozen_string_literal: true
+
 require 'rspec'
 require 'webmock/rspec'
 require 'faraday'
-
-module EnvCompatibility
-  def faraday_env(env)
-    if defined?(Faraday::Env)
-      Faraday::Env.from(env)
-    else
-      env
-    end
-  end
-end
-
-module ResponseMiddlewareExampleGroup
-  def self.included(base)
-    base.let(:options) { {} }
-    base.let(:headers) { {} }
-    base.let(:middleware) do
-      described_class.new(lambda { |env|
-        Faraday::Response.new(env)
-      }, options)
-    end
-  end
-
-  def process(body, content_type = nil, options = {})
-    env = {
-      body: body, request: options,
-      request_headers: Faraday::Utils::Headers.new,
-      response_headers: Faraday::Utils::Headers.new(headers)
-    }
-    env[:response_headers]['content-type'] = content_type if content_type
-    yield(env) if block_given?
-    middleware.call(faraday_env(env))
-  end
-end
+require 'faraday_hal_middleware'
 
 RSpec.configure do |config|
-  config.include EnvCompatibility
-  config.include ResponseMiddlewareExampleGroup, type: :response
   config.expect_with :rspec do |c|
     c.syntax = :expect
   end

data/spec/unit/{encode_hal_json_spec.rb → request_spec.rb}

@@ -1,12 +1,15 @@
-require 'faraday_middleware/request/encode_hal_json'
+# frozen_string_literal: true
 
-describe FaradayMiddleware::EncodeHalJson do
-  let(:middleware) { described_class.new(->(env) { env }) }
+require 'spec_helper'
+
+# mostly https://github.com/lostisland/faraday/blob/main/spec/faraday/request/json_spec.rb
+describe Faraday::HalJson::Request, type: :request do
+  let(:middleware) { described_class.new(->(env) { Faraday::Response.new(env) }) }
 
   def process(body, content_type = nil)
     env = { body: body, request_headers: Faraday::Utils::Headers.new }
     env[:request_headers]['content-type'] = content_type if content_type
-    middleware.call(faraday_env(env))
+    middleware.call(Faraday::Env.from(env)).env
   end
 
   def result_body

data/spec/unit/response_spec.rb

@@ -0,0 +1,192 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+# copy of https://github.com/lostisland/faraday/blob/main/spec/faraday/response/json_spec.rb
+describe Faraday::HalJson::Response do
+  let(:options) { {} }
+  let(:headers) { {} }
+  let(:middleware) do
+    described_class.new(lambda { |env|
+      Faraday::Response.new(env)
+    }, **options)
+  end
+
+  def process(body, content_type = 'application/json', options = {})
+    env = {
+      body: body, request: options,
+      request_headers: Faraday::Utils::Headers.new,
+      response_headers: Faraday::Utils::Headers.new(headers)
+    }
+    env[:response_headers]['content-type'] = content_type if content_type
+    yield(env) if block_given?
+    middleware.call(Faraday::Env.from(env))
+  end
+
+  context 'no type matching' do
+    it "doesn't change nil body" do
+      expect(process(nil).body).to be_nil
+    end
+
+    it 'nullifies empty body' do
+      expect(process('').body).to be_nil
+    end
+
+    it 'parses json body' do
+      response = process('{"a":1}')
+      expect(response.body).to eq('a' => 1)
+      expect(response.env[:raw_body]).to be_nil
+    end
+  end
+
+  context 'with preserving raw' do
+    let(:options) { { preserve_raw: true } }
+
+    it 'parses json body' do
+      response = process('{"a":1}')
+      expect(response.body).to eq('a' => 1)
+      expect(response.env[:raw_body]).to eq('{"a":1}')
+    end
+  end
+
+  context 'with default regexp type matching' do
+    it 'parses json body of correct type' do
+      response = process('{"a":1}', 'application/x-json')
+      expect(response.body).to eq('a' => 1)
+    end
+
+    it 'ignores json body of incorrect type' do
+      response = process('{"a":1}', 'text/json-xml')
+      expect(response.body).to eq('{"a":1}')
+    end
+  end
+
+  context 'with array type matching' do
+    let(:options) { { content_type: %w[a/b c/d] } }
+
+    it 'parses json body of correct type' do
+      expect(process('{"a":1}', 'a/b').body).to be_a(Hash)
+      expect(process('{"a":1}', 'c/d').body).to be_a(Hash)
+    end
+
+    it 'ignores json body of incorrect type' do
+      expect(process('{"a":1}', 'a/d').body).not_to be_a(Hash)
+    end
+  end
+
+  it 'chokes on invalid json' do
+    expect { process('{!') }.to raise_error(Faraday::ParsingError)
+  end
+
+  it 'includes the response on the ParsingError instance' do
+    process('{') { |env| env[:response] = Faraday::Response.new }
+    raise 'Parsing should have failed.'
+  rescue Faraday::ParsingError => e
+    expect(e.response).to be_a(Faraday::Response)
+  end
+
+  context 'HEAD responses' do
+    it "nullifies the body if it's only one space" do
+      response = process(' ')
+      expect(response.body).to be_nil
+    end
+
+    it "nullifies the body if it's two spaces" do
+      response = process(' ')
+      expect(response.body).to be_nil
+    end
+  end
+
+  context 'JSON options' do
+    let(:body) { '{"a": 1}' }
+    let(:result) { { a: 1 } }
+    let(:options) do
+      {
+        parser_options: {
+          symbolize_names: true
+        }
+      }
+    end
+
+    it 'passes relevant options to JSON parse' do
+      expect(JSON).to receive(:parse)
+        .with(body, options[:parser_options])
+        .and_return(result)
+
+      response = process(body)
+      expect(response.body).to eq(result)
+    end
+  end
+
+  context 'with decoder' do
+    let(:decoder) do
+      double('Decoder').tap do |e|
+        allow(e).to receive(:load) { |s, opts| JSON.parse(s, opts) }
+      end
+    end
+
+    let(:body) { '{"a": 1}' }
+    let(:result) { { a: 1 } }
+
+    context 'when decoder is passed as object' do
+      let(:options) do
+        {
+          parser_options: {
+            decoder: decoder,
+            option: :option_value,
+            symbolize_names: true
+          }
+        }
+      end
+
+      it 'passes relevant options to specified decoder\'s load method' do
+        expect(decoder).to receive(:load)
+          .with(body, { option: :option_value, symbolize_names: true })
+          .and_return(result)
+
+        response = process(body)
+        expect(response.body).to eq(result)
+      end
+    end
+
+    context 'when decoder is passed as an object-method pair' do
+      let(:options) do
+        {
+          parser_options: {
+            decoder: [decoder, :load],
+            option: :option_value,
+            symbolize_names: true
+          }
+        }
+      end
+
+      it 'passes relevant options to specified decoder\'s method' do
+        expect(decoder).to receive(:load)
+          .with(body, { option: :option_value, symbolize_names: true })
+          .and_return(result)
+
+        response = process(body)
+        expect(response.body).to eq(result)
+      end
+    end
+
+    context 'when decoder is not passed' do
+      let(:options) do
+        {
+          parser_options: {
+            symbolize_names: true
+          }
+        }
+      end
+
+      it 'passes relevant options to JSON parse' do
+        expect(JSON).to receive(:parse)
+          .with(body, { symbolize_names: true })
+          .and_return(result)
+
+        response = process(body)
+        expect(response.body).to eq(result)
+      end
+    end
+  end
+end

metadata

@@ -1,29 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: faraday_hal_middleware
 version: !ruby/object:Gem::Version
-  version: 0.1.1
+  version: 0.2.0
 platform: ruby
 authors:
 - Koen Punt
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-05-10 00:00:00.000000000 Z
+date: 2024-02-01 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: faraday_middleware
+  name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.9'
+        version: '2.0'
 description: Faraday Middleware for JSON HAL requests and responses.
 email:
 - koen@fetch.nl
@@ -31,11 +31,12 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/lint.yml"
+- ".github/workflows/test.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
-- ".travis.yml"
 - CHANGELOG.md
 - CONTRIBUTING.md
 - Gemfile
@@ -44,18 +45,20 @@ files:
 - RELEASING.md
 - Rakefile
 - faraday_hal_middleware.gemspec
+- lib/faraday/hal_json.rb
+- lib/faraday/hal_json/request.rb
+- lib/faraday/hal_json/response.rb
+- lib/faraday/hal_json/version.rb
 - lib/faraday_hal_middleware.rb
-- lib/faraday_hal_middleware/version.rb
-- lib/faraday_middleware/request/encode_hal_json.rb
-- lib/faraday_middleware/response/parse_hal_json.rb
 - spec/spec_helper.rb
-- spec/unit/encode_hal_json_spec.rb
-- spec/unit/parse_hal_json_spec.rb
+- spec/unit/request_spec.rb
+- spec/unit/response_spec.rb
 homepage: https://github.com/fetch/faraday_hal_middleware
 licenses:
 - MIT
-metadata: {}
-post_install_message: 
+metadata:
+  rubygems_mfa_required: 'true'
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -70,11 +73,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.6
+signing_key:
 specification_version: 4
 summary: Faraday Middleware for JSON HAL requests and responses.
-test_files:
-- spec/spec_helper.rb
-- spec/unit/encode_hal_json_spec.rb
-- spec/unit/parse_hal_json_spec.rb
+test_files: []

data/.travis.yml

@@ -1,6 +0,0 @@
-language: ruby
-
-cache: bundler
-
-rvm:
-  - 2.6.6

data/lib/faraday_hal_middleware/version.rb

@@ -1,3 +0,0 @@
-module FaradayHalMiddleware
-  VERSION = '0.1.1'.freeze
-end

data/lib/faraday_middleware/request/encode_hal_json.rb

@@ -1,52 +0,0 @@
-require 'faraday'
-
-module FaradayMiddleware
-  # Request middleware that encodes the body as JSON.
-  #
-  # Processes only requests with matching Content-type or those without a type.
-  # If a request doesn't have a type but has a body, it sets the Content-type
-  # to JSON MIME-type.
-  #
-  # Doesn't try to encode bodies that already are in string form.
-  class EncodeHalJson < Faraday::Middleware
-    CONTENT_TYPE = 'Content-Type'.freeze
-    MIME_TYPE    = 'application/hal+json'.freeze
-
-    dependency do
-      require 'json' unless defined?(::JSON)
-    end
-
-    def call(env)
-      match_content_type(env) do |data|
-        env[:body] = encode data
-      end
-      @app.call env
-    end
-
-    def encode(data)
-      ::JSON.dump data
-    end
-
-    def match_content_type(env)
-      if process_request?(env)
-        env[:request_headers][CONTENT_TYPE] ||= MIME_TYPE
-        yield env[:body] unless env[:body].respond_to?(:to_str)
-      end
-    end
-
-    def process_request?(env)
-      type = request_type(env)
-      has_body?(env) && (type.empty? || (type == MIME_TYPE))
-    end
-
-    def has_body?(env)
-      (body = env[:body]) && !(body.respond_to?(:to_str) && body.empty?)
-    end
-
-    def request_type(env)
-      type = env[:request_headers][CONTENT_TYPE].to_s
-      type = type.split(';', 2).first if type.index(';')
-      type
-    end
-  end
-end

data/lib/faraday_middleware/response/parse_hal_json.rb

@@ -1,7 +0,0 @@
-require 'faraday_middleware/response/parse_json'
-
-module FaradayMiddleware
-  # JSON+HAL is just JSON
-  class ParseHalJson < ParseJson
-  end
-end

data/spec/unit/parse_hal_json_spec.rb

@@ -1,79 +0,0 @@
-require 'faraday_middleware/response/parse_hal_json'
-
-describe FaradayMiddleware::ParseHalJson, type: :response do
-  context 'no type matching' do
-    it "doesn't change nil body" do
-      expect(process(nil).body).to be_nil
-    end
-
-    it 'nullifies empty body' do
-      expect(process('').body).to be_nil
-    end
-
-    it 'parses json body' do
-      response = process('{"a":1}')
-      expect(response.body).to eq('a' => 1)
-      expect(response.env[:raw_body]).to be_nil
-    end
-  end
-
-  context 'with preserving raw' do
-    let(:options) { { preserve_raw: true } }
-
-    it 'parses json body' do
-      response = process('{"a":1}')
-      expect(response.body).to eq('a' => 1)
-      expect(response.env[:raw_body]).to eq('{"a":1}')
-    end
-
-    it 'can opt out of preserving raw' do
-      response = process('{"a":1}', nil, preserve_raw: false)
-      expect(response.env[:raw_body]).to be_nil
-    end
-  end
-
-  context 'with regexp type matching' do
-    let(:options) { { content_type: /\bjson$/ } }
-
-    it 'parses json body of correct type' do
-      response = process('{"a":1}', 'application/x-json')
-      expect(response.body).to eq('a' => 1)
-    end
-
-    it 'ignores json body of incorrect type' do
-      response = process('{"a":1}', 'text/json-xml')
-      expect(response.body).to eq('{"a":1}')
-    end
-  end
-
-  context 'with array type matching' do
-    let(:options) { { content_type: %w[a/b c/d] } }
-
-    it 'parses json body of correct type' do
-      expect(process('{"a":1}', 'a/b').body).to be_a(Hash)
-      expect(process('{"a":1}', 'c/d').body).to be_a(Hash)
-    end
-
-    it 'ignores json body of incorrect type' do
-      expect(process('{"a":1}', 'a/d').body).not_to be_a(Hash)
-    end
-  end
-
-  it 'chokes on invalid json' do
-    ['{!', '"a'].each do |data|
-      expect { process(data) }.to raise_error(Faraday::ParsingError)
-    end
-  end
-
-  context 'HEAD responses' do
-    it "nullifies the body if it's only one space" do
-      response = process(' ')
-      expect(response.body).to be_nil
-    end
-
-    it "nullifies the body if it's two spaces" do
-      response = process(' ')
-      expect(response.body).to be_nil
-    end
-  end
-end