survey-gizmo-ruby 0.6.0 → 0.7.0

data/README.md

@@ -28,7 +28,7 @@ Integrate with the [Survey Gizmo API](http://developer.surveygizmo.com/resources
 	
 ## Adding API Objects
 
-Currently, the following API objects are included in the gem: `Survey`, `Question`, `Option`, `Page`. If you want to use something that isn't included you can easily write a class that handles it. Here's an example of the `SurveyGizmo::API::Survey` class:
+Currently, the following API objects are included in the gem: `Survey`, `Question`, `Option`, `Page`, `Response`. If you want to use something that isn't included you can easily write a class that handles it. Here's an example of the `SurveyGizmo::API::Survey` class:
 
 	class Survey
 	  # the base where most of the methods for handling the API are stored

data/VERSION

@@ -1 +1 @@
-0.6.0
+0.7.0

data/lib/survey_gizmo/resource.rb

@@ -52,6 +52,14 @@ module SurveyGizmo
         resource
       end
       
+      # Deleted the Resource from Survey Gizmo
+      # @param [Hash] conditions
+      # @return [Boolean]
+      def destroy(conditions)
+        response = Response.new SurveyGizmo.delete(handle_route(:delete, conditions))
+        response.ok?
+      end
+      
       # Define the path where a resource is located
       # @param [String] path 
       #   the path in Survey Gizmo for the resource
@@ -103,7 +111,9 @@ module SurveyGizmo
         path = @paths[key]
         raise "No routes defined for `#{key}` in #{self.name}" unless path
         options = interp.last.is_a?(Hash) ? interp.pop : path.scan(/:(\w+)/).inject({}){|hash, k| hash.merge(k.to_sym => interp.shift) }
-        path.gsub(/:(\w+)/){|m| options[$1.to_sym] }
+        path.gsub(/:(\w+)/) do |m| 
+          options[$1.to_sym].tap{ |result| raise(SurveyGizmo::URLError, "Missing parameters in request: `#{m}`") unless result }
+        end
       end
     end
     

data/lib/survey_gizmo/survey_gizmo.rb

@@ -20,6 +20,8 @@ module SurveyGizmo
   
   format :json
   
+  URLError = Class.new(RuntimeError)
+  
   # The base uri for this version of the API is $1
   base_uri 'https://restapi.surveygizmo.com/v1'
   

data/spec/resource_spec.rb

@@ -11,7 +11,7 @@ describe "Survey Gizmo Resource" do
     let(:create_attributes){ {:title => 'Spec', :test_id => 5} }
     let(:get_attributes)   { create_attributes.merge(:id => 1) }
     let(:update_attributes){ {:title => 'Updated'} }
-    let(:first_params){ {:id => 1} }
+    let(:first_params){ {:id => 1, :test_id => 5} }
     let(:uri_paths){ 
       { 
         :get => '/test/1',
@@ -28,18 +28,20 @@ describe "Survey Gizmo Resource" do
 
     it '#reload' do
       stub_request(:get, /#{@base}/).to_return(json_response(true, get_attributes))
-      obj = described_class.new(create_attributes)
-      obj.attributes.reject{|k,v| v.blank? }.should == create_attributes
+      obj = described_class.new(get_attributes.merge(update_attributes))
+      obj.attributes.reject{|k,v| v.blank? }.should == get_attributes.merge(update_attributes)
       obj.reload
       obj.attributes.reject{|k,v| v.blank? }.should == get_attributes
     end
 
     it '#valid?'
-  
-    it "should track descendants" do
-      SurveyGizmo::Resource.descendants.should include(SurveyGizmoSpec::ResourceTest)
-    end
       
+    it "should raise an error if params are missing" do
+      lambda {
+        SurveyGizmoSpec::ResourceTest.destroy(:test_id => 5)
+      }.should raise_error(SurveyGizmo::URLError, 'Missing parameters in request: `:id`')
+    end
+    
     it_should_behave_like 'an API object'
     it_should_behave_like 'an object with errors'
   end
@@ -65,7 +67,7 @@ describe "Survey Gizmo Resource" do
       create_attributes.merge(:id => 1)
     }
     let(:update_attributes){ {:survey_id => 1234, :page_id => 1, :title => 'Updated'} }
-    let(:first_params){ {:id => 1, :survey_id => 1234} }
+    let(:first_params){ {:id => 1, :survey_id => 1234, :page_id => 1} }
     let(:uri_paths){ 
       { :get =>    '/survey/1234/surveyquestion/1',
         :create => '/survey/1234/surveypage/1/surveyquestion',

data/spec/support/spec_shared_api_object.rb

@@ -3,7 +3,12 @@ shared_examples_for 'an API object' do
     SurveyGizmo.setup(:user => 'test@test.com', :password => 'password')
   end
   
-  context "create" do
+  it "should be descendant of SurveyGizmo::Resource" do
+    SurveyGizmo::Resource.descendants.should include(described_class)
+  end
+  
+  
+  context "#create" do
     it "should make a request" do
       stub_api_call(:put)
       described_class.create(create_attributes)
@@ -23,7 +28,7 @@ shared_examples_for 'an API object' do
     end
   end
   
-  context "get" do
+  context "#get" do
     it "should make a request" do
       stub_request(:get, /#{@base}/).to_return(json_response(true, get_attributes))
       described_class.first(first_params)
@@ -42,7 +47,7 @@ shared_examples_for 'an API object' do
     end
   end
   
-  context "update" do
+  context "instance#update" do
     before(:each) do
       @obj = described_class.new(get_attributes)
       @obj.__send__(:clean!)
@@ -73,7 +78,7 @@ shared_examples_for 'an API object' do
     
   end
   
-  context "destroy" do
+  context "instance#destroy" do
     before(:each) do
       @obj = described_class.new(get_attributes)
       @obj.__send__(:clean!)
@@ -103,7 +108,20 @@ shared_examples_for 'an API object' do
     end
   end
   
-  context '#save' do
+  context '#destroy', :focused => true do
+    it "should make a request" do
+      stub_api_call(:delete)
+      described_class.destroy(first_params)
+      a_request(:delete, /#{@base}#{uri_paths[:delete]}/).should have_been_made
+    end
+    
+    it "should return result" do
+      stub_api_call(:delete)
+      described_class.destroy(first_params).should be_true
+    end
+  end
+  
+  context 'instance#save' do
     it "should call create on a new resource" do
       stub_api_call(:put)
       obj = described_class.new(create_attributes)

data/spec/support/spec_shared_object_with_errors.rb

@@ -8,7 +8,7 @@ shared_examples_for 'an object with errors' do
 
   context "class methods" do
     it { described_class.first(get_attributes).should be_nil }
-    it { described_class.all(get_attributes).should be_empty }
+    it { described_class.all(get_attributes).should be_empty }    
   end
 
   context "instance methods" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: survey-gizmo-ruby
 version: !ruby/object:Gem::Version
-  version: 0.6.0
+  version: 0.7.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,11 +9,11 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2011-12-13 00:00:00.000000000Z
+date: 2012-03-02 00:00:00.000000000Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
-  requirement: &70195630027920 !ruby/object:Gem::Requirement
+  requirement: &70321870190520 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -21,10 +21,10 @@ dependencies:
         version: 3.0.1
   type: :runtime
   prerelease: false
-  version_requirements: *70195630027920
+  version_requirements: *70321870190520
 - !ruby/object:Gem::Dependency
   name: i18n
-  requirement: &70195630026500 !ruby/object:Gem::Requirement
+  requirement: &70321870172640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -32,10 +32,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70195630026500
+  version_requirements: *70321870172640
 - !ruby/object:Gem::Dependency
   name: virtus
-  requirement: &70195630025000 !ruby/object:Gem::Requirement
+  requirement: &70321870170640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -43,10 +43,10 @@ dependencies:
         version: 0.0.5
   type: :runtime
   prerelease: false
-  version_requirements: *70195630025000
+  version_requirements: *70321870170640
 - !ruby/object:Gem::Dependency
   name: httparty
-  requirement: &70195630006520 !ruby/object:Gem::Requirement
+  requirement: &70321870169380 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -54,10 +54,10 @@ dependencies:
         version: '0'
   type: :runtime
   prerelease: false
-  version_requirements: *70195630006520
+  version_requirements: *70321870169380
 - !ruby/object:Gem::Dependency
   name: bluecloth
-  requirement: &70195630005200 !ruby/object:Gem::Requirement
+  requirement: &70321870167280 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ! '>='
@@ -65,10 +65,10 @@ dependencies:
         version: '0'
   type: :development
   prerelease: false
-  version_requirements: *70195630005200
+  version_requirements: *70321870167280
 - !ruby/object:Gem::Dependency
   name: yard
-  requirement: &70195630002980 !ruby/object:Gem::Requirement
+  requirement: &70321870160460 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -76,10 +76,10 @@ dependencies:
         version: 0.7.0
   type: :development
   prerelease: false
-  version_requirements: *70195630002980
+  version_requirements: *70321870160460
 - !ruby/object:Gem::Dependency
   name: bundler
-  requirement: &70195629996380 !ruby/object:Gem::Requirement
+  requirement: &70321870158060 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -87,10 +87,10 @@ dependencies:
         version: 1.0.0
   type: :development
   prerelease: false
-  version_requirements: *70195629996380
+  version_requirements: *70321870158060
 - !ruby/object:Gem::Dependency
   name: jeweler
-  requirement: &70195629994020 !ruby/object:Gem::Requirement
+  requirement: &70321870156640 !ruby/object:Gem::Requirement
     none: false
     requirements:
     - - ~>
@@ -98,7 +98,7 @@ dependencies:
         version: 1.6.4
   type: :development
   prerelease: false
-  version_requirements: *70195629994020
+  version_requirements: *70321870156640
 description: ''
 email: self@ripthejacker.com
 executables: []
@@ -148,7 +148,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: -3511147689201295286
+      hash: 746384856005912275
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements: