From c880ba31b81e88face2b7013a4b4185176fcb72a Mon Sep 17 00:00:00 2001
From: Andrew Tomaka <atomaka@gmail.com>
Date: Thu, 6 Aug 2015 15:10:31 -0400
Subject: [PATCH 1/2] Creaet simple user profile

---
 app/controllers/users_controller.rb           |  5 ++++
 app/models/subcreddit.rb                      | 11 ++-------
 app/models/user.rb                            |  9 +++++++-
 app/validators/sluguuidless_validator.rb      |  7 ++++++
 app/views/application/_sidebar.html.slim      |  2 +-
 app/views/comments/_comment.html.slim         |  4 +++-
 app/views/posts/_post.html.slim               |  4 +++-
 app/views/subcreddits/show.html.slim          |  4 +++-
 app/views/users/show.html.slim                |  5 ++++
 config/application.rb                         |  3 +++
 config/routes.rb                              |  2 +-
 .../20150806191452_add_friendly_id_to_user.rb |  6 +++++
 db/schema.rb                                  |  5 +++-
 spec/controllers/users_controller_spec.rb     | 19 +++++++++++++++
 spec/features/users/profile_spec.rb           | 23 +++++++++++++++++++
 spec/models/user_spec.rb                      | 11 +++++++++
 16 files changed, 104 insertions(+), 16 deletions(-)
 create mode 100644 app/validators/sluguuidless_validator.rb
 create mode 100644 app/views/users/show.html.slim
 create mode 100644 db/migrate/20150806191452_add_friendly_id_to_user.rb
 create mode 100644 spec/features/users/profile_spec.rb

diff --git a/app/controllers/users_controller.rb b/app/controllers/users_controller.rb
index b1e6401..c8cd616 100644
--- a/app/controllers/users_controller.rb
+++ b/app/controllers/users_controller.rb
@@ -1,4 +1,9 @@
 class UsersController < ApplicationController
+  def show
+    @user = User.friendly.find(params[:id])
+    @comments = @user.comments
+  end
+
   def new
     @user = User.new
   end
diff --git a/app/models/subcreddit.rb b/app/models/subcreddit.rb
index f1aa3d1..ea37b6a 100644
--- a/app/models/subcreddit.rb
+++ b/app/models/subcreddit.rb
@@ -16,13 +16,12 @@ class Subcreddit < ActiveRecord::Base
     presence: true,
     format: /\A(?! )[a-z0-9 ]*(?<! )\z/i,
     uniqueness: true, #{ case_sensitive: false },
-    length: { minimum: 3, maximum: 21 }
+    length: { minimum: 3, maximum: 21 },
+    sluguuidless: true
 
   validates :closed,
     format: /\A[01]?\z/
 
-  validate :slug_does_not_have_uuid
-
   def closed?
     self.closed_at != nil
   end
@@ -36,10 +35,4 @@ class Subcreddit < ActiveRecord::Base
       self.closed_at = nil
     end
   end
-
-  def slug_does_not_have_uuid
-    if self.slug.match /([a-z0-9]+\-){4}[a-z0-9]+\z/
-      errors.add(:name, 'must be unique')
-    end
-  end
 end
diff --git a/app/models/user.rb b/app/models/user.rb
index ab848a4..ead07a5 100644
--- a/app/models/user.rb
+++ b/app/models/user.rb
@@ -1,10 +1,17 @@
 class User < ActiveRecord::Base
+  extend FriendlyId
+
   has_secure_password
 
+  has_many :comments
+  has_many :posts
+
+  friendly_id :username, use: :slugged
+
   before_save :downcase_email
 
   validates :email, presence: true, uniqueness: true
-  validates :username, presence: true, uniqueness: true
+  validates :username, presence: true, uniqueness: true, sluguuidless: true
   validates :password, length: { minimum: 8 }
 
   private
