only return supplier id on destroy

paviliondev · Sep 17, 2024 · c5b8b08 · c5b8b08
1 parent e5bf5f8
commit c5b8b08
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/app/controllers/discourse_subscription_client/suppliers_controller.rb b/app/controllers/discourse_subscription_client/suppliers_controller.rb
@@ -47,7 +47,7 @@ def authorize_callback
 
     def destroy
       if @supplier.destroy_authorization
-        render json: success_json.merge(supplier: @supplier.reload)
+        render json: success_json.merge(supplier_id: @supplier.id)
       else
         render json: failed_json
       end

diff --git a/spec/requests/discourse_subscription_client/suppliers_controller_spec.rb b/spec/requests/discourse_subscription_client/suppliers_controller_spec.rb
@@ -84,6 +84,7 @@
 
       delete "/admin/plugins/subscription-client/suppliers/authorize", params: { supplier_id: supplier.id }
       expect(response.status).to eq(200)
+      expect(response.parsed_body["supplier_id"]).to eq(supplier.id)
       expect(supplier.authorized?).to eq(false)
     end
   end