From 1fd3510b32cee73b376efb3d316673a6c70f9475 Mon Sep 17 00:00:00 2001 From: Claire Date: Tue, 5 Aug 2025 11:42:58 +0200 Subject: [PATCH] Change the quote revocation REST API endpoint to return updated quote post (#35677) --- app/controllers/api/v1/statuses/quotes_controller.rb | 2 +- spec/requests/api/v1/statuses/quotes_spec.rb | 6 ++++++ 2 files changed, 7 insertions(+), 1 deletion(-) diff --git a/app/controllers/api/v1/statuses/quotes_controller.rb b/app/controllers/api/v1/statuses/quotes_controller.rb index 7dd91e9a2ee..962855884ec 100644 --- a/app/controllers/api/v1/statuses/quotes_controller.rb +++ b/app/controllers/api/v1/statuses/quotes_controller.rb @@ -19,7 +19,7 @@ class Api::V1::Statuses::QuotesController < Api::V1::Statuses::BaseController RevokeQuoteService.new.call(@quote) - render_empty # TODO: do we want to return something? an updated status? + render json: @quote.status, serializer: REST::StatusSerializer end private diff --git a/spec/requests/api/v1/statuses/quotes_spec.rb b/spec/requests/api/v1/statuses/quotes_spec.rb index bbf697ce323..9456556ce99 100644 --- a/spec/requests/api/v1/statuses/quotes_spec.rb +++ b/spec/requests/api/v1/statuses/quotes_spec.rb @@ -108,6 +108,12 @@ RSpec.describe 'API V1 Statuses Quotes' do expect(response) .to have_http_status(200) + expect(response.content_type) + .to start_with('application/json') + expect(response.parsed_body) + .to match( + a_hash_including(id: quote.status.id.to_s, quote: a_hash_including(state: 'revoked')) + ) end end