graphql 1.9.6 → 1.9.7

data/spec/integration/mongoid/star_trek/schema.rb

@@ -55,7 +55,6 @@ module StarTrek
 
   class CustomBaseEdgeType < GraphQL::Types::Relay::BaseEdge
     node_type(BaseType)
-    graphql_name "CustomBaseEdge"
     field :upcased_name, String, null: true
     field :upcased_parent_name, String, null: true
     field :edge_class_name, String, null: true
@@ -413,7 +412,7 @@ module StarTrek
     end
 
     def self.id_from_object(object, type, ctx)
-      GraphQL::Schema::UniqueWithinType.encode(type.name, object.id)
+      GraphQL::Schema::UniqueWithinType.encode(type.graphql_name, object.id)
     end
 
     lazy_resolve(LazyWrapper, :value)

data/spec/integration/rails/graphql/input_object_spec.rb

@@ -0,0 +1,19 @@
+# frozen_string_literal: true
+require "spec_helper"
+
+describe GraphQL::Schema::InputObject do
+  let(:input_object) do
+    Dummy::DairyProductInput.new(
+      { source: 'COW',  fatContent: 0.8 },
+      defaults_used: Set.new,
+      context: GraphQL::Query::NullContext)
+  end
+
+  describe '#to_json' do
+    it 'returns JSON serialized representation of the variables hash' do
+      # Regression note: Previously, calling `to_json` on input objects caused stack too deep errors
+      assert_equal input_object.to_json, { source: "COW", fat_content: 0.8 }.to_json
+    end
+  end
+end
+

data/spec/integration/rails/graphql/relay/array_connection_spec.rb

@@ -65,35 +65,7 @@ describe GraphQL::Relay::ArrayConnection do
       assert_equal("NQ", result["data"]["rebels"]["ships"]["pageInfo"]["endCursor"])
     end
 
-    it "provides bidirectional_pagination" do
-      result = star_wars_query(query_string, "first" => 1)
-      last_cursor = get_last_cursor(result)
-
-      # When going forwards, bi-directional pagination
-      # returns `true` even for `hasPreviousPage`
-      result = star_wars_query(query_string, "first" => 1, "after" => last_cursor)
-      assert_equal(true, get_page_info(result, "ships")["hasNextPage"])
-      assert_equal(false, get_page_info(result, "ships")["hasPreviousPage"])
-
-      result = with_bidirectional_pagination {
-        star_wars_query(query_string, "first" => 3, "after" => last_cursor)
-      }
-      assert_equal(true, get_page_info(result, "ships")["hasNextPage"])
-      assert_equal(true, get_page_info(result, "ships")["hasPreviousPage"])
-
-      # When going backwards, bi-directional pagination
-      # returns true for `hasNextPage`
-      last_cursor = get_last_cursor(result)
-      result = star_wars_query(query_string, "last" => 1, "before" => last_cursor)
-      assert_equal(false, get_page_info(result, "ships")["hasNextPage"])
-      assert_equal(true, get_page_info(result, "ships")["hasPreviousPage"])
 
-      result = with_bidirectional_pagination {
-        star_wars_query(query_string, "last" => 2, "before" => last_cursor)
-      }
-      assert_equal(true, get_page_info(result, "ships")["hasNextPage"])
-      assert_equal(true, get_page_info(result, "ships")["hasPreviousPage"])
-    end
 
     it 'slices the result' do
       result = star_wars_query(query_string, "first" => 1)
@@ -286,5 +258,52 @@ describe GraphQL::Relay::ArrayConnection do
         assert_equal(first_second_and_third_names, get_names(result))
       end
     end
