@@ -10,7 +10,7 @@ module Spec # add_stubs) if +stubs+ is passed. def mock_model(model_class, options_and_stubs = {}) id = options_and_stubs[:id] || next_id - options_and_stubs.reverse_merge!({ + options_and_stubs = options_and_stubs.reverse_merge({ :id => id, :to_param => id.to_s, :new_record? => false, lib/spec/rails/mocks.rb @@ -25,6 +25,14 @@ describe "mock_model" do end end +describe "mock_mode with params" do + it "should not mutate its parameters" do + params = {:a => 'b'} + model = mock_model(MockableModel, params) + params.should == {:a => 'b'} + end +end + describe "mock_model with stubbed id", :type => :view do before(:each) do @model = mock_model(MockableModel, :id => 1) spec/rails/mocks/mock_model_spec.rb 2b25c672537fa3e4ce0341263e890aaa052a8b3e Reg Vos jerry.vos@gmail.com http://github.com/dchelimsky/rspec-rails/commit/705355b75b7555940a1d49442ffe5ef5ccccd7a7 705355b75b7555940a1d49442ffe5ef5ccccd7a7 2008-08-11T15:00:03-07:00 2008-06-12T22:31:03-07:00 don't mutate hash passed to mock_model a890fb741b61c79dfd33ee8c95a64dd96c277932 Pat Maddox pat.maddox@gmail.com