Add updated date to page header

Add spec for page header component

Author: rossta

app/content/layouts/article.html.erb

@@ -4,7 +4,8 @@
     <%= render Pages::Header.new(
       title: current_page.data.title!,
       description: current_page.data.description,
-      published_on: current_page.data.published&.to_date
+      published_on: current_page.data.published&.to_date,
+      updated_on: current_page.data.updated&.to_date
     ) %>
     <%- if current_page.data.toc -%>
       <aside>

app/views/components/pages/header.rb

@@ -4,10 +4,11 @@ class Pages::Header < ApplicationComponent
 
   attr_accessor :current_page
 
-  def initialize(title: nil, description: nil, published_on: nil)
+  def initialize(title: nil, description: nil, published_on: nil, updated_on: nil)
     @title = title
     @description = description
     @published_on = published_on
+    @updated_on = updated_on
   end
 
   def view_template
@@ -19,11 +20,27 @@ def view_template
           h1 { @title }
         end
         p(class: "description") { @description } if @description
-        if @published_on
+        if @published_on || @updated_on
           span(class: "block") do
-            # <time datetime="2024-03-13T00:00:00Z" itemprop="datePublished" class="dt-published"> March 13th, 2024 </time>
-            em do
-              time_tag @published_on, itemprop: "datePublished", class: "dt-published"
+            if @published_on && @updated_on
+              plain "Published:"
+              whitespace
+            end
+            if @published_on
+              em do
+                time_tag @published_on, itemprop: "datePublished", class: "dt-published"
+              end
+            end
+            if @updated_on
+              if @published_on
+                plain " // "
+              end
+              plain "Updated:"
+              whitespace
+
+              em do
+                time_tag @updated_on, itemprop: "dateModified", class: "dt-modified"
+              end
             end
           end
         end

spec/views/components/pages/header_spec.rb

@@ -0,0 +1,31 @@
+require "rails_helper"
+
+RSpec.describe Pages::Header, type: :view do
+  def render(**kwargs)
+    described_class.new(**kwargs).call(view_context: view)
+  end
+
+  describe "#call" do
+    it "renders the header" do
+      expect(render(title: "Hello")).to have_css(".page-header h1", text: "Hello")
+    end
+
+    it "renders the description" do
+      expect(render(description: "Describe me")).to have_css(".page-header p", text: "Describe me")
+    end
+
+    it "renders the published date" do
+      expect(render(published_on: Date.today)).to have_css("time.dt-published", text: Date.today.to_fs(:long))
+    end
+
+    it "renders the updated date" do
+      expect(render(updated_on: Date.today)).to have_css("time.dt-modified", text: Date.today.to_fs(:long))
+    end
+
+    it "renders the published and updated date" do
+      rendered = render(published_on: Date.yesterday, updated_on: Date.today)
+      expect(rendered).to have_css("time.dt-published", text: Date.yesterday.to_fs(:long))
+      expect(rendered).to have_css("time.dt-modified", text: Date.today.to_fs(:long))
+    end
+  end
+end