+
+    describe "bidirectional pagination" do
+      it "provides bidirectional_pagination" do
+        result = star_wars_query(query_string, "first" => 1)
+        last_cursor = get_last_cursor(result)
+
+        # When going forwards, bi-directional pagination
+        # returns `true` even for `hasPreviousPage`
+        result = star_wars_query(query_string, "first" => 1, "after" => last_cursor)
+        assert_equal(true, get_page_info(result, "ships")["hasNextPage"])
+        assert_equal(false, get_page_info(result, "ships")["hasPreviousPage"])
+
+        result = with_bidirectional_pagination {
+          star_wars_query(query_string, "first" => 3, "after" => last_cursor)
+        }
+        assert_equal(true, get_page_info(result, "ships")["hasNextPage"])
+        assert_equal(true, get_page_info(result, "ships")["hasPreviousPage"])
+
+        # When going backwards, bi-directional pagination
+        # returns true for `hasNextPage`
+        last_cursor = get_last_cursor(result)
+        result = star_wars_query(query_string, "last" => 1, "before" => last_cursor)
+        assert_equal(false, get_page_info(result, "ships")["hasNextPage"])
+        assert_equal(true, get_page_info(result, "ships")["hasPreviousPage"])
+
+        result = with_bidirectional_pagination {
+          star_wars_query(query_string, "last" => 2, "before" => last_cursor)
+        }
+        assert_equal(true, get_page_info(result, "ships")["hasNextPage"])
+        assert_equal(true, get_page_info(result, "ships")["hasPreviousPage"])
+      end
+
+      it "returns correct page info when the before cursor belongs to the last element in the array" do
+        result = with_bidirectional_pagination{
+          star_wars_query(query_string, "last" => 1)
+        }
+
+        last_cursor = get_last_cursor(result)
+
+        result = with_bidirectional_pagination{
+          star_wars_query(query_string, "before" => last_cursor, "last" => 1)
+        }
+
+        assert_equal(true, get_page_info(result, "ships")["hasNextPage"])
+        assert_equal(true, get_page_info(result, "ships")["hasPreviousPage"])
+      end
+    end
   end
 end

data/spec/integration/rails/graphql/schema_spec.rb

@@ -129,6 +129,24 @@ describe GraphQL::Schema do
     end
   end
 
+  describe "#disable_introspection_entry_points" do
+    it "enables entry points by default" do
+      refute_empty empty_schema.introspection_system.entry_points
+    end
+
+    describe "when disable_introspection_entry_points is configured" do
+      let(:schema) do
+        GraphQL::Schema.define do
+          disable_introspection_entry_points
+        end
+      end
+
+      it "clears entry points" do
+        assert_empty schema.introspection_system.entry_points
+      end
+    end
+  end
+
   describe "object_from_id" do
     describe "when the hook wasn't implemented" do
       it "raises not implemented" do

data/spec/integration/tmp/app/graphql/types/date_type.rb

@@ -0,0 +1,14 @@
+# typed: true
+module Types
+  class DateType < Types::BaseScalar
+    def self.coerce_input(input_value, context)
+      # Override this to prepare a client-provided GraphQL value for your Ruby code
+      input_value
+    end
+
+    def self.coerce_result(ruby_value, context)
+      # Override this to serialize a Ruby value for the GraphQL response
+      ruby_value.to_s
+    end
+  end
+end

data/spec/integration/tmp/dummy/Gemfile

@@ -0,0 +1,50 @@
+source 'https://rubygems.org'
+
+git_source(:github) do |repo_name|
+  repo_name = "#{repo_name}/#{repo_name}" unless repo_name.include?("/")
+  "https://github.com/#{repo_name}.git"
+end
+
+
+# Bundle edge Rails instead: gem 'rails', github: 'rails/rails'
+gem 'rails', '~> 5.0.7', '>= 5.0.7.2'
+# Use Puma as the app server
+gem 'puma', '~> 3.0'
+# Use SCSS for stylesheets
+gem 'sass-rails', '~> 5.0'
+# Use Uglifier as compressor for JavaScript assets
+gem 'uglifier', '>= 1.3.0'
+# Use CoffeeScript for .coffee assets and views
+gem 'coffee-rails', '~> 4.2'
+# See https://github.com/rails/execjs#readme for more supported runtimes
+# gem 'therubyracer', platforms: :ruby
+
+# Use jquery as the JavaScript library
+gem 'jquery-rails'
+# Turbolinks makes navigating your web application faster. Read more: https://github.com/turbolinks/turbolinks
+gem 'turbolinks', '~> 5'
+# Build JSON APIs with ease. Read more: https://github.com/rails/jbuilder
+gem 'jbuilder', '~> 2.5'
+# Use Redis adapter to run Action Cable in production
+# gem 'redis', '~> 3.0'
+# Use ActiveModel has_secure_password
+# gem 'bcrypt', '~> 3.1.7'
+
+# Use Capistrano for deployment
+# gem 'capistrano-rails', group: :development
+
+group :development, :test do
+  # Call 'byebug' anywhere in the code to stop execution and get a debugger console
+  gem 'byebug', platform: :mri
+end
+
+group :development do
+  # Access an IRB console on exception pages or by using <%= console %> anywhere in the code.
+  gem 'web-console', '>= 3.3.0'
+  gem 'listen', '~> 3.0.5'
+end
+
+# Windows does not include zoneinfo files, so bundle the tzinfo-data gem
+gem 'tzinfo-data', platforms: [:mingw, :mswin, :x64_mingw, :jruby]
+
+gem 'graphiql-rails', group: :development

