aboutsummaryrefslogtreecommitdiffstats
path: root/activerecord/test/cases
diff options
context:
space:
mode:
authorAaron Patterson <aaron.patterson@gmail.com>2012-05-31 11:25:19 -0700
committerAaron Patterson <aaron.patterson@gmail.com>2012-05-31 11:25:19 -0700
commit5f91ea3dc1ed3fa1c6be2cff7de1d1663990b0c3 (patch)
treeed5851f76e6fb1852c3f37aed743250cbdb7dd62 /activerecord/test/cases
parent51af1e25789728248078db6b17989b252ac2e53c (diff)
parent060c91cd59ab86583a8f2f52142960d3433f62f5 (diff)
downloadrails-5f91ea3dc1ed3fa1c6be2cff7de1d1663990b0c3.tar.gz
rails-5f91ea3dc1ed3fa1c6be2cff7de1d1663990b0c3.tar.bz2
rails-5f91ea3dc1ed3fa1c6be2cff7de1d1663990b0c3.zip
Merge branch 'master-sec'
* master-sec: Strip [nil] from parameters hash. Thanks to Ben Murphy for reporting this! predicate builder should not recurse for determining where columns. Thanks to Ben Murphy for reporting this
Diffstat (limited to 'activerecord/test/cases')
-rw-r--r--activerecord/test/cases/relation/where_test.rb19
1 files changed, 19 insertions, 0 deletions
diff --git a/activerecord/test/cases/relation/where_test.rb b/activerecord/test/cases/relation/where_test.rb
new file mode 100644
index 0000000000..90c690e266
--- /dev/null
+++ b/activerecord/test/cases/relation/where_test.rb
@@ -0,0 +1,19 @@
+require "cases/helper"
+require 'models/post'
+
+module ActiveRecord
+ class WhereTest < ActiveRecord::TestCase
+ fixtures :posts
+
+ def test_where_error
+ assert_raises(ActiveRecord::StatementInvalid) do
+ Post.where(:id => { 'posts.author_id' => 10 }).first
+ end
+ end
+
+ def test_where_with_table_name
+ post = Post.first
+ assert_equal post, Post.where(:posts => { 'id' => post.id }).first
+ end
+ end
+end