From d3ba98ab1b561349f20f474369de70eda2952b9d Mon Sep 17 00:00:00 2001 From: Loic Nageleisen Date: Wed, 1 Mar 2017 17:02:19 +0100 Subject: [PATCH] fix join without clause --- lib/rebel/sql.rb | 2 +- test/test_raw.rb | 4 ++++ 2 files changed, 5 insertions(+), 1 deletion(-) diff --git a/lib/rebel/sql.rb b/lib/rebel/sql.rb index e2bdeaf..b4cb7d6 100644 --- a/lib/rebel/sql.rb +++ b/lib/rebel/sql.rb @@ -78,7 +78,7 @@ module Rebel::SQL end def on?(*clause) - clause ? on(clause) : self + clause.any? ? on(clause) : self end def and(*clause) diff --git a/test/test_raw.rb b/test/test_raw.rb index a7d702b..9d61aca 100644 --- a/test/test_raw.rb +++ b/test/test_raw.rb @@ -58,4 +58,8 @@ class TestRaw < Minitest::Test assert_str_equal(Rebel::SQL.where?(Rebel::SQL.name(:foo).eq(1).or(Rebel::SQL.name(:bar).eq(2)), Rebel::SQL.name(:baz).eq(3)), 'WHERE ("foo" = 1 OR "bar" = 2) AND "baz" = 3') assert_str_equal(Rebel::SQL.where?(Rebel::SQL.name(:foo).eq(1).or(Rebel::SQL.name(:bar).eq(2))), 'WHERE "foo" = 1 OR "bar" = 2') end + + def test_join + assert_str_equal(Rebel::SQL.join(:foo), 'JOIN "foo"') + end end