arel_extensions 1.3.4 → 1.3.6

data/lib/arel_extensions/visitors/mysql.rb

@@ -14,6 +14,52 @@ module ArelExtensions
         '%M' => '%i', '%S' => '%S', '%L' =>   '', '%N' => '%f', '%z' => ''
       }.freeze
 
+      # This helper method did not exist in rails < 5.2
+      if !Arel::Visitors::MySQL.method_defined?(:collect_nodes_for)
+        def collect_nodes_for(nodes, collector, spacer, connector = ", ")
+          if nodes&.any?
+            collector << spacer
+            inject_join nodes, collector, connector
+          end
+        end
+      end
+
+        # The whole purpose of this override is to fix the behavior of RollUp.
+        # All other databases treat RollUp sanely, execpt MySQL which requires
+        # that it figures as the last element of a GROUP BY.
+        def visit_Arel_Nodes_SelectCore(o, collector)
+          collector << "SELECT"
+
+          collector = collect_optimizer_hints(o, collector) if self.respond_to?(:collect_optimizer_hinsts)
+          collector = maybe_visit o.set_quantifier, collector
+
+          collect_nodes_for o.projections, collector, " "
+
+          if o.source && !o.source.empty?
+            collector << " FROM "
+            collector = visit o.source, collector
+          end
+
+          # The actual work
+          groups = o.groups
+          rollup = groups.select { |g| g.expr.class == Arel::Nodes::RollUp }.map { |r| r.expr.value }
+          if rollup && !rollup.empty?
+            groups = o.groups.reject { |g| g.expr.class == Arel::Nodes::RollUp }
+            groups << Arel::Nodes::RollUp.new(rollup)
+          end
+          # FIN
+
+          collect_nodes_for o.wheres, collector, " WHERE ", " AND "
+          collect_nodes_for groups, collector, " GROUP BY "            # Look ma, I'm viring a group
+          collect_nodes_for o.havings, collector, " HAVING ", " AND "
+          collect_nodes_for o.windows, collector, " WINDOW "
+
+          if o.respond_to?(:comment)
+            maybe_visit o.comment, collector 
+          else
+            collector
+          end
+        end
 
       # Math functions
       def visit_ArelExtensions_Nodes_Log10 o, collector