data/spec/integration/tmp/dummy/README.md

@@ -0,0 +1,24 @@
+# README
+
+This README would normally document whatever steps are necessary to get the
+application up and running.
+
+Things you may want to cover:
+
+* Ruby version
+
+* System dependencies
+
+* Configuration
+
+* Database creation
+
+* Database initialization
+
+* How to run the test suite
+
+* Services (job queues, cache servers, search engines, etc.)
+
+* Deployment instructions
+
+* ...

data/spec/integration/tmp/dummy/Rakefile

@@ -0,0 +1,6 @@
+# Add your own tasks in files placed in lib/tasks ending in .rake,
+# for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
+
+require_relative 'config/application'
+
+Rails.application.load_tasks

data/spec/integration/tmp/dummy/app/assets/config/manifest.js

@@ -0,0 +1,3 @@
+//= link_tree ../images
+//= link_directory ../javascripts .js
+//= link_directory ../stylesheets .css

data/spec/integration/tmp/dummy/app/assets/javascripts/application.js

@@ -0,0 +1,16 @@
+// This is a manifest file that'll be compiled into application.js, which will include all the files
+// listed below.
+//
+// Any JavaScript/Coffee file within this directory, lib/assets/javascripts, vendor/assets/javascripts,
+// or any plugin's vendor/assets/javascripts directory can be referenced here using a relative path.
+//
+// It's not advisable to add code directly here, but if you do, it'll appear at the bottom of the
+// compiled file. JavaScript code in this file should be added after the last require_* statement.
+//
+// Read Sprockets README (https://github.com/rails/sprockets#sprockets-directives) for details
+// about supported directives.
+//
+//= require jquery
+//= require jquery_ujs
+//= require turbolinks
+//= require_tree .

data/spec/integration/tmp/dummy/app/assets/javascripts/cable.js

@@ -0,0 +1,13 @@
+// Action Cable provides the framework to deal with WebSockets in Rails.
+// You can generate new channels where WebSocket features live using the rails generate channel command.
+//
+//= require action_cable
+//= require_self
+//= require_tree ./channels
+
+(function() {
+  this.App || (this.App = {});
+
+  App.cable = ActionCable.createConsumer();
+
+}).call(this);

data/spec/integration/tmp/dummy/app/assets/stylesheets/application.css

@@ -0,0 +1,15 @@
+/*
+ * This is a manifest file that'll be compiled into application.css, which will include all the files
+ * listed below.
+ *
+ * Any CSS and SCSS file within this directory, lib/assets/stylesheets, vendor/assets/stylesheets,
+ * or any plugin's vendor/assets/stylesheets directory can be referenced here using a relative path.
+ *
+ * You're free to add application-wide styles to this file and they'll appear at the bottom of the
+ * compiled file so the styles you add here take precedence over styles defined in any other CSS/SCSS
+ * files in this directory. Styles in this file should be added after the last require_* statement.
+ * It is generally better to create a new file per style scope.
+ *
+ *= require_tree .
+ *= require_self
+ */

data/spec/integration/tmp/dummy/app/channels/application_cable/channel.rb

@@ -0,0 +1,5 @@
+# typed: strong
+module ApplicationCable
+  class Channel < ActionCable::Channel::Base
+  end
+end

