search_object_graphql 0.2 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 3f77197a83b86e6ec05a3cb7f0f814bcd8e2d9d5590319b995657eccc64700f3
-  data.tar.gz: f31f8c4374c23da84bcccc42b6fc8307e08661cab1c67bed324aea90c2780faf
+  metadata.gz: c82b2028e4c0234e7f8109dbe66cb37746efc7c39e17d7e31a02b10f3e975c5e
+  data.tar.gz: b9d2f3bb799b65943fbcd3d777a75e9a1c3348dd7a33b0064212e373f6bd7422
 SHA512:
-  metadata.gz: 223b83055eaa7e7630694aee66c69abaa01d28a00f091535613b26c7d2f037370f6953ebcaba9d534315fcc381d0aecfbda06b0e0b25b9a0e015b01a233f29c8
-  data.tar.gz: 24a16c64a585f012bf10a10d0deefe724fec63338b7d901571595ccd92c7f8643373391f63165c2c30bb52d23a7d0e258f936dc7039ff06840140d303f9c5549
+  metadata.gz: a321cd52dfab7394ab746afa2b4d288be0a1beeec9eaa4286989fd0d63a4427f3153aa8aec2ebc42431daea0513b2604d6e123512a22e1cd394ef00ed330dc1c
+  data.tar.gz: 0e6a7c3a3a8a3695b4b2797aa0bcb26c5b98d3bd0bc8d91a605d2e26bafe6c58d001eb260d81fcc88d53c0f3b28f945bd6387ce3860bcd48412f781e38c4be1f

data/.rubocop.yml

@@ -7,15 +7,15 @@ AllCops:
     - search_object.gemspec
 
 # Disables "Line is too long"
-LineLength:
+Metrics/LineLength:
   Enabled: false
 
 # Disables Module has too many lines
-ModuleLength:
+Metrics/ModuleLength:
   Enabled: false
 
 # Disables "Missing top-level class documentation comment"
-Documentation:
+Style/Documentation:
   Enabled: false
 
 # Disables "Use each_with_object instead of inject"
@@ -49,3 +49,9 @@ RSpec/ExampleLength:
 # Disables "Too many expectations."
 RSpec/MultipleExpectations:
   Enabled: false
+
+Metrics/CyclomaticComplexity:
+  Enabled: false
+
+Metrics/PerceivedComplexity:
+  Enabled: false

data/.ruby-version

@@ -1 +1 @@
-2.5.1
+2.6.2

data/.travis.yml

@@ -1,8 +1,9 @@
 language: ruby
 rvm:
-  - 2.4.1
   - 2.5.2
   - 2.6.0
+  - 2.7.1
+  - 3.0.0
 script:
   - bundle exec rubocop
   - bundle exec rspec spec

data/CHANGELOG.md

@@ -1,5 +1,29 @@
 # Changelog
 
+## Version 1.0.1
+
+* __[fix]__ `camelize` defaults to false when not specified (@haines)
+
+## Version 1.0.0
+
+* __[break]__ Removed support for defining types via `type` method (@rstankov)
+* __[break]__ Require `GraphQL::Schema::Resolver` inheritance (@rstankov)
+* __[break]__ Removed support for legacy `GraphQL::Function` (@rstankov)
+* __[break]__ `type` creates type based on `GraphQL::Schema::Object`, not the deprecated `GraphQL::ObjectType.define` (@rstankov)
+
+## Version 0.3.2
+
+* __[feature]__ Added `camelize` argument to `option`, *`true` by default* (@glenbray)
+
+## Version 0.3.1
+
+* __[fix]__ Support for GraphQL gem version v1.9.16 (@ardinusawan)
+
+## Version 0.3
+
+* __[feature]__ Allow passing `required` key to option definition (@vfonic)
+* __[fix]__ Support for GraphQL gem enums (@Postmodum37)
+
 ## Version 0.2
 
 * Added support for GraphQL::Schema::Resolver (@rstankov)

data/README.md

@@ -39,6 +39,14 @@ Or install it yourself as:
 
     $ gem install search_object_graphql
 
+
+**Require manually in your project**
+
+```ruby
+require 'search_object'
+require 'search_object/plugin/graphql'
+```
+
 ## Dependencies
 
 - `SearchObject` >= 1.2
