graphql_devise 0.12.3 → 0.13.0

data/spec/rails_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 ENV['RAILS_ENV'] ||= 'test'
@@ -42,4 +44,5 @@ RSpec.configure do |config|
   config.before(:suite) do
     ActionController::Base.allow_forgery_protection = true
   end
+  config.before { ActionMailer::Base.deliveries.clear }
 end

data/spec/requests/graphql_controller_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe GraphqlDevise::GraphqlController do

data/spec/requests/mutations/additional_mutations_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Additional Mutations' do

data/spec/requests/mutations/additional_queries_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Additional Queries' do

data/spec/requests/mutations/login_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Login Requests' do

data/spec/requests/mutations/logout_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Logout Requests' do

data/spec/requests/mutations/resend_confirmation_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Resend confirmation' do

data/spec/requests/mutations/send_password_reset_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Send Password Reset Requests' do

data/spec/requests/mutations/sign_up_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Sign Up process' do

data/spec/requests/mutations/update_password_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Update Password Requests' do

data/spec/requests/queries/check_password_token_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Check Password Token Requests' do

data/spec/requests/queries/confirm_account_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe 'Account confirmation' do
@@ -26,7 +28,7 @@ RSpec.describe 'Account confirmation' do
       user.send_confirmation_instructions(
         template_path: ['graphql_devise/mailer'],
         controller:    'graphql_devise/graphql',
-        action:        'auth'
+        schema_url:    'http://not-using-this-value.com/gql'
       )
     end
 

data/spec/requests/user_controller_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe "Integrations with the user's controller" do
@@ -34,7 +36,7 @@ RSpec.describe "Integrations with the user's controller" do
       it 'raises an invalid resource_name error' do
         expect { post_request('/api/v1/failing') }.to raise_error(
           GraphqlDevise::Error,
-          'Invalid resource_name `fail` provided to `graphql_context`. Possible values are: [:user, :admin, :guest, :users_customer].'
+          'Invalid resource_name `fail` provided to `graphql_context`. Possible values are: [:user, :admin, :guest, :users_customer, :schema_user].'
         )
       end
     end
@@ -55,9 +57,17 @@ RSpec.describe "Integrations with the user's controller" do
       context 'when user is authenticated' do
         let(:headers) { user.create_new_auth_token }
 
-        it 'allow to perform the query' do
+        it 'allows to perform the query' do
           expect(json_response[:data][:privateField]).to eq('Field will always require authentication')
         end
+
+        context 'when using a SchemaUser' do
+          let(:headers) { create(:schema_user, :confirmed).create_new_auth_token }
+
+          it 'allows to perform the query' do
+            expect(json_response[:data][:privateField]).to eq('Field will always require authentication')
+          end
+        end
       end
 
       context 'when user is not authenticated' do
@@ -75,7 +85,7 @@ RSpec.describe "Integrations with the user's controller" do
       context 'when user is authenticated' do
         let(:headers) { user.create_new_auth_token }
 
-        it 'allow to perform the query' do
+        it 'allows to perform the query' do
           expect(json_response[:data][:privateField]).to eq('Field will always require authentication')
         end
       end
@@ -105,7 +115,7 @@ RSpec.describe "Integrations with the user's controller" do
       context 'when user is authenticated' do
         let(:headers) { user.create_new_auth_token }
 
-        it 'allow to perform the query' do
+        it 'allows to perform the query' do
           expect(json_response[:data][:dummyMutation]).to eq('Necessary so GraphQL gem does not complain about empty mutation type')
         end
       end
@@ -125,7 +135,7 @@ RSpec.describe "Integrations with the user's controller" do
       context 'when user is authenticated' do
         let(:headers) { user.create_new_auth_token }
 
-        it 'allow to perform the query' do
+        it 'allows to perform the query' do
           expect(json_response[:data][:dummyMutation]).to eq('Necessary so GraphQL gem does not complain about empty mutation type')
         end
       end
@@ -160,7 +170,7 @@ RSpec.describe "Integrations with the user's controller" do
       context 'when user is authenticated' do
         let(:headers) { user.create_new_auth_token }
 
