Merge pull request #6752 from costajohnt/fix/error-page-button

fix: require button click to trigger intentional test exception

Author: FireLemons

app/controllers/error_controller.rb

@@ -2,8 +2,12 @@
 
 class ErrorController < ApplicationController
   skip_before_action :authenticate_user!
+  skip_after_action :verify_authorized
 
   def index
+  end
+
+  def create
     raise StandardError.new "This is an intentional test exception"
   end
 end

app/views/error/index.html.erb

@@ -1,6 +1,7 @@
 <div class="card text-center">
   <div class="card-body">
-    <h1 class="card-title" style="color:red">We're sorry, but something went wrong.</h1>
-    <p class="card-text" style="color:grey">If you are the application owner check the logs for more information.</p>
+    <h1 class="card-title">Intentional Error Test</h1>
+    <p class="card-text" style="color:grey">Click the button below to trigger an intentional test exception.</p>
+    <%= button_to "Trigger Exception", error_path, method: :post, class: "btn btn-danger", data: { turbo: false } %>
   </div>
 </div>

config/routes.rb

@@ -241,6 +241,7 @@
   end
 
   get "/error", to: "error#index"
+  post "/error", to: "error#create"
 
   namespace :api do
     namespace :v1 do

spec/requests/error_spec.rb

@@ -1,9 +1,19 @@
 require "rails_helper"
 
 RSpec.describe "/error", type: :request do
-  it "raises an error causing an internal server error" do
-    expect {
+  describe "GET /error" do
+    it "renders the error test page" do
       get error_path
-    }.to raise_error(StandardError, /This is an intentional test exception/)
+
+      expect(response).to be_successful
+    end
+  end
+
+  describe "POST /error" do
+    it "raises an error causing an internal server error" do
+      expect {
+        post error_path
+      }.to raise_error(StandardError, /This is an intentional test exception/)
+    end
   end
 end