@@ -49,7 +57,7 @@ Or install it yourself as:
 Just include the ```SearchObject.module``` and define your search options and their types:
 
 ```ruby
-class PostResolver
+class PostResolver < GraphQL::Schema::Resolver
   include SearchObject.module(:graphql)
 
   type [PostType], null: false
@@ -86,7 +94,7 @@ You can find example of most important features and plugins - [here](https://git
 Search object itself can be documented, as well as its options:
 
 ```ruby
-class PostResolver
+class PostResolver < GraphQL::Schema::Resolver
   include SearchObject.module(:graphql)
 
   description 'Lists all posts'
@@ -99,7 +107,7 @@ end
 ### Default Values
 
 ```ruby
-class PostResolver
+class PostResolver < GraphQL::Schema::Resolver
   include SearchObject.module(:graphql)
 
   scope { Post.all }
@@ -113,7 +121,7 @@ end
 Sometimes you want to scope posts based on parent object, it is accessible as `object` property:
 
 ```ruby
-class PostResolver
+class PostResolver < GraphQL::Schema::Resolver
   include SearchObject.module(:graphql)
 
   # lists only posts from certain category
@@ -160,7 +168,7 @@ end
 Search objects can be used as [Relay Connections](https://graphql-ruby.org/relay/connections.html):
 
 ```ruby
-class PostResolver
+class PostResolver < GraphQL::Schema::Resolver
   include SearchObject.module(:graphql)
 
   type PostType.connection_type, null: false
@@ -173,22 +181,6 @@ end
 field :posts, resolver: PostResolver
 ```
 
-### Legacy Function Support
-
-```ruby
-class PostResolver
-  include SearchObject.module(:graphql)
-
-  type [PostType], null: false
-
-  # ...
-end
-```
-
-```ruby
-field :posts, function: PostResolver
-```
-
 ## Contributing
 
 1. Fork it

data/example/app/controllers/graphql_controller.rb

@@ -7,10 +7,10 @@ class GraphqlController < ApplicationController
                             context: {},
                             operation_name: params[:operationName])
     render json: result
-  rescue StandardError => error
-    raise error unless Rails.env.development?
+  rescue StandardError => e
+    raise e unless Rails.env.development?
 
-    handle_error_in_development error
+    handle_error_in_development e
   end
 
   private

data/example/app/graphql/resolvers/base_resolver.rb

@@ -0,0 +1,6 @@
+# frozen_string_literal: true
+
+module Resolvers
+  class BaseResolver < GraphQL::Schema::Resolver
+  end
+end

data/example/app/graphql/resolvers/base_search_resolver.rb

@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 module Resolvers
-  class BaseSearchResolver
+  class BaseSearchResolver < BaseResolver
     include SearchObject.module(:graphql)
 
     def escape_search_term(term)

data/example/app/graphql/resolvers/category_search.rb

@@ -2,7 +2,7 @@
 
 module Resolvers
   class CategorySearch < Resolvers::BaseSearchResolver
-    type Types::CategoryType.connection_type
+    type Types::CategoryType.connection_type, null: false
     description 'Lists categories'
 
     class OrderEnum < Types::BaseEnum

data/example/app/graphql/resolvers/post_search.rb

@@ -2,7 +2,7 @@
 
 module Resolvers
   class PostSearch < Resolvers::BaseSearchResolver
-    type Types::PostType.connection_type
+    type Types::PostType.connection_type, null: false
     description 'Lists posts'
 
     class OrderEnum < Types::BaseEnum

data/example/app/graphql/types/category_type.rb

@@ -4,6 +4,6 @@ module Types
   class CategoryType < BaseObject
     field :id, ID, null: false
     field :name, String, null: false
-    field :posts, function: Resolvers::PostSearch
+    field :posts, resolver: Resolvers::PostSearch
   end
 end

data/example/app/graphql/types/query_type.rb

@@ -2,7 +2,7 @@
 
 module Types
   class QueryType < Types::BaseObject
-    field :categories, function: Resolvers::CategorySearch
+    field :categories, resolver: Resolvers::CategorySearch
     field :posts, resolver: Resolvers::PostSearch
   end
 end

data/lib/search_object/plugin/graphql.rb

@@ -4,18 +4,19 @@ module SearchObject
   module Plugin
     module Graphql
       def self.included(base)
+        raise NotIncludedInResolverError, base unless base.ancestors.include? GraphQL::Schema::Resolver
+
         base.include SearchObject::Plugin::Enum
-        base.include ::GraphQL::Schema::Member::GraphQLTypeNames
         base.extend ClassMethods
       end
 
       attr_reader :object, :context
 
