Add missing logout specs

Author: mcelicalderon

app/controllers/graphql_devise/graphql_controller.rb

@@ -18,6 +18,8 @@ def auth
       render json: result
     end
 
+    attr_reader :client_id, :token
+
     private
 
     def execute_params(item)
@@ -26,7 +28,6 @@ def execute_params(item)
         variables:      ensure_hash(item[:variables]),
         context:        {
           current_resource: @resource,
-          token:            @token,
           controller:       self,
           resource_class:   resource_class
         }

app/graphql/graphql_devise/mutations/base.rb

@@ -1,3 +1,5 @@
+require 'devise_token_auth/version'
+
 module GraphqlDevise
   module Mutations
     class Base < GraphQL::Schema::Mutation
@@ -27,8 +29,12 @@ def current_resource
         context[:current_resource]
       end
 
-      def token
-        context[:token]
+      def client
+        if DeviseTokenAuth::VERSION <= '1'
+          controller.client_id
+        else
+          controller.token.client if controller.token.present?
+        end
       end
     end
   end

app/graphql/graphql_devise/mutations/logout.rb

@@ -5,15 +5,13 @@ class Logout < Base
       field :errors,  [String], null: false
 
       def resolve
-        client = token.client if token.client
+        if current_resource && client && current_resource.tokens[client]
+          current_resource.tokens.delete(client)
+          current_resource.save!
 
-        if current_user && token.client && current_user.tokens[client]
-          user.tokens.delete(client)
-          user.save!
+          yield current_resource if block_given?
 
-          yield user if block_given?
-
-          { success: true, errors: [] }
+          { success: true, errors: [], authenticable: current_resource }
         else
           { success: false, errors: [I18n.t('graphql_devise.user_not_found')] }
         end

lib/graphql_devise/rails/routes.rb

@@ -29,11 +29,8 @@ def mount_graphql_devise_for(resource, opts = {})
         else
           new_mutation = Class.new(mutation)
           new_mutation.graphql_name("#{resource}#{action.to_s.titleize}")
-          new_mutation.field(
-            :authenticable,
-            authenticable_type,
-            null: true
-          )
+          new_mutation.field(:authenticable, authenticable_type, null: true)
+
           new_mutation
         end
 

spec/requests/mutations/logout_spec.rb

@@ -0,0 +1,46 @@
+require 'rails_helper'
+
+RSpec.describe 'Logout Requests' do
+  include_context 'with graphql query request'
+
+  let(:user) { create(:user, :confirmed) }
+  let(:query) do
+    <<-GRAPHQL
+      mutation {
+        userLogout{
+          authenticable { email }
+          success
+          errors
+        }
+      }
+    GRAPHQL
+  end
+
+  before { post '/api/v1/graphql_auth', *graphql_params }
+
+  context 'when user is logged in' do
+    let(:headers) { user.create_new_auth_token }
+
+    it 'logs out the user' do
+      expect(response).not_to include_auth_headers
+      expect(user.reload.tokens.keys).to be_empty
+      expect(json_response[:data][:userLogout]).to match(
+        success:       true,
+        errors:        [],
+        authenticable: { email: user.email }
+      )
+    end
+  end
+
+  context 'when user is not logged in' do
+    it 'returns an error' do
+      expect(response).not_to include_auth_headers
+      expect(user.reload.tokens.keys).to be_empty
+      expect(json_response[:data][:userLogout]).to match(
+        success:       false,
+        errors:        ['User was not found or was not logged in.'],
+        authenticable: nil
+      )
+    end
+  end
+end

spec/support/contexts/graphql_request.rb

@@ -1,10 +1,11 @@
 RSpec.shared_context 'with graphql query request' do
+  let(:headers)   { {} }
   let(:variables) { {} }
   let(:graphql_params) do
     if Rails::VERSION::MAJOR >= 5
-      [{ params: { query: query, variables: variables } }]
+      [{ params: { query: query, variables: variables }, headers: headers }]
     else
-      [{ query: query, variables: variables }]
+      [{ query: query, variables: variables }, headers]
     end
   end
 end