Improve routing

app/controllers/hotsheet/home_controller.rb

@@ -0,0 +1,9 @@
+# frozen_string_literal: true
+
+class Hotsheet::HomeController < Hotsheet::ApplicationController
+  def show; end
+
+  def error
+    render "error", status: :not_found
+  end
+end

app/controllers/hotsheet/sheets_controller.rb

@@ -18,14 +18,12 @@ def update
     end
   end
 
-  def error
-    render "error", status: :not_found
-  end
-
   private
 
   def set_sheet
     @sheet = Hotsheet.sheets[params[:sheet_name]]
+
+    render "hotsheet/home/error", status: :not_found if @sheet.nil?
   end
 
   def set_column

app/views/hotsheet/shared/_nav.html.erb

@@ -6,7 +6,7 @@
     <% Hotsheet.sheets.each_with_index do |entry, index| %>
       <li>
         <% sheet_name, sheet = entry %>
-        <%= link_to sheet.model.model_name.human(count: 2), sheet_path_for(sheet_name),
+        <%= link_to sheet.model.model_name.human(count: 2), sheets_path(sheet_name),
                     class: ("active" if params[:sheet_name] == sheet_name), data: { x: index } %>
       </li>
     <% end %>

config/routes.rb

@@ -1,12 +1,11 @@
 # frozen_string_literal: true
 
 Hotsheet::Engine.routes.draw do
-  next unless defined? Rails::Server
+  root "home#show"
 
-  Hotsheet.sheets.each_key do |sheet_name|
-    resources sheet_name, sheet_name:, controller: :sheets, only: %i[index update]
+  scope ":sheet_name" do
+    resources controller: :sheets, only: %i[index update], as: :sheets
   end
 
-  root "sheets#root"
-  match "*path", to: "sheets#error", via: :all
+  match "*path", to: "home#error", via: :all
 end

spec/requests/hotsheet/home_controller_spec.rb

@@ -0,0 +1,13 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe Hotsheet::HomeController do
+  describe "#error" do
+    it "shows an error page" do
+      get "/hotsheet/invalid/route"
+      expect(response).to be_not_found
+      expect(response.body).to include I18n.t("hotsheet.errors.not_found")
+    end
+  end
+end

spec/requests/hotsheet/sheets_controller_spec.rb

@@ -11,15 +11,23 @@
   before { prepare { config } }
 
   describe "#index" do
-    it "shows a table with all values" do
-      get hotsheet.users_path
+    it "shows a spreadsheet with all values" do
+      get hotsheet.sheets_path :users
       expect(response).to be_successful
       expect(response.body).to include user.name
     end
+
+    context "when the sheet does not exist" do
+      it "shows an error page" do
+        get "/hotsheet/authors"
+        expect(response).to be_not_found
+        expect(response.body).to include I18n.t("hotsheet.errors.not_found")
+      end
+    end
   end
 
   describe "#update" do
-    let(:path) { hotsheet.user_path(user.id) }
+    let(:path) { hotsheet.sheet_path :users, user }
 
     it "updates the resource" do
       expect { put path, params: { column_name: :name, value: "Bob" } }
@@ -43,7 +51,7 @@
     end
 
     context "with nonexistent id" do
-      let(:path) { hotsheet.user_path(0) }
+      let(:path) { hotsheet.sheet_path :users, 0 }
 
       it "does not update the resource" do
         expect { put path, params: { column_name: :name, value: "Bob" } }.not_to change(user, :reload)
@@ -65,12 +73,4 @@
       end
     end
   end
-
-  describe "#error" do
-    it "shows an error page" do
-      get "/hotsheet/authors"
-      expect(response).to be_not_found
-      expect(response.body).to include I18n.t("hotsheet.errors.not_found")
-    end
-  end
 end

spec/support/utils.rb

@@ -1,8 +1,6 @@
 # frozen_string_literal: true
 
 def prepare(&)
-  stub_const "Rails::Server", true
   Hotsheet.instance_variable_set :@sheets, nil # reset memoized sheets
   yield
-  Rails.application.reload_routes!
 end

spec/system/hotsheet/sheets_controller_spec.rb

@@ -10,7 +10,7 @@
   before { prepare { Hotsheet.configure { sheet :User } } }
 
   it "updates the cell value" do
-    visit hotsheet.users_path
+    visit hotsheet.sheets_path :users
     find(".cell", text: user.name).click.send_keys(:enter).fill_in(with: "Admin").send_keys :enter
 
     expect { find(".cell", text: user.email).click }