fix: adjust cookie function name to clarify purpose

Author: macite

app/api/authentication_api.rb

@@ -80,7 +80,7 @@ class AuthenticationApi < Grape::API
       # Return user details
       present :user, user, with: Entities::UserEntity
       present :auth_token, token.authentication_token
-      add_refresh_cookie_to_response(remember)
+      set_refresh_cookie_in_response(remember)
     end
   end
 
@@ -278,7 +278,7 @@ class AuthenticationApi < Grape::API
         # Respond user details with new auth token
         present :user, user, with: Entities::UserEntity
         present :auth_token, token.authentication_token
-        add_refresh_cookie_to_response(params[:remember])
+        set_refresh_cookie_in_response(params[:remember])
       end
     end
   end
@@ -360,7 +360,7 @@ class AuthenticationApi < Grape::API
     end
 
     # Remove the refresh token cookie - if remember is false
-    add_refresh_cookie_to_response(false) if params[:remember]
+    set_refresh_cookie_in_response(false) unless params[:remember]
     present nil
   end
 

app/helpers/authentication_helpers.rb

@@ -62,7 +62,7 @@ def authenticated_via_refresh_token?
       true
     when :token_expired, :error, :missing_details
       # Token expired - remove cookies
-      add_refresh_cookie_to_response(false)
+      set_refresh_cookie_in_response(false)
       false
     end
   end
@@ -205,7 +205,10 @@ def db_auth?
     Doubtfire::Application.config.auth_method == :database
   end
 
-  def add_refresh_cookie_to_response(remember)
+  # rubocop:disable Naming/AccessorMethodName
+  def set_refresh_cookie_in_response(remember)
+    # rubocop:enable Naming/AccessorMethodName
+
     if remember
       token = current_user.auth_tokens.where(token_type: :refresh_token).last