Update bad request error, specs

department-of-veterans-affairs · Jan 21, 2025 · edafbb1 · edafbb1
1 parent 0177441
commit edafbb1
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 6 deletions.
diff --git a/...l/app/controllers/concerns/accredited_representative_portal/power_of_attorney_requests.rb b/...l/app/controllers/concerns/accredited_representative_portal/power_of_attorney_requests.rb
@@ -24,9 +24,9 @@ module PowerOfAttorneyRequests
           )
         end
 
-        rescue_from Common::Exceptions::BadRequest do |e|
+        rescue_from ActionController::BadRequest do |e|
           render(
-            json: { errors: e.errors },
+            json: { errors: [e.message] },
             status: :bad_request
           )
         end

diff --git a/...rtal/spec/requests/accredited_representative_portal/v0/power_of_attorney_requests_spec.rb b/...rtal/spec/requests/accredited_representative_portal/v0/power_of_attorney_requests_spec.rb
@@ -67,9 +67,6 @@ def load_response_fixture(path_suffix)
   end
 
   before do
-    AccreditedRepresentativePortal::PowerOfAttorneyForm.delete_all
-    AccreditedRepresentativePortal::PowerOfAttorneyRequestResolution.delete_all
-    AccreditedRepresentativePortal::PowerOfAttorneyRequest.delete_all
     Flipper.enable(:accredited_representative_portal_pilot)
     login_as(test_user)
     travel_to(time)
@@ -207,7 +204,7 @@ def load_response_fixture(path_suffix)
 
       it 'throws an error if any other status filter provided' do
         get('/accredited_representative_portal/v0/power_of_attorney_requests?status=delete_all')
-        expect(response).to have_http_status(:internal_server_error)
+        expect(response).to have_http_status(:bad_request)
       end
     end
   end