graphql_devise 0.7.0 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e14199edb2f13b7bc5998a9b56895559f24a823e0764b81fa6e3d9f30baabe63
-  data.tar.gz: bb649528f0e55edac087bb29badd8a849a17c4904fdd3804509cdbdcdc803ea8
+  metadata.gz: 0b6f3ff9d48c3f0aead4020e78b67b67d4a4f006d0567e46c9d25e2d4ec75027
+  data.tar.gz: c8b9e08819e1b67ee96728190a3a99063ad6370fb2764c14f5d7ab170ee8dbbc
 SHA512:
-  metadata.gz: 9446b37c593b01f806e426d6b23d596a3c043daafecc5410876db20369831484c734c9916bd2df124409e6081d7d2a3c28adbe4feed456541558d2ac8260f3b1
-  data.tar.gz: 0dc4c8c206fd10d5de9c3247f72cac7d40f9374e085e641c3d8e57259fa1d7fb587dd4d1e5f8c36045d7fb10954cbc538f9ef80b0b45becaa56adafd2e184903
+  metadata.gz: 1492eb845e6ad50959eee321dbd0feb7efa2ebd8ff37784080715305e2d072109dd0575e2ecbb7ac584f868156a7c4d646101edc4962bc88a86d0005acbfc085
+  data.tar.gz: 811254816bf9d2ca63d9064339c0d85bf7073ec7266953f2b8caa42e0be3924277a4a483c05bd7ff60326aeb284fb3632bd53fad2cf2912481c16d3d5d550368

data/CHANGELOG.md

@@ -1,5 +1,13 @@
 # Changelog
 
