diff options
author | Ryuta Kamizono <kamipo@gmail.com> | 2016-07-31 13:26:14 +0900 |
---|---|---|
committer | Ryuta Kamizono <kamipo@gmail.com> | 2016-07-31 13:37:35 +0900 |
commit | d14a5defeb0c9f2972f90d83cb625beed8105d76 (patch) | |
tree | 9fe5b2e735a2b2aec339a6628515037396542cfa /activerecord/test | |
parent | 815b730b1b79158511f9f4c8465c476b9fe9b7e0 (diff) | |
download | rails-d14a5defeb0c9f2972f90d83cb625beed8105d76.tar.gz rails-d14a5defeb0c9f2972f90d83cb625beed8105d76.tar.bz2 rails-d14a5defeb0c9f2972f90d83cb625beed8105d76.zip |
Remove internal `sanitize` method
Currently internal `sanitize`/`quote_value` method is only used for `quoted_id`.
Simply it is enough to use `connection.quote` public API instead.
Diffstat (limited to 'activerecord/test')
-rw-r--r-- | activerecord/test/cases/finder_test.rb | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/activerecord/test/cases/finder_test.rb b/activerecord/test/cases/finder_test.rb index 6eaaa30cd0..80fed50dbe 100644 --- a/activerecord/test/cases/finder_test.rb +++ b/activerecord/test/cases/finder_test.rb @@ -877,11 +877,6 @@ class FinderTest < ActiveRecord::TestCase assert_kind_of Time, Topic.where(["id = :id", { id: 1 }]).first.written_on end - def test_string_sanitation - assert_not_equal "'something ' 1=1'", ActiveRecord::Base.sanitize("something ' 1=1") - assert_equal "'something; select table'", ActiveRecord::Base.sanitize("something; select table") - end - def test_count_by_sql assert_equal(0, Entrant.count_by_sql("SELECT COUNT(*) FROM entrants WHERE id > 3")) assert_equal(1, Entrant.count_by_sql(["SELECT COUNT(*) FROM entrants WHERE id > ?", 2])) |