diff --git a/app/validators/sluguuidless_validator.rb b/app/validators/sluguuidless_validator.rb
new file mode 100644
index 0000000..7e1d65b
--- /dev/null
+++ b/app/validators/sluguuidless_validator.rb
@@ -0,0 +1,7 @@
+class SluguuidlessValidator < ActiveModel::EachValidator
+  def validate_each(record, attribute, value)
+    if record.slug.match /([a-z0-9]+\-){4}[a-z0-9]+\z/
+      record.errors[attribute] << 'must be unique'
+    end
+  end
+end
diff --git a/app/views/application/_sidebar.html.slim b/app/views/application/_sidebar.html.slim
index 66663e0..aebf356 100644
--- a/app/views/application/_sidebar.html.slim
+++ b/app/views/application/_sidebar.html.slim
@@ -18,4 +18,4 @@
     .title Moderators
     .box
       ul
-        li= @subcreddit.owner_username
+        li= link_to @subcreddit.owner_username, @subcreddit.owner
diff --git a/app/views/comments/_comment.html.slim b/app/views/comments/_comment.html.slim
index edba596..32dbc97 100644
--- a/app/views/comments/_comment.html.slim
+++ b/app/views/comments/_comment.html.slim
@@ -1,5 +1,7 @@
 .comment
-  p.details= "#{comment.user_username} X points #{distance_of_time_in_words comment.created_at, Time.now} ago"
+  p.details
+    == link_to comment.user_username, comment.user
+    = " X points #{distance_of_time_in_words comment.created_at, Time.now} ago"
   p.content= comment.content
   ul.links.list-inline
     li= link_to 'permalink', subcreddit_post_comment_path(subcreddit, post, comment)
diff --git a/app/views/posts/_post.html.slim b/app/views/posts/_post.html.slim
index 8a1fa93..b278d45 100644
--- a/app/views/posts/_post.html.slim
+++ b/app/views/posts/_post.html.slim
@@ -1,7 +1,9 @@
 .posts.contents
   .post.show
     p.title= link_to post.title, [post.subcreddit, post]
-    p.details= "submitted #{distance_of_time_in_words post.created_at, Time.now} ago by #{post.user_username}"
+    p.details
+      = "submitted #{distance_of_time_in_words post.created_at, Time.now} ago by "
+      == link_to post.user_username, post.user
     p.content= post.content
     ul.links.list-inline
       li= link_to "#{post.comments_count} comments", subcreddit_post_path(post.subcreddit, post)
diff --git a/app/views/subcreddits/show.html.slim b/app/views/subcreddits/show.html.slim
index 4cea55a..cb109f2 100644
--- a/app/views/subcreddits/show.html.slim
+++ b/app/views/subcreddits/show.html.slim
@@ -7,7 +7,9 @@
         li
           .post
             p.title= link_to post.title, subcreddit_post_path(@subcreddit, post)
-            p.details= "submitted #{distance_of_time_in_words post.created_at, Time.now} ago by #{post.user_username}"
+            p.details
+              = "submitted #{distance_of_time_in_words post.created_at, Time.now} ago by "
+              == link_to post.user_username, post.user
             ul.links.list-inline
               li= link_to "#{post.comments_count} comments", subcreddit_post_path(@subcreddit, post)
               li= link_to 'share', ''
diff --git a/app/views/users/show.html.slim b/app/views/users/show.html.slim
new file mode 100644
index 0000000..edd0b6a
--- /dev/null
+++ b/app/views/users/show.html.slim
@@ -0,0 +1,5 @@
+h1 #{@user.username}
+
+.comments.contents
+  - @comments.each do |comment|
+    == render 'comments/comment', subcreddit: comment.post.subcreddit, post: comment.post, comment: comment
diff --git a/config/application.rb b/config/application.rb
index 99b1eac..b205fd1 100644
--- a/config/application.rb
+++ b/config/application.rb
@@ -22,5 +22,8 @@ module Creddit
 
     # Do not swallow errors in after_commit/after_rollback callbacks.
     config.active_record.raise_in_transactional_callbacks = true
+
+    # So our console continues to work
+    config.autoload_paths += %W["#{config.root}/app/validators/"]
   end
 end
diff --git a/config/routes.rb b/config/routes.rb
index c5026cb..f036f0a 100644
--- a/config/routes.rb
+++ b/config/routes.rb
@@ -10,7 +10,7 @@ Rails.application.routes.draw do
   end
 
   resources :user_sessions, only: [:new, :create, :destroy]
