From 411c1245961869701f1dbfa4baccead3a2b2fd76 Mon Sep 17 00:00:00 2001 From: Michael Crumm Date: Fri, 31 Aug 2018 15:17:35 -0700 Subject: [PATCH] Fix return values for delete calls in tests --- test/stripe/connect/account_test.exs | 3 +-- test/stripe/connect/recipient_test.exs | 3 +-- test/stripe/core_resources/customer_test.exs | 5 ++--- test/stripe/relay/sku_test.exs | 5 ++--- test/stripe/subscriptions/coupon_test.exs | 5 ++--- test/stripe/subscriptions/plan_test.exs | 3 +-- test/stripe/subscriptions/subscription_item_test.exs | 3 +-- 7 files changed, 10 insertions(+), 17 deletions(-) diff --git a/test/stripe/connect/account_test.exs b/test/stripe/connect/account_test.exs index d96831df..f4d4a59a 100644 --- a/test/stripe/connect/account_test.exs +++ b/test/stripe/connect/account_test.exs @@ -24,9 +24,8 @@ defmodule Stripe.AccountTest do end test "is deletable" do - assert {:ok, %{deleted: deleted, id: _id}} = Stripe.Account.delete("acct_123") + assert {:ok, %Stripe.Account{}} = Stripe.Account.delete("acct_123") assert_stripe_requested(:delete, "/v1/accounts/acct_123") - assert deleted == true end test "is listable" do diff --git a/test/stripe/connect/recipient_test.exs b/test/stripe/connect/recipient_test.exs index 2cc51060..a232e437 100644 --- a/test/stripe/connect/recipient_test.exs +++ b/test/stripe/connect/recipient_test.exs @@ -19,9 +19,8 @@ defmodule Stripe.RecipientTest do end test "is deletable" do - assert {:ok, %{deleted: deleted, id: _id}} = Stripe.Recipient.delete("recip_123") + assert {:ok, %Stripe.Recipient{}} = Stripe.Recipient.delete("recip_123") assert_stripe_requested(:delete, "/v1/recipients/recip_123") - assert deleted == true end test "is listable" do diff --git a/test/stripe/core_resources/customer_test.exs b/test/stripe/core_resources/customer_test.exs index ee79cf25..0354d980 100644 --- a/test/stripe/core_resources/customer_test.exs +++ b/test/stripe/core_resources/customer_test.exs @@ -17,11 +17,10 @@ defmodule Stripe.CustomerTest do assert_stripe_requested(:post, "/v1/customers/cus_123") end - test "is deleteable" do + test "is deleteble" do {:ok, customer} = Stripe.Customer.retrieve("cus_123") - assert {:ok, %{deleted: deleted, id: _id}} = Stripe.Customer.delete(customer) + assert {:ok, %Stripe.Customer{}} = Stripe.Customer.delete(customer) assert_stripe_requested(:delete, "/v1/customers/#{customer.id}") - assert deleted === true end test "is listable" do diff --git a/test/stripe/relay/sku_test.exs b/test/stripe/relay/sku_test.exs index 909e73cb..eb4639de 100644 --- a/test/stripe/relay/sku_test.exs +++ b/test/stripe/relay/sku_test.exs @@ -24,10 +24,9 @@ defmodule Stripe.SkuTest do assert_stripe_requested(:post, "/v1/skus/sku_123") end - test "is deleteable" do - assert {:ok, %{deleted: deleted, id: _id}} = Stripe.Sku.delete("sku_123") + test "is deletable" do + assert {:ok, %Stripe.Sku{}} = Stripe.Sku.delete("sku_123") assert_stripe_requested(:delete, "/v1/skus/sku_123/delete") - assert deleted end test "is listable" do diff --git a/test/stripe/subscriptions/coupon_test.exs b/test/stripe/subscriptions/coupon_test.exs index ad794d7f..8c144981 100644 --- a/test/stripe/subscriptions/coupon_test.exs +++ b/test/stripe/subscriptions/coupon_test.exs @@ -24,9 +24,8 @@ defmodule Stripe.CouponTest do assert_stripe_requested(:post, "/v1/coupons/25OFF") end - test "is deleteable" do - assert {:ok, %{deleted: deleted, id: _id}} = Stripe.Coupon.delete("25OFF") + test "is deletable" do + assert {:ok, %Stripe.Coupon{}} = Stripe.Coupon.delete("25OFF") assert_stripe_requested(:delete, "/v1/coupons/25OFF") - assert deleted == true end end diff --git a/test/stripe/subscriptions/plan_test.exs b/test/stripe/subscriptions/plan_test.exs index b317211d..c848c92e 100644 --- a/test/stripe/subscriptions/plan_test.exs +++ b/test/stripe/subscriptions/plan_test.exs @@ -52,9 +52,8 @@ defmodule Stripe.PlanTest do describe "delete/2" do test "deletes a Plan" do {:ok, plan} = Stripe.Plan.retrieve("sapphire-elite") - assert {:ok, %{deleted: deleted, id: _id}} = Stripe.Plan.delete(plan) + assert {:ok, %Stripe.Plan{}} = Stripe.Plan.delete(plan) assert_stripe_requested(:delete, "/v1/plans/#{plan.id}") - assert deleted === true end end diff --git a/test/stripe/subscriptions/subscription_item_test.exs b/test/stripe/subscriptions/subscription_item_test.exs index f39b7d62..675e1752 100644 --- a/test/stripe/subscriptions/subscription_item_test.exs +++ b/test/stripe/subscriptions/subscription_item_test.exs @@ -30,9 +30,8 @@ defmodule Stripe.SubscriptionItemTest do describe "delete/2" do test "deletes a subscription" do {:ok, subscription_item} = Stripe.SubscriptionItem.retrieve("sub_123") - assert {:ok, %{deleted: deleted, id: _id}} = Stripe.SubscriptionItem.delete("sub_123") + assert {:ok, %Stripe.SubscriptionItem{}} = Stripe.SubscriptionItem.delete("sub_123") assert_stripe_requested(:delete, "/v1/subscriptions/#{subscription_item.id}") - assert deleted === true end end