Merge pull request #144 from ryanhertz/destroyable

added destroy class method to Destroyable

Author: ryanhertz

CHANGELOG

@@ -1,3 +1,6 @@
+v1.4.20
+ - added destroy class method to Destroyable
+
 v1.4.19
  - create SparkQLErrors attribute for SparkApi::Response and populate when present
  - create Errors attribute for SparkApi::Response and populate when present

VERSION

@@ -1 +1 @@
-1.4.19
+1.4.20

lib/spark_api/models/concerns/destroyable.rb

@@ -4,6 +4,19 @@ module Concerns
 
       module Destroyable
 
+        def self.included(base)
+          base.extend(ClassMethods)
+        end
+
+        module ClassMethods
+
+          def destroy(id, arguments = {})
+            connection.delete("#{path}/#{id}", arguments)
+          end
+
+        end
+
+
         def destroy(arguments = {})
           self.errors = []
           begin

spec/unit/spark_api/models/concerns/destroyable_spec.rb

@@ -14,15 +14,32 @@ class MyExampleModel < Base
     @model = MyExampleModel.first
   end
 
-  it "should not be destroyed" do
-    @model.destroyed?.should eq(false)
+  describe 'destroyed?' do
+    
+    it "should not be destroyed" do
+      @model.destroyed?.should eq(false)
+    end
   end
 
-  it "should be destroyable" do
-    stub_api_delete("/some/place/20101230223226074201000000")
-    @model = MyExampleModel.first
-    @model.destroy
-    @model.destroyed?.should eq(true)
+  describe 'destroy' do
+
+    it "should be destroyable" do
+      stub_api_delete("/some/place/20101230223226074201000000")
+      @model = MyExampleModel.first
+      @model.destroy
+      @model.destroyed?.should eq(true)
+    end
+
+  end
+
+  describe 'destroy class method' do
+
+    it "allows you to destroy with only the id" do
+      stub_api_delete("/test/example/20101230223226074201000000")
+      MyExampleModel.destroy('20101230223226074201000000')
+      expect_api_request(:delete, "/test/example/20101230223226074201000000").to have_been_made.once
+    end
+
   end
 
 end