-  resources :users, only: [:new, :create]
+  resources :users, path: 'u', only: [:show, :new, :create]
 
   root to: 'subcreddits#index'
 end
diff --git a/db/migrate/20150806191452_add_friendly_id_to_user.rb b/db/migrate/20150806191452_add_friendly_id_to_user.rb
new file mode 100644
index 0000000..e78d3a8
--- /dev/null
+++ b/db/migrate/20150806191452_add_friendly_id_to_user.rb
@@ -0,0 +1,6 @@
+class AddFriendlyIdToUser < ActiveRecord::Migration
+  def change
+    add_column :users, :slug, :string
+    add_index :users, :slug, unique: true
+  end
+end
diff --git a/db/schema.rb b/db/schema.rb
index 6295055..dbeb484 100644
--- a/db/schema.rb
+++ b/db/schema.rb
@@ -11,7 +11,7 @@
 #
 # It's strongly recommended that you check this file into your version control system.
 
-ActiveRecord::Schema.define(version: 20150804145405) do
+ActiveRecord::Schema.define(version: 20150806191452) do
 
   create_table "comments", force: :cascade do |t|
     t.integer  "user_id"
@@ -84,6 +84,9 @@ ActiveRecord::Schema.define(version: 20150804145405) do
     t.string   "password_digest"
     t.datetime "created_at",      null: false
     t.datetime "updated_at",      null: false
+    t.string   "slug"
   end
 
+  add_index "users", ["slug"], name: "index_users_on_slug", unique: true
+
 end
diff --git a/spec/controllers/users_controller_spec.rb b/spec/controllers/users_controller_spec.rb
index b01b3c4..f1ebc03 100644
--- a/spec/controllers/users_controller_spec.rb
+++ b/spec/controllers/users_controller_spec.rb
@@ -1,6 +1,25 @@
 require 'rails_helper'
 
 describe UsersController, type: :controller do
+  describe '#show' do
+    let(:user) { create(:user) }
+    before(:each) { get :show, id: user }
+
+    it 'should render :show' do
+      expect(response).to render_template(:show)
+    end
+
+    it 'should assign user comments to @comments' do
+      comments = 5.times.collect { create(:comment, user: user) }
+
+      expect(assigns(:comments)).to eq(comments)
+    end
+
+    it 'should assign requested User to @user' do
+      expect(assigns(:user)).to eq(user)
+    end
+  end
+
   describe '#new' do
     it 'should render :new' do
       get :new
diff --git a/spec/features/users/profile_spec.rb b/spec/features/users/profile_spec.rb
new file mode 100644
index 0000000..038a870
--- /dev/null
+++ b/spec/features/users/profile_spec.rb
@@ -0,0 +1,23 @@
+require 'rails_helper'
+
+describe 'Profile', type: :feature do
+  let!(:user) { create(:user) }
+
+  it 'should display a user profile' do
+    visit user_path(user)
+
+    expect(page).to have_content(user.username)
+  end
+
+  context 'when the user has commented' do
+    let!(:comments) { 5.times.collect { create(:comment, user: user) } }
+
+    it 'should display user comments' do
+      visit user_path(user)
+
+      comments.each do |comment|
+        expect(page).to have_content(comment.content)
+      end
+    end
+  end
+end
diff --git a/spec/models/user_spec.rb b/spec/models/user_spec.rb
index 2df6728..e371da3 100644
--- a/spec/models/user_spec.rb
+++ b/spec/models/user_spec.rb
@@ -1,10 +1,15 @@
 require 'rails_helper'
 
+require 'securerandom'
+
 describe User, type: :model do
   let(:user) { build(:user) }
 
   it { should have_secure_password }
 
+  it { should have_many :comments }
+  it { should have_many :posts }
+
   context 'with valid data' do
     it 'should be valid' do
       expect(user).to be_valid
@@ -44,5 +49,11 @@ describe User, type: :model do
       user.password = 'a' * 4
       expect(user).to be_invalid
     end
+
+    it 'should not allow a slug with a UUID' do
+      user.slug = "test-#{SecureRandom.uuid}"
+
+      expect(user).to be_invalid
+    end
   end
 end

