From 368702aa967ebd8630b6ca103bd82af493ab6099 Mon Sep 17 00:00:00 2001
From: Chris Mason <47318+cmason@users.noreply.github.com>
Date: Wed, 30 Aug 2023 18:45:01 -0500
Subject: [PATCH] Localization (#61)

* Localize application views
* Update system tests to use authentication
* Swap selenium for cuprite
* Upgrade Rails to 7.0.7.2
---
 Gemfile                                      |   2 +-
 Gemfile.lock                                 | 129 ++++++++++---------
 app/controllers/sessions_controller.rb       |   4 +-
 app/helpers/application_helper.rb            |  13 ++
 app/helpers/users_helper.rb                  |   8 +-
 app/views/layouts/_navbar.html.erb           |  12 +-
 app/views/layouts/_og_meta.html.erb          |   8 +-
 app/views/layouts/application.html.erb       |   2 +-
 config/environments/test.rb                  |   1 +
 config/locales/en.yml                        |  36 ------
 config/locales/globals/en.yml                |   5 +
 config/locales/views/layouts/navbar/en.yml   |   5 +
 config/locales/views/quotes/en.yml           |   1 +
 config/locales/views/sessions/en.yml         |   6 +
 db/seeds.rb                                  |  15 ++-
 test/application_system_test_case.rb         |   4 +-
 test/controllers/sessions_controller_test.rb |  46 +------
 test/helpers/application_helper_test.rb      |  18 +++
 test/helpers/users_helper_test.rb            |  10 ++
 test/support/authentication_helpers.rb       |  51 ++++++++
 test/support/quote_helpers.rb                |   8 ++
 test/system/nav_bar_test.rb                  |  19 +++
 test/test_helper.rb                          |  11 +-
 23 files changed, 244 insertions(+), 170 deletions(-)
 delete mode 100644 config/locales/en.yml
 create mode 100644 config/locales/globals/en.yml
 create mode 100644 config/locales/views/layouts/navbar/en.yml
 create mode 100644 config/locales/views/quotes/en.yml
 create mode 100644 config/locales/views/sessions/en.yml
 create mode 100644 test/support/authentication_helpers.rb
 create mode 100644 test/support/quote_helpers.rb
 create mode 100644 test/system/nav_bar_test.rb

diff --git a/Gemfile b/Gemfile
index f7371c9..ba45d23 100644
--- a/Gemfile
+++ b/Gemfile
@@ -78,7 +78,7 @@ end
 group :test do
   # Use system testing [https://guides.rubyonrails.org/testing.html#system-testing]
   gem "capybara"
-  gem "selenium-webdriver", "~> 4.11"
+  gem "cuprite"
   gem "simplecov", require: false
 end
 
diff --git a/Gemfile.lock b/Gemfile.lock
index bda2619..83fe907 100644
--- a/Gemfile.lock
+++ b/Gemfile.lock
@@ -10,67 +10,67 @@ GIT
 GEM
   remote: https://rubygems.org/
   specs:
-    actioncable (7.0.6)
-      actionpack (= 7.0.6)
-      activesupport (= 7.0.6)
+    actioncable (7.0.7.2)
+      actionpack (= 7.0.7.2)
+      activesupport (= 7.0.7.2)
       nio4r (~> 2.0)
       websocket-driver (>= 0.6.1)
-    actionmailbox (7.0.6)
-      actionpack (= 7.0.6)
-      activejob (= 7.0.6)
-      activerecord (= 7.0.6)
-      activestorage (= 7.0.6)
-      activesupport (= 7.0.6)
+    actionmailbox (7.0.7.2)
+      actionpack (= 7.0.7.2)
+      activejob (= 7.0.7.2)
+      activerecord (= 7.0.7.2)
+      activestorage (= 7.0.7.2)
+      activesupport (= 7.0.7.2)
       mail (>= 2.7.1)
       net-imap
       net-pop
       net-smtp
-    actionmailer (7.0.6)
-      actionpack (= 7.0.6)
-      actionview (= 7.0.6)
-      activejob (= 7.0.6)
-      activesupport (= 7.0.6)
+    actionmailer (7.0.7.2)
+      actionpack (= 7.0.7.2)
+      actionview (= 7.0.7.2)
+      activejob (= 7.0.7.2)
+      activesupport (= 7.0.7.2)
       mail (~> 2.5, >= 2.5.4)
       net-imap
       net-pop
       net-smtp
       rails-dom-testing (~> 2.0)
-    actionpack (7.0.6)
-      actionview (= 7.0.6)
-      activesupport (= 7.0.6)
+    actionpack (7.0.7.2)
+      actionview (= 7.0.7.2)
+      activesupport (= 7.0.7.2)
       rack (~> 2.0, >= 2.2.4)
       rack-test (>= 0.6.3)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.0, >= 1.2.0)
