rspec-graphql_matchers 1.0.0.pre.0.1 → 1.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 2c995f57264270239040010ba138b480e7280b05e1ca75b54dbc4a03426ba42c
-  data.tar.gz: fcbc1410d22f83530270cd24db7cbe81541754a0b8cd58d27265dd7b248cdf3c
+  metadata.gz: d19eb108a03226d730c4ef1cc10afe2b54e2ae3e922e9617cd8d69dec4650d26
+  data.tar.gz: 49882ece0fb24f10899e6f4f923c2c639d9859ec6309c46135e7c3ea6f486a51
 SHA512:
-  metadata.gz: 150c4f842ffef148a7063803d09d4c5433f7ff63637e70fb9cafc2ddb3e6d480ccc1662af7f30aaf241791ae6c53e03e24698b8bd4055cf5dc2d47bd273ed10b
-  data.tar.gz: 2779881b25f48f04c8f276c4ce5ea6aee3e376f6058e83ecfa54b9c5e04c7e24c31127cb66c07f5a861b1da2af2fe8e12517fb369d52cb3edeb810e09d924552
+  metadata.gz: b0f0ec4e822a9ba1ae174edace4b3a77d76ece28df8c46fe128b75d02e97652191506890be7101c52b8c32644a6e48d401bf623c69a6fc60d81a434063281ae9
+  data.tar.gz: 56c4388cfa19bcc95967e2f16ec726a49a935c5ecccf5f30ae722384283cecac29ba3409c21dd925f40683f55f5d2fef7aa2ec872834980477498e7d015f619a

data/CHANGELOG.md

@@ -16,11 +16,14 @@
 
 ### Deprecations
 
--   `.with_metadata` and `.with_property` matchers for fields will be removed on the next release.
+-   `.with_metadata` and `.with_property` matchers for fields will be removed on the next release;
+-   `.accept_arguments` (plural form) will be removed on the next release;
+-   `.accept_argument` (singular form) receiving a hash with a single or multiple arguments will no longer be supported, use `accept_argument(name).of_type('MyType')` instead.
 
 ### New features
 
 -   Add support for Class-based type definition api (adds support for graphql-ruby v1.8.x). Please note that `.with_metadata` and `.with_property` support has been kept but will only work on fields defined using the legacy api.
+-   Implemented `accept_argument(arg_name).of_type('MyType')´ matcher, which returns much clearer error messages when the arguments are not found on the target type.
 
 ### Bug fixes
 

data/README.md

@@ -14,18 +14,17 @@ gem 'rspec-graphql_matchers'
 
 The matchers currently supported are:
 
--   `expect(a_graphql_object).to have_a_field(field_name).that_returns(valid_type)`
+-   `expect(a_graphql_object).to have_a_field(field_name).of_type(valid_type)`
 -   `expect(a_graphql_object).to implement(interface_name, ...)`
 -   `expect(a_mutation_type).to have_a_return_field(field_name).returning(valid_type)`
 -   `expect(a_mutation_type).to have_an_input_field(field_name).of_type(valid_type)`
 -   `expect(a_field).to be_of_type(valid_type)`
--   `expect(an_input).to accept_arguments(hash_of_arg_names_and_valid_types)`
--   `expect(an_input).to accept_arguments(hash_of_arg_names_and_valid_types)`
+-   `expect(an_input).to accept_argument(argument_name).of_type(valid_type)`
 
 Where a valid type for the expectation is either:
 
 -   A reference to the actual type you expect;
--   A String representation of a type: `"String!"`, `"Int!"`, `"[String]!"`
+-   [Recommended] A String representation of a type: `"String!"`, `"Int!"`, `"[String]!"`
     (note the exclamation mark at the end, as required by the [GraphQL specs](http://graphql.org/).
 
 For objects defined with the legacy `#define` api, testing `:property`, `:hash_key` and _metadata_ is also possible by chaining `.with_property`, `.with_hash_key` and `.with_metadata`. For example:
@@ -35,31 +34,22 @@ For objects defined with the legacy `#define` api, testing `:property`, `:hash_k
 
 ## Examples
 
-Given a `GraphQL::ObjectType` defined as
+Given a GraphQL object defined as
 
 ```ruby
-
-PostType = GraphQL::ObjectType.define do
-  name "Post"
+class PostType < GraphQL::Schema::Object
+  graphql_name "Post"
   description "A blog post"
 
-  interfaces [GraphQL::Relay::Node.interface]
-
-  field :id, !types.ID,
-             property: :post_id
+  implements GraphQL::Relay::Node.interface
 
-  field :comments,
-        !types[types.String],
-        hash_key: :post_comments
+  field :id, ID, null: false
+  field :comments, [String], null: false
+  field :isPublished, Boolean, null: true
 
-  field :isPublished,
-        admin_only: true
-
-  field :subposts, PostType do
-    type !PostType
-
-    argument :filter, types.String
-    argument :id, types.ID
+  field :subposts, PostType, null: true do
+    argument :filter, types.String, required: false
+    argument :id, types.ID, required: false
   end
 end
 ```
@@ -68,13 +58,11 @@ end
 
 ```ruby
 describe PostType do
-  it 'defines a field id of type ID!' do
-    expect(subject).to have_field(:id).that_returns(!types.ID)
-  end
+  subject { described_class }
 
-  # Fluent alternatives
-  it { is_expected.to have_field(:id).of_type("ID!") }
-  it { is_expected.to have_a_field(:id).returning("ID!") }
+  it { is_expected.to have_field(:id).of_type(!types.ID) }
+  it { is_expected.to have_field(:comments).of_type("[String!]!") }
+  it { is_expected.to have_field(:isPublished).of_type("Boolean") }
 end
 ```
 
@@ -92,80 +80,54 @@ describe PostType do
   describe 'subposts' do
     subject { PostType.fields['subposts'] }
 
-    # You can use your type object directly when building expectations
-    it 'has type PostType' do
-      expect(subject).to be_of_type(!PostType)
-    end
-
-    # Or as usual, a literal String
-    it { is_expected.to be_of_type('Post!') }
+    it { is_expected.to be_of_type('Post') }
   end
 end
 ```
 
