2017-07-09 13:41:28 -04:00
# frozen_string_literal: true
2011-06-06 14:17:44 -04:00
require " cases/helper "
2016-08-06 12:26:20 -04:00
require " models/post "
require " models/comment "
require " models/author "
require " models/rating "
2018-06-06 03:41:13 -04:00
require " models/categorization "
2010-11-30 17:10:55 -05:00
module ActiveRecord
class RelationTest < ActiveRecord :: TestCase
2018-06-06 03:41:13 -04:00
fixtures :posts , :comments , :authors , :author_addresses , :ratings , :categorizations
2010-11-30 17:47:55 -05:00
2010-11-30 17:10:55 -05:00
def test_construction
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass , table : :b )
Use separate Relation subclasses for each AR class
At present, ActiveRecord::Delegation compiles delegation methods on a
global basis. The compiled methods apply to all subsequent Relation
instances. This creates several problems:
1) After Post.all.recent has been called, User.all.respond_to?(:recent)
will be true, even if User.all.recent will actually raise an error due
to no User.recent method existing. (See #8080.)
2) Depending on the AR class, the delegation should do different things.
For example, if a Post.zip method exists, then Post.all.zip should call
it. But this will then result in User.zip being called by a subsequent
User.all.zip, even if User.zip does not exist, when in fact
User.all.zip should call User.all.to_a.zip. (There are various
variants of this problem.)
We are creating these compiled delegations in order to avoid method
missing and to avoid repeating logic on each invocation.
One way of handling these issues is to add additional checks in various
places to ensure we're doing the "right thing". However, this makes the
compiled methods signficantly slower. In which case, there's almost no
point in avoiding method_missing at all. (See #8127 for a proposed
solution which takes this approach.)
This is an alternative approach which involves creating a subclass of
ActiveRecord::Relation for each AR class represented. So, with this
patch, Post.all.class != User.all.class. This means that the delegations
are compiled for and only apply to a single AR class. A compiled method
for Post.all will not be invoked from User.all.
This solves the above issues without incurring significant performance
penalties. It's designed to be relatively seamless, however the downside
is a bit of complexity and potentially confusion for a user who thinks
that Post.all and User.all should be instances of the same class.
Benchmark
---------
require 'active_record'
require 'benchmark/ips'
class Post < ActiveRecord::Base
establish_connection adapter: 'sqlite3', database: ':memory:'
connection.create_table :posts
def self.omg
:omg
end
end
relation = Post.all
Benchmark.ips do |r|
r.report('delegation') { relation.omg }
r.report('constructing') { Post.all }
end
Before
------
Calculating -------------------------------------
delegation 4392 i/100ms
constructing 4780 i/100ms
-------------------------------------------------
delegation 144235.9 (±27.7%) i/s - 663192 in 5.038075s
constructing 182015.5 (±21.2%) i/s - 850840 in 5.005364s
After
-----
Calculating -------------------------------------
delegation 6677 i/100ms
constructing 6260 i/100ms
-------------------------------------------------
delegation 166828.2 (±34.2%) i/s - 754501 in 5.001430s
constructing 116575.5 (±18.6%) i/s - 563400 in 5.036690s
Comments
--------
Bear in mind that the standard deviations in the above are huge, so we
can't compare the numbers too directly. However, we can conclude that
Relation construction has become a little slower (as we'd expect), but
not by a huge huge amount, and we can still construct a large number of
Relations quite quickly.
2012-11-23 07:36:22 -05:00
assert_equal FakeKlass , relation . klass
2010-11-30 17:10:55 -05:00
assert_equal :b , relation . table
2018-05-12 22:26:10 -04:00
assert_not relation . loaded , " relation is not loaded "
2010-11-30 17:10:55 -05:00
end
2012-06-03 01:20:49 -04:00
def test_responds_to_model_and_returns_klass
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
Use separate Relation subclasses for each AR class
At present, ActiveRecord::Delegation compiles delegation methods on a
global basis. The compiled methods apply to all subsequent Relation
instances. This creates several problems:
1) After Post.all.recent has been called, User.all.respond_to?(:recent)
will be true, even if User.all.recent will actually raise an error due
to no User.recent method existing. (See #8080.)
2) Depending on the AR class, the delegation should do different things.
For example, if a Post.zip method exists, then Post.all.zip should call
it. But this will then result in User.zip being called by a subsequent
User.all.zip, even if User.zip does not exist, when in fact
User.all.zip should call User.all.to_a.zip. (There are various
variants of this problem.)
We are creating these compiled delegations in order to avoid method
missing and to avoid repeating logic on each invocation.
One way of handling these issues is to add additional checks in various
places to ensure we're doing the "right thing". However, this makes the
compiled methods signficantly slower. In which case, there's almost no
point in avoiding method_missing at all. (See #8127 for a proposed
solution which takes this approach.)
This is an alternative approach which involves creating a subclass of
ActiveRecord::Relation for each AR class represented. So, with this
patch, Post.all.class != User.all.class. This means that the delegations
are compiled for and only apply to a single AR class. A compiled method
for Post.all will not be invoked from User.all.
This solves the above issues without incurring significant performance
penalties. It's designed to be relatively seamless, however the downside
is a bit of complexity and potentially confusion for a user who thinks
that Post.all and User.all should be instances of the same class.
Benchmark
---------
require 'active_record'
require 'benchmark/ips'
class Post < ActiveRecord::Base
establish_connection adapter: 'sqlite3', database: ':memory:'
connection.create_table :posts
def self.omg
:omg
end
end
relation = Post.all
Benchmark.ips do |r|
r.report('delegation') { relation.omg }
r.report('constructing') { Post.all }
end
Before
------
Calculating -------------------------------------
delegation 4392 i/100ms
constructing 4780 i/100ms
-------------------------------------------------
delegation 144235.9 (±27.7%) i/s - 663192 in 5.038075s
constructing 182015.5 (±21.2%) i/s - 850840 in 5.005364s
After
-----
Calculating -------------------------------------
delegation 6677 i/100ms
constructing 6260 i/100ms
-------------------------------------------------
delegation 166828.2 (±34.2%) i/s - 754501 in 5.001430s
constructing 116575.5 (±18.6%) i/s - 563400 in 5.036690s
Comments
--------
Bear in mind that the standard deviations in the above are huge, so we
can't compare the numbers too directly. However, we can conclude that
Relation construction has become a little slower (as we'd expect), but
not by a huge huge amount, and we can still construct a large number of
Relations quite quickly.
2012-11-23 07:36:22 -05:00
assert_equal FakeKlass , relation . model
2012-06-03 01:20:49 -04:00
end
2010-11-30 17:10:55 -05:00
def test_initialize_single_values
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2018-01-04 10:42:10 -05:00
( Relation :: SINGLE_VALUE_METHODS - [ :create_with ] ) . each do | method |
2010-11-30 17:10:55 -05:00
assert_nil relation . send ( " #{ method } _value " ) , method . to_s
end
2015-05-15 09:38:26 -04:00
value = relation . create_with_value
assert_equal ( { } , value )
assert_predicate value , :frozen?
2010-11-30 17:10:55 -05:00
end
def test_multi_value_initialize
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2010-11-30 17:10:55 -05:00
Relation :: MULTI_VALUE_METHODS . each do | method |
2015-05-15 09:38:26 -04:00
values = relation . send ( " #{ method } _values " )
assert_equal [ ] , values , method . to_s
assert_predicate values , :frozen? , method . to_s
2010-11-30 17:10:55 -05:00
end
end
def test_extensions
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2010-11-30 17:10:55 -05:00
assert_equal [ ] , relation . extensions
end
2010-11-30 17:29:35 -05:00
2010-11-30 17:47:55 -05:00
def test_empty_where_values_hash
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2010-11-30 17:29:35 -05:00
assert_equal ( { } , relation . where_values_hash )
end
2010-11-30 17:47:55 -05:00
def test_has_values
2018-01-24 14:06:30 -05:00
relation = Relation . new ( Post )
2017-07-16 02:19:06 -04:00
relation . where! ( id : 10 )
assert_equal ( { " id " = > 10 } , relation . where_values_hash )
2010-11-30 17:47:55 -05:00
end
def test_values_wrong_table
2018-01-24 14:06:30 -05:00
relation = Relation . new ( Post )
2014-12-29 13:21:56 -05:00
relation . where! Comment . arel_table [ :id ] . eq ( 10 )
2010-11-30 17:47:55 -05:00
assert_equal ( { } , relation . where_values_hash )
end
2010-11-30 17:56:32 -05:00
def test_tree_is_not_traversed
2018-01-24 14:06:30 -05:00
relation = Relation . new ( Post )
2014-12-29 13:21:56 -05:00
left = relation . table [ :id ] . eq ( 10 )
right = relation . table [ :id ] . eq ( 10 )
2017-12-07 12:37:02 -05:00
combine = left . or ( right )
2012-04-13 10:59:47 -04:00
relation . where! combine
2010-11-30 17:56:32 -05:00
assert_equal ( { } , relation . where_values_hash )
end
2010-11-30 17:29:35 -05:00
def test_scope_for_create
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2010-11-30 17:29:35 -05:00
assert_equal ( { } , relation . scope_for_create )
end
2010-11-30 18:44:30 -05:00
def test_create_with_value
2018-01-24 14:06:30 -05:00
relation = Relation . new ( Post )
2017-07-12 18:14:59 -04:00
relation . create_with_value = { hello : " world " }
assert_equal ( { " hello " = > " world " } , relation . scope_for_create )
2010-11-30 18:44:30 -05:00
end
def test_create_with_value_with_wheres
2018-01-24 14:06:30 -05:00
relation = Relation . new ( Post )
2010-11-30 18:45:32 -05:00
assert_equal ( { } , relation . scope_for_create )
2017-07-16 02:19:06 -04:00
relation . where! ( id : 10 )
2017-07-16 08:31:08 -04:00
assert_equal ( { " id " = > 10 } , relation . scope_for_create )
2010-11-30 18:45:32 -05:00
2016-08-16 03:30:11 -04:00
relation . create_with_value = { hello : " world " }
2017-07-16 08:31:08 -04:00
assert_equal ( { " hello " = > " world " , " id " = > 10 } , relation . scope_for_create )
2010-11-30 18:45:32 -05:00
end
2010-11-30 19:10:49 -05:00
2017-07-24 00:28:38 -04:00
def test_empty_scope
2018-01-24 14:06:30 -05:00
relation = Relation . new ( Post )
2018-01-25 18:14:09 -05:00
assert_predicate relation , :empty_scope?
2017-07-24 00:28:38 -04:00
relation . merge! ( relation )
2018-01-25 18:14:09 -05:00
assert_predicate relation , :empty_scope?
2019-04-05 23:00:17 -04:00
assert_not_predicate NullPost . all , :empty_scope?
assert_not_predicate FirstPost . all , :empty_scope?
2017-07-24 00:28:38 -04:00
end
2013-07-23 18:02:43 -04:00
def test_bad_constants_raise_errors
assert_raises ( NameError ) do
ActiveRecord :: Relation :: HelloWorld
end
end
2010-11-30 19:10:49 -05:00
def test_empty_eager_loading?
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2018-01-25 18:14:09 -05:00
assert_not_predicate relation , :eager_loading?
2010-11-30 19:10:49 -05:00
end
def test_eager_load_values
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2012-04-13 10:59:47 -04:00
relation . eager_load! :b
2018-01-25 18:14:09 -05:00
assert_predicate relation , :eager_loading?
2010-11-30 19:10:49 -05:00
end
2012-01-13 18:56:07 -05:00
def test_references_values
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2012-01-13 18:56:07 -05:00
assert_equal [ ] , relation . references_values
relation = relation . references ( :foo ) . references ( :omg , :lol )
2016-08-06 12:26:20 -04:00
assert_equal [ " foo " , " omg " , " lol " ] , relation . references_values
2012-01-13 18:56:07 -05:00
end
def test_references_values_dont_duplicate
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2012-01-13 18:56:07 -05:00
relation = relation . references ( :foo ) . references ( :foo )
2016-08-06 12:26:20 -04:00
assert_equal [ " foo " ] , relation . references_values
2012-01-13 18:56:07 -05:00
end
2012-01-16 16:14:34 -05:00
2016-08-06 12:26:20 -04:00
test " merging a hash into a relation " do
2018-01-24 14:06:30 -05:00
relation = Relation . new ( Post )
2016-08-16 03:30:11 -04:00
relation = relation . merge where : { name : :lol } , readonly : true
2012-04-13 07:10:51 -04:00
2016-10-28 23:05:58 -04:00
assert_equal ( { " name " = > :lol } , relation . where_clause . to_h )
2012-04-13 07:10:51 -04:00
assert_equal true , relation . readonly_value
end
2016-08-06 12:26:20 -04:00
test " merging an empty hash into a relation " do
2018-01-24 14:06:30 -05:00
assert_equal Relation :: WhereClause . empty , Relation . new ( FakeKlass ) . merge ( { } ) . where_clause
2012-04-13 07:10:51 -04:00
end
2012-04-13 07:15:43 -04:00
2016-08-06 12:26:20 -04:00
test " merging a hash with unknown keys raises " do
assert_raises ( ArgumentError ) { Relation :: HashMerger . new ( nil , omg : " lol " ) }
2012-04-13 07:15:43 -04:00
end
2012-04-13 11:17:28 -04:00
2016-08-06 12:26:20 -04:00
test " merging nil or false raises " do
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2015-02-06 14:50:45 -05:00
e = assert_raises ( ArgumentError ) do
relation = relation . merge nil
end
2016-08-06 12:26:20 -04:00
assert_equal " invalid argument: nil. " , e . message
2015-02-06 14:50:45 -05:00
e = assert_raises ( ArgumentError ) do
relation = relation . merge false
end
2016-08-06 12:26:20 -04:00
assert_equal " invalid argument: false. " , e . message
2015-02-06 14:50:45 -05:00
end
2016-08-06 12:26:20 -04:00
test " # values returns a dup of the values " do
2018-01-24 14:06:30 -05:00
relation = Relation . new ( Post ) . where! ( name : :foo )
2012-04-13 11:17:28 -04:00
values = relation . values
values [ :where ] = nil
2015-01-25 19:35:46 -05:00
assert_not_nil relation . where_clause
2012-04-13 11:17:28 -04:00
end
2016-08-06 12:26:20 -04:00
test " relations can be created with a values hash " do
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass , values : { select : [ :foo ] } )
Introduce `Relation::WhereClause`
The way that bind values are currently stored on Relation is a mess.
They can come from `having`, `where`, or `join`. I'm almost certain that
`having` is actually broken, and calling `where` followed by `having`
followed by `where` will completely scramble the binds.
Joins don't actually add the bind parameters to the relation itself, but
instead add it onto an accessor on the arel AST which is undocumented,
and unused in Arel itself. This means that the bind values must always
be accessed as `relation.arel.bind_values + relation.bind_values`.
Anything that doesn't is likely broken (and tons of bugs have come up
for exactly that reason)
The result is that everything dealing with `Relation` instances has to
know far too much about the internals. The binds are split, combined,
and re-stored in non-obvious ways that makes it difficult to change
anything about the internal representation of `bind_values`, and is
extremely prone to bugs.
So the goal is to move a lot of logic off of `Relation`, and into
separate objects. This is not the same as what is currently done with
`JoinDependency`, as `Relation` knows far too much about its internals,
and vice versa. Instead these objects need to be black boxes that can
have their implementations swapped easily.
The end result will be two classes, `WhereClause` and `JoinClause`
(`having` will just re-use `WhereClause`), and there will be a single
method to access the bind values of a `Relation` which will be
implemented as
```
join_clause.binds + where_clause.binds + having_clause.binds
```
This is the first step towards that refactoring, with the internal
representation of where changed, and an intermediate representation of
`where_values` and `bind_values` to let the refactoring take small
steps. These will be removed shortly.
2015-01-25 16:54:18 -05:00
assert_equal [ :foo ] , relation . select_values
2012-04-13 11:17:28 -04:00
end
2012-04-22 05:34:55 -04:00
2016-08-06 12:26:20 -04:00
test " merging a hash interpolates conditions " do
2013-08-30 19:48:20 -04:00
klass = Class . new ( FakeKlass ) do
def self . sanitize_sql ( args )
2016-08-06 12:26:20 -04:00
raise unless args == [ " foo = ? " , " bar " ]
" foo = bar "
2013-08-30 19:48:20 -04:00
end
end
2012-04-22 05:34:55 -04:00
2018-01-24 14:06:30 -05:00
relation = Relation . new ( klass )
2018-07-26 04:33:49 -04:00
relation . merge! ( where : [ " foo = ? " , " bar " ] )
Refactor Active Record to let Arel manage bind params
A common source of bugs and code bloat within Active Record has been the
need for us to maintain the list of bind values separately from the AST
they're associated with. This makes any sort of AST manipulation
incredibly difficult, as any time we want to potentially insert or
remove an AST node, we need to traverse the entire tree to find where
the associated bind parameters are.
With this change, the bind parameters now live on the AST directly.
Active Record does not need to know or care about them until the final
AST traversal for SQL construction. Rather than returning just the SQL,
the Arel collector will now return both the SQL and the bind parameters.
At this point the connection adapter will have all the values that it
had before.
A bit of this code is janky and something I'd like to refactor later. In
particular, I don't like how we're handling associations in the
predicate builder, the special casing of `StatementCache::Substitute` in
`QueryAttribute`, or generally how we're handling bind value replacement
in the statement cache when prepared statements are disabled.
This also mostly reverts #26378, as it moved all the code into a
location that I wanted to delete.
/cc @metaskills @yahonda, this change will affect the adapters
Fixes #29766.
Fixes #29804.
Fixes #26541.
Close #28539.
Close #24769.
Close #26468.
Close #26202.
There are probably other issues/PRs that can be closed because of this
commit, but that's all I could find on the first few pages.
2017-07-24 08:19:35 -04:00
assert_equal Relation :: WhereClause . new ( [ " foo = bar " ] ) , relation . where_clause
2012-04-22 05:34:55 -04:00
end
2013-04-10 08:34:47 -04:00
2013-09-11 08:18:31 -04:00
def test_merging_readonly_false
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2013-09-11 08:18:31 -04:00
readonly_false_relation = relation . readonly ( false )
# test merging in both directions
assert_equal false , relation . merge ( readonly_false_relation ) . readonly_value
assert_equal false , readonly_false_relation . merge ( relation ) . readonly_value
end
2013-05-29 23:36:19 -04:00
def test_relation_merging_with_merged_joins_as_symbols
2013-04-10 08:34:47 -04:00
special_comments_with_ratings = SpecialComment . joins ( :ratings )
posts_with_special_comments_with_ratings = Post . group ( " posts.id " ) . joins ( :special_comments ) . merge ( special_comments_with_ratings )
2016-08-17 11:16:36 -04:00
assert_equal ( { 4 = > 2 } , authors ( :david ) . posts . merge ( posts_with_special_comments_with_ratings ) . count )
end
def test_relation_merging_with_merged_symbol_joins_keeps_inner_joins
2017-06-21 20:13:02 -04:00
queries = capture_sql { Author . joins ( :posts ) . merge ( Post . joins ( :comments ) ) . to_a }
2016-08-17 11:16:36 -04:00
nb_inner_join = queries . sum { | sql | sql . scan ( / INNER \ s+JOIN /i ) . size }
assert_equal 2 , nb_inner_join , " Wrong amount of INNER JOIN in query "
assert queries . none? { | sql | / LEFT \ s+(OUTER)? \ s+JOIN /i . match? ( sql ) } , " Shouldn't have any LEFT JOIN in query "
end
def test_relation_merging_with_merged_symbol_joins_has_correct_size_and_count
# Has one entry per comment
merged_authors_with_commented_posts_relation = Author . joins ( :posts ) . merge ( Post . joins ( :comments ) )
post_ids_with_author = Post . joins ( :author ) . pluck ( :id )
manual_comments_on_post_that_have_author = Comment . where ( post_id : post_ids_with_author ) . pluck ( :id )
assert_equal manual_comments_on_post_that_have_author . size , merged_authors_with_commented_posts_relation . count
assert_equal manual_comments_on_post_that_have_author . size , merged_authors_with_commented_posts_relation . to_a . size
2013-04-10 08:34:47 -04:00
end
2018-06-06 03:41:13 -04:00
def test_relation_merging_with_merged_symbol_joins_is_aliased
categorizations_with_authors = Categorization . joins ( :author )
queries = capture_sql { Post . joins ( :author , :categorizations ) . merge ( Author . select ( :id ) ) . merge ( categorizations_with_authors ) . to_a }
nb_inner_join = queries . sum { | sql | sql . scan ( / INNER \ s+JOIN /i ) . size }
assert_equal 3 , nb_inner_join , " Wrong amount of INNER JOIN in query "
# using `\W` as the column separator
2018-10-04 06:56:00 -04:00
assert queries . any? { | sql | %r[ INNER \ s+JOIN \ s+ #{ Regexp . escape ( Author . quoted_table_name ) } \ s+ \ Wauthors_categorizations \ W ]i . match? ( sql ) } , " Should be aliasing the child INNER JOINs in query "
2018-06-06 03:41:13 -04:00
end
def test_relation_with_merged_joins_aliased_works
categorizations_with_authors = Categorization . joins ( :author )
posts_with_joins_and_merges = Post . joins ( :author , :categorizations )
. merge ( Author . select ( :id ) ) . merge ( categorizations_with_authors )
author_with_posts = Author . joins ( :posts ) . ids
categorizations_with_author = Categorization . joins ( :author ) . ids
posts_with_author_and_categorizations = Post . joins ( :categorizations ) . where ( author_id : author_with_posts , categorizations : { id : categorizations_with_author } ) . ids
assert_equal posts_with_author_and_categorizations . size , posts_with_joins_and_merges . count
assert_equal posts_with_author_and_categorizations . size , posts_with_joins_and_merges . to_a . size
end
2013-12-28 21:46:21 -05:00
def test_relation_merging_with_joins_as_join_dependency_pick_proper_parent
post = Post . create! ( title : " haha " , body : " huhu " )
comment = post . comments . create! ( body : " hu " )
3 . times { comment . ratings . create! }
2013-12-31 18:04:11 -05:00
relation = Post . joins ( :comments ) . merge Comment . joins ( :ratings )
2013-12-28 21:46:21 -05:00
2013-12-31 18:04:11 -05:00
assert_equal 3 , relation . where ( id : post . id ) . pluck ( :id ) . size
2013-12-28 21:46:21 -05:00
end
2015-11-17 09:57:48 -05:00
def test_merge_raises_with_invalid_argument
assert_raises ArgumentError do
2018-01-24 14:06:30 -05:00
relation = Relation . new ( FakeKlass )
2015-11-17 09:57:48 -05:00
relation . merge ( true )
end
end
2013-04-13 21:19:17 -04:00
def test_respond_to_for_non_selected_element
2013-04-20 03:26:07 -04:00
post = Post . select ( :title ) . first
2018-01-24 21:14:10 -05:00
assert_not_respond_to post , :body , " post should not respond_to?(:body) since invoking it raises exception "
2013-04-20 03:26:07 -04:00
2013-05-07 05:36:14 -04:00
silence_warnings { post = Post . select ( " 'title' as post_title " ) . first }
2018-01-24 21:14:10 -05:00
assert_not_respond_to post , :title , " post should not respond_to?(:body) since invoking it raises exception "
2013-04-13 21:19:17 -04:00
end
2015-05-30 14:26:41 -04:00
def test_select_quotes_when_using_from_clause
2015-09-21 13:36:55 -04:00
skip_if_sqlite3_version_includes_quoting_bug
2015-05-30 14:26:41 -04:00
quoted_join = ActiveRecord :: Base . connection . quote_table_name ( " join " )
selected = Post . select ( :join ) . from ( Post . select ( " id as #{ quoted_join } " ) ) . map ( & :join )
assert_equal Post . pluck ( :id ) , selected
end
2015-09-21 13:36:55 -04:00
def test_selecting_aliased_attribute_quotes_column_name_when_from_is_used
skip_if_sqlite3_version_includes_quoting_bug
klass = Class . new ( ActiveRecord :: Base ) do
self . table_name = :test_with_keyword_column_name
alias_attribute :description , :desc
end
klass . create! ( description : " foo " )
assert_equal [ " foo " ] , klass . select ( :description ) . from ( klass . all ) . map ( & :desc )
2019-04-09 09:38:08 -04:00
assert_equal [ " foo " ] , klass . reselect ( :description ) . from ( klass . all ) . map ( & :desc )
2015-09-21 13:36:55 -04:00
end
2013-05-29 23:36:19 -04:00
def test_relation_merging_with_merged_joins_as_strings
join_string = " LEFT OUTER JOIN #{ Rating . quoted_table_name } ON #{ SpecialComment . quoted_table_name } .id = #{ Rating . quoted_table_name } .comment_id "
special_comments_with_ratings = SpecialComment . joins join_string
posts_with_special_comments_with_ratings = Post . group ( " posts.id " ) . joins ( :special_comments ) . merge ( special_comments_with_ratings )
2016-10-28 23:05:58 -04:00
assert_equal ( { 2 = > 1 , 4 = > 3 , 5 = > 1 } , authors ( :david ) . posts . merge ( posts_with_special_comments_with_ratings ) . count )
2013-05-29 23:36:19 -04:00
end
2014-06-25 21:32:14 -04:00
2017-11-10 13:54:10 -05:00
def test_relation_merging_keeps_joining_order
authors = Author . where ( id : 1 )
posts = Post . joins ( :author ) . merge ( authors )
comments = Comment . joins ( :post ) . merge ( posts )
ratings = Rating . joins ( :comment ) . merge ( comments )
assert_equal 3 , ratings . count
end
2019-03-14 15:57:14 -04:00
def test_relation_with_annotation_includes_comment_in_to_sql
post_with_annotation = Post . where ( id : 1 ) . annotate ( " foo " )
assert_match %r{ = 1 / \ * foo \ */ } , post_with_annotation . to_sql
end
def test_relation_with_annotation_includes_comment_in_sql
post_with_annotation = Post . where ( id : 1 ) . annotate ( " foo " )
assert_sql ( %r{ / \ * foo \ */ } ) do
assert post_with_annotation . first , " record should be found "
end
end
def test_relation_with_annotation_chains_sql_comments
post_with_annotation = Post . where ( id : 1 ) . annotate ( " foo " ) . annotate ( " bar " )
assert_sql ( %r{ / \ * foo \ */ / \ * bar \ */ } ) do
assert post_with_annotation . first , " record should be found "
end
end
def test_relation_with_annotation_filters_sql_comment_delimiters
post_with_annotation = Post . where ( id : 1 ) . annotate ( " **//foo//** " )
assert_match %r{ = 1 / \ * foo \ */ } , post_with_annotation . to_sql
end
def test_relation_with_annotation_includes_comment_in_count_query
post_with_annotation = Post . annotate ( " foo " )
all_count = Post . all . to_a . count
assert_sql ( %r{ / \ * foo \ */ } ) do
assert_equal all_count , post_with_annotation . count
end
end
def test_relation_without_annotation_does_not_include_an_empty_comment
log = capture_sql do
Post . where ( id : 1 ) . first
end
assert_not_predicate log , :empty?
assert_predicate log . select { | query | query . match? ( %r{ / \ * } ) } , :empty?
end
def test_relation_with_optimizer_hints_filters_sql_comment_delimiters
post_with_hint = Post . where ( id : 1 ) . optimizer_hints ( " **//BADHINT//** " )
assert_match %r{ BADHINT } , post_with_hint . to_sql
assert_no_match %r{ \ */BADHINT } , post_with_hint . to_sql
assert_no_match %r{ \ *//BADHINT } , post_with_hint . to_sql
assert_no_match %r{ BADHINT/ \ * } , post_with_hint . to_sql
assert_no_match %r{ BADHINT// \ * } , post_with_hint . to_sql
post_with_hint = Post . where ( id : 1 ) . optimizer_hints ( " /*+ BADHINT */ " )
assert_match %r{ / \ * \ + BADHINT \ */ } , post_with_hint . to_sql
end
2019-07-18 18:21:20 -04:00
def test_does_not_duplicate_optimizer_hints_on_merge
escaped_table = Post . connection . quote_table_name ( " posts " )
expected = " SELECT /*+ OMGHINT */ #{ escaped_table } .* FROM #{ escaped_table } "
query = Post . optimizer_hints ( " OMGHINT " ) . merge ( Post . optimizer_hints ( " OMGHINT " ) ) . to_sql
assert_equal expected , query
end
2014-06-25 21:32:14 -04:00
class EnsureRoundTripTypeCasting < ActiveRecord :: Type :: Value
def type
:string
end
2017-11-13 15:24:28 -05:00
def cast ( value )
raise value unless value == " value from user "
" cast value "
end
2015-02-17 13:29:51 -05:00
def deserialize ( value )
2014-06-25 21:32:14 -04:00
raise value unless value == " type cast for database "
" type cast from database "
end
2015-02-17 15:35:23 -05:00
def serialize ( value )
2017-11-13 15:24:28 -05:00
raise value unless value == " cast value "
2014-06-25 21:32:14 -04:00
" type cast for database "
end
end
class UpdateAllTestModel < ActiveRecord :: Base
2016-08-06 12:26:20 -04:00
self . table_name = " posts "
2014-06-25 21:32:14 -04:00
attribute :body , EnsureRoundTripTypeCasting . new
end
def test_update_all_goes_through_normal_type_casting
UpdateAllTestModel . update_all ( body : " value from user " , type : nil ) # No STI
assert_equal " type cast from database " , UpdateAllTestModel . first . body
end
2015-05-30 14:26:41 -04:00
2018-04-11 20:08:13 -04:00
def test_skip_preloading_after_arel_has_been_generated
assert_nothing_raised do
relation = Comment . all
relation . arel
relation . skip_preloading!
end
end
2019-09-21 17:42:12 -04:00
test " no queries on empty IN " do
Post . send ( :load_schema )
assert_no_queries do
Post . where ( id : [ ] ) . load
end
end
test " can unscope empty IN " do
Post . send ( :load_schema )
2019-09-30 21:38:00 -04:00
assert_queries 1 do
2019-09-21 17:42:12 -04:00
Post . where ( id : [ ] ) . unscope ( where : :id ) . load
end
end
2015-05-30 14:26:41 -04:00
private
2016-08-06 13:55:02 -04:00
def skip_if_sqlite3_version_includes_quoting_bug
if sqlite3_version_includes_quoting_bug?
skip <<-ERROR.squish
2016-08-22 21:24:08 -04:00
You are using an outdated version of SQLite3 which has a bug in
quoted column names . Please update SQLite3 and rebuild the sqlite3
ruby gem
ERROR
2016-08-06 13:55:02 -04:00
end
2015-09-21 13:36:55 -04:00
end
2016-08-06 13:55:02 -04:00
def sqlite3_version_includes_quoting_bug?
if current_adapter? ( :SQLite3Adapter )
selected_quoted_column_names = ActiveRecord :: Base . connection . exec_query (
'SELECT "join" FROM (SELECT id AS "join" FROM posts) subquery'
) . columns
[ " join " ] != selected_quoted_column_names
end
2015-05-30 14:26:41 -04:00
end
2010-11-30 17:10:55 -05:00
end
end