-    actiontext (7.0.6)
-      actionpack (= 7.0.6)
-      activerecord (= 7.0.6)
-      activestorage (= 7.0.6)
-      activesupport (= 7.0.6)
+    actiontext (7.0.7.2)
+      actionpack (= 7.0.7.2)
+      activerecord (= 7.0.7.2)
+      activestorage (= 7.0.7.2)
+      activesupport (= 7.0.7.2)
       globalid (>= 0.6.0)
       nokogiri (>= 1.8.5)
-    actionview (7.0.6)
-      activesupport (= 7.0.6)
+    actionview (7.0.7.2)
+      activesupport (= 7.0.7.2)
       builder (~> 3.1)
       erubi (~> 1.4)
       rails-dom-testing (~> 2.0)
       rails-html-sanitizer (~> 1.1, >= 1.2.0)
-    activejob (7.0.6)
-      activesupport (= 7.0.6)
+    activejob (7.0.7.2)
+      activesupport (= 7.0.7.2)
       globalid (>= 0.3.6)
-    activemodel (7.0.6)
-      activesupport (= 7.0.6)
-    activerecord (7.0.6)
-      activemodel (= 7.0.6)
-      activesupport (= 7.0.6)
-    activestorage (7.0.6)
-      actionpack (= 7.0.6)
-      activejob (= 7.0.6)
-      activerecord (= 7.0.6)
-      activesupport (= 7.0.6)
+    activemodel (7.0.7.2)
+      activesupport (= 7.0.7.2)
+    activerecord (7.0.7.2)
+      activemodel (= 7.0.7.2)
+      activesupport (= 7.0.7.2)
+    activestorage (7.0.7.2)
+      actionpack (= 7.0.7.2)
+      activejob (= 7.0.7.2)
+      activerecord (= 7.0.7.2)
+      activesupport (= 7.0.7.2)
       marcel (~> 1.0)
       mini_mime (>= 1.1.0)
-    activesupport (7.0.6)
+    activesupport (7.0.7.2)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
@@ -111,6 +111,9 @@ GEM
     crass (1.0.6)
     cssbundling-rails (1.2.0)
       railties (>= 6.0.0)
+    cuprite (0.14.3)
+      capybara (~> 3.0)
+      ferrum (~> 0.13.0)
     date (3.3.3)
     debug (1.8.0)
       irb (>= 1.5.0)
@@ -123,6 +126,11 @@ GEM
       faraday-net_http (>= 2.0, < 3.1)
       ruby2_keywords (>= 0.0.4)
     faraday-net_http (3.0.2)
+    ferrum (0.13)
+      addressable (~> 2.5)
+      concurrent-ruby (~> 1.1)
+      webrick (~> 1.7)
+      websocket-driver (>= 0.6, < 0.8)
     flay (2.13.1)
       erubi (~> 1.10)
       path_expander (~> 1.0)
@@ -176,7 +184,7 @@ GEM
     marcel (1.0.2)
     matrix (0.4.2)
     method_source (1.0.0)
-    mini_mime (1.1.2)
+    mini_mime (1.1.5)
     minitest (5.19.0)
     msgpack (1.7.2)
     multi_xml (0.6.0)
@@ -195,9 +203,9 @@ GEM
     net-smtp (0.3.3)
       net-protocol
     nio4r (2.5.9)
-    nokogiri (1.15.3-x86_64-darwin)
+    nokogiri (1.15.4-x86_64-darwin)
       racc (~> 1.4)