-Keep in mind that when using strings as type expectation you have to use the
-type name (`Post`) and not the constant name (`PostType`).
-
-Using your type objects directly is riskier than the string representation, since
-renaming the graphql name of an object is potentially a breaking change that
-wouldn't get caught by your test suite.
-
-You can also use the built-in [graphql-ruby](https://github.com/rmosolgo/graphql-ruby) scalar types:
+### 3) For objects defined using the legacy `#define` api, you can also use `with_property`, `with_hash_key` and `with_metadata`:
 
 ```ruby
-# ensure you have the GraphQL type definer available in your tests
-types = GraphQL::Define::TypeDefiner.instance
+PostTypeWithDefineApi = GraphQL::ObjectType.define do
+  name "DefinedPost"
 
-describe PostType do
-  describe 'comments' do
-    subject { PostType.fields['comments'] }
-    it { is_expected.to be_of_type(!types[types.String]) }
-    it { is_expected.to be_of_type('[String]!') }
-  end
-end
-```
+  interfaces [GraphQL::Relay::Node.interface]
 
-### 3) Test a specific field with `with_property`, `with_hash_key` and `with_metadata`
+  field :id, !types.ID, property: :post_id
+  field :comments, !types[types.String], hash_key: :post_comments
+  field :isPublished, admin_only: true
+end
 
-```ruby
-describe PostType do
-  it { is_expected.to have_a_field(:id).with_property(:post_id) }
+describe PostTypeWithDefineApi do
+  it { is_expected.to have_a_field(:id).of_type('ID!').with_property(:post_id) }
   it { is_expected.to have_a_field(:comments).with_hash_key(:post_comments) }
   it { is_expected.to have_a_field(:isPublished).with_metadata(admin_only: true) }
 end
 ```
 
-### 4) Test the arguments accepted by a field with `accept_arguments` matcher:
+### 4) Test the arguments accepted by a field with `accept_argument` matcher:
 
 ```ruby
 describe PostType do
   describe 'subposts' do
     subject { PostType.fields['subposts'] }
 
-    let(:a_whole_bunch_of_args) do
-      { filter: 'String', id: types.Int, pippo: 'Float', posts: PostType }
-    end
-
     it 'accepts a filter and an id argument, of types String and ID' do
-      expect(subject).to accept_arguments(filter: types.String, id: types.ID)
+      expect(subject).to accept_argument(:filter).of_type('String')
+      expect(subject).to accept_argument(:id).of_type('ID')
     end
 
-    # You can also test if a field does not accept args. Not quite useful :D.
-    it { is_expected.not_to accept_arguments(a_whole_bunch_of_args) }
+    it { is_expected.not_to accept_argument(:weirdo) }
   end
 end
 ```
 
-The spec will only pass if all attributes/types specified in the hash are
-defined on the field.
-
-For better fluency, `accept_arguments` is also available in singular form, as
-`accept_argument`.
-
 ### 5) Test an object's interface implementations:
 
 ```ruby
 describe PostType do
+  subject { described_class }
+
   it 'implements interface Node' do
     expect(subject).to implement('Node')
   end
@@ -178,6 +140,8 @@ end
 
 ## TODO
 
+-   Support GraphQL 1.9.x;
+-   Check the method used for resolving a field;
 -   New matchers!
 
 ## Contributing

data/lib/rspec/graphql_matchers/accept_argument.rb

@@ -0,0 +1,73 @@
+require_relative 'base_matcher'
+require_relative './have_a_field_matchers/of_type'
+
+module RSpec
+  module GraphqlMatchers
+    class AcceptArgument < BaseMatcher
+      def initialize(expected_arg_name)
+        @expectations = []
+
+        if expected_arg_name.is_a?(Hash)
+          (expected_arg_name, expected_type) = expected_arg_name.to_a.first
+          of_type(expected_type)
+
+          warn 'DEPRECATION WARNING: using accept_arguments with a hash will be '\
+               'deprecated on the next major release. Use the format ' \
+               "`accept_argument(:argument_name).of_type('ExpectedType!') instead.`"
+        end
+
+        @expected_arg_name = expected_arg_name.to_s
+      end
+
+      def matches?(graph_object)
+        @graph_object = graph_object
+
+        @actual_argument = field_arguments[@expected_arg_name]
+        return false if @actual_argument.nil?
+
+        @results = @expectations.select do |matcher|
+          !matcher.matches?(@actual_argument)
+        end
+
+        @results.empty?
+      end
+
+      def of_type(expected_field_type)
+        @expectations << HaveAFieldMatchers::OfType.new(expected_field_type)
+        self
+      end
+
+      def failure_message
+        base_msg = "expected #{member_name(@graph_object)} " \
+          "to accept argument `#{@expected_arg_name}`" \
+
+        return "#{base_msg} #{failure_messages.join(', ')}" if @actual_argument
+
+        "#{base_msg} but no argument was found with that name"
+      end
+
+      def description
+        ["accept argument `#{@expected_arg_name}`"].concat(descriptions).join(', ')
+      end
+
+      private
+
+      def descriptions
+        @results.map(&:description)
+      end
+
+      def failure_messages
+        @results.map(&:failure_message)
+      end
+
+      def field_arguments
+        if @graph_object.respond_to?(:arguments)
+          @graph_object.public_send(:arguments)
+        else
+          raise "Invalid object #{@graph_object} provided to accept_argument " \
+            'matcher. It does not seem to be a valid GraphQL object type.'
+        end
+      end
+    end
+  end
+end

data/lib/rspec/graphql_matchers/accept_arguments.rb

@@ -1,5 +1,5 @@
 require_relative 'base_matcher'
-require 'pry'
+
 module RSpec
   module GraphqlMatchers
     class AcceptArguments < BaseMatcher

data/lib/rspec/graphql_matchers/base_matcher.rb

@@ -10,10 +10,13 @@ module RSpec
       end
 
       def types_match?(actual_type, expected_type)
-        actual_type = actual_type.to_graphql if actual_type.respond_to?(:to_graphql)
-        expected_type = expected_type.to_graphql if expected_type.respond_to?(:to_graphql)
+        expected_type.nil? || type_name(expected_type) == type_name(actual_type)
+      end
+
+      def type_name(a_type)
+        a_type = a_type.to_graphql if a_type.respond_to?(:to_graphql)
 
-        expected_type.nil? || expected_type.to_s == actual_type.to_s
+        a_type.to_s
       end
     end
   end

data/lib/rspec/graphql_matchers/be_of_type.rb

@@ -1,4 +1,5 @@
 require_relative 'base_matcher'
+require 'pry'
 
 module RSpec
   module GraphqlMatchers
@@ -11,12 +12,12 @@ module RSpec
 
       def matches?(actual_sample)
         @sample = actual_sample
-        sample.respond_to?(:type) && sample.type.to_s == @expected.to_s
+        sample.respond_to?(:type) && types_match?(sample.type, @expected)
       end
 
       def failure_message
         "expected field '#{member_name(sample)}' to be of type '#{expected}', " \
-        "but it was '#{sample.type}'"
+        "but it was '#{type_name(sample.type)}'"
       end
 
       def description

data/lib/rspec/graphql_matchers/have_a_field.rb

@@ -7,18 +7,10 @@ require_relative './have_a_field_matchers/with_hash_key'
 module RSpec
   module GraphqlMatchers
     class HaveAField < BaseMatcher
-      DESCRIPTIONS = {
-        type: 'of type `%s`',
-        property: 'reading from the `%s` property',
-        hash_key: 'reading from the `%s` hash_key',
-        metadata: 'with metadata `%s`'
-      }.freeze
-
       def initialize(expected_field_name, fields = :fields)
         @expected_field_name = expected_field_name.to_s
         @fields = fields.to_sym
         @expectations = []
-        @descriptions = []
       end
 
       def matches?(graph_object)
@@ -62,7 +54,7 @@ module RSpec
         base_msg = "expected #{member_name(@graph_object)} " \
           "to define field `#{@expected_field_name}`" \
 
-        return "#{base_msg} #{descriptions.join(', ')}" if @actual_field
+        return "#{base_msg} #{failure_messages.join(', ')}" if @actual_field
 
         "#{base_msg} but no field was found with that name"
       end
@@ -77,6 +69,10 @@ module RSpec
         @results.map(&:description)
       end
 
+      def failure_messages
+        @results.map(&:failure_message)
+      end
+
       def field_collection
         if @graph_object.respond_to?(@fields)
           @graph_object.public_send(@fields)

data/lib/rspec/graphql_matchers/have_a_field_matchers/of_type.rb

@@ -5,6 +5,10 @@ module RSpec
         def description
           "of type `#{expected}`"
         end
+
+        def failure_message
+          "#{description}, but it was `#{type_name(sample.type)}`"
+        end
       end
     end
   end

data/lib/rspec/graphql_matchers/have_a_field_matchers/with_hash_key.rb

@@ -11,7 +11,12 @@ module RSpec
         end
 
         def matches?(actual_field)
-          get_hash_key(actual_field) == @expected_hash_key.to_sym
+          @actual_hash_key = get_hash_key(actual_field)
+          @actual_hash_key == @expected_hash_key.to_sym
+        end
+
+        def failure_message
+          "#{description}, but it was `#{@actual_hash_key}`"
         end
 
         private

data/lib/rspec/graphql_matchers/have_a_field_matchers/with_metadata.rb

@@ -11,8 +11,13 @@ module RSpec
         end
 
         def matches?(actual_field)
+          @actual_metadata = actual_field.metadata
           actual_field.metadata == @expected_metadata
         end
+
+        def failure_message
+          "#{description}, but it was `#{@actual_metadata}`"
+        end
       end
     end
   end

data/lib/rspec/graphql_matchers/have_a_field_matchers/with_property.rb

@@ -11,7 +11,12 @@ module RSpec
         end
 
         def matches?(actual_field)
-          actual_field.property.to_sym == @expected_property_name.to_sym
+          @actual_property = actual_field.property.to_sym
+          @actual_property == @expected_property_name.to_sym
+        end
+
+        def failure_message
+          "#{description}, but it was `#{@actual_property}`"
         end
       end
     end

data/lib/rspec/graphql_matchers/implement.rb

@@ -3,8 +3,8 @@ require_relative 'base_matcher'
 module RSpec
   module GraphqlMatchers
     class Implement < BaseMatcher
-      def initialize(interface_names)
-        @expected = interface_names.map(&:to_s)
+      def initialize(interfaces)
+        @expected = interfaces.map {|interface| interface_name(interface) }
       end
 
       def matches?(graph_object)
@@ -33,13 +33,13 @@ module RSpec
 
       def actual
         if @graph_object.respond_to?(:interfaces)
-          @graph_object.interfaces.map do |interface|
+          return @graph_object.interfaces.map do |interface|
             interface_name(interface)
           end
-        else
-          raise "Invalid object #{@graph_object} provided to #{matcher_name} " \
-            'matcher. It does not seem to be a valid GraphQL object type.'
         end
+
+        raise "Invalid object #{@graph_object} provided to #{matcher_name} " \
+          'matcher. It does not seem to be a valid GraphQL object type.'
       end
 
       def interface_name(interface)

data/lib/rspec/graphql_matchers/matchers.rb

@@ -1,6 +1,7 @@
 require 'rspec/matchers'
 require 'rspec/graphql_matchers/be_of_type'
 require 'rspec/graphql_matchers/accept_arguments'
+require 'rspec/graphql_matchers/accept_argument'
 require 'rspec/graphql_matchers/have_a_field'
 require 'rspec/graphql_matchers/implement'
 
@@ -10,10 +11,13 @@ module RSpec
       RSpec::GraphqlMatchers::BeOfType.new(expected)
     end
 
+    def accept_argument(expected_argument)
+      RSpec::GraphqlMatchers::AcceptArgument.new(expected_argument)
+     end
+
     def accept_arguments(expected_args)
       RSpec::GraphqlMatchers::AcceptArguments.new(expected_args)
     end
-    alias accept_argument accept_arguments
 
     # rubocop:disable Style/PredicateName
     def have_a_field(field_name)

data/lib/rspec/graphql_matchers/version.rb

@@ -1,5 +1,5 @@
 module Rspec
   module GraphqlMatchers
-    VERSION = '1.0.0-0.1'.freeze
+    VERSION = '1.0'.freeze
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rspec-graphql_matchers
 version: !ruby/object:Gem::Version
-  version: 1.0.0.pre.0.1
+  version: '1.0'
 platform: ruby
 authors:
 - Samuel Brandão
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-06-18 00:00:00.000000000 Z
+date: 2019-06-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql
@@ -108,6 +108,7 @@ files:
 - bin/console
 - bin/setup
 - lib/rspec/graphql_matchers.rb
+- lib/rspec/graphql_matchers/accept_argument.rb
 - lib/rspec/graphql_matchers/accept_arguments.rb
 - lib/rspec/graphql_matchers/base_matcher.rb
 - lib/rspec/graphql_matchers/be_of_type.rb
@@ -120,6 +121,7 @@ files:
 - lib/rspec/graphql_matchers/matchers.rb
 - lib/rspec/graphql_matchers/types_helper.rb
 - lib/rspec/graphql_matchers/version.rb
+- rspec-graphql_matchers-1.0.0.pre.0.1.gem
 - rspec-graphql_matchers.gemspec
 homepage: https://github.com/khamusa/rspec-graphql_matchers
 licenses:
@@ -136,9 +138,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements: []
 rubyforge_project: 
 rubygems_version: 2.7.8