Move mutation assign

Author: David Revelo

lib/graphql_devise/engine.rb

@@ -3,5 +3,13 @@
 module GraphqlDevise
   class Engine < ::Rails::Engine
     isolate_namespace GraphqlDevise
+
+    config.before_initialize do
+      GraphqlDevise::Schema.mutation(GraphqlDevise::Types::MutationType)
+
+      if GraphqlDevise::Types::QueryType.fields.blank?
+        GraphqlDevise::Types::QueryType.field(:dummy, resolver: GraphqlDevise::Resolvers::Dummy)
+      end
+    end
   end
 end

lib/graphql_devise/rails/routes.rb

@@ -51,8 +51,12 @@ def mount_graphql_devise_for(resource, opts = {})
         authenticatable_type: authenticatable_type
       )
 
-      add_gql_devise_mutations!(prepared_mutations, additional_mutations)
-      add_gql_devise_queries!(prepared_queries, additional_queries)
+      prepared_mutations.merge(additional_mutations).each do |action, mutation|
+        GraphqlDevise::Types::MutationType.field(action, mutation: mutation)
+      end
+      prepared_queries.merge(additional_queries).each do |action, resolver|
+        GraphqlDevise::Types::QueryType.field(action, resolver: resolver)
+      end
 
       Devise.mailer.helper(GraphqlDevise::MailerHelper)
 
@@ -71,27 +75,5 @@ def validate_gql_devise_operations!(param_operations)
         **param_operations
       )
     end
-
-    def add_gql_devise_mutations!(prepared, additional)
-      all_mutations = prepared.merge(additional)
-
-      all_mutations.each do |action, mutation|
-        GraphqlDevise::Types::MutationType.field(action, mutation: mutation)
-      end
-
-      if all_mutations.present? && GraphqlDevise::Schema.try(:mutation).nil?
-        GraphqlDevise::Schema.mutation(GraphqlDevise::Types::MutationType)
-      end
-    end
-
-    def add_gql_devise_queries!(prepared, additional)
-      prepared.merge(additional).each do |action, resolver|
-        GraphqlDevise::Types::QueryType.field(action, resolver: resolver)
-      end
-
-      if (prepared.blank? || additional.present?) && GraphqlDevise::Types::QueryType.fields.blank?
-        GraphqlDevise::Types::QueryType.field(:dummy, resolver: GraphqlDevise::Resolvers::Dummy)
-      end
-    end
   end
 end

spec/requests/mutations/login_spec.rb

@@ -138,7 +138,7 @@
       GRAPHQL
     end
 
-    before { post_request }
+    before { post_request('/api/v1/guest/graphql_auth') }
 
     it 'works alongside the user mount point' do
       expect(json_response[:data][:guestLogin]).to include(

spec/requests/mutations/sign_up_spec.rb

@@ -123,10 +123,7 @@
     end
 
     it 'works without the confirmable module' do
-      expect {
-        post_request('/api/v1/guest/graphql_auth')
-        pp json_response
-      }.to change(Guest, :count).by(1)
+      expect { post_request('/api/v1/guest/graphql_auth') }.to change(Guest, :count).by(1)
     end
   end
 end