-    nokogiri (1.15.3-x86_64-linux)
+    nokogiri (1.15.4-x86_64-linux)
       racc (~> 1.4)
     oauth2 (2.0.9)
       faraday (>= 0.17.3, < 3.0)
@@ -221,20 +229,20 @@ GEM
       rack (>= 1.2.0)
     rack-test (2.1.0)
       rack (>= 1.3)
-    rails (7.0.6)
-      actioncable (= 7.0.6)
-      actionmailbox (= 7.0.6)
-      actionmailer (= 7.0.6)
-      actionpack (= 7.0.6)
-      actiontext (= 7.0.6)
-      actionview (= 7.0.6)
-      activejob (= 7.0.6)
-      activemodel (= 7.0.6)
-      activerecord (= 7.0.6)
-      activestorage (= 7.0.6)
-      activesupport (= 7.0.6)
+    rails (7.0.7.2)
+      actioncable (= 7.0.7.2)
+      actionmailbox (= 7.0.7.2)
+      actionmailer (= 7.0.7.2)
+      actionpack (= 7.0.7.2)
+      actiontext (= 7.0.7.2)
+      actionview (= 7.0.7.2)
+      activejob (= 7.0.7.2)
+      activemodel (= 7.0.7.2)
+      activerecord (= 7.0.7.2)
+      activestorage (= 7.0.7.2)
+      activesupport (= 7.0.7.2)
       bundler (>= 1.15.0)
-      railties (= 7.0.6)
+      railties (= 7.0.7.2)
     rails-dom-testing (2.2.0)
       activesupport (>= 5.0.0)
       minitest
@@ -242,9 +250,9 @@ GEM
     rails-html-sanitizer (1.6.0)
       loofah (~> 2.21)
       nokogiri (~> 1.14)
-    railties (7.0.6)
-      actionpack (= 7.0.6)
-      activesupport (= 7.0.6)
+    railties (7.0.7.2)
+      actionpack (= 7.0.7.2)
+      activesupport (= 7.0.7.2)
       method_source
       rake (>= 12.2)
       thor (~> 1.0)
@@ -295,11 +303,6 @@ GEM
       simplecov (>= 0.22.0)
       tty-which (~> 0.5.0)
       virtus (~> 2.0)
-    rubyzip (2.3.2)
-    selenium-webdriver (4.11.0)
-      rexml (~> 3.2, >= 3.2.5)
-      rubyzip (>= 1.2.2, < 3.0)
-      websocket (~> 1.0)
     sexp_processor (4.17.0)
     simplecov (0.22.0)
       docile (~> 1.1)
@@ -359,7 +362,7 @@ GEM
       activemodel (>= 6.0.0)
       bindex (>= 0.4.0)
       railties (>= 6.0.0)
-    websocket (1.2.9)
+    webrick (1.8.1)
     websocket-driver (0.7.6)
       websocket-extensions (>= 0.1.0)
     websocket-extensions (0.1.5)
@@ -379,6 +382,7 @@ DEPENDENCIES
   bundler-leak
   capybara
   cssbundling-rails
+  cuprite
   debug
   jbuilder
   jsbundling-rails
@@ -388,7 +392,6 @@ DEPENDENCIES
   rails (~> 7.0.6)
   rubocop-minitest
   rubycritic
-  selenium-webdriver (~> 4.11)
   simplecov
   slack_sign_in!
   sprockets-rails
diff --git a/app/controllers/sessions_controller.rb b/app/controllers/sessions_controller.rb
index 0b1eaae..c48c83d 100644
--- a/app/controllers/sessions_controller.rb
+++ b/app/controllers/sessions_controller.rb
@@ -5,7 +5,7 @@ def create
     if slack_authorization.successful?
       user = find_or_create_user(slack_authorization.identity)
       create_session_for_user(user)
-      flash[:notice] = t(:signed_in, name: user.name)
+      flash[:notice] = t(".success", name: user.name)
     else
       logger.error "Slack authorization failed: #{slack_authorization.inspect}"
       flash[:alert] = t(slack_authorization.error)
@@ -15,7 +15,7 @@ def create
 
   def destroy
     session[:user_id] = nil