-        it 'allow to perform the query' do
+        it 'allows to perform the query' do
           expect(json_response[:data][:user]).to match(
             email: user.email,
             id:    user.id
@@ -183,7 +193,7 @@ RSpec.describe "Integrations with the user's controller" do
       context 'when user is authenticated' do
         let(:headers) { user.create_new_auth_token }
 
-        it 'allow to perform the query' do
+        it 'allows to perform the query' do
           expect(json_response[:data][:user]).to match(
             email: user.email,
             id:    user.id
@@ -193,7 +203,7 @@ RSpec.describe "Integrations with the user's controller" do
 
       context 'when user is not authenticated' do
         # Interpreter schema fields are public unless specified otherwise (plugin setting)
-        it 'allow to perform the query' do
+        it 'allows to perform the query' do
           expect(json_response[:data][:user]).to match(
             email: user.email,
             id:    user.id
@@ -202,4 +212,43 @@ RSpec.describe "Integrations with the user's controller" do
       end
     end
   end
+
+  describe 'updateUser' do
+    let(:headers) { user.create_new_auth_token }
+    let(:query) do
+      <<-GRAPHQL
+        mutation {
+          updateUser(email: "updated@gmail.com", name: "updated name") {
+            user { email name }
+          }
+        }
+      GRAPHQL
+    end
+
+    it 'requires new email confirmation' do
+      original_email = user.email
+
+      expect do
+        post_request('/api/v1/graphql?test=value')
+        user.reload
+      end.to not_change(user, :email).from(original_email).and(
+        change(user, :unconfirmed_email).from(nil).to('updated@gmail.com')
+      ).and(
+        not_change(user, :uid).from(original_email)
+      ).and(
+        change(user, :name).from(user.name).to('updated name')
+      )
+
+      email = Nokogiri::HTML(ActionMailer::Base.deliveries.last.body.encoded)
+      link  = email.css('a').first
+      expect(link['href']).to include('/api/v1/graphql')
+
+      expect do
+        get link['href']
+        user.reload
+      end.to change(user, :email).from(original_email).to('updated@gmail.com').and(
+        change(user, :uid).from(original_email).to('updated@gmail.com')
+      )
+    end
+  end
 end

data/spec/services/mount_method/operation_preparer_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OperationPreparer do

data/spec/services/mount_method/operation_preparers/custom_operation_preparer_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OperationPreparers::CustomOperationPreparer do

data/spec/services/mount_method/operation_preparers/default_operation_preparer_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OperationPreparers::DefaultOperationPreparer do

data/spec/services/mount_method/operation_preparers/gql_name_setter_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OperationPreparers::GqlNameSetter do

data/spec/services/mount_method/operation_preparers/mutation_field_setter_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OperationPreparers::MutationFieldSetter do

data/spec/services/mount_method/operation_preparers/resolver_type_setter_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OperationPreparers::ResolverTypeSetter do

data/spec/services/mount_method/operation_preparers/resource_name_setter_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OperationPreparers::ResourceNameSetter do

data/spec/services/mount_method/operation_sanitizer_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rails_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OperationSanitizer do

data/spec/services/mount_method/option_sanitizer_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OptionSanitizer do

data/spec/services/mount_method/option_sanitizers/array_checker_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OptionSanitizers::ArrayChecker do

data/spec/services/mount_method/option_sanitizers/class_checker_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OptionSanitizers::ClassChecker do

data/spec/services/mount_method/option_sanitizers/hash_checker_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OptionSanitizers::HashChecker do

data/spec/services/mount_method/option_sanitizers/string_checker_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OptionSanitizers::StringChecker do

data/spec/services/mount_method/option_validators/provided_operations_validator_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OptionValidators::ProvidedOperationsValidator do

data/spec/services/mount_method/option_validators/skip_only_validator_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OptionValidators::SkipOnlyValidator do

data/spec/services/mount_method/option_validators/supported_operations_validator_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OptionValidators::SupportedOperationsValidator do

data/spec/services/mount_method/options_validator_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::MountMethod::OptionsValidator do

data/spec/services/resource_loader_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::ResourceLoader do

data/spec/services/schema_plugin_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 RSpec.describe GraphqlDevise::SchemaPlugin do

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 if ENV['CI'] && !ENV['SKIP_COVERALLS']
   require 'simplecov'
   require 'coveralls'

data/spec/support/contexts/graphql_request.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec.shared_context 'with graphql query request' do
   let(:headers)   { {} }
   let(:variables) { {} }

data/spec/support/factory_bot.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec.configure do |config|
   config.include FactoryBot::Syntax::Methods
 

data/spec/support/matchers/auth_headers_matcher.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define :include_auth_headers do
   match do |response|
     auth_headers = %w[uid access-token client].map { |key| response.headers[key] }

data/spec/support/matchers/not_change_matcher.rb

@@ -1 +1,3 @@
+# frozen_string_literal: true
+
 RSpec::Matchers.define_negated_matcher :not_change, :change

data/spec/support/requests/auth_helpers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Requests
   module AuthHelpers
     def auth_headers_for(user)

data/spec/support/requests/json_helpers.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module Requests
   module JsonHelpers
     def json_response

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: graphql_devise
 version: !ruby/object:Gem::Version
-  version: 0.12.3
+  version: 0.13.0
 platform: ruby
 authors:
 - Mario Celi
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2020-06-20 00:00:00.000000000 Z
+date: 2020-06-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: devise_token_auth
@@ -314,6 +314,7 @@ files:
 - lib/graphql_devise/errors/error_codes.rb
 - lib/graphql_devise/errors/execution_error.rb
 - lib/graphql_devise/errors/user_error.rb
+- lib/graphql_devise/model/with_email_updater.rb
 - lib/graphql_devise/mount_method/operation_preparer.rb
 - lib/graphql_devise/mount_method/operation_preparers/custom_operation_preparer.rb
 - lib/graphql_devise/mount_method/operation_preparers/default_operation_preparer.rb
@@ -362,6 +363,7 @@ files:
 - spec/dummy/app/graphql/mutations/login.rb
 - spec/dummy/app/graphql/mutations/register_confirmed_user.rb
 - spec/dummy/app/graphql/mutations/sign_up.rb
+- spec/dummy/app/graphql/mutations/update_user.rb
 - spec/dummy/app/graphql/resolvers/public_user.rb
 - spec/dummy/app/graphql/resolvers/user_show.rb
 - spec/dummy/app/graphql/types/base_object.rb
@@ -374,6 +376,7 @@ files:
 - spec/dummy/app/models/admin.rb
 - spec/dummy/app/models/application_record.rb
 - spec/dummy/app/models/guest.rb
+- spec/dummy/app/models/schema_user.rb
 - spec/dummy/app/models/user.rb
 - spec/dummy/app/models/users.rb
 - spec/dummy/app/models/users/customer.rb
@@ -412,14 +415,18 @@ files:
 - spec/dummy/db/migrate/20190916012505_create_admins.rb
 - spec/dummy/db/migrate/20191013213045_create_guests.rb
 - spec/dummy/db/migrate/20200321121807_create_users_customers.rb
+- spec/dummy/db/migrate/20200621182414_remove_uncofirmed_email_from_admins.rb
+- spec/dummy/db/migrate/20200623003142_create_schema_users.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/seeds.rb
 - spec/dummy/public/robots.txt
 - spec/factories/admins.rb
 - spec/factories/guests.rb
+- spec/factories/schema_users.rb
 - spec/factories/users.rb
 - spec/factories/users_customers.rb
 - spec/generators/graphql_devise/install_generator_spec.rb
+- spec/graphql_devise/model/with_email_updater_spec.rb
 - spec/graphql_devise_spec.rb
 - spec/models/user_spec.rb
 - spec/rails_helper.rb
@@ -497,6 +504,7 @@ test_files:
 - spec/dummy/app/graphql/mutations/login.rb
 - spec/dummy/app/graphql/mutations/register_confirmed_user.rb
 - spec/dummy/app/graphql/mutations/sign_up.rb
+- spec/dummy/app/graphql/mutations/update_user.rb
 - spec/dummy/app/graphql/resolvers/public_user.rb
 - spec/dummy/app/graphql/resolvers/user_show.rb
 - spec/dummy/app/graphql/types/base_object.rb
@@ -509,6 +517,7 @@ test_files:
 - spec/dummy/app/models/admin.rb
 - spec/dummy/app/models/application_record.rb
 - spec/dummy/app/models/guest.rb
+- spec/dummy/app/models/schema_user.rb
 - spec/dummy/app/models/user.rb
 - spec/dummy/app/models/users.rb
 - spec/dummy/app/models/users/customer.rb
@@ -547,14 +556,18 @@ test_files:
 - spec/dummy/db/migrate/20190916012505_create_admins.rb
 - spec/dummy/db/migrate/20191013213045_create_guests.rb
 - spec/dummy/db/migrate/20200321121807_create_users_customers.rb
+- spec/dummy/db/migrate/20200621182414_remove_uncofirmed_email_from_admins.rb
+- spec/dummy/db/migrate/20200623003142_create_schema_users.rb
 - spec/dummy/db/schema.rb
 - spec/dummy/db/seeds.rb
 - spec/dummy/public/robots.txt
 - spec/factories/admins.rb
 - spec/factories/guests.rb
+- spec/factories/schema_users.rb
 - spec/factories/users.rb
 - spec/factories/users_customers.rb
 - spec/generators/graphql_devise/install_generator_spec.rb
+- spec/graphql_devise/model/with_email_updater_spec.rb
 - spec/graphql_devise_spec.rb
 - spec/models/user_spec.rb
 - spec/rails_helper.rb