Permalink
Browse files

Merge pull request #106 from jhollinger/informix_joins

Patch Informix Visitor so that it includes joins
  • Loading branch information...
tenderlove committed Mar 14, 2012
2 parents 777b4b1 + 7cb1044 commit 660768f57cc1fd6e66a1f675b774f782e431cf13
Showing with 11 additions and 1 deletion.
  1. +1 −1 lib/arel/visitors/informix.rb
  2. +10 −0 test/visitors/test_informix.rb
@@ -15,7 +15,7 @@ def visit_Arel_Nodes_SelectStatement o
def visit_Arel_Nodes_SelectCore o
[
"#{o.projections.map { |x| visit x }.join ', '}",
- ("FROM #{visit o.froms}" if o.froms),
+ ("FROM #{visit(o.source)}" if o.source && !o.source.empty?),
("WHERE #{o.wheres.map { |x| visit x }.join ' AND ' }" unless o.wheres.empty?),
("GROUP BY #{o.groups.map { |x| visit x }.join ', ' }" unless o.groups.empty?),
(visit(o.having) if o.having),
@@ -37,6 +37,16 @@ module Visitors
sql.must_be_like "SELECT SKIP 1 LIMIT 1"
end
+ it 'uses INNER JOIN to perform joins' do
+ core = Nodes::SelectCore.new
+ table = Table.new(:posts)
+ core.source = Nodes::JoinSource.new(table, [table.create_join(Table.new(:comments))])
+
+ stmt = Nodes::SelectStatement.new([core])
+ sql = @visitor.accept(stmt)
+ sql.must_be_like 'SELECT FROM "posts" INNER JOIN "comments"'
+ end
+
end
end
end

0 comments on commit 660768f

Please sign in to comment.