Rubocop offense

Author: akhil-gautam

app/controllers/api/v1/users_controller.rb

@@ -3,7 +3,7 @@
 module Api
   module V1
     class UsersController < ApplicationController
-      skip_before_action :authenticate_user, only: [:create, :login, :verify]
+      skip_before_action :authenticate_user, only: %i[create login verify]
       before_action :set_user, only: [:update]
 
       # serves as signup
@@ -19,7 +19,7 @@ def create
       def update
         authorize @user
         if @user.update(user_params)
-          render json: { message: I18n.t('generic.update.success')}, status: :ok
+          render json: { message: I18n.t('generic.update.success') }, status: :ok
         else
           render json: { errors: @user.error_string }, status: :unprocessable_entity
         end

app/controllers/application_controller.rb

@@ -25,17 +25,15 @@ def authenticate_user
     end
   end
 
-  def current_user
-    @current_user
-  end
+  attr_reader :current_user
 
   private
 
   def record_not_found
-    return render json: { errors: I18n.t('generic.record_not_found') }, status: :not_found
+    render json: { errors: I18n.t('generic.record_not_found') }, status: :not_found
   end
 
   def unauthorized_access
-    return render json: { errors: I18n.t('authorization.error') }, status: :forbidden
+    render json: { errors: I18n.t('authorization.error') }, status: :forbidden
   end
 end

app/interactors/user/add_to_system.rb

@@ -1,7 +1,9 @@
 # frozen_string_literal: true
 
-class User::AddToSystem
-  include Interactor::Organizer
+module User
+  class AddToSystem
+    include Interactor::Organizer
 
-  organize User::Create, Mail::VerifyUser
+    organize User::Create, Mail::VerifyUser
+  end
 end

app/interactors/user/create.rb

@@ -1,18 +1,20 @@
 # frozen_string_literal: true
 
-class User::Create
-  include Interactor
+module User
+  class Create
+    include Interactor
 
-  def call
-    params = context.params.merge(
-      reset_password_token: SecureRandom.urlsafe_base64,
-      reset_password_sent_at: Time.now
-    )
-    user = User.new(params)
-    if user.save
-      context.user = user
-    else
-      context.fail!(error: { errors: user.error_string })
+    def call
+      params = context.params.merge(
+        reset_password_token: SecureRandom.urlsafe_base64,
+        reset_password_sent_at: Time.now
+      )
+      user = User.new(params)
+      if user.save
+        context.user = user
+      else
+        context.fail!(error: { errors: user.error_string })
+      end
     end
   end
 end

app/policies/user_policy.rb

@@ -1,5 +1,7 @@
+# frozen_string_literal: true
+
 class UserPolicy < ApplicationPolicy
   def update?
     user.id == record.id
   end
-end
+end

config/application.rb

@@ -1,7 +1,6 @@
 # frozen_string_literal: true
 
 require_relative 'boot'
-require_relative '../lib/health_check/middleware_healthcheck'
 
 require 'rails/all'
 
@@ -25,7 +24,6 @@ class Application < Rails::Application
     # Only loads a smaller set of middleware suitable for API only apps.
     # Middleware like session, flash, cookies can be added back manually.
     # Skip views, helpers and assets when generating a new resource.
-    # config.middleware.insert_after Rails::Rack::Logger, MiddlewareHealthcheck
     config.api_only = true
     config.active_job.queue_adapter = :good_job
   end

config/environments/test.rb

@@ -31,7 +31,7 @@
 
   config.active_job.queue_adapter = :good_job
   config.good_job.execution_mode = :inline
-  
+
   # Raise exceptions instead of rendering exception templates.
   config.action_dispatch.show_exceptions = false
 

coverage/.last_run.json

@@ -1,5 +1,5 @@
 {
   "result": {
-    "line": 89.0
+    "line": 93.93
   }
 }