kotovalexarian-likes-github/rails--rails
1
0
Fork 0
mirror of https://github.com/rails/rails.git synced 2022-11-09 12:12:34 -05:00
Code Releases Activity

Use instance_variable_defined? instead of instance_variables.include? to account for string/symbol differences between 1.8 and 1.9

Browse source 
git-svn-id: http://svn-commit.rubyonrails.org/rails/trunk@7650 5ecf4fe2-1ee6-0310-87b1-e25e094e27de
This commit is contained in:
Jeremy Kemper 2007-09-27 09:27:37 +00:00
parent d96e38be99
commit e36b9b95fe
2 changed files with 11 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

0
activesupport/test/core_ext/module/attr_accessor_with_default.rb
View file

22
activesupport/test/core_ext/module/attr_internal_test.rb
View file

@ -6,46 +6,46 @@ class AttrInternalTest < Test::Unit::TestCase
@instance = @target.new
end
def test_attr_internal_reader
def test_reader
assert_nothing_raised { @target.attr_internal_reader :foo }
assert !@instance.instance_variables.include?('@_foo')
assert !@instance.instance_variable_defined?('@_foo')
assert_raise(NoMethodError) { @instance.foo = 1 }
@instance.instance_variable_set('@_foo', 1)
assert_nothing_raised { assert_equal 1, @instance.foo }
end
def test_attr_internal_writer
def test_writer
assert_nothing_raised { @target.attr_internal_writer :foo }
assert !@instance.instance_variables.include?('@_foo')
assert !@instance.instance_variable_defined?('@_foo')
assert_nothing_raised { assert_equal 1, @instance.foo = 1 }
assert_equal 1, @instance.instance_variable_get('@_foo')
assert_raise(NoMethodError) { @instance.foo }
end
def test_attr_internal_accessor
def test_accessor
assert_nothing_raised { @target.attr_internal :foo }
assert !@instance.instance_variables.include?('@_foo')
assert !@instance.instance_variable_defined?('@_foo')
assert_nothing_raised { assert_equal 1, @instance.foo = 1 }
assert_equal 1, @instance.instance_variable_get('@_foo')
assert_nothing_raised { assert_equal 1, @instance.foo }
end
def test_attr_internal_naming_format
def test_naming_format
assert_equal '@_%s', @target.attr_internal_naming_format
assert_nothing_raised { @target.attr_internal_naming_format = '@abc%sdef' }
@target.attr_internal :foo
assert !@instance.instance_variables.include?('@_foo')
assert !@instance.instance_variables.include?('@abcfoodef')
assert !@instance.instance_variable_defined?('@_foo')
assert !@instance.instance_variable_defined?('@abcfoodef')
assert_nothing_raised { @instance.foo = 1 }
assert !@instance.instance_variables.include?('@_foo')
assert @instance.instance_variables.include?('@abcfoodef')
assert !@instance.instance_variable_defined?('@_foo')
assert @instance.instance_variable_defined?('@abcfoodef')
ensure
@target.attr_internal_naming_format = '@_%s'
end