Allow setting current resource in tests

Author: David Revelo

lib/graphql_devise/schema_plugin.rb

@@ -40,18 +40,21 @@ def trace(event, trace_data)
     private
 
     def set_current_resource(context)
-      controller                 = context[:controller]
-      resource_names             = Array(context[:resource_name])
-      context[:current_resource] = resource_names.find do |resource_name|
-        unless Devise.mappings.key?(resource_name)
-          raise(
-            GraphqlDevise::Error,
-            "Invalid resource_name `#{resource_name}` provided to `graphql_context`. Possible values are: #{Devise.mappings.keys}."
-          )
+      controller     = context[:controller]
+      resource_names = Array(context[:resource_name])
+
+      if context[:current_resource].blank?
+        context[:current_resource] = resource_names.find do |resource_name|
+          unless Devise.mappings.key?(resource_name)
+            raise(
+              GraphqlDevise::Error,
+              "Invalid resource_name `#{resource_name}` provided to `graphql_context`. Possible values are: #{Devise.mappings.keys}."
+            )
+          end
+
+          found = controller.set_resource_by_token(resource_name)
+          break found if found
         end
-
-        found = controller.set_resource_by_token(resource_name)
-        break found if found
       end
 
       context

spec/graphql/user_queries_spec.rb

@@ -0,0 +1,149 @@
+
+require 'rails_helper'
+
+RSpec.describe 'Sign Up process' do
+  include_context 'with graphql schema test'
+
+  let(:schema) { DummySchema }
+  let(:user)   { create(:user, :confirmed) }
+
+  describe 'publicField' do
+    let(:query) do
+      <<-GRAPHQL
+        query {
+          publicField
+        }
+      GRAPHQL
+    end
+
+    context 'when using a regular schema' do
+      it 'does not require authentication' do
+        expect(response[:data][:publicField]).to eq('Field does not require authentication')
+      end
+    end
+  end
+
+  describe 'privateField' do
+    let(:query) do
+      <<-GRAPHQL
+        query {
+          privateField
+        }
+      GRAPHQL
+    end
+
+    context 'when using a regular schema' do
+      context 'when user is authenticated' do
+        let(:resource) { user }
+
+        it 'allows to perform the query' do
+          expect(response[:data][:privateField]).to eq('Field will always require authentication')
+        end
+
+        context 'when using a SchemaUser' do
+          let(:resource) { create(:schema_user, :confirmed) }
+
+          it 'allows to perform the query' do
+            expect(response[:data][:privateField]).to eq('Field will always require authentication')
+          end
+        end
+      end
+
+      context 'when user is not authenticated' do
+        it 'returns a must sign in error' do
+          expect(response[:errors]).to contain_exactly(
+            hash_including(
+              message:    'privateField field requires authentication',
+              extensions: { code: 'AUTHENTICATION_ERROR' }
+            )
+          )
+        end
+      end
+    end
+
+    context 'when using an interpreter schema' do
+      let(:schema) { InterpreterSchema }
+
+      context 'when user is authenticated' do
+        let(:resource) { user }
+
+        it 'allows to perform the query' do
+          expect(response[:data][:privateField]).to eq('Field will always require authentication')
+        end
+      end
+
+      context 'when user is not authenticated' do
+        it 'returns a must sign in error' do
+          expect(response[:errors]).to contain_exactly(
+            hash_including(
+              message:    'privateField field requires authentication',
+              extensions: { code: 'AUTHENTICATION_ERROR' }
+            )
+          )
+        end
+      end
+    end
+  end
+
+  describe 'user' do
+    let(:query) do
+      <<-GRAPHQL
+        query {
+          user(id: #{user.id}) {
+            id
+            email
+          }
+        }
+      GRAPHQL
+    end
+
+    context 'when using a regular schema' do
+      context 'when user is authenticated' do
+        let(:resource) { user }
+
+        it 'allows to perform the query' do
+          expect(response[:data][:user]).to match(
+            email: user.email,
+            id:    user.id
+          )
+        end
+      end
+
+      context 'when user is not authenticated' do
+        it 'returns a must sign in error' do
+          expect(response[:errors]).to contain_exactly(
+            hash_including(
+              message: 'user field requires authentication',
+              extensions: { code: 'AUTHENTICATION_ERROR' }
+            )
+          )
+        end
+      end
+    end
+
+    context 'when using an interpreter schema' do
+      let(:schema) { InterpreterSchema }
+
+      context 'when user is authenticated' do
+        let(:resource) { user }
+
+        it 'allows to perform the query' do
+          expect(response[:data][:user]).to match(
+            email: user.email,
+            id:    user.id
+          )
+        end
+      end
+
+      context 'when user is not authenticated' do
+        # Interpreter schema fields are public unless specified otherwise (plugin setting)
+        it 'allows to perform the query' do
+          expect(response[:data][:user]).to match(
+            email: user.email,
+            id:    user.id
+          )
+        end
+      end
+    end
+  end
+end

spec/support/contexts/schema_test.rb

@@ -0,0 +1,15 @@
+RSpec.shared_context 'with graphql schema test' do
+  let(:variables)  { {} }
+  let(:resource)   { nil }
+  let(:controller) { instance_double(Api::V1::GraphqlController) }
+  let(:context)    { { current_resource: resource, controller: controller } }
+  let(:response) do
+    schema.execute(query, context: context, variables: variables).deep_symbolize_keys
+  end
+
+  # before do
+  #   allow_any_instance_of(GraphqlDevise::Mutations::Login).to receive(:set_auth_headers)
+  #   allow(controller).to receive(:request).and_return(request)
+  #   # allow(controller).to receive(:response).and_return(response)
+  # end
+end