diff options
author | Aaron Patterson <aaron.patterson@gmail.com> | 2011-03-07 08:36:09 -0800 |
---|---|---|
committer | Aaron Patterson <aaron.patterson@gmail.com> | 2011-03-07 08:36:09 -0800 |
commit | 2fc53bec29bb660e501cf350ae07506082dcb110 (patch) | |
tree | 7cd8a27a81372589cf9c3c65c6f6e4904a521599 /test | |
parent | e7cdaedb6bc7d452142b9b8c42942d659efcd11a (diff) | |
parent | a85530820426f8baa4dbb1fa863313c6a6c69484 (diff) | |
download | rails-2fc53bec29bb660e501cf350ae07506082dcb110.tar.gz rails-2fc53bec29bb660e501cf350ae07506082dcb110.tar.bz2 rails-2fc53bec29bb660e501cf350ae07506082dcb110.zip |
Merge branch 'master' of github.com:rails/arel
* 'master' of github.com:rails/arel:
Add an #table_name method to Table and TableAlias, which always returns the actual table name, not the alias. Then fix ToSql#column_for to use this table name when checking whether the table exists (rather than before, where it was checking whether a table with the alias name exists, which was incorrect).
Diffstat (limited to 'test')
-rw-r--r-- | test/test_table.rb | 4 | ||||
-rw-r--r-- | test/visitors/test_to_sql.rb | 9 |
2 files changed, 13 insertions, 0 deletions
diff --git a/test/test_table.rb b/test/test_table.rb index 129d7ba736..1035540513 100644 --- a/test/test_table.rb +++ b/test/test_table.rb @@ -164,6 +164,10 @@ module Arel @relation.name.must_equal 'users' end + it "should have a table name" do + @relation.table_name.must_equal 'users' + end + it "should have an engine" do @relation.engine.must_equal Table.engine end diff --git a/test/visitors/test_to_sql.rb b/test/visitors/test_to_sql.rb index 3af316037a..c47fd57a28 100644 --- a/test/visitors/test_to_sql.rb +++ b/test/visitors/test_to_sql.rb @@ -281,6 +281,15 @@ module Arel } end end + + describe 'TableAlias' do + it "should use the underlying table for checking columns" do + test = Table.new(:users).alias('zomgusers')[:id].eq '3' + @visitor.accept(test).must_be_like %{ + "zomgusers"."id" = 3 + } + end + end end end end |