data/spec/integration/tmp/dummy/app/channels/application_cable/connection.rb

@@ -0,0 +1,5 @@
+# typed: strong
+module ApplicationCable
+  class Connection < ActionCable::Connection::Base
+  end
+end

data/spec/integration/tmp/dummy/app/controllers/application_controller.rb

@@ -0,0 +1,4 @@
+# typed: false
+class ApplicationController < ActionController::Base
+  protect_from_forgery with: :exception
+end

data/spec/integration/tmp/dummy/app/controllers/graphql_controller.rb

@@ -0,0 +1,44 @@
+# typed: false
+class GraphqlController < ApplicationController
+  def execute
+    variables = ensure_hash(params[:variables])
+    query = params[:query]
+    operation_name = params[:operationName]
+    context = {
+      # Query context goes here, for example:
+      # current_user: current_user,
+    }
+    result = DummySchema.execute(query, variables: variables, context: context, operation_name: operation_name)
+    render json: result
+  rescue => e
+    raise e unless Rails.env.development?
+    handle_error_in_development e
+  end
+
+  private
+
+  # Handle form data, JSON body, or a blank value
+  def ensure_hash(ambiguous_param)
+    case ambiguous_param
+    when String
+      if ambiguous_param.present?
+        ensure_hash(JSON.parse(ambiguous_param))
+      else
+        {}
+      end
+    when Hash, ActionController::Parameters
+      ambiguous_param
+    when nil
+      {}
+    else
+      raise ArgumentError, "Unexpected parameter: #{ambiguous_param}"
+    end
+  end
+
+  def handle_error_in_development(e)
+    logger.error e.message
+    logger.error e.backtrace.join("\n")
+
+    render json: { error: { message: e.message, backtrace: e.backtrace }, data: {} }, status: 500
+  end
+end

data/spec/integration/tmp/dummy/app/helpers/application_helper.rb

@@ -0,0 +1,3 @@
+# typed: strong
+module ApplicationHelper
+end

data/spec/integration/tmp/dummy/app/jobs/application_job.rb

@@ -0,0 +1,3 @@
+# typed: strong
+class ApplicationJob < ActiveJob::Base
+end

data/spec/integration/tmp/dummy/app/mailers/application_mailer.rb

@@ -0,0 +1,5 @@
+# typed: false
+class ApplicationMailer < ActionMailer::Base
+  default from: 'from@example.com'
+  layout 'mailer'
+end

data/spec/integration/tmp/dummy/app/mydirectory/dummy_schema.rb

@@ -0,0 +1,5 @@
+# typed: strict
+class DummySchema < GraphQL::Schema
+  mutation(Types::MutationType)
+  query(Types::QueryType)
+end

data/spec/integration/tmp/dummy/app/mydirectory/mutations/update_name.rb

@@ -0,0 +1,15 @@
+# typed: strong
+module Mutations
+  class UpdateName < GraphQL::Schema::RelayClassicMutation
+    # TODO: define return fields
+    # field :post, Types::PostType, null: false
+
+    # TODO: define arguments
+    # argument :name, String, required: true
+
+    # TODO: define resolve method
+    # def resolve(name:)
+    #   { post: ... }
+    # end
+  end
+end

data/spec/integration/tmp/dummy/app/mydirectory/types/base_enum.rb

@@ -0,0 +1,5 @@
+# typed: strong
+module Types
+  class BaseEnum < GraphQL::Schema::Enum
+  end
+end

data/spec/integration/tmp/dummy/app/mydirectory/types/base_input_object.rb

@@ -0,0 +1,5 @@
+# typed: strong
+module Types
+  class BaseInputObject < GraphQL::Schema::InputObject
+  end
+end

data/spec/integration/tmp/dummy/app/mydirectory/types/base_interface.rb

@@ -0,0 +1,6 @@
+# typed: strong
+module Types
+  module BaseInterface
+    include GraphQL::Schema::Interface
+  end
+end

data/spec/integration/tmp/dummy/app/mydirectory/types/base_object.rb

@@ -0,0 +1,5 @@
+# typed: strong
+module Types
+  class BaseObject < GraphQL::Schema::Object
+  end
+end