-    redirect_to root_path, notice: t(:signed_out)
+    redirect_to root_path, notice: t(".success")
   end
 
   private
diff --git a/app/helpers/application_helper.rb b/app/helpers/application_helper.rb
index 1ac9d90..2061413 100644
--- a/app/helpers/application_helper.rb
+++ b/app/helpers/application_helper.rb
@@ -6,4 +6,17 @@ def flash_message(type, message)
       message
     end
   end
+
+  def title
+    return t("letterkenny") unless content_for?(:title)
+    "#{content_for(:title)} | #{t("letterkenny")}"
+  end
+
+  def og_description
+    content_for?(:og_description) ? content_for(:og_description) : t("tagline")
+  end
+
+  def og_url
+    content_for?(:og_url) ? content_for(:og_url) : root_url
+  end
 end
diff --git a/app/helpers/users_helper.rb b/app/helpers/users_helper.rb
index 5b4f409..597e106 100644
--- a/app/helpers/users_helper.rb
+++ b/app/helpers/users_helper.rb
@@ -1,7 +1,13 @@
 module UsersHelper
   def avatar(user)
     content_tag :figure, class: "image is-32x32" do
-      image_tag user.avatar_url, class: "is-rounded", size: 32, alt: user.name
+      if user.avatar_url
+        image_tag user.avatar_url, class: "is-rounded", size: 32, alt: user.name
+      else
+        content_tag :span, class: "icon" do
+          tag.i class: "fas fa-user-secret is-size-3"
+        end
+      end
     end
   end
 end
diff --git a/app/views/layouts/_navbar.html.erb b/app/views/layouts/_navbar.html.erb
index f96b75a..74f907f 100644
--- a/app/views/layouts/_navbar.html.erb
+++ b/app/views/layouts/_navbar.html.erb
@@ -13,12 +13,12 @@
   <div id="main-nav" class="navbar-menu">
     <div class="navbar-start">
       <div class="navbar-item">
-        Letterkenny consists of hicks, skids, hockey players, and Christians. These are their quotes.
+        <%= t("tagline") %>
       </div>
     </div>
     <div class="navbar-end is-active">
       <% if current_user %>
-        <div class="navbar-item has-dropdown has-dropdown-up is-hoverable">
+        <div class="navbar-item has-dropdown has-dropdown-up is-hoverable signed-in-user">
           <div class="navbar-link">
           <span>
             <%= avatar(current_user) %>
@@ -27,17 +27,17 @@
           </div>
 
           <div class="navbar-dropdown">
-            <%= link_to root_path, title: "Admin", class: "navbar-item" do %>
+            <%= link_to root_path, title: t(".admin"), class: "navbar-item" do %>
               <span class="icon">
                 <i class="fas fa-screwdriver-wrench"></i>
               </span>
-              <span>Admin</span>
+              <span><%= t(".admin") %></span>
             <% end %>
-            <%= link_to destroy_session_path, title: "Sign out", class: "navbar-item" do %>
+            <%= link_to destroy_session_path, title: t(".signout"), class: "navbar-item" do %>
               <span class="icon">
                 <i class="fas fa-sign-out-alt"></i>
               </span>
-              <span>Sign Out</span>
+              <span><%= t(".signout") %></span>
             <% end %>
           </div>
         </div>
diff --git a/app/views/layouts/_og_meta.html.erb b/app/views/layouts/_og_meta.html.erb
index 029c174..a9903be 100644
--- a/app/views/layouts/_og_meta.html.erb
+++ b/app/views/layouts/_og_meta.html.erb
@@ -1,6 +1,6 @@
 <meta property="og:type" content="website" />
-<meta property="og:url" content="<%= content_for?(:og_url) ? yield(:og_url) : root_url %>" />
-<meta property="og:title" content="Letterkenny Quotes" />
-<meta property="og:description" content="<%= content_for?(:og_description) ? yield(:og_description) : "Random Letterkenny Quotes" %>" />
+<meta property="og:url" content="<%= og_url %>" />
+<meta property="og:title" content="<%=t("letterkenny")%>" />
+<meta property="og:description" content="<%= og_description %>" />
 <meta property="og:image" content="<%= image_url "letterkenny_logo.jpg" %>" />
