Skip to content

Commit 67186f3

Browse files
committed
Add FIXME for coupon spec failure
Ran out of time so added this to fix next time.
1 parent 8252473 commit 67186f3

File tree

1 file changed

+4
-0
lines changed
  • promotions/spec/models/solidus_promotions/promotion_handler

1 file changed

+4
-0
lines changed

promotions/spec/models/solidus_promotions/promotion_handler/coupon_spec.rb

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -400,6 +400,10 @@ def expect_adjustment_creation(adjustable:, promotion:)
400400
end
401401

402402
it "successfully removes the coupon code from the order" do
403+
# FIXME:
404+
# If we call `order.recalcualte` the test will pass.
405+
# But we think there are better ways to test this assertion, such as
406+
# that the adjustments have changed.
403407
subject.remove
404408
expect(subject.error).to eq nil
405409
expect(subject.success).to eq "The coupon code was successfully removed from this order."

0 commit comments

Comments
 (0)