rename ShowUser to FindUser and add error handling, start authenticat…

…ion_controller

Not currently passing spec, our log_in before_action currently must
take a User.

app/controllers/authenticated_controller.rb

@@ -0,0 +1,3 @@
+class AuthenticatedController < ApplicationController
+  before_action :log_in
+end

app/controllers/password_resets_controller.rb

@@ -1,6 +1,7 @@
-class PasswordResetsController < ApplicationController
-  before_action :find_user,        only: %i(edit update)
-  before_action :check_expiration, only: %i(edit update)
+class PasswordResetsController < AuthenticatedController
+  before_action      :find_user,        only: %i(edit update)
+  before_action      :check_expiration, only: %i(edit update)
+  skip_before_action :log_in,           only: %i(new edit create)
 
   def new
   end
@@ -25,7 +26,6 @@ def update
     update_password = UpdatePassword.call(user_params: user_params, user: @user)
 
     if update_password.success?
-      log_in update_password.user
       flash[:success] = "Password has been reset"
       redirect_to update_password.user
     else

app/controllers/sessions_controller.rb

@@ -1,4 +1,4 @@
-class SessionsController < ApplicationController
+class SessionsController < AuthenticatedController
   def new
   end
 
@@ -7,7 +7,6 @@ def create
                                    password: params[:session][:password])
 
     if valid_user.success?
-      log_in valid_user.user
       redirect_to valid_user.user
     else
       flash.now[:error] = "There was a problem signing in"

app/controllers/users_controller.rb

@@ -16,10 +16,7 @@ def create
   end
 
   def show
-    result = ShowUser.call(id: params[:id])
-    @user = result.user
-    @todos = result.todos
-    @todo = result.todo
+    @user = FindUser.call(id: params[:id], current_id: session[:user_id]).user
   end
 
   private

app/interactors/find_user.rb

@@ -0,0 +1,11 @@
+class FindUser
+  include Interactor
+
+  def call
+    begin
+      context.user = User.find(context.id)
+    rescue ActiveRecord::RecordNotFound
+      context.user = User.find(context.current_id)
+    end
+  end
+end

app/interactors/validate_user.rb

@@ -3,9 +3,11 @@ class ValidateUser
 
   def call
     context.user = User.find_by(email: context.email.downcase)
-    user = context.user
-    password = context.password
-    errors = context.user.errors
-    context.fail!(errors: errors) unless user && user.authenticate(password)
+    authenticate_user(context.user, context.password)
+  end
+
+  def authenticate_user(user, password)
+    return if user && user.authenticate(password)
+    context.fail!(errors: user.errors)
   end
 end

app/views/users/show.html.erb

@@ -2,6 +2,6 @@
   <h1><%= @user.name %></h1>
   <h3><%= @user.email %></h3>
   <p>Test subject since <%= @user.created_at.strftime("%B %d, %Y") %></p>
-  <%= render @todos %>
-  <%= render partial: "todos/form", locals: { user: @user, todo: @todo } %>
+  <%= render @user.todos %>
+  <%= render partial: "todos/form", locals: { user: @user, todo: @user.todos.new } %>
 </div>