added update_attributes to savable and #favorite? to saved_search

Author: ryanhertz

lib/spark_api/models/concerns/savable.rb

@@ -22,12 +22,17 @@ def save!(arguments = {})
 
         def create!(arguments = {})
           results = connection.post self.path, post_data.merge(params_for_save), arguments
-          update_attributes(results.first)
+          update_attributes_after_create(results.first)
           reset_dirty
           params_for_save.clear
           true
         end
 
+        def update_attributes(attrs = {})
+          attrs.each{|k,v| public_send(:"#{k}=", v)}
+          save!
+        end
+
         def update!(arguments = {})
           return true unless changed? && persisted?
           connection.put resource_uri, dirty_attributes, arguments
@@ -48,7 +53,7 @@ def post_data
 
         private 
 
-        def update_attributes(result)
+        def update_attributes_after_create(result)
           attributes['Id'] = result['Id'] ? result['Id'] : parse_id(result['ResourceUri'])
           result.delete('Id')
           attributes.merge! result

lib/spark_api/models/saved_search.rb

@@ -35,6 +35,10 @@ def self.tagged(tag, arguments={})
         collect(connection.get("/#{self.element_name}/tags/#{tag}", arguments))
       end
 
+      def favorite?
+        @attributes["Tags"] && @attributes["Tags"].include?( "Favorites")
+      end
+
       # list contacts (private role)
       def contacts
         return [] unless persisted?

spec/unit/spark_api/models/saved_search_spec.rb

@@ -165,6 +165,18 @@
     end
   end
 
+  describe "favorite?" do
+    it "should return true if the search has been tagged as a favorite" do
+      search = SavedSearch.new(Tags: ["Favorites"])
+      search.should be_favorite
+    end
+
+    it "should return false if the search has not been tagged as a favorite" do
+      search = SavedSearch.new
+      search.should_not be_favorite
+    end
+  end
+
   describe "can_have_newsfeed?" do
 
     it "should return false without at least three filter parameters" do