-<meta name="description" content="<%= content_for?(:og_description) ? yield(:og_description) : "Random Letterkenny Quotes" %>" />
+<meta name="description" content="<%= og_description %>" />
diff --git a/app/views/layouts/application.html.erb b/app/views/layouts/application.html.erb
index 3869aa4..a37e400 100644
--- a/app/views/layouts/application.html.erb
+++ b/app/views/layouts/application.html.erb
@@ -1,7 +1,7 @@
 <!DOCTYPE html>
 <html lang="en">
   <head>
-    <title>LetterKenny Quotes</title>
+    <title><%= title %></title>
     <meta name="viewport" content="width=device-width,initial-scale=1">
     <%= csrf_meta_tags %>
     <%= csp_meta_tag %>
diff --git a/config/environments/test.rb b/config/environments/test.rb
index 9de8427..b0e75c7 100644
--- a/config/environments/test.rb
+++ b/config/environments/test.rb
@@ -57,4 +57,5 @@
 
   # Annotate rendered view with file names.
   # config.action_view.annotate_rendered_view_with_filenames = true
+  config.slack_sign_in.client = SlackSignIn::TestClient
 end
diff --git a/config/locales/en.yml b/config/locales/en.yml
deleted file mode 100644
index 7affb2a..0000000
--- a/config/locales/en.yml
+++ /dev/null
@@ -1,36 +0,0 @@
-# Files in the config/locales directory are used for internationalization
-# and are automatically loaded by Rails. If you want to use locales other
-# than English, add the necessary files in this directory.
-#
-# To use the locales, use `I18n.t`:
-#
-#     I18n.t "hello"
-#
-# In views, this is aliased to just `t`:
-#
-#     <%= t("hello") %>
-#
-# To use a different locale, set it with `I18n.locale`:
-#
-#     I18n.locale = :es
-#
-# This would use the information in config/locales/es.yml.
-#
-# The following keys must be escaped otherwise they will not be retrieved by
-# the default I18n backend:
-#
-# true, false, on, off, yes, no
-#
-# Instead, surround them with single quotes.
-#
-# en:
-#   "true": "foo"
-#
-# To learn more, please read the Rails Internationalization guide
-# available at https://guides.rubyonrails.org/i18n.html.
-
-en:
-  hello: "Hello world"
-  signed_in: "How are ya now %{name}!"
-  signed_out: "We hate to see you pull a Letterkenny Leave"
-  invalid_scope_requested: "Permission to access Slack denied. Unable to authenticate."
diff --git a/config/locales/globals/en.yml b/config/locales/globals/en.yml
new file mode 100644
index 0000000..46f96c8
--- /dev/null
+++ b/config/locales/globals/en.yml
@@ -0,0 +1,5 @@
+en:
+  letterkenny: "LetterKenny Quotes"
+  tagline: "Letterkenny consists of hicks, skids, hockey players, and Christians. These are their quotes."
+  description: "Random Letterkenny Quotes"
+  invalid_scope_requested: "Permission to access Slack denied. Unable to authenticate."
diff --git a/config/locales/views/layouts/navbar/en.yml b/config/locales/views/layouts/navbar/en.yml
new file mode 100644
index 0000000..1a6cba8
--- /dev/null
+++ b/config/locales/views/layouts/navbar/en.yml
@@ -0,0 +1,5 @@
+en:
+  layouts:
+    navbar:
+      admin: "Admin"
+      signout: "Sign Out"
diff --git a/config/locales/views/quotes/en.yml b/config/locales/views/quotes/en.yml
new file mode 100644
index 0000000..63f1c3e
--- /dev/null
+++ b/config/locales/views/quotes/en.yml
@@ -0,0 +1 @@
+en:
diff --git a/config/locales/views/sessions/en.yml b/config/locales/views/sessions/en.yml
new file mode 100644
index 0000000..dbea7a8
--- /dev/null
+++ b/config/locales/views/sessions/en.yml
@@ -0,0 +1,6 @@
+en:
+  sessions:
+    create:
+      success: "How are ya now %{name}!"
+    destroy:
+      success: "Maybe this calls for a Letterkenny Leave"
diff --git a/db/seeds.rb b/db/seeds.rb
index 79e0433..d468d7c 100644
--- a/db/seeds.rb
+++ b/db/seeds.rb
@@ -46,7 +46,8 @@
   {text: "Real men finish what they started."},
   {text: "It's a hard life pickin stones and pullin teets, but sure as gods got sandals, it sure beats fightin dudes with treasure trails."},
   {text: "Do you guys do CrossFit? You can CrossFuck off!"},
