Squelch block parameter warning during tests

lib/mogli/model.rb

@@ -3,7 +3,7 @@ class Model < Hashie::Dash
     def client=(val)
       @client=val
     end
-    
+
     def client
       @client || Mogli::Client.new
     end
@@ -12,20 +12,20 @@ def initialize(hash={},client=nil)
       self.client=client
       super(hash||{})
     end
-    
+
     def post_params
       post_params = {}
       self.class.creation_keys.each do |key|
         post_params[key] = self[key]
       end
       post_params
     end
-    
+
     def destroy
       client.delete(id)
       freeze
     end
-    
+
     def self.included(other)
       other.extend(ClassMethods)
     end
@@ -34,48 +34,49 @@ def method_missing(method, *args)
       method_as_s = method.to_s
       if method_as_s.to_s[-1].chr == "="
         warn_about_invalid_property(method_as_s.chop)
-      else 
+      else
         super
       end
     end
     def warn_about_invalid_property(property)
       puts "Warning: property #{property} doesn't exist for class #{self.class.name}"
     end
-          
+
     def self.define_properties(*args)
       args.each do |arg|
         property arg
       end
     end
-    
+
     def self.creation_properties(*args)
       @creation_properties = args
     end
-    
+
     def self.creation_keys
       @creation_properties || []
     end
-    
+
     def self.hash_populating_accessor(method_name,*klass)
       define_method "#{method_name}=" do |hash|
         instance_variable_set("@#{method_name}",client.map_data(hash,klass))
       end
       define_method "#{method_name}" do
         instance_variable_get "@#{method_name}"
       end
-      
+
       add_creation_method(method_name,klass)
-      
+
     end
-    
+
     def self.add_creation_method(name,klass)
-      define_method "#{name}_create" do |arg|
+      define_method "#{name}_create" do |*args|
+        arg = args.first
         params = arg.nil? ? {} : arg.post_params
         klass_to_send = arg.nil? ? nil : klass
         client.post("#{id}/#{name}", klass_to_send, params)
       end
     end
-    
+
     def self.has_association(name,klass)
       define_method name do |*fields|
         body_args = fields.empty? ? {} : {:fields => fields}
@@ -85,23 +86,23 @@ def self.has_association(name,klass)
         end
         return ret
       end
-      
+
       add_creation_method(name,klass)
     end
-    
+
     def fetch()
       raise ArgumentError.new("You cannot fetch models without a populated id attribute") if id.nil?
       other = self.class.find(id,client) 
       merge!(other) if other
     end
-    
+
     def self.recognize?(data)
       true
     end
-    
+
     def self.find(id,client=nil, *fields)
       body_args = fields.empty? ? {} : {:fields => fields}
       (client||Mogli::Client.new).get_and_map(id,self, body_args)
     end
   end
-end
+end

spec/model_spec.rb

@@ -3,31 +3,31 @@ class TestModel < Mogli::Model
   define_properties :id, :other_property
   creation_properties :other_property
   has_association :comments, "Comment"
-  
+
   hash_populating_accessor :from, "User"
   hash_populating_accessor :activities, "Activity"
 end
 
-  
+
 describe Mogli::Model do
-  
+
   let :model do
     model = TestModel.new(:id=>1)
     model.client = mock_client
     model
   end
-  
+
   let :mock_client do
     Mogli::Client.new
   end
-  
-  
+
+
   it "allow setting the client" do
     user = Mogli::Model.new
     user.client = "client"
     user.client.should == "client"
   end
-  
+
   it "has a define proeprties method" do
     model.respond_to?(:id).should be_true
     model.respond_to?(:other_property).should be_true
@@ -43,9 +43,9 @@ class TestModel < Mogli::Model
     model.comments
     model.comments
   end
-  
+
   describe "creation" do
-    
+
     it "has a create method on associations" do
       mock_client.stub!(:post)
       model.comments_create(Mogli::Activity.new)
@@ -55,62 +55,62 @@ class TestModel < Mogli::Model
       mock_client.should_receive(:post).with("1/comments","Comment",:message=>"my message")
       model.comments_create(Mogli::Comment.new(:message=>"my message"))
     end
-  
+
   end
-  
+
   it "populates from as the user from a hash" do
     model.from = {"id" => "12451752", "name" => "Mike Mangino"}
     model.from.should be_an_instance_of(Mogli::User)
     model.from.id.should == "12451752"
     model.from.name.should == "Mike Mangino"
   end
-    
+
   it "populates activities from a hash array" do
     model.activities = {"data"=>[{"id"=>1,"message"=>"message1"},{"id"=>2}]}
     model.activities.size.should == 2
     model.activities.each {|c| c.should be_an_instance_of(Mogli::Activity)}
     model.activities.first.id.should == 1
     model.activities.first.client.should == mock_client
   end
-  
+
   it "allows deleting the post" do
     mock_client.should_receive(:delete).with(1)
     model.destroy
   end
-  
+
   it "freezes deleted objects" do
     mock_client.should_receive(:delete).with(1)
     model.destroy
     model.should be_frozen
   end
-  
+
   it "knows which attributes are posted" do
-    TestModel.new(:id=>1,:other_property=>2).post_params.should == {:other_property=>2}      
+    TestModel.new(:id=>1,:other_property=>2).post_params.should == {:other_property=>2}
   end
-  
+
   it "will allow updating status with no object" do
     mock_client.should_receive(:post).once.with("1/comments",nil,{}).and_return([])
-    model.comments_create 
+    model.comments_create
   end
-  
+
   it "emits warnings when properties that don't exist are written to" do
     model.should_receive(:warn_about_invalid_property).with("doesnt_exist")
     model.doesnt_exist=1
   end
-  
+
   describe "Fetching" do
-  
+
     it "fetches data for a model with an id " do
       Mogli::Client.should_receive(:get).with("https://graph.facebook.com/1", :query=>{}).and_return({:id=>1,:other_property=>2})
       model.fetch
       model.other_property.should == 2
     end
-    
+
     it "raises an exception when there is no id" do
       lambda do
         TestModel.new.fetch
       end.should raise_error(ArgumentError)
     end
   end
-  
-end
+
+end