github/rails
1
1
Fork 0
mirror of https://github.com/github/rails.git synced 2026-01-10 07:07:54 -05:00
Code Issues Packages Projects Releases Wiki Activity

Avoid interpreted as grouped expression warnings

Browse Source
This commit is contained in:
Santiago Pastorino
2010-07-10 19:53:39 -03:00
parent f33ee69d59
commit 16c14f7465
2 changed files with 4 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
activemodel/test/cases/mass_assignment_security/sanitizer_test.rb
View File

@@ -31,7 +31,7 @@ class SanitizerTest < ActiveModel::TestCase
log = StringIO.new
@sanitizer.logger = Logger.new(log)
@sanitizer.sanitize(original_attributes)
assert (log.string =~ /admin/), "Should log removed attributes: #{log.string}"
assert_match(/admin/, log.string, "Should log removed attributes: #{log.string}")
end
end

6
activemodel/test/cases/mass_assignment_security_test.rb
View File

@@ -35,10 +35,10 @@ class MassAssignmentSecurityTest < ActiveModel::TestCase
assert_equal Set.new([ 'credit_rating', 'administrator', 'phone_number', 'name']), LooseDescendantSecond.protected_attributes,
'Running attr_protected twice in one class should merge the protections'
assert (TightPerson.protected_attributes - TightPerson.attributes_protected_by_default).blank?
assert_blank TightPerson.protected_attributes - TightPerson.attributes_protected_by_default
assert_equal Set.new([ 'name', 'address' ]), TightPerson.accessible_attributes
assert (TightDescendant.protected_attributes - TightDescendant.attributes_protected_by_default).blank?
assert_blank TightDescendant.protected_attributes - TightDescendant.attributes_protected_by_default
assert_equal Set.new([ 'name', 'address', 'phone_number' ]), TightDescendant.accessible_attributes
end
@@ -49,4 +49,4 @@ class MassAssignmentSecurityTest < ActiveModel::TestCase
assert_equal sanitized, { }
end
end
end