clean up files after rubocop linting

Author: simonfranzen

Rakefile

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 # Add your own tasks in files placed in lib/tasks ending in .rake,
 # for example lib/tasks/capistrano.rake, and they will automatically be available to Rake.
 

app/channels/application_cable/channel.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ApplicationCable
   class Channel < ActionCable::Channel::Base
   end

app/channels/application_cable/connection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module ApplicationCable
   class Connection < ActionCable::Connection::Base
   end

app/controllers/application_controller.rb

@@ -1,3 +1,4 @@
+# frozen_string_literal: true
+
 class ApplicationController < ActionController::API
-  
 end

app/controllers/graphql_controller.rb

@@ -1,55 +1,65 @@
+# frozen_string_literal: true
+
+# GraphQL API entry point
 class GraphqlController < ApplicationController
+  # Executes the graphql request
+  def execute
+    query, variables, operation_name = read_query_params()
+    context = set_context()
+    result = GraphqlSchema.execute(
+      query,
+      variables: variables,
+      context: context,
+      operation_name: operation_name
+    )
+    render json: result
+  rescue StandardError => e
+    raise e unless Rails.env.development?
 
-    def execute
-      variables = ensure_hash(params[:variables])
-      query = params[:query]
-      operation_name = params[:operationName]
-      context = {
-       current_user: current_user,
-       login: method(:sign_in)
-      }
-      result = GraphqlSchema.execute(
-        query, 
-        variables: variables, 
-        context: context, 
-        operation_name: operation_name
-      )
-      render json: result
-    rescue => e
-      raise e unless Rails.env.development?
-      handle_error_in_development e
-    end
-  
-    private
-    # Handle form data, JSON body, or a blank value
-    def ensure_hash(ambiguous_param)
-      case ambiguous_param
-      when String
-        if ambiguous_param.present?
-          ensure_hash(JSON.parse(ambiguous_param))
-        else
-          {}
-        end
-      when Hash, ActionController::Parameters
-        ambiguous_param
-      when nil
-        {}
-      else
-        raise ArgumentError, "Unexpected parameter: #{ambiguous_param}"
-      end
-    end
-  
-    def handle_error_in_development(e)
-      logger.error e.message
-      logger.error e.backtrace.join("\n")
-  
-      render json: { 
-        error: { 
-          message: e.message, 
-          backtrace: e.backtrace 
-        }, 
-        data: {} 
-      }, status: 500
+    handle_error_in_development e
+  end
+
+  private
+
+  def read_query_params
+    [
+      params[:query],
+      ensure_hash(params[:variables]),
+      params[:operationName]
+    ]
+  end
+
+  def set_context
+    {
+      current_user: current_user
+      # login: method(:sign_in)
+    }
+  end
+
+  # Handle form data, JSON body, or a blank value
+  def ensure_hash(ambiguous_param)
+    case ambiguous_param
+    when String
+      ambiguous_param.present? ? ensure_hash(JSON.parse(ambiguous_param)) : {}
+    when Hash, ActionController::Parameters
+      ambiguous_param
+    when nil
+      {}
+    else
+      raise ArgumentError, "Unexpected parameter: #{ambiguous_param}"
     end
   end
-  
+
+  def handle_error_in_development(err)
+    logger.error err.message
+    logger.error err.backtrace.join("\n")
+
+    render json: {
+      error: {
+        message: err.message,
+        backtrace: err.backtrace
+      },
+      data: {}
+    }, status: 500
+  end
+end

app/graphql/graphql_schema.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
+# Entry point for graphql schema
 class GraphqlSchema < GraphQL::Schema
-    query(Types::QueryType)
-    mutation(Types::MutationType)
-  end
-  
+  query(Types::QueryType)
+  mutation(Types::MutationType)
+end

app/graphql/mutations/user/login.rb

@@ -1,19 +1,24 @@
-class Mutations::User::Login < GraphQL::Schema::Mutation
+# frozen_string_literal: true
 