-  {text: "What’s up with your fuckin body hair big shoots? You look like a 12-year-old dutch girl."}
+  {text: "What’s up with your fuckin body hair big shoots? You look like a 12-year-old dutch girl."},
+  {text: "It sure is nice to be back in the comfort zone."}
 ])
 
 daryl.quotes.create([
@@ -55,7 +56,8 @@
   {text: "How're we going to fuck this pig?"},
   {text: "Your neighbor put a Corvette engine in a Ford Ranger. Like, what a fuckin' basic."},
   {text: "Closest you’re gettin’ to any action this weekend is givin’ the dairy cow’s teets a good scrubbin’."},
-  {text: "Can't help gettin' a semi while bouncing around in a semi truck."}
+  {text: "Can't help gettin' a semi while bouncing around in a semi truck."},
+  {text: "Oh, bother."}
 ])
 
 katy.quotes.create([
@@ -86,7 +88,8 @@
   {text: "Out of the way I’m growing a tail!"},
   {text: "Youse guys ever take the dirt road home?"},
   {text: "That's what I appreciates about you."},
-  {text: "I'm so upset about my perennials."}
+  {text: "I'm so upset about my perennials."},
+  {text: "You are obliged to partakes in some socials stimulations from times to time, lest the young lady becomes rastless."}
 ])
 
 gail.quotes.create([
@@ -94,13 +97,15 @@
   {text: "You wanna 68? That's when you go down on me... And I'll owe ya one"},
   {text: "How many planets are there? Only seven after I destroy Uranus"},
   {text: "I was flicking more bean than a Starbucks barista!"},
-  {text: "Something about that old goat just really grinds my pepper."}
+  {text: "Something about that old goat just really grinds my pepper."},
+  {text: "No tushy play unless I ask for it - and I will ask for it."}
 ])
 
 stewart.quotes.create([
   {text: "Shirt tucker!"},
   {text: "Wonderous!"},
-  {text: "One does not simply hop on or off the Anabolia Express. No. You ride that juice train all the way to the end of the line."}
+  {text: "One does not simply hop on or off the Anabolia Express. No. You ride that juice train all the way to the end of the line."},
+  {text: "Ah, sweet hubris."}
 ])
 
 glen.quotes.create([
diff --git a/test/application_system_test_case.rb b/test/application_system_test_case.rb
index 33264b9..62c618a 100644
--- a/test/application_system_test_case.rb
+++ b/test/application_system_test_case.rb
@@ -1,5 +1,7 @@
 require "test_helper"
+require "capybara/cuprite"
 
 class ApplicationSystemTestCase < ActionDispatch::SystemTestCase
-  driven_by :selenium, using: :headless_chrome, screen_size: [1400, 1400]
+  driven_by :cuprite, screen_size: [1400, 1400], options:
+    {js_errors: true, inspector: true}
 end
diff --git a/test/controllers/sessions_controller_test.rb b/test/controllers/sessions_controller_test.rb
index 9f2b1ae..62f12cf 100644
--- a/test/controllers/sessions_controller_test.rb
+++ b/test/controllers/sessions_controller_test.rb
@@ -10,7 +10,7 @@ def setup
 
   test "should login user from slack" do
     get create_session_url, env: slack_successful_authentication(@user)
-    assert_equal t(:signed_in, name: @user.name), flash[:notice]
+    assert_equal t("sessions.create.success", name: @user.name), flash[:notice]
     assert_equal @user.id, session[:user_id]
     assert_redirected_to root_path
   end
@@ -25,7 +25,7 @@ def setup
   test "should be able to log out" do
     get destroy_session_url
     assert_nil session[:user_id]
-    assert_equal t(:signed_out), flash[:notice]
+    assert_equal t("sessions.destroy.success"), flash[:notice]
     assert_redirected_to root_path
   end
 
@@ -33,51 +33,11 @@ def setup
     assert_difference("User.count") do
       get create_session_url, env: slack_successful_authentication(new_user)
     end
-    assert_equal t(:signed_in, name: new_user.name), flash[:notice]
+    assert_equal t("sessions.create.success", name: new_user.name), flash[:notice]
     assert_not_nil session[:user_id]
     assert_redirected_to root_path
   end
 
-  # The slack_sign_in gem passes the results of the slack sign in flow to the
-  # controller as a flash message. We can simulate failed or successful authentication
-  # attempts by crafting the flash message, avoiding having to do the oauth dance
-  # during testing.
-
-  def slack_successful_authentication(user)
-    {
-      ActionDispatch::Flash::KEY => ActionDispatch::Flash::FlashHash.new(
-        slack_sign_in: {"success" => {
-          "ok" => true,
-          "scope" => "identity.basic,identity.email,identity.avatar,identity.team",
-          "user_id" => user.slack_user_id,
-          "team_id" => user.slack_team_id,
-          "enterprise_id" => nil,
-          "user" => {
-            "name" => user.name,
-            "id" => user.slack_user_id,
-            "image_24" => "https://example.com/24.png",
-            "image_32" => "https://example.com/32.png",
-            "image_48" => "https://example.com/48.png",
-            "image_72" => "https://example.com/72.png",
-            "image_192" => "https://example.com/192.png",
-            "image_512" => "https://example.com/512.png"
-          },
-          "team" => {
-            "id" => user.slack_team_id
-          }
-        }}
-      )
-    }
-  end
-
-  def slack_failed_authentication
-    {ActionDispatch::Flash::KEY => ActionDispatch::Flash::FlashHash.new(
-      {slack_sign_in: {
-        "error" => "invalid_scope_requested"
-      }}
-    )}
-  end
-
   def new_user
     id = SecureRandom.uuid
     User.new(name: "New Guy", slack_unique_id: id, slack_user_id: id, slack_team_id: id, avatar_url: "https://example.com/32.png")
diff --git a/test/helpers/application_helper_test.rb b/test/helpers/application_helper_test.rb
index 2cd374f..2165b0b 100644
--- a/test/helpers/application_helper_test.rb
+++ b/test/helpers/application_helper_test.rb
@@ -24,4 +24,22 @@ class ApplicationHelperTest < ActionView::TestCase
       flash_message(:wat, message)
     )
   end
+
+  test "formats page specific title" do
+    content_for(:title) { "Page Title" }
+    assert_equal "Page Title | #{I18n.t("letterkenny")}", title
+  end
+
+  test "returns app name when page title is missing" do
+    assert_equal I18n.t("letterkenny"), title
+  end
+
+  test "formats page specific open graph description" do
+    content_for(:og_description) { "OG Description" }
+    assert_equal "OG Description", og_description
+  end
+
+  test "returns tagline when og_description is missing" do
+    assert_equal I18n.t("tagline"), og_description
+  end
 end
diff --git a/test/helpers/users_helper_test.rb b/test/helpers/users_helper_test.rb
index 0424596..a172ff8 100644
--- a/test/helpers/users_helper_test.rb
+++ b/test/helpers/users_helper_test.rb
@@ -9,4 +9,14 @@ class UsersHelperTest < ActionView::TestCase
       avatar(user)
     )
   end
