graphlient 0.5.0 → 0.7.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: c049e978a0c48e45fe84d49470bc890ae3b6ea86b86aab8158322616c62968dd
-  data.tar.gz: c7307c2b5c6ac6ac2b55ee4d416bbfffee1e29ed0826916254077add10370ee9
+  metadata.gz: aa3b9416b7374b0c3881224177f2a5646432e27ce9fcfdb2454b31ba318e231a
+  data.tar.gz: 6ca64471c0129dc021248fa309ed187bde0c9f9e58cf07a98d9072768e5446fe
 SHA512:
-  metadata.gz: 627d0d04d5ecea5e4c33ea0e59b15ac46fc458f76f179ac624d3519036865ad29267f629b1b3fda7592a50c846ea2cf06cc58a8f188a06caee44715268c18d07
-  data.tar.gz: 9742adef6879fbd08b00350674dced4b4886e476b45388d6d1a1afaaebba440153632e878649ff68beb223b158f365dd180b6db76e8463a13dbbeb4cdf8aa88d
+  metadata.gz: f035b28bfd747c72574b124cd5130e92798ac6b14d9000b73c6ce25357489f23ded22ca11dd74cc777c96dfeada948f86a9dd3b754838abf230cf82849e8f893
+  data.tar.gz: e6fd794359759c72051ddd7114fbe85c1110d0b743f55cd521cdcbb56fc9e6cecd4a70b6a6383f76d5304fd542d889e965d28738394995282768794059067b6c

data/.github/ISSUE_TEMPLATE/bug_report.md

@@ -0,0 +1,38 @@
+---
+name: Bug report
+about: Create a report to help us improve
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+**Describe the bug**
+A clear and concise description of what the bug is.
+
+**To Reproduce**
+Steps to reproduce the behavior:
+1. Go to '...'
+2. Click on '....'
+3. Scroll down to '....'
+4. See error
+
+**Expected behavior**
+A clear and concise description of what you expected to happen.
+
+**Screenshots**
+If applicable, add screenshots to help explain your problem.
+
+**Desktop (please complete the following information):**
+ - OS: [e.g. iOS]
+ - Browser [e.g. chrome, safari]
+ - Version [e.g. 22]
+
+**Smartphone (please complete the following information):**
+ - Device: [e.g. iPhone6]
+ - OS: [e.g. iOS8.1]
+ - Browser [e.g. stock browser, safari]
+ - Version [e.g. 22]
+
+**Additional context**
+Add any other context about the problem here.

data/.github/ISSUE_TEMPLATE/feature_request.md

@@ -0,0 +1,20 @@
+---
+name: Feature request
+about: Suggest an idea for this project
+title: ''
+labels: ''
+assignees: ''
+
+---
+
+**Is your feature request related to a problem? Please describe.**
+A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
+
+**Describe the solution you'd like**
+A clear and concise description of what you want to happen.
+
+**Describe alternatives you've considered**
+A clear and concise description of any alternative solutions or features you've considered.
+
+**Additional context**
+Add any other context or screenshots about the feature request here.

data/.github/workflows/ci.yml

@@ -0,0 +1,29 @@
+name: CI
+
+on: [push, pull_request]
+
+permissions:
+  contents: read
+
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        entry:
+          - { ruby: 2.7.2 }
+          - { ruby: 3.0.0 }
+          - { ruby: 3.1.2 }
+          - { ruby: "ruby-head", ignore: true }
+          - { ruby: "jruby-9.1.17", ignore: true }
+          - { ruby: "jruby-head", ignore: true }
+    steps:
+    - uses: actions/checkout@v3
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: ${{ matrix.entry.ruby }}
+        bundler-cache: true
+    - name: Run tests
+      continue-on-error: ${{ matrix.entry.ignore || false }}
+      run: bundle exec rspec spec/

data/.github/workflows/danger.yml