-  null true
-  description "Login for users"
-  argument :email, String, required: true
-  argument :password, String, required: true
-  payload_type Types::UserType 
+module Mutations
+  module User
+    # Login for users
+    class Login < GraphQL::Schema::Mutation
+      null true
+      description 'Login for users'
+      argument :email, String, required: true
+      argument :password, String, required: true
+      payload_type Types::UserType
 
-  def resolve(email:, password:)
-    user = User.find_for_authentication(email: email)
-    return nil if !user
-    
-    is_valid_for_auth = user.valid_for_authentication?{
-      user.valid_password?(password)
-    }
-    return is_valid_for_auth ? user : nil
-  end
+      def resolve(email:, password:)
+        user = ::User.find_for_authentication(email: email)
+        return nil unless user
 
-end
+        is_valid_for_auth = user.valid_for_authentication? do
+          user.valid_password?(password)
+        end
+        is_valid_for_auth ? user : nil
+      end
+    end
+  end
+end

app/graphql/mutations/user/logout.rb

@@ -1,15 +1,21 @@
-class Mutations::User::Logout < GraphQL::Schema::Mutation
+# frozen_string_literal: true
 
-  null true
-  description "Logout for users"
-  payload_type Boolean
+module Mutations
+  module User
+    # Logout for users
+    class Logout < GraphQL::Schema::Mutation
+      null true
+      description 'Logout for users'
+      payload_type Boolean
 
-  def resolve
-    if context[:current_user]
-      context[:current_user].update(jti: SecureRandom.uuid)
-      return true
-    end 
-    false
+      def resolve
+        user = context[:current_user]
+        if user
+          user.update(jti: SecureRandom.uuid)
+          return true
+        end
+        false
+      end
+    end
   end
-
-end
+end

app/graphql/mutations/user/resend_unlock_instructions.rb

@@ -1,14 +1,20 @@
-class Mutations::User::ResendUnlockInstructions < GraphQL::Schema::Mutation
+# frozen_string_literal: true
 
-  null false
-  description "Unlock the user account"
-  argument :email, String, required: true
-  payload_type Boolean
+module Mutations
+  module User
+    # Unlock the user account
+    class ResendUnlockInstructions < GraphQL::Schema::Mutation
+      null false
+      description 'Unlock the user account'
+      argument :email, String, required: true
+      payload_type Boolean
 
-  def resend_unlock_instructions(email:)
-    user = User.find_by_email(email)
-    return false if !user
-    user.resend_unlock_instructions
-  end
+      def resend_unlock_instructions(email:)
+        user = ::User.find_by(email: email)
+        return false unless user
 
-end
+        user.resend_unlock_instructions
+      end
+    end
+  end
+end

app/graphql/mutations/user/reset_password.rb

@@ -1,16 +1,22 @@
-class Mutations::User::ResetPassword < GraphQL::Schema::Mutation
+# frozen_string_literal: true
 
-  null true
-  description "Set the new password"
-  argument :password, String, required: true
-  argument :passwordConfirmation, String, required: true
-  argument :resetPasswordToken, String, required: true
-  payload_type Boolean
+module Mutations
+  module User
+    # Set the new password
+    class ResetPassword < GraphQL::Schema::Mutation
+      null true
+      description 'Set the new password'
+      argument :password, String, required: true
+      argument :passwordConfirmation, String, required: true
+      argument :resetPasswordToken, String, required: true
+      payload_type Boolean
 
-  def resolve(password:, password_confirmation:, reset_password_token:)
-    user = User.with_reset_password_token(reset_password_token)
-    return false if !user
-    user.reset_password(password, password_confirmation)
-  end  
+      def resolve(password:, password_confirmation:, reset_password_token:)
+        user = ::User.with_reset_password_token(reset_password_token)
+        return false unless user
 
-end
+        user.reset_password(password, password_confirmation)
+      end
+    end
+  end
+end