-      def initialize(filters: {}, object: nil, context: {}, scope: nil)
+      def initialize(filters: {}, object: nil, context: {}, scope: nil, field: nil)
         @object = object
         @context = context
 
-        super filters: filters, scope: scope
+        super filters: filters, scope: scope, field: field
       end
 
       # NOTE(rstankov): GraphQL::Schema::Resolver interface
@@ -26,108 +27,29 @@ module SearchObject
       end
 
       module ClassMethods
-        KEYS = %i[type default description].freeze
         def option(name, options = {}, &block)
-          config[:arguments] ||= {}
-          config[:arguments][name.to_s] = KEYS.inject({}) do |acc, key|
-            acc[key] = options[key] if options.key?(key)
-            acc
-          end
-
           type = options.fetch(:type) { raise MissingTypeDefinitionError, name }
-          options[:enum] = type.values.keys if type.respond_to?(:values)
 
-          super(name, options, &block)
-        end
+          argument_options = { required: options[:required] || false }
+          argument_options[:camelize] = options[:camelize] if options.include?(:camelize)
+          argument_options[:default_value] = options[:default] if options.include?(:default)
+          argument_options[:description] = options[:description] if options.include?(:description)
 
-        def type(value = :default, null: true, &block)
-          return config[:type] if value == :default && !block_given?
+          argument(name.to_s, type, **argument_options)
 
-          config[:type] = block_given? ? GraphQL::ObjectType.define(&block) : value
-          config[:null] = null
-        end
-
-        def complexity(value = :default)
-          return config[:complexity] || 1 if value == :default
+          options[:enum] = type.values.map { |value, enum_value| enum_value.value || value } if type.respond_to?(:values)
 
-          config[:complexity] = value
-        end
-
-        def description(value = :default)
-          return config[:description] if value == :default
-
-          config[:description] = value
-        end
-
-        def deprecation_reason(value = :default)
-          return config[:deprecation_reason] if value == :default
-
-          config[:deprecation_reason] = value
-        end
-
-        # NOTE(rstankov): GraphQL::Function interface (deprecated in favour of GraphQL::Schema::Resolver)
-        # Documentation - http://graphql-ruby.org/guides
-        def call(object, args, context)
-          new(filters: args.to_h, object: object, context: context).results
+          super(name, options, &block)
         end
 
-        # NOTE(rstankov): Used for GraphQL::Function
         def types
           GraphQL::Define::TypeDefiner.instance
         end
+      end
 
-        # NOTE(rstankov): Used for GraphQL::Function
-        def arguments
-          (config[:arguments] || {}).inject({}) do |acc, (name, options)|
-            argument = GraphQL::Argument.new
-            argument.name = name.to_s
-            argument.type = options.fetch(:type) { raise MissingTypeDefinitionError, name }
-            argument.default_value = options[:default] if options.key? :default
-            argument.description = options[:description] if options.key? :description
-
-            acc[name] = argument
-            acc
-          end
-        end
-
-        # NOTE(rstankov): Used for GraphQL::Schema::Resolver
-        def field_options
-          {
-            type: type,
-            description: description,
-            extras: [],
-            resolver_method: :resolve_with_support,
-            resolver_class: self,
-            deprecation_reason: deprecation_reason,
-            arguments: (config[:arguments] || {}).inject({}) do |acc, (name, options)|
-              acc[name] = ::GraphQL::Schema::Argument.new(
-                name: name.to_s,
-                type: options.fetch(:type) { raise MissingTypeDefinitionError, name },
-                description: options[:description],
-                required: !!options[:required],
-                default_value: options.fetch(:default) { ::GraphQL::Schema::Argument::NO_DEFAULT },
-                owner: self
-              )
-              acc
-            end,
-            null: !!config[:null],
-            complexity: complexity
-          }
-        end
-
-        # NOTE(rstankov): Used for GraphQL::Schema::Resolver
-        def visible?(_context)
-          true
-        end
-
-        # NOTE(rstankov): Used for GraphQL::Schema::Resolver
-        def accessible?(_context)
-          true
-        end
-
-        # NOTE(rstankov): Used for GraphQL::Schema::Resolver
-        def authorized?(_object, _context)
-          true
+      class NotIncludedInResolverError < ArgumentError
+        def initialize(base)
+          super "#{base.name} should inherit from GraphQL::Schema::Resolver. Current ancestors #{base.ancestors}"
         end
       end
 

data/lib/search_object/plugin/graphql/version.rb

