Merge pull request #265 from joyofrails/feat/enhance-search-index

Enhance search index

Author: rossta

app/jobs/pages/search_index_refresh_job.rb

@@ -0,0 +1,7 @@
+module Pages
+  class SearchIndexRefreshJob < ApplicationJob
+    def perform
+      Page.refresh_search_index
+    end
+  end
+end

app/models/page.rb

@@ -30,7 +30,7 @@ def self.search(query)
       .where("pages_search_index MATCH ?", query)
   end
 
-  def self.rebuild_search_index
+  def self.refresh_search_index
     find_each(&:update_in_search_index)
   end
 
@@ -43,7 +43,15 @@ def body
   end
 
   def body_html
-    ApplicationController.render(inline: body, type: resource.handler, layout: false)
+    ApplicationController.render(
+      inline: body,
+      type: (resource.handler.to_sym == :mdrb) ? :"mdrb-atom" : resource.handler,
+      layout: false,
+      content_type: "application/atom+xml",
+      assigns: {
+        format: :atom
+      }
+    )
   end
 
   def body_text

app/views/components/markdown/application.rb

@@ -12,11 +12,13 @@ def default_commonmarker_options
     }
   end
 
-  def code_block(source, language = "", **attributes)
+  def code_block(source, metadata = "", **attributes)
+    language, _ = metadata.split(":", 2)
     render CodeBlock::Code.new(source, language: language, **attributes)
   end
 
   def image(src, alt: "", title: "")
+    title, _ = title.split("|", 2)
     figure do
       image_tag(src, alt: alt, title: title)
       figcaption { title }

lib/tasks/deploy.rake

@@ -0,0 +1,6 @@
+namespace :deploy do
+  desc "Post deploy script"
+  task finish: :environment do
+    Pages::SearchIndexRefreshJob.perform_later
+  end
+end

spec/jobs/pages/search_index_refresh_job_spec.rb

@@ -0,0 +1,11 @@
+require "rails_helper"
+
+RSpec.describe Pages::SearchIndexRefreshJob, type: :job do
+  it "refreshes the search index" do
+    allow(Page).to receive(:refresh_search_index)
+
+    described_class.perform_now
+
+    expect(Page).to have_received(:refresh_search_index)
+  end
+end

spec/models/page_spec.rb

@@ -7,11 +7,11 @@
     expect(page.resource).to eq Sitepress.site.get("/")
   end
 
-  describe ".rebuild_search_index" do
+  describe ".refresh_search_index" do
     it "doesn’t blow up" do
       Page.find_or_create_by!(request_path: "/")
 
-      Page.rebuild_search_index
+      Page.refresh_search_index
     end
   end
 
@@ -25,7 +25,7 @@
     it "works after rebuilding index" do
       page = Page.find_or_create_by!(request_path: "/")
 
-      Page.rebuild_search_index
+      Page.refresh_search_index
 
       expect(Page.search("Joy of Rails")).to include page
     end