raggedstaff/openfoodnetwork
1
0
Fork 0
mirror of https://github.com/openfoodfoundation/openfoodnetwork synced 2026-02-06 22:36:07 +00:00
Code Issues Packages Projects Releases Wiki Activity

Remove duplicate specs

Browse Source 
The second of these is flaky; moreover, the OrdersController doesn't specifically have a method that they're testing. I think the ready_to_capture behaviors is adequately covered by 006236b8dd/spec/serializers/api/admin/order_serializer_spec.rb (L39-L68)
This commit is contained in:
Andy Brett
2021-02-27 15:05:11 -08:00
parent 006236b8dd
commit 448c0e0ce3
1 changed files with 0 additions and 41 deletions
Download Patch File Download Diff File Expand all files Collapse all files

41
spec/controllers/api/orders_controller_spec.rb
View File

@@ -157,47 +157,6 @@ module Api
expect(json_response['pagination']).to eq pagination_data
end
end
context "when there is a pending payment requiring authorization" do
let!(:pending_payment) do
create(
:payment,
order: order1,
state: 'pending',
amount: 123.45,
cvv_response_message: "https://stripe.com/redirect"
)
end
before do
allow(controller).to receive(:spree_current_user) { distributor.owner }
end
it "returns false" do
get :index
expect(json_response['orders'].first['ready_to_capture']).to eq(false)
end
end
context "when there is a pending payment but it does not require authorization" do
let!(:pending_payment) do
create(
:payment,
order: order1,
state: 'pending',
amount: 123.45,
)
end
before do
allow(controller).to receive(:spree_current_user) { distributor.owner }
end
it "returns true" do
get :index
expect(json_response['orders'].first['ready_to_capture']).to eq(true)
end
end
end
describe "#show" do