+## [v0.8.0](https://github.com/graphql-devise/graphql_devise/tree/v0.8.0) (2019-11-26)
+
+[Full Changelog](https://github.com/graphql-devise/graphql_devise/compare/v0.7.0...v0.8.0)
+
+**Breaking changes:**
+
+- Set standard to use authenticatable per Devise's coding standard. [\#46](https://github.com/graphql-devise/graphql_devise/pull/46) ([aarona](https://github.com/aarona))
+
 ## [v0.7.0](https://github.com/graphql-devise/graphql_devise/tree/v0.7.0) (2019-11-25)
 
 [Full Changelog](https://github.com/graphql-devise/graphql_devise/compare/v0.6.0...v0.7.0)

data/README.md

@@ -61,7 +61,7 @@ Rails.application.routes.draw do
   mount_graphql_devise_for(
     'User',
     at: 'api/v1',
-    authenticable_type: Types::MyCustomUserType,
+    authenticatable_type: Types::MyCustomUserType,
     operations: {
       login: Mutations::Login
     },
@@ -81,8 +81,8 @@ and [queries](https://github.com/graphql-devise/graphql_devise/blob/b5985036e01e
 All mutations and queries are built so you can extend default behavior just by extending
 our default classes and yielding your customized code after calling `super`, example
 [here](https://github.com/graphql-devise/graphql_devise/blob/b5985036e01ea064e43e457b4f0c8516f172471c/spec/dummy/app/graphql/mutations/login.rb#L6).
-1. `authenticable_type`: By default, the gem will add an `authenticable` field to every mutation
-and an `authenticable` type to every query. Gem will try to use `Types::<model>Type` by
+1. `authenticatable_type`: By default, the gem will add an `authenticatable` field to every mutation
+and an `authenticatable` type to every query. Gem will try to use `Types::<model>Type` by
 default, so in our example you could define `Types::UserType` and every query and mutation
 will use it. But, you can override this type with this option like in the example.
 1. `skip`: An array of the operations that should not be available in the authentication schema. All these operations are
@@ -170,7 +170,7 @@ Here is a list of the available mutations and queries assuming your mounted mode
 1. `userSendResetPassword(email: String!, redirectUrl: String!): UserSendReserPasswordPayload`
 1. `userResendConfirmation(email: String!, redirectUrl: String!): UserResendConfirmationPayload`
 
-    The `UserResendConfirmationPayload` will return the `authenticable` resource that was sent the confirmation instructions but also has a `message: String!` that can be used to notify a user what to do after the instructions were sent to them
+    The `UserResendConfirmationPayload` will return the `authenticatable` resource that was sent the confirmation instructions but also has a `message: String!` that can be used to notify a user what to do after the instructions were sent to them
 
 #### Queries
 1. `userConfirmAccount(confirmationToken: String!, redirectUrl: String!): User`

data/app/graphql/graphql_devise/mutations/login.rb

@@ -17,7 +17,7 @@ module GraphqlDevise
 
           yield resource if block_given?
 
-          { authenticable: resource}
+          { authenticatable: resource}
         elsif resource && !active_for_authentication?(resource)
           if locked?(resource)
             raise_user_error(I18n.t('graphql_devise.mailer.unlock_instructions.account_lock_msg'))

data/app/graphql/graphql_devise/mutations/logout.rb

@@ -10,7 +10,7 @@ module GraphqlDevise
 
           yield current_resource if block_given?
 
-          { authenticable: current_resource }
+          { authenticatable: current_resource }
         else
           raise_user_error(I18n.t('graphql_devise.user_not_found'))
         end

data/app/graphql/graphql_devise/mutations/resend_confirmation.rb

@@ -20,7 +20,7 @@ module GraphqlDevise
           )
 
           {
-            authenticable: resource,
+            authenticatable: resource,
             message:       I18n.t('graphql_devise.confirmations.send_instructions', email: email)
           }
         else

data/app/graphql/graphql_devise/mutations/send_password_reset.rb

@@ -17,7 +17,7 @@ module GraphqlDevise
           )
 
           if resource.errors.empty?
-            { authenticable: resource }
+            { authenticatable: resource }
           else
             raise_user_error_list(I18n.t('graphql_devise.invalid_resource'), errors: resource.errors.full_messages)
           end

data/app/graphql/graphql_devise/mutations/sign_up.rb

@@ -33,7 +33,7 @@ module GraphqlDevise
 
           set_auth_headers(resource) if resource.active_for_authentication?
 
-          { authenticable: resource }
+          { authenticatable: resource }
         else
           clean_up_passwords(resource)
           raise_user_error_list(

data/app/graphql/graphql_devise/mutations/update_password.rb

@@ -20,7 +20,7 @@ module GraphqlDevise
 
           yield current_resource if block_given?
 
-          { authenticable: current_resource }
+          { authenticatable: current_resource }
         else
           raise_user_error_list(
             I18n.t('graphql_devise.passwords.update_password_error'),

data/app/graphql/graphql_devise/resolvers/confirm_account.rb

@@ -25,7 +25,7 @@ module GraphqlDevise
           end
 
           controller.redirect_to(redirect_to_link)
-          { authenticable: resource }
+          { authenticatable: resource }
         else
           raise_user_error(I18n.t('graphql_devise.confirmations.invalid_token'))
         end

data/app/graphql/graphql_devise/types/{authenticable_type.rb → authenticatable_type.rb}

@@ -1,6 +1,6 @@
 module GraphqlDevise
   module Types
-    class AuthenticableType < GraphQL::Schema::Object
+    class AuthenticatableType < GraphQL::Schema::Object
       field :email, String, null: false
     end
   end

data/lib/graphql_devise/rails/routes.rb

@@ -39,9 +39,9 @@ module ActionDispatch::Routing
         skip:   [:sessions, :registrations, :passwords, :confirmations, :omniauth_callbacks, :unlocks]
       )
 
-      authenticable_type = opts[:authenticable_type] ||
-                           "Types::#{resource}Type".safe_constantize ||
-                           GraphqlDevise::Types::AuthenticableType
+      authenticatable_type = opts[:authenticatable_type] ||
+                             "Types::#{resource}Type".safe_constantize ||
+                             GraphqlDevise::Types::AuthenticatableType
 
       used_mutations = if only_operations.present?
         default_mutations.slice(*only_operations)
@@ -54,7 +54,7 @@ module ActionDispatch::Routing
         else
           new_mutation = Class.new(mutation)
           new_mutation.graphql_name("#{resource}#{action.to_s.camelize(:upper)}")
-          new_mutation.field(:authenticable, authenticable_type, null: true)
+          new_mutation.field(:authenticatable, authenticatable_type, null: true)
 
           new_mutation
         end
@@ -74,7 +74,7 @@ module ActionDispatch::Routing
         else
           new_query = Class.new(query)
           new_query.graphql_name("#{resource}#{action.to_s.camelize(:upper)}")
-          new_query.type(authenticable_type, null: true)
+          new_query.type(authenticatable_type, null: true)
 
           new_query
         end

data/lib/graphql_devise/version.rb

@@ -1,3 +1,3 @@
 module GraphqlDevise
-  VERSION = '0.7.0'.freeze
+  VERSION = '0.8.0'.freeze
 end

data/spec/dummy/app/graphql/mutations/login.rb

@@ -8,7 +8,7 @@ module Mutations
         user.reload
       end
 
-      original_payload.merge(user: original_payload[:authenticable])
+      original_payload.merge(user: original_payload[:authenticatable])
     end
   end
 end

data/spec/dummy/app/graphql/mutations/sign_up.rb

@@ -6,7 +6,7 @@ module Mutations
 
     def resolve(email:, **attrs)
       original_payload = super
-      original_payload.merge(user: original_payload[:authenticable])
+      original_payload.merge(user: original_payload[:authenticatable])
     end
   end
 end

data/spec/dummy/config/routes.rb

@@ -6,9 +6,9 @@ Rails.application.routes.draw do
 
   mount_graphql_devise_for(
     'Admin',
-    authenticable_type: Types::CustomAdminType,
-    skip:               [:sign_up, :check_password_token],
-    at:                 '/api/v1/admin/graphql_auth'
+    authenticatable_type: Types::CustomAdminType,
+    skip:                 [:sign_up, :check_password_token],
+    at:                   '/api/v1/admin/graphql_auth'
   )
 
   mount_graphql_devise_for(

data/spec/requests/mutations/login_spec.rb

@@ -100,7 +100,7 @@ RSpec.describe 'Login Requests' do
             email: "#{admin.email}",
             password: "#{password}"
           ) {
-            authenticable { email customField }
+            authenticatable { email customField }
           }
         }
       GRAPHQL
@@ -110,7 +110,7 @@ RSpec.describe 'Login Requests' do
 
     it 'works alongside the user mount point' do
       expect(json_response[:data][:adminLogin]).to include(
-        authenticable: { email: admin.email, customField: "email: #{admin.email}" }
+        authenticatable: { email: admin.email, customField: "email: #{admin.email}" }
       )
     end
   end
@@ -124,7 +124,7 @@ RSpec.describe 'Login Requests' do
             email: "#{guest.email}",
             password: "#{password}"
           ) {
-            authenticable { email }
+            authenticatable { email }
           }
         }
       GRAPHQL
@@ -134,7 +134,7 @@ RSpec.describe 'Login Requests' do
 
     it 'works alongside the user mount point' do
       expect(json_response[:data][:guestLogin]).to include(
-        authenticable: { email: guest.email }
+        authenticatable: { email: guest.email }
       )
     end
   end

data/spec/requests/mutations/logout_spec.rb

@@ -8,7 +8,7 @@ RSpec.describe 'Logout Requests' do
     <<-GRAPHQL
       mutation {
         userLogout {
-          authenticable { email }
+          authenticatable { email }
         }
       }
     GRAPHQL
@@ -23,7 +23,7 @@ RSpec.describe 'Logout Requests' do
       expect(response).not_to include_auth_headers
       expect(user.reload.tokens.keys).to be_empty
       expect(json_response[:data][:userLogout]).to match(
-        authenticable: { email: user.email }
+        authenticatable: { email: user.email }
       )
       expect(json_response[:errors]).to be_nil
     end
@@ -45,7 +45,7 @@ RSpec.describe 'Logout Requests' do
       <<-GRAPHQL
         mutation {
           adminLogout {
-            authenticable { email }
+            authenticatable { email }
           }
         }
       GRAPHQL
@@ -57,7 +57,7 @@ RSpec.describe 'Logout Requests' do
       expect(response).not_to include_auth_headers
       expect(admin.reload.tokens.keys).to be_empty
       expect(json_response[:data][:adminLogout]).to match(
-        authenticable: { email: admin.email }
+        authenticatable: { email: admin.email }
       )
       expect(json_response[:errors]).to be_nil
     end

data/spec/requests/mutations/resend_confirmation_spec.rb

@@ -15,7 +15,7 @@ RSpec.describe 'Resend confirmation' do
           redirectUrl:"#{redirect}"
         ) {
           message
-          authenticable {
+          authenticatable {
             id
             email
           }
@@ -28,7 +28,7 @@ RSpec.describe 'Resend confirmation' do
     it 'sends an email to the user with confirmation url and returns a success message' do
       expect { post_request }.to change(ActionMailer::Base.deliveries, :count).by(1)
       expect(json_response[:data][:userResendConfirmation]).to include(
-        authenticable: {
+        authenticatable: {
           id: id,
           email: email
         },

data/spec/requests/mutations/send_password_reset_spec.rb

@@ -13,7 +13,7 @@ RSpec.describe 'Send Password Reset Requests' do
           email:       "#{email}",
           redirectUrl: "#{redirect_url}"
         ) {
-          authenticable {
+          authenticatable {
             email
           }
         }

data/spec/requests/mutations/sign_up_spec.rb

@@ -87,7 +87,7 @@ RSpec.describe 'Sign Up process' do
             passwordConfirmation: "#{password}"
             confirmSuccessUrl:    "#{redirect}"
           ) {
-            authenticable {
+            authenticatable {
               email
             }
           }
@@ -114,7 +114,7 @@ RSpec.describe 'Sign Up process' do
             passwordConfirmation: "#{password}"
             confirmSuccessUrl:    "#{redirect}"
           ) {
-            authenticable {
+            authenticatable {
               email
             }
           }

data/spec/requests/mutations/update_password_spec.rb

@@ -10,7 +10,7 @@ RSpec.describe 'Update Password Requests' do
 
       expect(response).to include_auth_headers
       expect(json_response[:data][:userUpdatePassword]).to match(
-        authenticable: { email: user.email }
+        authenticatable: { email: user.email }
       )
       expect(json_response[:errors]).to be_nil
       expect(user).to be_valid_password(password)
@@ -33,7 +33,7 @@ RSpec.describe 'Update Password Requests' do
           passwordConfirmation: "#{password_confirmation}",
           currentPassword: "#{current_password}"
         ) {
-          authenticable { email }
+          authenticatable { email }
         }
       }
     GRAPHQL

data/spec/requests/queries/check_password_token_spec.rb

@@ -26,7 +26,7 @@ RSpec.describe 'Check Password Token Requests' do
       context 'when redirect_url is not provided' do
         let(:redirect_url) { nil }
 
-        it 'returns authenticable and credentials in the headers' do
+        it 'returns authenticatable and credentials in the headers' do
           get_request
 
           expect(response).to include_auth_headers

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: graphql_devise
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.8.0
 platform: ruby
 authors:
 - Mario Celi
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2019-11-25 00:00:00.000000000 Z
+date: 2019-11-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: devise_token_auth
@@ -270,7 +270,7 @@ files:
 - app/graphql/graphql_devise/resolvers/check_password_token.rb
 - app/graphql/graphql_devise/resolvers/confirm_account.rb
 - app/graphql/graphql_devise/schema.rb
-- app/graphql/graphql_devise/types/authenticable_type.rb
+- app/graphql/graphql_devise/types/authenticatable_type.rb
 - app/graphql/graphql_devise/types/mutation_type.rb
 - app/graphql/graphql_devise/types/query_type.rb
 - app/helpers/graphql_devise/application_helper.rb