From b68960efe5e9dae1f09c4b544bdd689559f18ef6 Mon Sep 17 00:00:00 2001
From: Andrew Tomaka <atomaka@gmail.com>
Date: Thu, 6 Aug 2015 16:31:08 -0400
Subject: [PATCH 2/2] Refactor page header code

---
 app/helpers/application_helper.rb       |  4 ++++
 app/views/application/_navbar.html.slim |  3 +--
 app/views/comments/show.html.slim       |  1 +
 app/views/posts/show.html.slim          |  1 +
 app/views/subcreddits/show.html.slim    |  1 +
 app/views/users/show.html.slim          |  2 +-
 spec/helpers/application_helper_spec.rb | 16 ++++++++++++++++
 7 files changed, 25 insertions(+), 3 deletions(-)

diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 4b0e707..1fae468 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -3,6 +3,10 @@ module ApplicationHelper
     bootstrap_classes[flash_type] || flash_type.to_s
   end
 
+  def page_header
+    content_for(:title) || ''
+  end
+
   private
 
   def bootstrap_classes
diff --git a/app/views/application/_navbar.html.slim b/app/views/application/_navbar.html.slim
index 754f55f..437764b 100644
--- a/app/views/application/_navbar.html.slim
+++ b/app/views/application/_navbar.html.slim
@@ -10,8 +10,7 @@
       = link_to 'Creddit', root_path, class: 'navbar-brand'
     .collapse.navbar-collapse
       ul.nav.navbar-nav
-        - if @subcreddit && @subcreddit.name
-          li= link_to @subcreddit.name, @subcreddit
+        li= page_header
       ul.nav.navbar-nav.navbar-right
         - if logged_in?
           li= link_to 'Sign Out', signout_path
diff --git a/app/views/comments/show.html.slim b/app/views/comments/show.html.slim
index b55d49f..420f1ba 100644
--- a/app/views/comments/show.html.slim
+++ b/app/views/comments/show.html.slim
@@ -1,3 +1,4 @@
+- content_for(:title, link_to(@subcreddit.name, @subcreddit))
 == render 'posts/post', post: @post
 .alert.alert-info.in-page
   p you are viewing a single comment's thread.
diff --git a/app/views/posts/show.html.slim b/app/views/posts/show.html.slim
index 2a6c2e7..5a86e55 100644
--- a/app/views/posts/show.html.slim
+++ b/app/views/posts/show.html.slim
@@ -1,3 +1,4 @@
+- content_for(:title, link_to(@subcreddit.name, @subcreddit))
 == render 'post', post: @post
 - if @post.comments?
   .title= "all #{@post.comments_count} comments"
diff --git a/app/views/subcreddits/show.html.slim b/app/views/subcreddits/show.html.slim
index cb109f2..6901c4d 100644
--- a/app/views/subcreddits/show.html.slim
+++ b/app/views/subcreddits/show.html.slim
@@ -1,3 +1,4 @@
+- content_for(:title) { link_to @subcreddit.name, @subcreddit }
 - if @subcreddit.closed?
   = "Board has been closed"
 - else
diff --git a/app/views/users/show.html.slim b/app/views/users/show.html.slim
index edd0b6a..bef5346 100644
--- a/app/views/users/show.html.slim
+++ b/app/views/users/show.html.slim
@@ -1,4 +1,4 @@
-h1 #{@user.username}
+- content_for(:title, link_to(@user.username, @user))
 
 .comments.contents
   - @comments.each do |comment|
diff --git a/spec/helpers/application_helper_spec.rb b/spec/helpers/application_helper_spec.rb
index 59b4ce6..44dae05 100644
--- a/spec/helpers/application_helper_spec.rb
+++ b/spec/helpers/application_helper_spec.rb
@@ -26,4 +26,20 @@ describe ApplicationHelper do
       end
     end
   end
+
+  describe '#page_header' do
+    context 'when title is provided' do
+      it 'should return the set title' do
+        allow(helper).to receive(:content_for).with(:title).and_return('Test')
+
+        expect(helper.page_header).to eq 'Test'
+      end
+    end
+
+    context 'when title is not provided' do
+      it 'should return blank' do
+        expect(helper.page_header).to be_blank
+      end
+    end
+  end
 end