aboutsummaryrefslogtreecommitdiffstats
path: root/test/test_update_manager.rb
diff options
context:
space:
mode:
authorSean Griffin <sean@thoughtbot.com>2014-11-29 17:22:17 -0700
committerSean Griffin <sean@thoughtbot.com>2014-11-29 17:22:17 -0700
commit98fc25991137ee09b6800578117f8c1c322680f2 (patch)
tree038381ae4465d6ea9950ecda9ef10822fd3ff9c6 /test/test_update_manager.rb
parent7508284800f67b4611c767bff9eae7045674b66f (diff)
downloadrails-98fc25991137ee09b6800578117f8c1c322680f2.tar.gz
rails-98fc25991137ee09b6800578117f8c1c322680f2.tar.bz2
rails-98fc25991137ee09b6800578117f8c1c322680f2.zip
Remove `engine` from `TreeManager` and subclasses
This constructor parameter was unused for everything except the convenience methods `to_sql` and `where_sql`. We can pass the engine into those methods directly.
Diffstat (limited to 'test/test_update_manager.rb')
-rw-r--r--test/test_update_manager.rb26
1 files changed, 13 insertions, 13 deletions
diff --git a/test/test_update_manager.rb b/test/test_update_manager.rb
index d636ab548f..f41dc46e7d 100644
--- a/test/test_update_manager.rb
+++ b/test/test_update_manager.rb
@@ -4,13 +4,13 @@ module Arel
describe 'update manager' do
describe 'new' do
it 'takes an engine' do
- Arel::UpdateManager.new Table.engine
+ Arel::UpdateManager.new
end
end
it "should not quote sql literals" do
table = Table.new(:users)
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.table table
um.set [[table[:name], Arel::Nodes::BindParam.new]]
um.to_sql.must_be_like %{ UPDATE "users" SET "name" = ? }
@@ -18,7 +18,7 @@ module Arel
it 'handles limit properly' do
table = Table.new(:users)
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.key = 'id'
um.take 10
um.table table
@@ -29,7 +29,7 @@ module Arel
describe 'set' do
it "updates with null" do
table = Table.new(:users)
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.table table
um.set [[table[:name], nil]]
um.to_sql.must_be_like %{ UPDATE "users" SET "name" = NULL }
@@ -37,7 +37,7 @@ module Arel
it 'takes a string' do
table = Table.new(:users)
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.table table
um.set Nodes::SqlLiteral.new "foo = bar"
um.to_sql.must_be_like %{ UPDATE "users" SET foo = bar }
@@ -45,7 +45,7 @@ module Arel
it 'takes a list of lists' do
table = Table.new(:users)
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.table table
um.set [[table[:id], 1], [table[:name], 'hello']]
um.to_sql.must_be_like %{
@@ -55,25 +55,25 @@ module Arel
it 'chains' do
table = Table.new(:users)
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.set([[table[:id], 1], [table[:name], 'hello']]).must_equal um
end
end
describe 'table' do
it 'generates an update statement' do
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.table Table.new(:users)
um.to_sql.must_be_like %{ UPDATE "users" }
end
it 'chains' do
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.table(Table.new(:users)).must_equal um
end
it 'generates an update statement with joins' do
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
table = Table.new(:users)
join_source = Arel::Nodes::JoinSource.new(
@@ -89,7 +89,7 @@ module Arel
describe 'where' do
it 'generates a where clause' do
table = Table.new :users
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.table table
um.where table[:id].eq(1)
um.to_sql.must_be_like %{
@@ -99,7 +99,7 @@ module Arel
it 'chains' do
table = Table.new :users
- um = Arel::UpdateManager.new Table.engine
+ um = Arel::UpdateManager.new
um.table table
um.where(table[:id].eq(1)).must_equal um
end
@@ -108,7 +108,7 @@ module Arel
describe 'key' do
before do
@table = Table.new :users
- @um = Arel::UpdateManager.new Table.engine
+ @um = Arel::UpdateManager.new
@um.key = @table[:foo]
end