final corrections to the mass-assignment security tests #325

Merged
merged 3 commits into from Apr 27, 2011

3 participants

@joshk

And this found fix the CI failures

tenderlove and others added some commits Apr 27, 2011
@tenderlove tenderlove Merged pull request #325 from joshk/mass_assignment_tests_fixed.
final corrections to the mass-assignment security tests
1cd5a08
@misfo misfo prevent errors when passing a frozen string as a param to ActionContr…
…oller::TestCase#process

since ActionDispatch::Http::Parameters#encode_params will force encoding on all params strings (when using an encoding aware Ruby), dup all strings passed into process.  This prevents modification of params passed in and, more importantly, doesn't barf when a frozen string is passed
thanks and high fives to kinsteronline
9f6cafd
@joshk joshk some test renaming to avoid collisions, and some annoying issues betw…
…een dbs
78e581b
@tenderlove tenderlove merged commit 78e581b into rails:master Apr 27, 2011
@myronmarston myronmarston pushed a commit to myronmarston/rails that referenced this pull request Apr 27, 2011
@tenderlove tenderlove Merged pull request #325 from joshk/mass_assignment_tests_fixed.
final corrections to the mass-assignment security tests
e190569
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment