lint fixes

app/controllers/case_logs_controller.rb

@@ -1,7 +1,7 @@
 class CaseLogsController < ApplicationController
   skip_before_action :verify_authenticity_token, if: :json_api_request?
   before_action :authenticate, if: :json_api_request?
-  # TODO determine if it's worth splitting out an API controller
+  # TODO: determine if it's worth splitting out an API controller
   before_action :authenticate_user!, unless: :json_api_request?
 
   def index

config/environments/development.rb

@@ -38,7 +38,7 @@ Rails.application.configure do
 
   config.action_mailer.perform_caching = false
 
-  config.action_mailer.default_url_options = { host: 'localhost', port: 3000 }
+  config.action_mailer.default_url_options = { host: "localhost", port: 3000 }
 
   # Print deprecation notices to the Rails logger.
   config.active_support.deprecation = :log

config/initializers/devise.rb

@@ -24,7 +24,7 @@ Devise.setup do |config|
   # Configure the e-mail address which will be shown in Devise::Mailer,
   # note that it will be overwritten if you use your own mailer class
   # with default "from" parameter.
-  config.mailer_sender = 'please-change-me-at-config-initializers-devise@example.com'
+  config.mailer_sender = "please-change-me-at-config-initializers-devise@example.com"
 
   # Configure the class responsible to send e-mails.
   # config.mailer = 'Devise::Mailer'
@@ -36,7 +36,7 @@ Devise.setup do |config|
   # Load and configure the ORM. Supports :active_record (default) and
   # :mongoid (bson_ext recommended) by default. Other ORMs may be
   # available as additional gems.
-  require 'devise/orm/active_record'
+  require "devise/orm/active_record"
 
   # ==> Configuration for any authentication mechanism
   # Configure which keys are used when authenticating a user. The default is

db/migrate/20211101103235_devise_create_users.rb

@@ -32,7 +32,6 @@ class DeviseCreateUsers < ActiveRecord::Migration[6.1]
       # t.string   :unlock_token # Only if unlock strategy is :email or :both
       # t.datetime :locked_at
 
-
       t.timestamps null: false
     end
 

spec/features/user_spec.rb

@@ -1,18 +1,26 @@
 require "rails_helper"
 RSpec.describe "User Features" do
-    let!(:user) { FactoryBot.create(:user) }
+  let!(:user) { FactoryBot.create(:user) }
-    context "A user navigating to case logs" do
+  context "A user navigating to case logs" do
-        it " is required to log in" do
+    it " is required to log in" do
-          visit("/case_logs")
+      visit("/case_logs")
-          expect(page).to have_current_path("/users/sign_in")
+      expect(page).to have_current_path("/users/sign_in")
-        end
+    end
+
+    it " is redirected to case logs after signing in" do
+      visit("/case_logs")
+      fill_in("user_email", with: "test@example.com")
+      fill_in("user_password", with: "pAssword1")
+      click_button("Log in")
+      expect(page).to have_current_path("/case_logs")
+    end
+  end
 
-        it " is redirected to case logs after signing in" do
+  context "A user who has forgotten their password" do
-          visit("/case_logs")
+    it " is redirected to the forgotten password page when they click the forgot password link" do
-          fill_in("user_email", with: "test@example.com")
+      visit("/case_logs")
-          fill_in("user_password", with: "pAssword1")
+      click_link("Forgot your password?")
-          click_button("Log in")
+      expect(page).to have_current_path("/users/password/new")
-          expect(page).to have_current_path("/case_logs")
-        end 
     end
+  end
 end

spec/support/devise.rb

@@ -1,9 +1,9 @@
-require_relative './controller_macros' 
+require_relative "./controller_macros"
 
 RSpec.configure do |config|
   # For Devise > 4.1.1
-  config.include Devise::Test::ControllerHelpers, :type => :controller
+  config.include Devise::Test::ControllerHelpers, type: :controller
   # Use the following instead if you are on Devise <= 4.1.1
   # config.include Devise::TestHelpers, :type => :controller
-  config.extend ControllerMacros, :type => :controller
+  config.extend ControllerMacros, type: :controller
 end