@@ -126,7 +172,7 @@ module ArelExtensions
         collector << 'CONCAT('
         o.expressions.each_with_index { |arg, i|
           collector << COMMA if i != 0
-          if (arg.is_a?(Numeric)) || (arg.is_a?(Arel::Attributes::Attribute))
+          if arg.is_a?(Numeric) || arg.is_a?(Arel::Attributes::Attribute)
             collector << 'CAST('
             collector = visit arg, collector
             collector << ' AS char)'
@@ -138,6 +184,11 @@ module ArelExtensions
         collector
       end
 
+      def visit_Arel_Nodes_RollUp(o, collector)
+        visit o.expr, collector
+        collector << " WITH ROLLUP"
+      end
+
       def visit_ArelExtensions_Nodes_GroupConcat o, collector
         collector << 'GROUP_CONCAT('
         collector = visit o.left, collector
@@ -201,7 +252,18 @@ module ArelExtensions
       end
 
       def visit_ArelExtensions_Nodes_Format o, collector
-        case o.col_type
+        # One use case we met is
+        # `case…when…then(valid_date).else(Arel.null).format(…)`.
+        #
+        # In this case, `o.col_type` is `nil` but we have a legitimate type in
+        # the expression to be formatted.  The following is a best effort to
+        # infer the proper type.
+        type =
+          o.col_type.nil? && !o.expressions[0].return_type.nil? \
+          ? o.expressions[0].return_type \
+          : o.col_type
+
+        case type
         when :date, :datetime, :time
           fmt = ArelExtensions::Visitors::strftime_to_format(o.iso_format, DATE_FORMAT_DIRECTIVES)
           collector << 'DATE_FORMAT('
@@ -398,15 +460,20 @@ module ArelExtensions
             Arel::Nodes::NamedFunction.new('FORMAT', [col.abs] + params)
           end
 
-        repeated_char = (o.width == 0) ? Arel.quoted('') : ArelExtensions::Nodes::Case.new.
-          when(Arel.quoted(o.width).abs - (number.length + sign_length) > 0).
-          then(Arel.quoted(
-              o.flags.include?('-') ? ' ' : (o.flags.include?('0') ? '0' : ' ')
-            ).repeat(Arel.quoted(o.width).abs - (number.length + sign_length))
-          ).
-          else('')
-        before = (!o.flags.include?('0')) && (!o.flags.include?('-')) ? repeated_char : ''
-        middle = (o.flags.include?('0')) && (!o.flags.include?('-'))  ? repeated_char : ''
+        repeated_char =
+          if o.width == 0
+            Arel.quoted('')
+          else
+            Arel
+              .when(Arel.quoted(o.width).abs - (number.length + sign_length) > 0)
+              .then(Arel.quoted(
+                      o.flags.include?('-') ? ' ' : (o.flags.include?('0') ? '0' : ' ')
+                    ).repeat(Arel.quoted(o.width).abs - (number.length + sign_length))
+                   )
+              .else('')
+          end
+        before = !o.flags.include?('0') && !o.flags.include?('-') ? repeated_char : ''
+        middle = o.flags.include?('0') && !o.flags.include?('-')  ? repeated_char : ''
         after  = o.flags.include?('-') ? repeated_char : ''
         full_number = ArelExtensions::Nodes::Concat.new([
             before,

data/lib/arel_extensions/visitors/oracle.rb

@@ -11,7 +11,7 @@ module ArelExtensions
         '%H' => 'HH24', '%k' => '', '%I' => 'HH', '%l' => '', '%P' => 'am', '%p' => 'AM',                   # hours
         '%M' => 'MI', '%S' => 'SS', '%L' => 'MS', '%N' => 'US', '%z' => 'tz'                                # seconds, subseconds
       }
-      NUMBER_COMMA_MAPPING = { 'en_US' => '.,', 'fr_FR' => ',', 'sv_SE' => ', ' }
+      NUMBER_COMMA_MAPPING = {'en_US' => '.,', 'fr_FR' => ',', 'sv_SE' => ', '}
 
       def visit_ArelExtensions_Nodes_Log10 o, collector
         collector << 'LOG('
@@ -126,6 +126,11 @@ module ArelExtensions
       collector
       end
 
+      def visit_Arel_Nodes_RollUp(o, collector)
+        collector << "ROLLUP"
+        grouping_array_or_grouping_element o, collector
+      end
+
       def visit_ArelExtensions_Nodes_GroupConcat o, collector
         collector << '(LISTAGG('
         collector = visit o.left, collector
@@ -455,7 +460,7 @@ module ArelExtensions
           src_tz, dst_tz = o.time_zone.first
           collector << ' as timestamp) at time zone '
           collector = visit Arel.quoted(src_tz), collector
-          collecto < ' at time zone '
+          collector < ' at time zone '
           collector = visit Arel.quoted(dst_tz), collector
         when String
           collector << ' as timestamp) at time zone '
@@ -587,7 +592,7 @@ module ArelExtensions
       alias_method(:old_visit_Arel_Nodes_TableAlias, :visit_Arel_Nodes_TableAlias) rescue nil
       def visit_Arel_Nodes_TableAlias o, collector
         if o.name.length > 30
-          o = Arel::Table.new(o.table_name).alias(Base64.urlsafe_encode64(Digest::MD5.new.digest(o.name)).tr('=', '').tr('-', '_'))
+          o = Arel::Table.new(o.table_name).alias(Arel.shorten(o.name))
         end
         old_visit_Arel_Nodes_TableAlias(o, collector)
       end
@@ -601,7 +606,7 @@ module ArelExtensions
         else
           collector = visit o.left, collector
         end
-        quote = o.right.to_s =~ /(\A["].*["]\z)|\A[a-zA-Z_]*\z/ ? '' : '"'
+        quote = /(\A".*"\z)|\A[a-zA-Z_]*\z/.match?(o.right.to_s) ? '' : '"'
         collector << " AS #{quote}"
         collector = visit o.right, collector
         collector << "#{quote}"
@@ -627,10 +632,10 @@ module ArelExtensions
         comma_in_format = o.precision == 0 ? '' : 'D'
         nines_after = (1..o.precision - 1).map{'9'}.join('') + '0'
         if comma.length == 1
-          options = Arel.quoted("NLS_NUMERIC_CHARACTERS = '" + comma + " '")
+          options = Arel.quoted("NLS_NUMERIC_CHARACTERS = '#{comma} '")
           nines_before = ('999' * 4 + '990')
         else
-          options = Arel.quoted("NLS_NUMERIC_CHARACTERS = '" + comma + "'")
+          options = Arel.quoted("NLS_NUMERIC_CHARACTERS = '#{comma}'")
           nines_before = ('999G' * 4 + '990')
         end
         sign = Arel.when(col < 0).
@@ -643,7 +648,7 @@ module ArelExtensions
         if o.scientific_notation
           number = Arel::Nodes::NamedFunction.new('TO_CHAR', [
                 Arel.quoted(col.abs),
-                Arel.quoted('FM' + nines_before + comma_in_format + nines_after + 'EEEE'),
+                Arel.quoted("FM#{nines_before}#{comma_in_format}#{nines_after}EEEE"),
                 options
               ])
           if o.type == 'e'
@@ -652,20 +657,25 @@ module ArelExtensions
         else
           number = Arel::Nodes::NamedFunction.new('TO_CHAR', [
                 Arel.quoted(col.abs),
-                Arel.quoted('FM' + nines_before + comma_in_format + nines_after),
+                Arel.quoted("FM#{nines_before}#{comma_in_format}#{nines_after}"),
                 options
               ])
         end
 
-        repeated_char = (o.width == 0) ? Arel.quoted('') : ArelExtensions::Nodes::Case.new.
-          when(Arel.quoted(o.width).abs - (number.length + sign_length) > 0).
-          then(Arel.quoted(
-              o.flags.include?('-') ? ' ' : (o.flags.include?('0') ? '0' : ' ')
-            ).repeat(Arel.quoted(o.width).abs - (number.length + sign_length))
-          ).
-          else('')
-        before = (!o.flags.include?('0')) && (!o.flags.include?('-')) ? repeated_char : ''
-        middle = (o.flags.include?('0')) && (!o.flags.include?('-'))  ? repeated_char : ''
+        repeated_char =
+          if o.width == 0
+            Arel.quoted('')
+          else
+            Arel
+              .when(Arel.quoted(o.width).abs - (number.length + sign_length) > 0)
+              .then(Arel.quoted(
+                      o.flags.include?('-') ? ' ' : (o.flags.include?('0') ? '0' : ' ')
+                    ).repeat(Arel.quoted(o.width).abs - (number.length + sign_length))
+                   )
+              .else('')
+          end
+        before = !o.flags.include?('0') && !o.flags.include?('-') ? repeated_char : ''
+        middle = o.flags.include?('0') && !o.flags.include?('-')  ? repeated_char : ''
         after  = o.flags.include?('-') ? repeated_char : ''
         full_number = ArelExtensions::Nodes::Concat.new([
             before,
@@ -700,6 +710,18 @@ module ArelExtensions
         collector << ')'
         collector
       end
+
+      # Utilized by GroupingSet, Cube & RollUp visitors to
+      # handle grouping aggregation semantics
+      def grouping_array_or_grouping_element(o, collector)
+        if o.expr.is_a? Array
+          collector << "( "
+          visit o.expr, collector
+          collector << " )"
+        else
+          visit o.expr, collector
+        end
+      end
     end
   end
 end

data/lib/arel_extensions/visitors/postgresql.rb

@@ -425,32 +425,37 @@ module ArelExtensions
                 Arel::Nodes::NamedFunction.new('TRIM', [
                   Arel::Nodes::NamedFunction.new('TO_CHAR', [
                     Arel.when(col.not_eq 0).then(col.abs / Arel.quoted(10).pow(col.abs.log10.floor)).else(1),
-                    Arel.quoted('FM' + nines_before + '"' + comma + '"V' + nines_after)
+                    Arel.quoted("FM#{nines_before}\"#{comma}\"V#{nines_after}")
                   ])]),
                 o.type,
                 Arel::Nodes::NamedFunction.new('TRIM', [
                   Arel::Nodes::NamedFunction.new('TO_CHAR', [
                     Arel.when(col.not_eq 0).then(col.abs.log10.floor).else(0),
-                    Arel.quoted('FM' + nines_before)
+                    Arel.quoted("FM#{nines_before}")
                   ])])
               ])
           else
             Arel::Nodes::NamedFunction.new('TRIM', [
               Arel::Nodes::NamedFunction.new('TO_CHAR', [
                 Arel.quoted(col.abs),
-                Arel.quoted('FM' + nines_before + '"' + comma + '"V' + nines_after)
+                Arel.quoted("FM#{nines_before}\"#{comma}\"V#{nines_after}")
               ])])
           end
 
-        repeated_char = (o.width == 0) ? Arel.quoted('') : ArelExtensions::Nodes::Case.new.
-          when(Arel.quoted(o.width).abs - (number.length + sign_length) > 0).
-          then(Arel.quoted(
-              o.flags.include?('-') ? ' ' : (o.flags.include?('0') ? '0' : ' ')
-            ).repeat(Arel.quoted(o.width).abs - (number.length + sign_length))
-          ).
-          else('')
-        before = (!o.flags.include?('0')) && (!o.flags.include?('-')) ? repeated_char : ''
-        middle = (o.flags.include?('0')) && (!o.flags.include?('-'))  ? repeated_char : ''
+        repeated_char =
+          if o.width == 0
+            Arel.quoted('')
+          else
+            Arel
+              .when(Arel.quoted(o.width).abs - (number.length + sign_length) > 0)
+              .then(Arel.quoted(
+                      o.flags.include?('-') ? ' ' : (o.flags.include?('0') ? '0' : ' ')
+                    ).repeat(Arel.quoted(o.width).abs - (number.length + sign_length))
+                   )
+              .else('')
+          end
+        before = !o.flags.include?('0') && !o.flags.include?('-') ? repeated_char : ''
+        middle = o.flags.include?('0') && !o.flags.include?('-')  ? repeated_char : ''
         after  = o.flags.include?('-') ? repeated_char : ''
         full_number = ArelExtensions::Nodes::Concat.new([
             before,

data/lib/arel_extensions/visitors/sqlite.rb

@@ -325,16 +325,16 @@ module ArelExtensions
 
       def get_time_converted element
         if element.is_a?(Time)
-          return Arel::Nodes::NamedFunction.new('STRFTIME', [element, '%H:%M:%S'])
+          Arel::Nodes::NamedFunction.new('STRFTIME', [element, '%H:%M:%S'])
         elsif element.is_a?(Arel::Attributes::Attribute)
           col = Arel.column_of(element.relation.table_name, element.name.to_s)
           if col && (col.type == :time)
-            return Arel::Nodes::NamedFunction.new('STRFTIME', [element, '%H:%M:%S'])
+            Arel::Nodes::NamedFunction.new('STRFTIME', [element, '%H:%M:%S'])
           else
-            return element
+            element
           end
         else
-          return element
+          element
         end
       end
 

data/lib/arel_extensions/visitors/to_sql.rb

@@ -649,7 +649,10 @@ module ArelExtensions
 
       def visit_ArelExtensions_Nodes_JsonGroup o, collector
         if o.as_array
-          res = Arel.quoted('[') + (o.orders ? o.dict.group_concat(', ', order: Array(o.orders)) : o.dict.group_concat(', ')).coalesce('') + ']'
+          res =
+            Arel.quoted('[') \
+            + (o.orders ? o.dict.group_concat(', ', order: Array(o.orders)) : o.dict.group_concat(', ')).coalesce('') \
+            + ']'
           collector = visit res, collector
         else
           res = Arel.quoted('{')

data/lib/arel_extensions/visitors.rb

@@ -19,6 +19,14 @@ if defined?(Arel::Visitors::SQLServer)
   end
 end
 
+if defined?(Arel::Visitors::DepthFirst)
+  class Arel::Visitors::DepthFirst
+    def visit_Arel_SelectManager o
+        visit o.ast
+    end
+  end
+end
+
 if defined?(Arel::Visitors::MSSQL)
   class Arel::Visitors::MSSQL
     include ArelExtensions::Visitors::MSSQL

data/lib/arel_extensions.rb

@@ -1,4 +1,5 @@
 require 'arel'
+require 'base64'
 
 require 'arel_extensions/railtie' if defined?(Rails::Railtie)
 
@@ -77,6 +78,8 @@ require 'arel_extensions/nodes/case'
 require 'arel_extensions/nodes/soundex'
 require 'arel_extensions/nodes/cast'
 require 'arel_extensions/nodes/json'
+require 'arel_extensions/nodes/rollup'
+require 'arel_extensions/nodes/select'
 
 # It seems like the code in lib/arel_extensions/visitors.rb that is supposed
 # to inject ArelExtension is not enough. Different versions of the sqlserver
@@ -131,6 +134,10 @@ module Arel
     ArelExtensions::Nodes::Rand.new
   end
 
+  def self.rollup(*args)
+    Arel::Nodes::RollUp.new(args)
+  end
+
   def self.shorten s
     Base64.urlsafe_encode64(Digest::MD5.new.digest(s)).tr('=', '').tr('-', '_')
   end
@@ -268,3 +275,22 @@ class Arel::Nodes::TableAlias
     end
   end
 end
+
+
+class Arel::Attributes::Attribute
+  def to_sql(engine = Arel::Table.engine)
+    collector = Arel::Collectors::SQLString.new
+    collector = engine.connection.visitor.accept self, collector
+    collector.value
+  end
+
+  def rollup
+    Arel::Nodes::RollUp.new([self])
+  end
+end
+
+class Arel::Nodes::Node
+  def rollup
+    Arel::Nodes::RollUp.new([self])
+  end
+end

data/test/arelx_test_helper.rb

@@ -7,7 +7,7 @@ require 'active_record'
 require 'support/fake_record'
 
 def colored(color, msg)
-  ENV['TERM'] =~ /^xterm|-256color$/ ? "\x1b[#{color}m#{msg}\x1b[89m\x1b[0m" : "#{msg}"
+  /^xterm|-256color$/.match?(ENV['TERM']) ? "\x1b[#{color}m#{msg}\x1b[89m\x1b[0m" : "#{msg}"
 end
 
 YELLOW = '33'

data/test/real_db_test.rb

@@ -14,7 +14,7 @@ def setup_db
     ActiveRecord::Base.default_timezone = :utc
   end
   @cnx = ActiveRecord::Base.connection
-  if ActiveRecord::Base.connection.adapter_name =~ /sqlite/i
+  if /sqlite/i.match?(ActiveRecord::Base.connection.adapter_name)
     $sqlite = true
     db = @cnx.raw_connection
     if !$load_extension_disabled
@@ -82,7 +82,7 @@ class ListTest < Minitest::Test
   end
 
   def test_coalesce
-    if @cnx.adapter_name =~ /pgsql/i
+    if /pgsql/i.match?(@cnx.adapter_name)
       assert_equal 100, User.where(User.arel_table[:name].eq('Test')).select((User.arel_table[:age].coalesce(100)).as('res')).first.res
         assert_equal 'Camille', User.where(User.arel_table[:name].eq('Camille')).select((User.arel_table[:name].coalesce('Null', 'default')).as('res')).first.res
     else
@@ -128,7 +128,7 @@ class ListTest < Minitest::Test
   end
 
   def test_isnull
-    if ActiveRecord::Base.connection.adapter_name =~ /pgsql/i
+    if /pgsql/i.match?(ActiveRecord::Base.connection.adapter_name)
       assert_equal 100, User.where(User.arel_table[:name].eq('Test')).select((User.arel_table[:age].isnull(100)).as('res')).first.res
     else
       assert_equal 'default', User.where(User.arel_table[:name].eq('Test')).select((User.arel_table[:age].isnull('default')).as('res')).first.res
@@ -196,8 +196,8 @@ class ListTest < Minitest::Test
   end
 
   def test_replace
-    assert_equal 'LucaX', User.where(User.arel_table[:name].eq('Lucas')).select(((User.arel_table[:name]).replace('s', 'X')).as('res')).first.res
-    assert_equal 'replace', User.where(User.arel_table[:name].eq('Lucas')).select(((User.arel_table[:name]).replace(User.arel_table[:name], 'replace')).as('res')).first.res
+    assert_equal 'LucaX', User.where(User.arel_table[:name].eq('Lucas')).select((User.arel_table[:name].replace('s', 'X')).as('res')).first.res
+    assert_equal 'replace', User.where(User.arel_table[:name].eq('Lucas')).select((User.arel_table[:name].replace(User.arel_table[:name], 'replace')).as('res')).first.res
   end
 
   def test_round

data/test/support/fake_record.rb

@@ -66,7 +66,7 @@ module FakeRecord
     end
 
     def in_clause_length
-      10000
+      10_000
     end
 
     def quote thing, column = nil

data/test/visitors/test_bulk_insert_oracle.rb

@@ -7,10 +7,10 @@ module ArelExtensions
         @conn = FakeRecord::Base.new
         @visitor = Arel::Visitors::Oracle.new @conn.connection
         @table = Arel::Table.new(:users)
-        @cols = ['name', 'comments', 'created_at']
+        @cols = %w[name comments created_at]
         @data = [
-          ['nom1', 'sdfdsfdsfsdf', '2016-01-01'],
-          ['nom2', 'sdfdsfdsfsdf', '2016-01-01']
+          %w[nom1 sdfdsfdsfsdf 2016-01-01],
+          %w[nom2 sdfdsfdsfsdf 2016-01-01]
         ]
       end
 

data/test/visitors/test_bulk_insert_sqlite.rb

@@ -8,7 +8,7 @@ module ArelExtensions
         @visitor = Arel::Visitors::SQLite.new @conn.connection
         @table = Arel::Table.new(:users)
         Arel::Table.engine = @conn
-        @cols = ['id', 'name', 'comments', 'created_at']
+        @cols = %w[id name comments created_at]
         @data = [
           [23, 'nom1', 'sdfdsfdsfsdf', '2016-01-01'],
           [25, 'nom2', 'sdfdsfdsfsdf', '2016-01-01']

data/test/visitors/test_bulk_insert_to_sql.rb

@@ -8,7 +8,7 @@ module ArelExtensions
         Arel::Table.engine = @conn
         @visitor = Arel::Visitors::ToSql.new @conn.connection
         @table = Arel::Table.new(:users)
-        @cols = ['id', 'name', 'comments', 'created_at']
+        @cols = %w[id name comments created_at]
         @data = [
           [23, 'nom1', 'sdfdsfdsfsdf', '2016-01-01'],
           [25, 'nom2', 'sdfdsfdsfsdf', '2016-01-01']

data/test/visitors/test_to_sql.rb

@@ -11,7 +11,7 @@ module ArelExtensions
           # test become flaky.
           #
           # The first time `Arel::Table.engine` is called
-          # from `ArelExtenstions::column_of_via_arel_table(table_name, column_name)`
+          # from `ArelExtensions.column_of_via_arel_table(table_name, column_name)`
           # in   `lib/arel_extensions/helpers.rb`
           # will almost always fail. It's important to note that when the test
           # fails, it's always on 1 test case, and every subsequent test that
@@ -273,13 +273,13 @@ module ArelExtensions
         c = @table.project(@table[:name])
         _(compile(c.union_all(c)))
           .must_be_like %{(SELECT "users"."name" FROM "users") UNION ALL (SELECT "users"."name" FROM "users")}
-        _((c.union_all(c)).to_sql)
+        _(c.union_all(c).to_sql)
           .must_be_like %{(SELECT "users"."name" FROM "users") UNION ALL (SELECT "users"."name" FROM "users")}
-        _((c.union_all(c.union_all(c))).to_sql)
+        _(c.union_all(c.union_all(c)).to_sql)
           .must_be_like %{(SELECT "users"."name" FROM "users") UNION ALL (SELECT "users"."name" FROM "users") UNION ALL (SELECT "users"."name" FROM "users")}
-        _(((c.union_all(c)).union_all(c)).to_sql)
+        _((c.union_all(c)).union_all(c).to_sql)
           .must_be_like %{(SELECT "users"."name" FROM "users") UNION ALL (SELECT "users"."name" FROM "users") UNION ALL (SELECT "users"."name" FROM "users")}
-        _((c.union_all(c).union_all(c)).to_sql)
+        _(c.union_all(c).union_all(c).to_sql)
           .must_be_like %{(SELECT "users"."name" FROM "users") UNION ALL (SELECT "users"."name" FROM "users") UNION ALL (SELECT "users"."name" FROM "users")}
         _((c.union_all(c)).as('union_table').to_sql)
           .must_be_like %{((SELECT "users"."name" FROM "users") UNION ALL (SELECT "users"."name" FROM "users")) union_table}
@@ -576,7 +576,7 @@ module ArelExtensions
         end
       end
 
-      puts 'AREL VERSION : ' + Arel::VERSION.to_s
+      puts "AREL VERSION: #{Arel::VERSION}"
     end
   end
 end