@@ -3,7 +3,7 @@
 module SearchObject
   module Plugin
     module Graphql
-      VERSION = '0.2'
+      VERSION = '1.0.1'
     end
   end
 end

data/spec/search_object/plugin/graphql_spec.rb

@@ -5,17 +5,17 @@ require 'graphql'
 require 'ostruct'
 require 'search_object/plugin/graphql'
 
-describe SearchObject::Plugin::Graphql do
-  Post = Struct.new(:id) do
-    def to_json
-      { 'id' => id }
-    end
+Post = Struct.new(:id) do
+  def to_json(_options = {})
+    { 'id' => id }
   end
+end
 
-  class PostType < GraphQL::Schema::Object
-    field :id, ID, null: false
-  end
+class PostType < GraphQL::Schema::Object
+  field :id, ID, null: false
+end
 
+describe SearchObject::Plugin::Graphql do
   def define_schema(&block)
     query_type = Class.new(GraphQL::Schema::Object) do
       graphql_name 'Query'
@@ -31,7 +31,7 @@ describe SearchObject::Plugin::Graphql do
   end
 
   def define_search_class(&block)
-    Class.new do
+    Class.new(GraphQL::Schema::Resolver) do
       include SearchObject.module(:graphql)
 
       scope { [] }
@@ -52,6 +52,12 @@ describe SearchObject::Plugin::Graphql do
     end
   end
 
+  it 'requires class to inherit from GraphQL::Schema::Resolver' do
+    expect do
+      Class.new { include SearchObject.module(:graphql) }
+    end.to raise_error SearchObject::Plugin::Graphql::NotIncludedInResolverError
+  end
+
   it 'can be used as GraphQL::Schema::Resolver' do
     post_type = Class.new(GraphQL::Schema::Object) do
       graphql_name 'Post'
@@ -62,7 +68,7 @@ describe SearchObject::Plugin::Graphql do
     search_object = define_search_class do
       scope { [Post.new('1'), Post.new('2'), Post.new('3')] }
 
-      type [post_type]
+      type [post_type], null: 1
 
       option(:id, type: !types.ID) { |scope, value| scope.select { |p| p.id == value } }
     end
@@ -80,34 +86,6 @@ describe SearchObject::Plugin::Graphql do
     )
   end
 
-  it 'can be used as GraphQL::Function' do
-    post_type = GraphQL::ObjectType.define do
-      name 'Post'
-
-      field :id, !types.ID
-    end
-
-    search_object = define_search_class do
-      scope { [Post.new('1'), Post.new('2'), Post.new('3')] }
-
-      type types[post_type]
-
-      option(:id, type: !types.ID) { |scope, value| scope.select { |p| p.id == value } }
-    end
-
-    schema = define_schema do
-      field :posts, function: search_object
-    end
-
-    result = schema.execute '{ posts(id: "2") { id } }'
-
-    expect(result).to eq(
-      'data' => {
-        'posts' => [Post.new('2').to_json]
-      }
-    )
-  end
-
   it 'can access to parent object' do
     search_object = define_search_class do
       scope { object.posts }
@@ -164,81 +142,6 @@ describe SearchObject::Plugin::Graphql do
     )
   end
 
-  it 'can define a custom type' do
-    schema = define_search_class_and_return_schema do
-      type do
-        name 'Test'
-
-        field :title, types.String
-      end
-
-      description 'Test description'
-    end
-
-    result = schema.execute <<-SQL
-      {
-        __type(name: "Query") {
-          name
-          fields {
-            name
-            deprecationReason
-            type {
-              name
-              fields {
-                name
-              }
-            }
-          }
-        }
-      }
-    SQL
-
-    expect(result).to eq(
-      'data' => {
-        '__type' => {
-          'name' => 'Query',
-          'fields' => [{
-            'name' => 'posts',
-            'deprecationReason' => nil,
-            'type' => {
-              'name' => 'Test',
-              'fields' => [{
-                'name' => 'title'
-              }]
-            }
-          }]
-        }
-      }
-    )
-  end
-
-  it 'can be marked as deprecated' do
-    schema = define_search_class_and_return_schema do
-      type [PostType]
-      deprecation_reason 'Not needed any more'
-    end
-
-    result = schema.execute <<-QUERY
-      {
-        __type(name: "Query") {
-          name
-          fields {
-            name
-          }
-        }
-      }
-    QUERY
-
-    expect(result.to_h).to eq(
-      'data' => {
-        '__type' => {
-          'name' => 'Query',
-          'fields' => []
-        }
-      }
-    )
-  end
-
   describe 'option' do
     it 'converts GraphQL::Schema::Enum to SearchObject enum' do
       schema = define_search_class_and_return_schema do
