Merge pull request #1690 from vijaydev/mattr_accessor_changes

Added instance_accessor: false to Module#mattr_accessor

1 files changed, 7 insertions, 0 deletions

diff --git a/activesupport/test/core_ext/module/attribute_accessor_test.rb b/activesupport/test/core_ext/module/attribute_accessor_test.rb
index 118fb070a0..29889b51e0 100644
--- a/activesupport/test/core_ext/module/attribute_accessor_test.rb
+++ b/activesupport/test/core_ext/module/attribute_accessor_test.rb
@@ -7,6 +7,7 @@ class ModuleAttributeAccessorTest < Test::Unit::TestCase
       mattr_accessor :foo
       mattr_accessor :bar, :instance_writer => false
       mattr_reader   :shaq, :instance_reader => false
+      mattr_accessor :camp, :instance_accessor => false
     end
     @class = Class.new
     @class.instance_eval { include m }
@@ -37,4 +38,10 @@ class ModuleAttributeAccessorTest < Test::Unit::TestCase
     assert_respond_to @module, :shaq
     assert !@object.respond_to?(:shaq)
   end
+
+  def test_should_not_create_instance_accessors
+    assert_respond_to @module, :camp
+    assert !@object.respond_to?(:camp)
+    assert !@object.respond_to?(:camp=)
+  end
 end