@@ -0,0 +1,22 @@
+name: Danger
+
+on: pull_request
+
+jobs:
+  danger:
+    runs-on: ubuntu-latest
+    env:
+      BUNDLE_GEMFILE: ${{ github.workspace }}/Gemfile.danger
+    steps:
+    - uses: actions/checkout@v3
+      with:
+        fetch-depth: 0
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 2.7.2
+        bundler-cache: true
+    - run: |
+        # the personal token is public, this is ok, base64 encode to avoid tripping Github
+        TOKEN=$(echo -n NWY1ZmM5MzEyMzNlYWY4OTZiOGU3MmI3MWQ3Mzk0MzgxMWE4OGVmYwo= | base64 --decode)
+        DANGER_GITHUB_API_TOKEN=$TOKEN bundle exec danger --verbose

data/.github/workflows/rubocop.yml

@@ -0,0 +1,19 @@
+name: Rubocop
+
+on: [push, pull_request]
+
+permissions:
+  contents: read
+
+jobs:
+  rubocop:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v3
+    - name: Set up Ruby
+      uses: ruby/setup-ruby@v1
+      with:
+        ruby-version: 2.7.2
+        bundler-cache: true
+    - name: Run rubocop
+      run: bundle exec rubocop

data/CHANGELOG.md

@@ -1,74 +1,87 @@
-### 0.6.0 (Next)
+### (Next)
 * Your contribution here.
 