+
+  test "should return a default avatar for users without an avatar" do
+    user = users(:slack)
+    user.avatar_url = nil
+
+    assert_dom_equal(
+      %(<figure class="image is-32x32"><span class="icon"><i class="fas fa-user-secret is-size-3"></i></span></figure>),
+      avatar(user)
+    )
+  end
 end
diff --git a/test/support/authentication_helpers.rb b/test/support/authentication_helpers.rb
new file mode 100644
index 0000000..ba32896
--- /dev/null
+++ b/test/support/authentication_helpers.rb
@@ -0,0 +1,51 @@
+module AuthenticationHelpers
+  def log_in
+    visit root_url
+    assert has_button?("Sign in with Slack")
+    click_on "Sign in with Slack"
+  end
+
+  def log_out
+    visit destroy_session_url
+  end
+
+  # The slack_sign_in gem passes the results of the slack sign in flow to the
+  # controller as a flash message. We can simulate failed or successful authentication
+  # attempts by crafting the flash message, avoiding having to do the oauth dance
+  # during testing.
+
+  def slack_successful_authentication(user)
+    {
+      ActionDispatch::Flash::KEY => ActionDispatch::Flash::FlashHash.new(
+        slack_sign_in: {"success" => {
+          "ok" => true,
+          "scope" => "identity.basic,identity.email,identity.avatar,identity.team",
+          "user_id" => user.slack_user_id,
+          "team_id" => user.slack_team_id,
+          "enterprise_id" => nil,
+          "user" => {
+            "name" => user.name,
+            "id" => user.slack_user_id,
+            "image_24" => "https://example.com/24.png",
+            "image_32" => "https://example.com/32.png",
+            "image_48" => "https://example.com/48.png",
+            "image_72" => "https://example.com/72.png",
+            "image_192" => "https://example.com/192.png",
+            "image_512" => "https://example.com/512.png"
+          },
+          "team" => {
+            "id" => user.slack_team_id
+          }
+        }}
+      )
+    }
+  end
+
+  def slack_failed_authentication
+    {ActionDispatch::Flash::KEY => ActionDispatch::Flash::FlashHash.new(
+      {slack_sign_in: {
+        "error" => "invalid_scope_requested"
+      }}
+    )}
+  end
+end
diff --git a/test/support/quote_helpers.rb b/test/support/quote_helpers.rb
new file mode 100644
index 0000000..e2ec2e9
--- /dev/null
+++ b/test/support/quote_helpers.rb
@@ -0,0 +1,8 @@
+module QuotesHelpers
+  def assert_quote_rendered
+    assert_selector "h1.quote"
+    assert has_link?("Random Quote")
+    assert(has_link?("Wayne Quote") || has_link?("Katy Quote"), "Missing random character quote button")
+    assert has_link?("Permalink")
+  end
+end
diff --git a/test/system/nav_bar_test.rb b/test/system/nav_bar_test.rb
new file mode 100644
index 0000000..45af44f
--- /dev/null
+++ b/test/system/nav_bar_test.rb
@@ -0,0 +1,19 @@
+require "application_system_test_case"
+
+class NavBarTest < ApplicationSystemTestCase
+  test "when logged out the sign in with slack button is seen" do
+    visit root_url
+    assert_selector "button.slack-sign-in"
+  end
+
+  test "when logged in can see the admin link" do
+    log_in
+    assert_selector "div.signed-in-user"
+    find("div.signed-in-user").click
+    assert_selector "a.navbar-item", text: "Admin"
+    assert_selector "a.navbar-item", text: "Sign Out"
+    click_on "Sign Out"
+    refute_selector "div.signed-in-user"
+    assert_selector "button.slack-sign-in"
+  end
+end
diff --git a/test/test_helper.rb b/test/test_helper.rb
index 6ca2c48..2024c3b 100644
--- a/test/test_helper.rb
+++ b/test/test_helper.rb
@@ -6,6 +6,8 @@
 require "minitest/autorun"
 require "minitest/pride"
 
+Dir[Rails.root.join("test", "support", "**", "*.rb")].each { |f| require f }
+
 class ActiveSupport::TestCase
   # Run tests in parallel with specified workers
   parallelize(workers: :number_of_processors)
@@ -14,11 +16,6 @@ class ActiveSupport::TestCase
   fixtures :all
 
   # Add more helper methods to be used by all tests here...
-  def assert_quote_rendered
-    assert_selector "h1.quote"
-    assert has_link?("Random Quote")
-    assert(has_link?("Wayne Quote") || has_link?("Katy Quote"), "Missing random character quote button")
-    assert has_link?("Permalink")
-    # assert_match(%r{/quotes/\d+}, current_path, "should redirect to quote_path")
-  end
+  include AuthenticationHelpers
+  include QuotesHelpers
 end