@@ -271,8 +174,8 @@ describe SearchObject::Plugin::Graphql do
 
     it 'converts GraphQL::EnumType to SearchObject enum' do
       schema = define_search_class_and_return_schema do
-        enum_type = GraphQL::EnumType.define do
-          name 'TestEnum'
+        enum_type = Class.new(GraphQL::Schema::Enum) do
+          graphql_name 'TestEnum'
 
           value 'PRICE'
           value 'DATE'
@@ -314,9 +217,57 @@ describe SearchObject::Plugin::Graphql do
       )
     end
 
+    it 'sets default_value on the argument' do
+      schema = define_search_class_and_return_schema do
+        type PostType, null: true
+
+        option('option', type: types.String, default: 'default') { [] }
+      end
+
+      result = schema.execute <<~GRAPHQL
+        {
+          __type(name: "Query") {
+            name
+            fields {
+              args {
+                name
+                defaultValue
+              }
+            }
+          }
+        }
+      GRAPHQL
+
+      expect(result).to eq(
+        'data' => {
+          '__type' => {
+            'name' => 'Query',
+            'fields' => [{
+              'args' => [{
+                'name' => 'option',
+                'defaultValue' => '"default"'
+              }]
+            }]
+          }
+        }
+      )
+    end
+
+    it 'accepts "required"' do
+      schema = define_search_class_and_return_schema do
+        option(:id, type: types.String, required: true) do |_scope, value|
+          [Post.new(value)]
+        end
+      end
+
+      result = schema.execute '{ posts { id } }'
+
+      expect(result['errors'][0]['message']).to eq("Field 'posts' is missing required arguments: id")
+    end
+
     it 'accepts description' do
       schema = define_search_class_and_return_schema do
-        type PostType
+        type PostType, null: true
 
         option('option', type: types.String, description: 'what this argument does') { [] }
       end
@@ -350,6 +301,74 @@ describe SearchObject::Plugin::Graphql do
       )
     end
 
+    it 'accepts camelize' do
+      schema = define_search_class_and_return_schema do
+        type PostType, null: true
+
+        option('option_field', type: types.String, camelize: false)
+      end
+
+      result = schema.execute <<-SQL
+        {
+          __type(name: "Query") {
+            name
+            fields {
+              args {
+                name
+              }
+            }
+          }
+        }
+      SQL
+
+      expect(result.to_h).to eq(
+        'data' => {
+          '__type' => {
+            'name' => 'Query',
+            'fields' => [{
+              'args' => [{
+                'name' => 'option_field'
+              }]
+            }]
+          }
+        }
+      )
+    end
+
+    it 'does not override the default camelize option' do
+      schema = define_search_class_and_return_schema do
+        type PostType, null: true
+
+        option('option_field', type: types.String)
+      end
+
+      result = schema.execute <<~GRAPHQL
+        {
+          __type(name: "Query") {
+            name
+            fields {
+              args {
+                name
+              }
+            }
+          }
+        }
+      GRAPHQL
+
+      expect(result.to_h).to eq(
+        'data' => {
+          '__type' => {
+            'name' => 'Query',
+            'fields' => [{
+              'args' => [{
+                'name' => 'optionField'
+              }]
+            }]
+          }
+        }
+      )
+    end
+
     it 'raises error when no type is given' do
       expect { define_search_class { option :name } }.to raise_error described_class::MissingTypeDefinitionError
     end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: search_object_graphql
 version: !ruby/object:Gem::Version
-  version: '0.2'
+  version: 1.0.1
 platform: ruby
 authors:
 - Radoslav Stankov
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2019-06-19 00:00:00.000000000 Z
+date: 2021-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: graphql
@@ -134,6 +134,7 @@ files:
 - example/app/controllers/application_controller.rb
 - example/app/controllers/graphql_controller.rb
 - example/app/graphql/mutations/.keep
+- example/app/graphql/resolvers/base_resolver.rb
 - example/app/graphql/resolvers/base_search_resolver.rb
 - example/app/graphql/resolvers/category_search.rb
 - example/app/graphql/resolvers/post_search.rb
@@ -204,8 +205,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.7.6
+rubygems_version: 3.0.3
 signing_key: 
 specification_version: 4
 summary: Maps search objects to GraphQL resolvers