Add missing logout specs

Author: mcelicalderon

app/graphql/graphql_devise/mutations/logout.rb

@@ -7,13 +7,13 @@ class Logout < Base
       def resolve
         client = token.client if token.client
 
-        if current_user && token.client && current_user.tokens[client]
-          user.tokens.delete(client)
-          user.save!
+        if current_resource && token.client && current_resource.tokens[client]
+          current_resource.tokens.delete(client)
+          current_resource.save!
 
-          yield user if block_given?
+          yield current_resource 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