Revert "Updated test to no longer expect create to use turbo, forgot to set http status"

Benjamin-Couey · Benjamin-Couey · commit 7a4332405cf2 · 2025-02-05T06:50:39.000-07:00
This reverts commit 0204a22.
diff --git a/app/controllers/distributions_controller.rb b/app/controllers/distributions_controller.rb
@@ -7,7 +7,7 @@ class DistributionsController < ApplicationController
   include DateRangeHelper
   include DistributionHelper
 
-  # before_action :enable_turbo!, only: %i[show]
+  before_action :enable_turbo!, only: %i[show]
   skip_before_action :authenticate_user!, only: %i(calendar)
   skip_before_action :authorize_user, only: %i(calendar)
 
@@ -129,7 +129,7 @@ def create
 
       flash_error = insufficient_error_message(result.error.message)
       flash.now[:error] = flash_error
-      render :new, status: :bad_request
+      render :new
     end
   end
 
diff --git a/spec/controllers/distributions_controller_spec.rb b/spec/controllers/distributions_controller_spec.rb
@@ -219,7 +219,7 @@
             }
           }
         end
-        subject { post :create, params: params }
+        subject { post :create, params: params.merge(format: :turbo_stream) }
 
         it "flashes an error" do
           expect(subject).to have_http_status(:bad_request)

Original file line number	Diff line number	Diff line change
`@@ -219,7 +219,7 @@`
`219`	`219`	`}`
`220`	`220`	`}`
`221`	`221`	`end`
`222`		`- subject { post :create, params: params }`
	`222`	`+ subject { post :create, params: params.merge(format: :turbo_stream) }`
`223`	`223`
`224`	`224`	`it "flashes an error" do`
`225`	`225`	`expect(subject).to have_http_status(:bad_request)`