Merge pull request #368 from stripe/rasmus-support_product_sku_deletion

Support deleting SKUs and products.

lib/stripe/product.rb

@@ -3,6 +3,7 @@ module Stripe
     extend Stripe::APIOperations::List
     extend Stripe::APIOperations::Create
     include Stripe::APIOperations::Update
+    include Stripe::APIOperations::Delete
 
     # Keep APIResource#url as `api_url` to avoid letting the external URL
     # replace the Stripe URL.

lib/stripe/sku.rb

@@ -3,6 +3,7 @@ module Stripe
     extend Stripe::APIOperations::List
     extend Stripe::APIOperations::Create
     include Stripe::APIOperations::Update
+    include Stripe::APIOperations::Delete
 
   end
 end

test/stripe/product_test.rb

@@ -11,12 +11,14 @@ module Stripe
       end
     end
 
-    should "products should not be deletable" do
-      assert_raises NoMethodError do
-        @mock.expects(:get).once.returns(make_response(make_product))
-        p = Stripe::Product.retrieve("test_product")
-        p.delete
-      end
+    should "products should be deletable" do
+      @mock.expects(:get).once.returns(make_response(make_product))
+      @mock.expects(:delete).once.
+        returns(make_response(make_product(:deleted => true)))
+
+      p = Stripe::Product.retrieve("test_product")
+      p.delete
+      assert p.deleted
     end
 
     should "products should be updateable" do

test/stripe/sku_test.rb

@@ -12,12 +12,13 @@ module Stripe
       end
     end
 
-    should "SKUs should not be deletable" do
-      assert_raises NoMethodError do
-        @mock.expects(:get).once.returns(make_response(make_sku))
-        p = Stripe::SKU.retrieve("test_product")
-        p.delete
-      end
+    should "SKUs should be deletable" do
+      @mock.expects(:get).once.returns(make_response(make_sku))
+      @mock.expects(:delete).once.returns(make_response(make_sku(:deleted => true)))
+
+      s = Stripe::SKU.retrieve("test_sku")
+      s.delete
+      assert s.deleted
     end
 
   end