-### 0.5.0 (12/28/2020)
+### 0.7.0 (2022/10/11)
+* [#98](https://github.com/ashkan18/graphlient/pull/98): Bring back danger checks and improve them - [@ashkan18](https://github.com/ashkan18).
+* [#94](https://github.com/ashkan18/graphlient/pull/94): Enabled fragments - [@rellampec](https://github.com/rellampec).
+* [#95](https://github.com/ashkan18/graphlient/pull/95): Upgrade faraday dependency to version 2 - [@kirillkaiumov](https://github.com/kirillkaiumov).
+
+### 0.6.0 (2022/06/11)
+
+* [#87](https://github.com/ashkan18/graphlient/pull/87): Raised `ExecutionError` with partial error response - [@QQism](https://github.com/QQism).
+* [#90](https://github.com/ashkan18/graphlient/pull/90): Added support for Ruby 3.1 - [@QQism](https://github.com/QQism).
+* [#90](https://github.com/ashkan18/graphlient/pull/90): Dropped support for Ruby 2.5 - [@QQism](https://github.com/QQism).
+* [#91](https://github.com/ashkan18/graphlient/pull/91): Update GHA for `danger` with right permissions - [@QQism](https://github.com/QQism).
+* [#89](https://github.com/ashkan18/graphlient/pull/89): Replace Travis CI with Github Actions - [@QQism](https://github.com/QQism).
+
+### 0.5.0 (2020/12/28)
 
 * [#81](https://github.com/ashkan18/graphlient/pull/81): Make graphlient run on ruby 3.0 - [@Burgestrand](https://github.com/Burgestrand).
 * [#79](https://github.com/ashkan18/graphlient/pull/79): Added client testing docs - [@GabrielDzul](https://github.com/GabrielDzul).
 
-### 0.4.0 (5/22/2020)
+### 0.4.0 (2020/05/22)
 
 * [#72](https://github.com/ashkan18/graphlient/pull/72): Add http_options - [@neroleung](https://github.com/neroleung).
 * [#71](https://github.com/ashkan18/graphlient/issues/70): Add `Graphlient::Errors::TimeoutError` - [@BenDrozdoff](https://github.com/BenDrozdoff).
 * [#75](https://github.com/ashkan18/graphlient/pull/75): Support Faraday 1.x - [@jfhinchcliffe](https://github.com/jfhinchcliffe).
 * [#78](https://github.com/ashkan18/graphlient/pull/78): Add description of timeout values - [@sap1enza](https://github.com/sap1enza).
 
-### 0.3.7 (11/14/2019)
+### 0.3.7 (2019/11/14)
 
 * [#68](https://github.com/ashkan18/graphlient/pull/68): Add `Graphlient::Errors::ConnectionFailedError` - [@neroleung](https://github.com/neroleung).
 
-### 0.3.6 (07/23/2019)
+### 0.3.6 (2019/07/23)
 
 * [#63](https://github.com/ashkan18/graphlient/pull/63): Remove unused method for attribute with typo - [@ashkan18](https://github.com/ashkan18).
 * [#62](https://github.com/ashkan18/graphlient/pull/62): Fix typo preventing access to response object on error - [@jmondo](https://github.com/jmondo).
 
-### 0.3.4 (01/31/2019)
+### 0.3.4 (2019/01/31)
 
 * [#56](https://github.com/ashkan18/graphlient/pull/56): Remove safe navigation usage to retain support for Ruby 2.2 - [@avinoth](https://github.com/avinoth).
 * [#57](https://github.com/ashkan18/graphlient/pull/57): Add support for parsing queries from a String - [@ateamlunchbox](https://github.com/ateamlunchbox).
 
-### 0.3.3 (09/23/2018)
+### 0.3.3 (2018/09/23)
 
 * [#50](https://github.com/ashkan18/graphlient/pull/50): More detailed error responses - [@ashkan18](https://github.com/ashkan18).
 
-### 0.3.2 (07/03/2018)
+### 0.3.2 (2018/07/03)
 
 * [#46](https://github.com/ashkan18/graphlient/pull/46): Fix issue with gathering error details when trying `to_s` on `GraphQLError` - [@ashkan18](https://github.com/ashkan18).
 * [#45](https://github.com/ashkan18/graphlient/pull/45): Drop Support for Ruby 2.2 and Lock RuboCop - [@jonallured](https://github.com/jonallured).
 
-### 0.3.1 (04/17/2018)
+### 0.3.1 (2018/04/17)
 
 * [#43](https://github.com/ashkan18/graphlient/pull/43): Allow to load and dump schema to json - [@povilasjurcys](https://github.com/povilasjurcys).
 
-### 0.3.0 (02/22/2018)
+### 0.3.0 (2018/02/22)
 
 * [#38](https://github.com/ashkan18/graphlient/pull/38): Add support for Ruby 2.5 - [@yuki24](https://github.com/yuki24).
 * [#39](https://github.com/ashkan18/graphlient/pull/39): Add support for Ruby 2.2 - [@yuki24](https://github.com/yuki24).
 * [#40](https://github.com/ashkan18/graphlient/pull/40): Add experimental support for JRuby - [@yuki24](https://github.com/yuki24).
 
-### 0.2.0 (11/09/2017)
+### 0.2.0 (2017/11/09)
 
 * [#33](https://github.com/ashkan18/graphlient/pull/33): Added dsl for supporting parametrized queries/mutations - [@ashkan18](https://github.com/ashkan18).
 * [#34](https://github.com/ashkan18/graphlient/issues/34): Fix: don't convert variables to `String` - [@dblock](https://github.com/dblock).
 
-### 0.1.0 (10/27/2017)
+### 0.1.0 (2017/10/27)
 
 * [#31](https://github.com/ashkan18/graphlient/issues/31): Fix: catch execution errors that don't contain field names - [@dblock](https://github.com/dblock).
 
-### 0.0.9 (10/26/2017)
+### 0.0.9 (2017/10/26)
 
 * [#28](https://github.com/ashkan18/graphlient/pull/28): Raise errors in `execute`, not only `query` - [@dblock](https://github.com/dblock).
 * [#29](https://github.com/ashkan18/graphlient/pull/29): Added `Graphlient::Adapters::HTTP::HTTPAdapter` that replaces Faraday with `Net::HTTP` - [@dblock](https://github.com/dblock).
 
-### 0.0.8 (10/26/2017)
+### 0.0.8 (2017/10/26)
 
 * [#27](https://github.com/ashkan18/graphlient/pull/27): Always raise an exception unless a query has succeeded - [@dblock](https://github.com/dblock).
 
-### 0.0.7 (10/24/2017)
+### 0.0.7 (2017/10/24)
 
 * [#26](https://github.com/ashkan18/graphlient/pull/26): Support String queries - [@dblock](https://github.com/dblock).
 
-### 0.0.6 (10/20/2017)
+### 0.0.6 (2017/10/20)
 
 * [#14](https://github.com/ashkan18/graphlient/pull/14): Switch to `graphql-client` for network calls and schema validation - [@ashkan18](https://github.com/ashkan18).
 * [#17](https://github.com/ashkan18/graphlient/pull/17): Specialize server errors as `Graphlient::Errors::Server` - [@dblock](https://github.com/dblock).
@@ -79,16 +92,16 @@
 * [#20](https://github.com/ashkan18/graphlient/pull/20): Added support for parameterized queries and mutations - [@dblock](https://github.com/dblock).
 * [#25](https://github.com/ashkan18/graphlient/pull/25): Added `client.parse` and `client.execute` to parse and execute queries separately - [@dblock](https://github.com/dblock).
 
-### 0.0.5 (10/5/2017)
+### 0.0.5 (2017/10/05)
 
 * [#11](https://github.com/ashkan18/graphlient/pull/11): Fixed query argument types - [@ashkan18](https://github.com/ashkan18).
 
-### 0.0.4 (10/4/2017)
+### 0.0.4 (2017/10/04)
 
 * [#8](https://github.com/ashkan18/graphlient/pull/8): Handle HTTP errors and raise `Graphlient::Errors::HTTP` on failure - [@dblock](https://github.com/dblock).
 * [#5](https://github.com/ashkan18/graphlient/pull/5): Added RuboCop, Ruby-style linter, CHANGELOG, CONTRIBUTING and RELEASING - [@dblock](https://github.com/dblock).
 * [#4](https://github.com/ashkan18/graphlient/pull/4): Refactored Graphlient::Client to take a URL and options, moved extensions - [@dblock](https://github.com/dblock).
 
-### 0.0.3 (10/3/2017)
+### 0.0.3 (2017/10/03)
 
 * Initial public release - [@ashkan18](https://github.com/ashkan18).

data/Dangerfile

@@ -1 +1,24 @@
-changelog.check
+# frozen_string_literal: true
+
+# --------------------------------------------------------------------------------------------------------------------
+# Has any changes happened inside the actual library code?
+# --------------------------------------------------------------------------------------------------------------------
+has_app_changes = !git.modified_files.grep(/lib/).empty?
+has_spec_changes = !git.modified_files.grep(/spec/).empty?
+
+# --------------------------------------------------------------------------------------------------------------------
+# You've made changes to lib, but didn't write any tests?
+# --------------------------------------------------------------------------------------------------------------------
+warn("There're library changes, but not tests. That's OK as long as you're refactoring existing code.", sticky: false) if has_app_changes && !has_spec_changes
+
+# --------------------------------------------------------------------------------------------------------------------
+# You've made changes to specs, but no library code has changed?
+# --------------------------------------------------------------------------------------------------------------------
+if !has_app_changes && has_spec_changes
+  message('We really appreciate pull requests that demonstrate issues, even without a fix. That said, the next step is to try and fix the failing tests!', sticky: false)
+end
+
+# --------------------------------------------------------------------------------------------------------------------
+# Have you updated CHANGELOG.md?
+# --------------------------------------------------------------------------------------------------------------------
+changelog.check!

data/Gemfile

@@ -10,11 +10,11 @@ end
 
 group :development do
   gem 'byebug', platform: :ruby
-  gem 'danger-changelog', '~> 0.2.1'
   gem 'rubocop', '0.56.0'
 end
 
 group :test do
+  gem 'faraday-rack', '~> 2.0'
   gem 'graphql', '~> 1.9'
   gem 'graphql-errors'
   gem 'rack-parser'

data/Gemfile.danger

@@ -0,0 +1,5 @@
+source 'https://rubygems.org'
+
+group :test do
+  gem 'danger-changelog', '~> 0.6.0'
+end

data/README.md

@@ -1,10 +1,25 @@
 # Graphlient
 
 [![Gem Version](https://badge.fury.io/rb/graphlient.svg)](https://badge.fury.io/rb/graphlient)
-[![Build Status](https://travis-ci.org/ashkan18/graphlient.svg?branch=master)](https://travis-ci.org/ashkan18/graphlient)
+[![Build Status](https://github.com/ashkan18/graphlient/actions/workflows/ci.yml/badge.svg)](https://github.com/ashkan18/graphlient/actions/workflows/ci.yml)
 
 A friendlier Ruby client for consuming GraphQL-based APIs. Built on top of your usual [graphql-client](https://github.com/github/graphql-client), but with better defaults, more consistent error handling, and using the [faraday](https://github.com/lostisland/faraday) HTTP client.
 
+# Table of Contents
+
+- [Installation](#installation)
+- [Usage](#usage)
+  - [Schema storing and loading on disk](#schema-storing-and-loading-on-disk)
+  - [Error Handling](#error-handling)
+  - [Executing Parameterized Queries and Mutations](#executing-parameterized-queries-and-mutations)
+  - [Parse and Execute Queries Separately](#parse-and-execute-queries-separately)
+  - [Dynamic vs. Static Queries](#dynamic-vs-static-queries)
+  - [Generate Queries with Graphlient::Query](#generate-queries-with-graphlientquery)
+  - [Create API Client Classes with Graphlient::Extension::Query](#create-api-client-classes-with-graphlientextensionquery)
+  - [Swapping the HTTP Stack](#swapping-the-http-stack)
+  - [Testing with Graphlient and RSpec](#testing-with-graphlient-and-rspec)
+-  [License](#license)
+
 ## Installation
 
 Add the following line to your Gemfile.
@@ -300,6 +315,62 @@ query.to_s
 # "\nquery {\n  invoice(id: 10){\n    line_items\n    }\n  }\n"
 ```
 
+### Use of Fragments
+
+[Fragments](https://github.com/github/graphql-client#defining-queries) should be referred by constant:
+
+```ruby
+module Fragments
+  Invoice = client.parse <<~'GRAPHQL'
+    fragment on Invoice {
+      id
+      feeInCents
+    }
+  GRAPHQL
+end
+```
+
+`Graphlient` offers the syntax below to refer to the original constant:
+  * Triple underscore `___` to refer to the fragment
+  * Double underscore `__` for namespace separator
+
+In this example, `Fragments::Invoice` would be referred as follows:
+
+```ruby
+invoice_query = client.parse do
+  query do
+    invoice(id: 10) do
+      id
+      ___Graphlient__InvoiceFragment
+    end
+  end
+end
+```
+
+The wrapped response only allows access to fields that have been explicitly asked for.
+In this example, while `id` has been referenced directly in the main query, `feeInCents` has been spread via fragment and trying to access it in the original wrapped response will throw [`GraphQL::Client::ImplicitlyFetchedFieldError`](https://github.com/github/graphql-client/blob/master/guides/implicitly-fetched-field-error.md) (to prevent data leaks between components).
+
+```ruby
+response = client.execute(invoice_query)
+result = response.data.invoice
+result.to_h
+# {"id" => 10, "feeInCents"=> 20000}
+result.id
+# 10
+result.fee_in_cents
+# raises GraphQL::Client::ImplicitlyFetchedFieldError
+```
+
+`feeInCents` cannot be fetched directly from the main query, but from the fragment as shown below:
+
+```ruby
+invoice = Fragments::Invoice.new(result)
+invoice.id
+# 10
+invoice.fee_in_cents
+# 20000
+```
+
 ### Create API Client Classes with Graphlient::Extension::Query
 
 You can include `Graphlient::Extensions::Query` in your class. This will add a new `method_missing` method to your context which will be used to generate GraphQL queries.
@@ -435,4 +506,3 @@ end
 ## License
 
 MIT License, see [LICENSE](LICENSE)
-

data/graphlient.gemspec

@@ -14,7 +14,6 @@ Gem::Specification.new do |s|
   s.homepage = 'http://github.com/ashkan18/graphlient'
   s.licenses = ['MIT']
   s.summary = 'A friendlier Ruby client for consuming GraphQL-based APIs.'
-  s.add_dependency 'faraday', '>= 1.0'
-  s.add_dependency 'faraday_middleware'
+  s.add_dependency 'faraday', '~> 2.0'
   s.add_dependency 'graphql-client'
 end

data/lib/graphlient/adapters/http/faraday_adapter.rb

@@ -1,5 +1,4 @@
 require 'faraday'
-require 'faraday_middleware'
 
 module Graphlient
   module Adapters

data/lib/graphlient/client.rb

@@ -67,7 +67,7 @@ module Graphlient
     end
 
     def errors_in_result?(response)
-      response.data && response.data.errors && response.data.errors.any?
+      response.data && response.data.errors && response.data.errors.all.any?
     end
   end
 end

data/lib/graphlient/query.rb

@@ -9,13 +9,15 @@ module Graphlient
 
     ROOT_NODES = %w[query mutation subscription].freeze
 
+    FRAGMENT_DEFITION = /___(?<const>[A-Z][a-zA-Z0-9_]*(__[A-Z][a-zA-Z0-9_]*)*)/
+
     attr_accessor :query_str
 
     def initialize(&block)
       @indents = 0
       @query_str = ''
       @variables = []
-      instance_eval(&block)
+      evaluate(&block)
     end
 
     def method_missing(method_name, *args, &block)
@@ -39,7 +41,24 @@ module Graphlient
 
     private
 
+    def evaluate(&block)
+      @last_block = block || self
+      (@context ||= {})[@last_block] ||= @last_block.binding
+      instance_eval(&block)
+    end
+
+    def resolve_fragment_constant(value)
+      return nil unless (match = value.to_s.match(FRAGMENT_DEFITION))
+      raw_const = match[:const].gsub('__', '::')
+      @context[@last_block].eval(raw_const).tap do |const|
+        msg = "Expected constant #{raw_const} to be GraphQL::Client::FragmentDefinition. Given #{const.class}"
+        raise Graphlient::Errors::Error, msg unless const.is_a? GraphQL::Client::FragmentDefinition
+      end
+    end
+
     def append_node(node, args, arg_processor: nil, &block)
+      node = "...#{resolve_fragment_constant(node)}".to_sym if node.to_s.start_with?('___')
+
       # add field
       @query_str << "\n#{indent}#{node}"
       # add filter
@@ -49,7 +68,7 @@ module Graphlient
       if block_given?
         @indents += 1
         @query_str << '{'
-        instance_eval(&block)
+        evaluate(&block)
         @query_str << '}'
         @indents -= 1
       end

data/lib/graphlient/version.rb

@@ -1,3 +1,3 @@
 module Graphlient
-  VERSION = '0.5.0'.freeze
+  VERSION = '0.7.0'.freeze
 end

data/spec/graphlient/adapters/http/faraday_adapter_spec.rb

@@ -19,8 +19,8 @@ describe Graphlient::Adapters::HTTP::FaradayAdapter do
       expect(client.http.connection.builder.handlers).to eq(
         [
           Faraday::Response::RaiseError,
-          FaradayMiddleware::EncodeJson,
-          FaradayMiddleware::ParseJson
+          Faraday::Request::Json,
+          Faraday::Response::Json
         ]
       )
     end
@@ -65,7 +65,8 @@ describe Graphlient::Adapters::HTTP::FaradayAdapter do
     before do
       stub_request(:post, url).to_return(
         status: 200,
-        body: DummySchema.execute(GraphQL::Introspection::INTROSPECTION_QUERY).to_json
+        body: DummySchema.execute(GraphQL::Introspection::INTROSPECTION_QUERY).to_json,
+        headers: { 'Content-Type' => 'application/json' }
       )
     end
 
@@ -90,7 +91,6 @@ describe Graphlient::Adapters::HTTP::FaradayAdapter do
 
     specify do
       expected_error_message = "Connection refused - #{error_message}"
-
       expect { client.schema }.to raise_error(Graphlient::Errors::ConnectionFailedError, expected_error_message)
     end
   end

data/spec/graphlient/client_query_spec.rb

@@ -25,6 +25,43 @@ describe Graphlient::Client do
         invoice = response.data.invoice
         expect(invoice.id).to eq '10'
       end
+
+      context 'with fragment' do
+        let(:invoice_fragment) do
+          client.parse <<~'GRAPHQL'
+            fragment on Invoice {
+              id
+              feeInCents
+            }
+          GRAPHQL
+        end
+
+        let(:invoice_fragment_const) do
+          stub_const('Graphlient::InvoiceFragment', invoice_fragment)
+        end
+
+        let(:query) do
+          invoice_fragment_const
+          client.parse do
+            query do
+              invoice(id: 10) do
+                ___Graphlient__InvoiceFragment
+              end
+            end
+          end
+        end
+
+        it '#parse' do
+          expect(query).to be_a GraphQL::Client::OperationDefinition
+        end
+
+        it '#execute' do
+          response = client.execute(query)
+          invoice = response.data.invoice
+          fragment = invoice_fragment.new(invoice)
+          expect(fragment.id).to eq '10'
+        end
+      end
     end
 
     context 'parameterized query' do
@@ -80,6 +117,29 @@ describe Graphlient::Client do
         GRAPHQL
       end
 
+      let(:execution_error_query) do
+        <<-GRAPHQL
+          query($id: Int) {
+            executionErrorInvoice(id: $id) {
+              id
+              feeInCents
+            }
+          }
+        GRAPHQL
+      end
+
+      let(:partial_success_query) do
+        <<-GRAPHQL
+          query {
+            someInvoices {
+              id
+              feeInCents
+              createdAt
+            }
+          }
+        GRAPHQL
+      end
+
       it '#execute' do
         response = client.execute(query, id: 42)
         invoice = response.data.invoice
@@ -97,10 +157,9 @@ describe Graphlient::Client do
 
       it 'fails on an execution error' do
         expect do
-          allow(OpenStruct).to receive(:new).and_raise StandardError, 'Unexpected error.'
-          client.execute(query, id: 42)
+          client.execute(execution_error_query, id: 42)
         end.to raise_error Graphlient::Errors::ExecutionError do |e|
-          expect(e.to_s).to eq 'invoice: Unexpected error.'
+          expect(e.to_s).to eq 'executionErrorInvoice: Execution Error'
         end
       end
 
@@ -119,6 +178,14 @@ describe Graphlient::Client do
           expect(e.response).to be_a GraphQL::Client::Response
         end
       end
+
+      it 'fails with a partial error response' do
+        expect do
+          client.execute(partial_success_query)
+        end.to raise_error Graphlient::Errors::ExecutionError do |e|
+          expect(e.response).to be_a GraphQL::Client::Response
+        end
+      end
     end
 
     context 'non-parameterized query' do

data/spec/graphlient/client_schema_spec.rb

@@ -7,13 +7,23 @@ describe Graphlient::Client do
 
   describe '#schema' do
     before do
-      stub_request(:post, url)
-        .to_return(body: DummySchema.execute(GraphQL::Introspection::INTROSPECTION_QUERY).to_json)
+      stub_request(:post, url).to_return(
+        body: DummySchema.execute(GraphQL::Introspection::INTROSPECTION_QUERY).to_json,
+        headers: { 'Content-Type' => 'application/json' }
+      )
     end
 
     context 'when server returns error' do
       before do
-        stub_request(:post, url).to_return(status: 500, body: { errors: [{ message: 'test message', extensions: { code: 'SOMETHING', timestamp: Time.now } }] }.to_json)
+        stub_request(:post, url).to_return(
+          status: 500,
+          body: {
+            errors: [
+              { message: 'test message', extensions: { code: 'SOMETHING', timestamp: Time.now } }
+            ]
+          }.to_json,
+          headers: { 'Content-Type' => 'application/json' }
+        )
       end
 
       it 'fails with an exception' do

data/spec/graphlient/schema_spec.rb

@@ -10,7 +10,10 @@ describe Graphlient::Schema do
     let!(:introspection_query_request) do
       stub_request(:post, url)
         .with(body: /query IntrospectionQuery/)
-        .to_return(body: DummySchema.execute(GraphQL::Introspection::INTROSPECTION_QUERY).to_json)
+        .to_return(
+          body: DummySchema.execute(GraphQL::Introspection::INTROSPECTION_QUERY).to_json,
+          headers: { 'Content-Type' => 'application/json' }
+        )
     end
 
     context 'when schema path is not given' do

data/spec/graphlient/webmock_client_query_spec.rb

@@ -27,7 +27,8 @@ describe 'App' do
   before do
     stub_request(:post, url).to_return(
       status: 200,
-      body: json_response
+      body: json_response,
+      headers: { 'Content-Type' => 'application/json' }
     )
   end
 

data/spec/spec_helper.rb

@@ -7,6 +7,7 @@ require 'byebug' if RUBY_ENGINE != 'jruby'
 require 'rack/test'
 require 'webmock/rspec'
 require 'vcr'
+require 'faraday/rack'
 
 Dir[File.join(File.dirname(__FILE__), 'support', '**/*.rb')].each do |file|
   require file

data/spec/support/mutations/create_invoice.rb

@@ -1,3 +1,4 @@
+require_relative '../types/invoice_type'
 class CreateInvoice < GraphQL::Schema::RelayClassicMutation
   null true
 

data/spec/support/queries/query.rb

@@ -10,6 +10,15 @@ class Query < GraphQL::Schema::Object
     argument :id, Integer, required: false
   end
 
+  field :execution_error_invoice, InvoiceType, null: false, extras: [:execution_errors] do
+    description 'Find invoice'
+    argument :id, Integer, required: false
+  end
+
+  field :some_invoices, [InvoiceType], null: true do
+    description 'List of invoices'
+  end
+
   def invoice(id: nil)
     return nil if id.nil?
     OpenStruct.new(
@@ -21,4 +30,18 @@ class Query < GraphQL::Schema::Object
   def not_null_invoice(*)
     nil
   end
+
+  def execution_error_invoice(id: nil, execution_errors:)
+    execution_errors.add(GraphQL::ExecutionError.new('Execution Error'))
+
+    invoice(id: id)
+  end
+
+  def some_invoices
+    [
+      OpenStruct.new(id: 0, fee_in_cents: 20_000),
+      OpenStruct.new(id: 1, fee_in_cents: 20_000),
+      OpenStruct.new(id: 2, fee_in_cents: 20_000)
+    ]
+  end
 end

data/spec/support/types/invoice_type.rb

@@ -4,4 +4,10 @@ class InvoiceType < GraphQL::Schema::Object
 
   field :id, ID, null: false
   field :fee_in_cents, Integer, null: true
+  field :created_at, String, null: true, extras: [:execution_errors]
+
+  def created_at(execution_errors:)
+    execution_errors.add(GraphQL::ExecutionError.new('This is a partial error'))
+    Time.now.iso8601
+  end
 end

metadata

@@ -1,43 +1,29 @@
 --- !ruby/object:Gem::Specification
 name: graphlient
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.7.0
 platform: ruby
 authors:
 - Ashkan Nasseri
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2020-12-28 00:00:00.000000000 Z
+date: 2022-10-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.0'
+        version: '2.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '1.0'
-- !ruby/object:Gem::Dependency
-  name: faraday_middleware
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.0'
 - !ruby/object:Gem::Dependency
   name: graphql-client
   requirement: !ruby/object:Gem::Requirement
@@ -58,15 +44,20 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/ISSUE_TEMPLATE/bug_report.md"
+- ".github/ISSUE_TEMPLATE/feature_request.md"
+- ".github/workflows/ci.yml"
+- ".github/workflows/danger.yml"
+- ".github/workflows/rubocop.yml"
 - ".gitignore"
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
-- ".travis.yml"
 - CHANGELOG.md
 - CONTRIBUTING.md
 - Dangerfile
 - Gemfile
+- Gemfile.danger
 - LICENSE
 - README.md
 - RELEASING.md
@@ -140,7 +131,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: 1.3.6
 requirements: []
-rubygems_version: 3.0.8
+rubygems_version: 3.1.4
 signing_key:
 specification_version: 4
 summary: A friendlier Ruby client for consuming GraphQL-based APIs.

data/.travis.yml

@@ -1,24 +0,0 @@
-language: ruby
-
-cache: bundler
-
-rvm:
-  - 2.5.0
-  - 2.7.2
-  - 3.0.0
-  - ruby-head
-  - jruby-9.1.16.0
-  - jruby-head
-
-before_install:
-  - gem update --system
-
-matrix:
-  include:
-    - rvm: 2.7.2
-      script:
-        - bundle exec danger
-  allow_failures:
-    - rvm: ruby-head
-    - rvm: jruby-9.1.16.0
-    - rvm: jruby-head