Fix URL to delete coupon redemptions (#137)

Author: everdance

mock/redemptions.go

@@ -22,7 +22,7 @@ type RedemptionsService struct {
 	OnRedeem      func(ctx context.Context, code string, r recurly.CouponRedemption) (*recurly.Redemption, error)
 	RedeemInvoked bool
 
-	OnDelete      func(ctx context.Context, accountCode string) error
+	OnDelete      func(ctx context.Context, accountCode, couponCode string) error
 	DeleteInvoked bool
 }
 
@@ -46,7 +46,7 @@ func (m *RedemptionsService) Redeem(ctx context.Context, code string, r recurly.
 	return m.OnRedeem(ctx, code, r)
 }
 
-func (m *RedemptionsService) Delete(ctx context.Context, accountCode string) error {
+func (m *RedemptionsService) Delete(ctx context.Context, accountCode, couponCode string) error {
 	m.DeleteInvoked = true
-	return m.OnDelete(ctx, accountCode)
+	return m.OnDelete(ctx, accountCode, couponCode)
 }

redemptions.go

@@ -43,7 +43,7 @@ type RedemptionsService interface {
 	// of the coupon. See Recurly's documentation for details.
 	//
 	// https://dev.recurly.com/docs/remove-a-coupon-from-an-account
-	Delete(ctx context.Context, accountCode string) error
+	Delete(ctx context.Context, accountCode, couponCode string) error
 }
 
 // Redemptions constants.
@@ -148,8 +148,8 @@ func (s *redemptionsImpl) Redeem(ctx context.Context, code string, r CouponRedem
 	return &dst, nil
 }
 
-func (s *redemptionsImpl) Delete(ctx context.Context, accountCode string) error {
-	path := fmt.Sprintf("/accounts/%s/redemption", accountCode)
+func (s *redemptionsImpl) Delete(ctx context.Context, accountCode, couponCode string) error {
+	path := fmt.Sprintf("/accounts/%s/redemptions/%s", accountCode, couponCode)
 	req, err := s.client.newRequest("DELETE", path, nil)
 	if err != nil {
 		return err

redemptions_test.go

@@ -156,11 +156,11 @@ func TestRedemptions_Delete(t *testing.T) {
 	client, s := recurly.NewTestServer()
 	defer s.Close()
 
-	s.HandleFunc("DELETE", "/v2/accounts/1/redemption", func(w http.ResponseWriter, r *http.Request) {
+	s.HandleFunc("DELETE", "/v2/accounts/1/redemptions/30_off", func(w http.ResponseWriter, r *http.Request) {
 		w.WriteHeader(http.StatusNoContent)
 	}, t)
 
-	if err := client.Redemptions.Delete(context.Background(), "1"); !s.Invoked {
+	if err := client.Redemptions.Delete(context.Background(), "1", "30_off"); !s.Invoked {
 		t.Fatal("expected fn invocation")
 	} else if err != nil {
 		t.Fatal(err)