flag_shih_tzu 0.3.13 → 0.3.14

data/lib/flag_shih_tzu/validators.rb

@@ -2,7 +2,7 @@
 unless defined?(::ActiveRecord)
   begin
     # If by some miracle it hasn't been loaded yet, try to load it.
-    require 'active_record'
+    require "active_record"
   rescue LoadError
     # If it fails to load, then assume the user is try to use flag_shih_tzu with some other database adapter
     warn "FlagShihTzu probably won't work unless you have some version of Active Record loaded. Versions >= 2.3 are supported."
@@ -12,13 +12,15 @@ end
 if defined?(::ActiveRecord) && ::ActiveRecord::VERSION::MAJOR >= 3
 
   module ActiveModel
+    # Open ActiveModel::Validations to define some additional ones
     module Validations
 
+      # A simple EachValidator that will check for the presence of the flags specified
       class PresenceOfFlagsValidator < EachValidator
         def validate_each(record, attribute, value)
           value = record.send(:read_attribute_for_validation, attribute)
           check_flag(record, attribute)
-          record.errors.add(attribute, :blank, options) if value.blank? or value == 0
+          record.errors.add(attribute, :blank, options) if value.blank? || value == 0
         end
 
         private
@@ -30,6 +32,7 @@ if defined?(::ActiveRecord) && ::ActiveRecord::VERSION::MAJOR >= 3
         end
       end
 
+      # Use these validators in your model
       module HelperMethods
         # Validates that the specified attributes are flags and are not blank.
         # Happens by default on save. Example:

data/lib/flag_shih_tzu/version.rb

@@ -1,3 +1,3 @@
 module FlagShihTzu
-  VERSION = "0.3.13"
+  VERSION = "0.3.14"
 end

data/test/database.yml

@@ -5,7 +5,7 @@ sqlite:
 mysql:
   adapter: mysql2
   database: flag_shih_tzu_test
-  username:
+  username: foss
   encoding: utf8
 postgresql:
   adapter: postgresql

data/test/flag_shih_tzu_test.rb

@@ -1,7 +1,7 @@
-require File.expand_path(File.dirname(__FILE__) + '/test_helper.rb')
+require File.expand_path(File.dirname(__FILE__) + "/test_helper.rb")
 
 class Spaceship < ActiveRecord::Base
-  self.table_name = 'spaceships'
+  self.table_name = "spaceships"
   include FlagShihTzu
 
   has_flags 1 => :warpdrive,
@@ -10,73 +10,91 @@ class Spaceship < ActiveRecord::Base
 end
 
 class SpaceshipWithoutNamedScopes < ActiveRecord::Base
-  self.table_name = 'spaceships'
+  self.table_name = "spaceships"
   include FlagShihTzu
 
-  has_flags(1 => :warpdrive, :named_scopes => false)
+  has_flags(1 => :warpdrive,
+            named_scopes: false)
 end
 
 class SpaceshipWithoutNamedScopesOldStyle < ActiveRecord::Base
-  self.table_name = 'spaceships'
+  self.table_name = "spaceships"
   include FlagShihTzu
 
-  has_flags({1 => :warpdrive}, :named_scopes => false)
+  has_flags({ 1 => :warpdrive },
+            named_scopes: false)
 end
 
 class SpaceshipWithCustomFlagsColumn < ActiveRecord::Base
-  self.table_name = 'spaceships_with_custom_flags_column'
+  self.table_name = "spaceships_with_custom_flags_column"
   include FlagShihTzu
 
-  has_flags(1 => :warpdrive, 2 => :hyperspace, :column => 'bits')
+  has_flags(1 => :warpdrive,
+            2 => :hyperspace,
+            column: "bits")
 end
 
 class SpaceshipWithColumnNameAsSymbol < ActiveRecord::Base
-  self.table_name = 'spaceships_with_custom_flags_column'
+  self.table_name = "spaceships_with_custom_flags_column"
   include FlagShihTzu
 
-  has_flags(1 => :warpdrive, 2 => :hyperspace, :column => :bits)
+  has_flags(1 => :warpdrive,
+            2 => :hyperspace,
+            column: :bits)
 end
 
 class SpaceshipWith2CustomFlagsColumn < ActiveRecord::Base
-  self.table_name = 'spaceships_with_2_custom_flags_column'
+  self.table_name = "spaceships_with_2_custom_flags_column"
   include FlagShihTzu
 
-  has_flags({ 1 => :warpdrive, 2 => :hyperspace }, :column => 'bits')
-  has_flags({ 1 => :jeanlucpicard, 2 => :dajanatroj }, :column => 'commanders')
+  has_flags(
+    { 1 => :warpdrive, 2 => :hyperspace },
+    column: "bits")
+  has_flags(
+    { 1 => :jeanlucpicard, 2 => :dajanatroj },
+    column: "commanders")
 end
 
 class SpaceshipWith3CustomFlagsColumn < ActiveRecord::Base
-  self.table_name = 'spaceships_with_3_custom_flags_column'
+  self.table_name = "spaceships_with_3_custom_flags_column"
   include FlagShihTzu
 
-  has_flags({ 1 => :warpdrive, 2 => :hyperspace }, :column => 'engines')
-  has_flags({ 1 => :photon, 2 => :laser, 3 => :ion_cannon, 4 => :particle_beam }, :column => 'weapons')
-  has_flags({ 1 => :power, 2 => :anti_ax_routine }, :column => 'hal3000')
+  has_flags({ 1 => :warpdrive,
+              2 => :hyperspace },
+            column: "engines")
+  has_flags({ 1 => :photon,
+              2 => :laser,
+              3 => :ion_cannon,
+              4 => :particle_beam },
+            column: "weapons")
+  has_flags({ 1 => :power,
+              2 => :anti_ax_routine },
+            column: "hal3000")
 end
 
 class SpaceshipWithInListQueryMode < ActiveRecord::Base
-  self.table_name = 'spaceships'
+  self.table_name = "spaceships"
   include FlagShihTzu
 
-  has_flags(1 => :warpdrive, 2 => :shields, :flag_query_mode => :in_list)
+  has_flags(1 => :warpdrive, 2 => :shields, flag_query_mode: :in_list)
 end
 
 class SpaceshipWithBitOperatorQueryMode < ActiveRecord::Base
-  self.table_name = 'spaceships'
+  self.table_name = "spaceships"
   include FlagShihTzu
 
-  has_flags(1 => :warpdrive, 2 => :shields, :flag_query_mode => :bit_operator)
+  has_flags(1 => :warpdrive, 2 => :shields, flag_query_mode: :bit_operator)
 end
 
 class SpaceshipWithBangMethods < ActiveRecord::Base
-  self.table_name = 'spaceships'
+  self.table_name = "spaceships"
   include FlagShihTzu
 
-  has_flags(1 => :warpdrive, 2 => :shields, :bang_methods => true)
+  has_flags(1 => :warpdrive, 2 => :shields, bang_methods: true)
 end
 
 class SpaceshipWithMissingFlags < ActiveRecord::Base
-  self.table_name = 'spaceships'
+  self.table_name = "spaceships"
   include FlagShihTzu
 
   has_flags 1 => :warpdrive,
@@ -88,44 +106,62 @@ end
 
 if (ActiveRecord::VERSION::MAJOR >= 3)
   class SpaceshipWithSymbolAndStringFlagColumns < ActiveRecord::Base
-    self.table_name = 'spaceships_with_symbol_and_string_flag_columns'
+    self.table_name = "spaceships_with_symbol_and_string_flag_columns"
     include FlagShihTzu
 
-    has_flags({ 1 => :warpdrive, 2 => :hyperspace }, :column => :peace, :check_for_column => true)
-    has_flags({ 1 => :photon, 2 => :laser, 3 => :ion_cannon, 4 => :particle_beam }, :column => :love, :check_for_column => true)
-    has_flags({ 1 => :power, 2 => :anti_ax_routine }, :column => 'happiness', :check_for_column => true)
+    has_flags({ 1 => :warpdrive,
+                2 => :hyperspace },
+              column: :peace,
+              check_for_column: true)
+    has_flags({ 1 => :photon,
+                2 => :laser,
+                3 => :ion_cannon,
+                4 => :particle_beam },
+              column: :love,
+              check_for_column: true)
+    has_flags({ 1 => :power,
+                2 => :anti_ax_routine },
+              column: "happiness",
+              check_for_column: true)
     validates_presence_of_flags :peace, :love
   end
 
   class SpaceshipWithValidationsAndCustomFlagsColumn < ActiveRecord::Base
-    self.table_name = 'spaceships_with_custom_flags_column'
+    self.table_name = "spaceships_with_custom_flags_column"
     include FlagShihTzu
 
-    has_flags(1 => :warpdrive, 2 => :hyperspace, :column => 'bits')
+    has_flags(1 => :warpdrive, 2 => :hyperspace, :column => "bits")
     validates_presence_of_flags :bits
   end
 
   class SpaceshipWithValidationsAnd3CustomFlagsColumn < ActiveRecord::Base
-    self.table_name = 'spaceships_with_3_custom_flags_column'
+    self.table_name = "spaceships_with_3_custom_flags_column"
     include FlagShihTzu
 
-    has_flags({ 1 => :warpdrive, 2 => :hyperspace }, :column => 'engines')
-    has_flags({ 1 => :photon, 2 => :laser, 3 => :ion_cannon, 4 => :particle_beam }, :column => 'weapons')
-    has_flags({ 1 => :power, 2 => :anti_ax_routine }, :column => 'hal3000')
+    has_flags(
+      { 1 => :warpdrive, 2 => :hyperspace },
+      column: "engines")
+    has_flags(
+      { 1 => :photon, 2 => :laser, 3 => :ion_cannon, 4 => :particle_beam },
+      column: "weapons")
+    has_flags(
+      { 1 => :power, 2 => :anti_ax_routine },
+      column: "hal3000")
 
     validates_presence_of_flags :engines, :weapons
   end
 
   class SpaceshipWithValidationsOnNonFlagsColumn < ActiveRecord::Base
-    self.table_name = 'spaceships_with_custom_flags_column'
+    self.table_name = "spaceships_with_custom_flags_column"
     include FlagShihTzu
 
-    has_flags(1 => :warpdrive, 2 => :hyperspace, :column => 'bits')
+    has_flags(1 => :warpdrive, 2 => :hyperspace, column: "bits")
     validates_presence_of_flags :id
   end
 end
 
-# table planets is missing intentionally to see if flagshihtzu handles missing tables gracefully
+# table planets is missing intentionally to see if
+#   flag_shih_tzu handles missing tables gracefully
 class Planet < ActiveRecord::Base
 end
 
@@ -139,13 +175,13 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     assert_raises ArgumentError do
       eval(<<-EOF
         class SpaceshipWithInvalidFlagKey < ActiveRecord::Base
-          self.table_name = 'spaceships'
+          self.table_name = "spaceships"
           include FlagShihTzu
 
           has_flags({ -1 => :error })
         end
-           EOF
-          )
+      EOF
+      )
     end
   end
 
@@ -153,14 +189,14 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     assert_raises ArgumentError do
       eval(<<-EOF
         class SpaceshipWithAlreadyUsedFlag < ActiveRecord::Base
-          self.table_name = 'spaceships_with_2_custom_flags_column'
+          self.table_name = "spaceships_with_2_custom_flags_column"
           include FlagShihTzu
 
-          has_flags({ 1 => :jeanluckpicard }, :column => 'bits')
-          has_flags({ 1 => :jeanluckpicard }, :column => 'commanders')
+          has_flags({ 1 => :jeanluckpicard }, column: "bits")
+          has_flags({ 1 => :jeanluckpicard }, column: "commanders")
         end
-           EOF
-          )
+      EOF
+      )
     end
   end
 
@@ -168,15 +204,15 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     assert_raises ArgumentError do
       eval(<<-EOF
         class SpaceshipWithAlreadyUsedMethod < ActiveRecord::Base
-          self.table_name = 'spaceships_with_2_custom_flags_column'
+          self.table_name = "spaceships_with_2_custom_flags_column"
           include FlagShihTzu
 
           def jeanluckpicard; end
 
-          has_flags({ 1 => :jeanluckpicard }, :column => 'bits')
+          has_flags({ 1 => :jeanluckpicard }, column: "bits")
         end
-           EOF
-          )
+      EOF
+      )
     end
   end
 
@@ -184,14 +220,22 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     assert_raises FlagShihTzu::DuplicateFlagColumnException do
       eval(<<-EOF
         class SpaceshipWithAlreadyUsedMethodByFlagshitzuStrict < ActiveRecord::Base
-          self.table_name = 'spaceships_with_2_custom_flags_column'
+          self.table_name = "spaceships_with_2_custom_flags_column"
           include FlagShihTzu
 
-          has_flags({ 1 => :jeanluckpicard }, :column => 'bits', :strict => true)
-          has_flags({ 1 => :jeanluckpicard }, :column => 'bits', :strict => true)
-        end
-           EOF
+          has_flags(
+            { 1 => :jeanluckpicard },
+            column: "bits",
+            strict: true
           )
+          has_flags(
+            { 1 => :jeanluckpicard },
+            column: "bits",
+            strict: true
+          )
+        end
+      EOF
+      )
     end
   end
 
@@ -199,14 +243,14 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     assert_nothing_raised ArgumentError do
       eval(<<-EOF
         class SpaceshipWithAlreadyUsedMethodByFlagshitzu < ActiveRecord::Base
-          self.table_name = 'spaceships_with_2_custom_flags_column'
+          self.table_name = "spaceships_with_2_custom_flags_column"
           include FlagShihTzu
 
-          has_flags({ 1 => :jeanluckpicard }, :column => 'bits')
-          has_flags({ 1 => :jeanluckpicard }, :column => 'bits')
+          has_flags({ 1 => :jeanluckpicard }, column: "bits")
+          has_flags({ 1 => :jeanluckpicard }, column: "bits")
         end
-           EOF
-          )
+      EOF
+      )
     end
   end
 
@@ -214,92 +258,138 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     assert_raises ArgumentError do
       eval(<<-EOF
         class SpaceshipWithInvalidFlagName < ActiveRecord::Base
-          self.table_name = 'spaceships'
+          self.table_name = "spaceships"
           include FlagShihTzu
 
-          has_flags({ 1 => 'error' })
+          has_flags({ 1 => "error" })
         end
-           EOF
-          )
+      EOF
+      )
     end
   end
 
   def test_should_define_a_sql_condition_method_for_flag_enabled
-    assert_equal "(spaceships.flags in (1,3,5,7))", Spaceship.warpdrive_condition
-    assert_equal "(spaceships.flags in (2,3,6,7))", Spaceship.shields_condition
-    assert_equal "(spaceships.flags in (4,5,6,7))", Spaceship.electrolytes_condition
+    assert_equal "(spaceships.flags in (1,3,5,7))",
+                 Spaceship.warpdrive_condition
+    assert_equal "(spaceships.flags in (2,3,6,7))",
+                 Spaceship.shields_condition
+    assert_equal "(spaceships.flags in (4,5,6,7))",
+                 Spaceship.electrolytes_condition
   end
 
   def test_should_define_a_sql_condition_method_for_flag_enabled_with_missing_flags
-    assert_equal "(spaceships.flags in (1,3,5,7))", SpaceshipWithMissingFlags.warpdrive_condition
-    assert_equal "(spaceships.flags in (4,5,6,7))", SpaceshipWithMissingFlags.electrolytes_condition
+    assert_equal "(spaceships.flags in (1,3,5,7))",
+                 SpaceshipWithMissingFlags.warpdrive_condition
+    assert_equal "(spaceships.flags in (4,5,6,7))",
+                 SpaceshipWithMissingFlags.electrolytes_condition
   end
 
   def test_should_accept_a_table_alias_option_for_sql_condition_method
-    assert_equal "(old_spaceships.flags in (1,3,5,7))", Spaceship.warpdrive_condition(:table_alias => 'old_spaceships')
+    assert_equal "(old_spaceships.flags in (1,3,5,7))",
+                 Spaceship.warpdrive_condition(table_alias: "old_spaceships")
   end
 
   def test_should_define_a_sql_condition_method_for_flag_enabled_with_2_colmns
-    assert_equal "(spaceships_with_2_custom_flags_column.bits in (1,3))", SpaceshipWith2CustomFlagsColumn.warpdrive_condition
-    assert_equal "(spaceships_with_2_custom_flags_column.bits in (2,3))", SpaceshipWith2CustomFlagsColumn.hyperspace_condition
-    assert_equal "(spaceships_with_2_custom_flags_column.commanders in (1,3))", SpaceshipWith2CustomFlagsColumn.jeanlucpicard_condition
-    assert_equal "(spaceships_with_2_custom_flags_column.commanders in (2,3))", SpaceshipWith2CustomFlagsColumn.dajanatroj_condition
+    assert_equal "(spaceships_with_2_custom_flags_column.bits in (1,3))",
+                 SpaceshipWith2CustomFlagsColumn.warpdrive_condition
+    assert_equal "(spaceships_with_2_custom_flags_column.bits in (2,3))",
+                 SpaceshipWith2CustomFlagsColumn.hyperspace_condition
+    assert_equal "(spaceships_with_2_custom_flags_column.commanders in (1,3))",
+                 SpaceshipWith2CustomFlagsColumn.jeanlucpicard_condition
+    assert_equal "(spaceships_with_2_custom_flags_column.commanders in (2,3))",
+                 SpaceshipWith2CustomFlagsColumn.dajanatroj_condition
   end
 
   def test_should_define_a_sql_condition_method_for_flag_not_enabled
-    assert_equal "(spaceships.flags not in (1,3,5,7))", Spaceship.not_warpdrive_condition
-    assert_equal "(spaceships.flags not in (2,3,6,7))", Spaceship.not_shields_condition
-    assert_equal "(spaceships.flags not in (4,5,6,7))", Spaceship.not_electrolytes_condition
+    assert_equal "(spaceships.flags not in (1,3,5,7))",
+                 Spaceship.not_warpdrive_condition
+    assert_equal "(spaceships.flags not in (2,3,6,7))",
+                 Spaceship.not_shields_condition
+    assert_equal "(spaceships.flags not in (4,5,6,7))",
+                 Spaceship.not_electrolytes_condition
   end
 
   def test_should_define_a_sql_condition_method_for_flag_not_enabled_with_missing_flags
-    assert_equal "(spaceships.flags not in (1,3,5,7))", SpaceshipWithMissingFlags.not_warpdrive_condition
-    assert_equal "(spaceships.flags not in (4,5,6,7))", SpaceshipWithMissingFlags.not_electrolytes_condition
+    assert_equal "(spaceships.flags not in (1,3,5,7))",
+                 SpaceshipWithMissingFlags.not_warpdrive_condition
+    assert_equal "(spaceships.flags not in (4,5,6,7))",
+                 SpaceshipWithMissingFlags.not_electrolytes_condition
   end
 
   def test_sql_condition_for_flag_with_custom_table_name_and_default_query_mode
-    assert_equal "(custom_spaceships.flags in (1,3,5,7))", Spaceship.send(:sql_condition_for_flag, :warpdrive, 'flags', true, 'custom_spaceships')
+    assert_equal "(custom_spaceships.flags in (1,3,5,7))",
+                 Spaceship.send(:sql_condition_for_flag,
+                                :warpdrive,
+                                "flags",
+                                true,
+                                "custom_spaceships")
   end
   def test_sql_condition_for_flag_with_in_list_query_mode
-    assert_equal "(spaceships.flags in (1,3))", SpaceshipWithInListQueryMode.send(:sql_condition_for_flag, :warpdrive, 'flags', true, 'spaceships')
+    assert_equal "(spaceships.flags in (1,3))",
+                 SpaceshipWithInListQueryMode.send(:sql_condition_for_flag,
+                                                   :warpdrive,
+                                                   "flags",
+                                                   true,
+                                                   "spaceships")
   end
   def test_sql_condition_for_flag_with_bit_operator_query_mode
-    assert_equal "(spaceships.flags & 1 = 1)", SpaceshipWithBitOperatorQueryMode.send(:sql_condition_for_flag, :warpdrive, 'flags', true, 'spaceships')
+    assert_equal "(spaceships.flags & 1 = 1)",
+                 SpaceshipWithBitOperatorQueryMode.send(:sql_condition_for_flag,
+                                                        :warpdrive,
+                                                        "flags",
+                                                        true,
+                                                        "spaceships")
   end
   def test_sql_in_for_flag
-    assert_equal [1,3,5,7], Spaceship.send(:sql_in_for_flag, :warpdrive, 'flags')
+    assert_equal [1, 3, 5, 7],
+                 Spaceship.send(:sql_in_for_flag, :warpdrive, "flags")
   end
   def test_sql_set_for_flag
-    assert_equal "flags = flags | 1", Spaceship.send(:sql_set_for_flag, :warpdrive, 'flags')
+    assert_equal "flags = flags | 1",
+                 Spaceship.send(:sql_set_for_flag, :warpdrive, "flags")
   end
 
   def test_should_define_a_sql_condition_method_for_flag_enabled_with_2_colmns_not_enabled
-    assert_equal "(spaceships_with_2_custom_flags_column.bits not in (1,3))", SpaceshipWith2CustomFlagsColumn.not_warpdrive_condition
-    assert_equal "(spaceships_with_2_custom_flags_column.bits not in (2,3))", SpaceshipWith2CustomFlagsColumn.not_hyperspace_condition
-    assert_equal "(spaceships_with_2_custom_flags_column.commanders not in (1,3))", SpaceshipWith2CustomFlagsColumn.not_jeanlucpicard_condition
-    assert_equal "(spaceships_with_2_custom_flags_column.commanders not in (2,3))", SpaceshipWith2CustomFlagsColumn.not_dajanatroj_condition
+    assert_equal "(spaceships_with_2_custom_flags_column.bits not in (1,3))",
+                 SpaceshipWith2CustomFlagsColumn.not_warpdrive_condition
+    assert_equal "(spaceships_with_2_custom_flags_column.bits not in (2,3))",
+                 SpaceshipWith2CustomFlagsColumn.not_hyperspace_condition
+    assert_equal "(spaceships_with_2_custom_flags_column.commanders not in (1,3))",
+                 SpaceshipWith2CustomFlagsColumn.not_jeanlucpicard_condition
+    assert_equal "(spaceships_with_2_custom_flags_column.commanders not in (2,3))",
+                 SpaceshipWith2CustomFlagsColumn.not_dajanatroj_condition
   end
 
   def test_should_define_a_sql_condition_method_for_flag_enabled_using_bit_operators
-    assert_equal "(spaceships.flags & 1 = 1)", SpaceshipWithBitOperatorQueryMode.warpdrive_condition
-    assert_equal "(spaceships.flags & 2 = 2)", SpaceshipWithBitOperatorQueryMode.shields_condition
+    assert_equal "(spaceships.flags & 1 = 1)",
+                 SpaceshipWithBitOperatorQueryMode.warpdrive_condition
+    assert_equal "(spaceships.flags & 2 = 2)",
+                 SpaceshipWithBitOperatorQueryMode.shields_condition
   end
 
   def test_should_define_a_sql_condition_method_for_flag_not_enabled_using_bit_operators
-    assert_equal "(spaceships.flags & 1 = 0)", SpaceshipWithBitOperatorQueryMode.not_warpdrive_condition
-    assert_equal "(spaceships.flags & 2 = 0)", SpaceshipWithBitOperatorQueryMode.not_shields_condition
+    assert_equal "(spaceships.flags & 1 = 0)",
+                 SpaceshipWithBitOperatorQueryMode.not_warpdrive_condition
+    assert_equal "(spaceships.flags & 2 = 0)",
+                 SpaceshipWithBitOperatorQueryMode.not_shields_condition
   end
 
   def test_should_define_a_named_scope_for_flag_enabled
-    assert_where_value "(spaceships.flags in (1,3,5,7))", Spaceship.warpdrive
-    assert_where_value "(spaceships.flags in (2,3,6,7))", Spaceship.shields
-    assert_where_value "(spaceships.flags in (4,5,6,7))", Spaceship.electrolytes
+    assert_where_value "(spaceships.flags in (1,3,5,7))",
+                       Spaceship.warpdrive
+    assert_where_value "(spaceships.flags in (2,3,6,7))",
+                       Spaceship.shields
+    assert_where_value "(spaceships.flags in (4,5,6,7))",
+                       Spaceship.electrolytes
   end
 
   def test_should_define_a_named_scope_for_flag_not_enabled
-    assert_where_value "(spaceships.flags not in (1,3,5,7))", Spaceship.not_warpdrive
-    assert_where_value "(spaceships.flags not in (2,3,6,7))", Spaceship.not_shields
-    assert_where_value "(spaceships.flags not in (4,5,6,7))", Spaceship.not_electrolytes
+    assert_where_value "(spaceships.flags not in (1,3,5,7))",
+                       Spaceship.not_warpdrive
+    assert_where_value "(spaceships.flags not in (2,3,6,7))",
+                       Spaceship.not_shields
+    assert_where_value "(spaceships.flags not in (4,5,6,7))",
+                       Spaceship.not_electrolytes
   end
 
   def test_should_define_a_dynamic_column_value_helpers_for_flags
@@ -308,27 +398,39 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
   end
 
   def test_should_define_a_named_scope_for_flag_enabled_with_2_columns
-    assert_where_value "(spaceships_with_2_custom_flags_column.bits in (1,3))", SpaceshipWith2CustomFlagsColumn.warpdrive
-    assert_where_value "(spaceships_with_2_custom_flags_column.bits in (2,3))", SpaceshipWith2CustomFlagsColumn.hyperspace
-    assert_where_value "(spaceships_with_2_custom_flags_column.commanders in (1,3))", SpaceshipWith2CustomFlagsColumn.jeanlucpicard
-    assert_where_value "(spaceships_with_2_custom_flags_column.commanders in (2,3))", SpaceshipWith2CustomFlagsColumn.dajanatroj
+    assert_where_value "(spaceships_with_2_custom_flags_column.bits in (1,3))",
+                       SpaceshipWith2CustomFlagsColumn.warpdrive
+    assert_where_value "(spaceships_with_2_custom_flags_column.bits in (2,3))",
+                       SpaceshipWith2CustomFlagsColumn.hyperspace
+    assert_where_value "(spaceships_with_2_custom_flags_column.commanders in (1,3))",
+                       SpaceshipWith2CustomFlagsColumn.jeanlucpicard
+    assert_where_value "(spaceships_with_2_custom_flags_column.commanders in (2,3))",
+                       SpaceshipWith2CustomFlagsColumn.dajanatroj
   end
 
   def test_should_define_a_named_scope_for_flag_not_enabled_with_2_columns
-    assert_where_value "(spaceships_with_2_custom_flags_column.bits not in (1,3))", SpaceshipWith2CustomFlagsColumn.not_warpdrive
-    assert_where_value "(spaceships_with_2_custom_flags_column.bits not in (2,3))", SpaceshipWith2CustomFlagsColumn.not_hyperspace
-    assert_where_value "(spaceships_with_2_custom_flags_column.commanders not in (1,3))", SpaceshipWith2CustomFlagsColumn.not_jeanlucpicard
-    assert_where_value "(spaceships_with_2_custom_flags_column.commanders not in (2,3))", SpaceshipWith2CustomFlagsColumn.not_dajanatroj
+    assert_where_value "(spaceships_with_2_custom_flags_column.bits not in (1,3))",
+                       SpaceshipWith2CustomFlagsColumn.not_warpdrive
+    assert_where_value "(spaceships_with_2_custom_flags_column.bits not in (2,3))",
+                       SpaceshipWith2CustomFlagsColumn.not_hyperspace
+    assert_where_value "(spaceships_with_2_custom_flags_column.commanders not in (1,3))",
+                       SpaceshipWith2CustomFlagsColumn.not_jeanlucpicard
+    assert_where_value "(spaceships_with_2_custom_flags_column.commanders not in (2,3))",
+                       SpaceshipWith2CustomFlagsColumn.not_dajanatroj
   end
 
   def test_should_define_a_named_scope_for_flag_enabled_using_bit_operators
-    assert_where_value "(spaceships.flags & 1 = 1)", SpaceshipWithBitOperatorQueryMode.warpdrive
-    assert_where_value "(spaceships.flags & 2 = 2)", SpaceshipWithBitOperatorQueryMode.shields
+    assert_where_value "(spaceships.flags & 1 = 1)",
+                       SpaceshipWithBitOperatorQueryMode.warpdrive
+    assert_where_value "(spaceships.flags & 2 = 2)",
+                       SpaceshipWithBitOperatorQueryMode.shields
   end
 
   def test_should_define_a_named_scope_for_flag_not_enabled_using_bit_operators
-    assert_where_value "(spaceships.flags & 1 = 0)", SpaceshipWithBitOperatorQueryMode.not_warpdrive
-    assert_where_value "(spaceships.flags & 2 = 0)", SpaceshipWithBitOperatorQueryMode.not_shields
+    assert_where_value "(spaceships.flags & 1 = 0)",
+                       SpaceshipWithBitOperatorQueryMode.not_warpdrive
+    assert_where_value "(spaceships.flags & 2 = 0)",
+                       SpaceshipWithBitOperatorQueryMode.not_shields
   end
 
   def test_should_work_with_raw_sql
@@ -342,10 +444,14 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     assert_equal true, spaceship.electrolytes
 
     if (ActiveRecord::VERSION::MAJOR <= 3)
-      Spaceship.update_all Spaceship.set_flag_sql(:warpdrive, true),
-                         ["id=?", spaceship.id]
+      Spaceship.update_all(
+        Spaceship.set_flag_sql(:warpdrive, true),
+        ["id=?", spaceship.id]
+      )
     else
-      Spaceship.where("id=?", spaceship.id).update_all Spaceship.set_flag_sql(:warpdrive, true)
+      Spaceship.where("id=?", spaceship.id).update_all(
+        Spaceship.set_flag_sql(:warpdrive, true)
+      )
     end
 
     spaceship.reload
@@ -365,10 +471,14 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     assert_equal true, spaceship.electrolytes
 
     if (ActiveRecord::VERSION::MAJOR <= 3)
-      Spaceship.update_all Spaceship.set_flag_sql(:shields, false),
-                           ["id=?", spaceship.id]
+      Spaceship.update_all(
+        Spaceship.set_flag_sql(:shields, false),
+        ["id=?", spaceship.id]
+      )
     else
-      Spaceship.where("id=?", spaceship.id).update_all Spaceship.set_flag_sql(:shields, false)
+      Spaceship.where("id=?", spaceship.id).update_all(
+        Spaceship.set_flag_sql(:shields, false)
+      )
     end
 
     spaceship.reload
@@ -401,9 +511,19 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
   end
 
   def test_should_return_the_correct_condition_with_chained_flags
-    assert_equal "(spaceships.flags in (3,7))", Spaceship.chained_flags_condition("flags", :warpdrive, :shields)
-    assert_equal "(spaceships.flags in (7))", Spaceship.chained_flags_condition("flags", :warpdrive, :shields, :electrolytes)
-    assert_equal "(spaceships.flags in (2,6))", Spaceship.chained_flags_condition("flags", :not_warpdrive, :shields)
+    assert_equal "(spaceships.flags in (3,7))",
+                 Spaceship.chained_flags_condition("flags",
+                                                   :warpdrive,
+                                                   :shields)
+    assert_equal "(spaceships.flags in (7))",
+                 Spaceship.chained_flags_condition("flags",
+                                                   :warpdrive,
+                                                   :shields,
+                                                   :electrolytes)
+    assert_equal "(spaceships.flags in (2,6))",
+                 Spaceship.chained_flags_condition("flags",
+                                                   :not_warpdrive,
+                                                   :shields)
   end
 
   def test_should_return_the_correct_number_of_items_with_chained_flags_with
@@ -423,12 +543,26 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     spaceship_4 = Spaceship.new
     spaceship_4.save!
     spaceship_4.reload
-    assert_equal 2, Spaceship.chained_flags_with("flags", :warpdrive).count
-    assert_equal 1, Spaceship.chained_flags_with("flags", :warpdrive, :shields).count
-    assert_equal 1, Spaceship.chained_flags_with("flags", :warpdrive, :not_shields).count
-    assert_equal 0, Spaceship.chained_flags_with("flags", :not_warpdrive, :shields, :electrolytes).count
-    assert_equal 1, Spaceship.chained_flags_with("flags", :not_warpdrive, :shields, :not_electrolytes).count
-    assert_equal 1, Spaceship.chained_flags_with("flags", :not_warpdrive, :not_shields, :not_electrolytes).count
+    assert_equal 2, Spaceship.chained_flags_with("flags",
+                                                 :warpdrive).count
+    assert_equal 1, Spaceship.chained_flags_with("flags",
+                                                 :warpdrive,
+                                                 :shields).count
+    assert_equal 1, Spaceship.chained_flags_with("flags",
+                                                 :warpdrive,
+                                                 :not_shields).count
+    assert_equal 0, Spaceship.chained_flags_with("flags",
+                                                 :not_warpdrive,
+                                                 :shields,
+                                                 :electrolytes).count
+    assert_equal 1, Spaceship.chained_flags_with("flags",
+                                                 :not_warpdrive,
+                                                 :shields,
+                                                 :not_electrolytes).count
+    assert_equal 1, Spaceship.chained_flags_with("flags",
+                                                 :not_warpdrive,
+                                                 :not_shields,
+                                                 :not_electrolytes).count
   end
 
   def test_should_not_define_named_scopes_if_not_wanted
@@ -442,15 +576,23 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     spaceship.enable_flag(:hyperspace)
     spaceship.save!
     spaceship.reload
-    assert_equal 3, spaceship.flags('bits')
-    assert_equal "(spaceships_with_custom_flags_column.bits in (1,3))", SpaceshipWithCustomFlagsColumn.warpdrive_condition
-    assert_equal "(spaceships_with_custom_flags_column.bits not in (1,3))", SpaceshipWithCustomFlagsColumn.not_warpdrive_condition
-    assert_equal "(spaceships_with_custom_flags_column.bits in (2,3))", SpaceshipWithCustomFlagsColumn.hyperspace_condition
-    assert_equal "(spaceships_with_custom_flags_column.bits not in (2,3))", SpaceshipWithCustomFlagsColumn.not_hyperspace_condition
-    assert_where_value "(spaceships_with_custom_flags_column.bits in (1,3))", SpaceshipWithCustomFlagsColumn.warpdrive
-    assert_where_value "(spaceships_with_custom_flags_column.bits not in (1,3))", SpaceshipWithCustomFlagsColumn.not_warpdrive
-    assert_where_value "(spaceships_with_custom_flags_column.bits in (2,3))", SpaceshipWithCustomFlagsColumn.hyperspace
-    assert_where_value "(spaceships_with_custom_flags_column.bits not in (2,3))", SpaceshipWithCustomFlagsColumn.not_hyperspace
+    assert_equal 3, spaceship.flags("bits")
+    assert_equal "(spaceships_with_custom_flags_column.bits in (1,3))",
+                 SpaceshipWithCustomFlagsColumn.warpdrive_condition
+    assert_equal "(spaceships_with_custom_flags_column.bits not in (1,3))",
+                 SpaceshipWithCustomFlagsColumn.not_warpdrive_condition
+    assert_equal "(spaceships_with_custom_flags_column.bits in (2,3))",
+                 SpaceshipWithCustomFlagsColumn.hyperspace_condition
+    assert_equal "(spaceships_with_custom_flags_column.bits not in (2,3))",
+                 SpaceshipWithCustomFlagsColumn.not_hyperspace_condition
+    assert_where_value %[(spaceships_with_custom_flags_column.bits in (1,3))],
+                       SpaceshipWithCustomFlagsColumn.warpdrive
+    assert_where_value %[(spaceships_with_custom_flags_column.bits not in (1,3))],
+                       SpaceshipWithCustomFlagsColumn.not_warpdrive
+    assert_where_value %[(spaceships_with_custom_flags_column.bits in (2,3))],
+                       SpaceshipWithCustomFlagsColumn.hyperspace
+    assert_where_value %[(spaceships_with_custom_flags_column.bits not in (2,3))],
+                       SpaceshipWithCustomFlagsColumn.not_hyperspace
   end
 
   def test_should_work_with_a_custom_flags_column_name_as_symbol
@@ -458,7 +600,7 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     spaceship.enable_flag(:warpdrive)
     spaceship.save!
     spaceship.reload
-    assert_equal 1, spaceship.flags('bits')
+    assert_equal 1, spaceship.flags("bits")
   end
 
   def test_should_not_error_out_when_table_is_not_present
@@ -474,10 +616,12 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
     assert_nothing_raised(ActiveRecord::StatementInvalid) do
       Planet.class_eval do
         # Now it has a table that exists, but the column does not.
-        self.table_name = 'spaceships'
+        self.table_name = "spaceships"
         include FlagShihTzu
 
-        has_flags({ 1 => :warpdrive, 2 => :hyperspace }, :column => :i_do_not_exist, :check_for_column => true)
+        has_flags({ 1 => :warpdrive, 2 => :hyperspace },
+                  column: :i_do_not_exist,
+                  check_for_column: true)
       end
     end
   end
@@ -485,8 +629,12 @@ class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
   private
 
   def assert_where_value(expected, scope)
-    assert_equal expected,
-      ActiveRecord::VERSION::MAJOR == 2 ? scope.proxy_options[:conditions] : scope.where_values.first
+    actual = if ActiveRecord::VERSION::MAJOR == 2
+               scope.proxy_options[:conditions]
+             else
+               scope.where_values.first
+             end
+    assert_equal expected, actual
   end
 
 end
@@ -553,8 +701,8 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     @big_spaceship.save!
     @big_spaceship.reload
 
-    assert_equal 1, @big_spaceship.flags('bits')
-    assert_equal 2, @big_spaceship.flags('commanders')
+    assert_equal 1, @big_spaceship.flags("bits")
+    assert_equal 2, @big_spaceship.flags("commanders")
 
     assert @big_spaceship.flag_enabled?(:warpdrive)
     assert !@big_spaceship.flag_enabled?(:hyperspace)
@@ -586,25 +734,29 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
   # --------------------------------------------------
 
   def test_should_define_an_all_flags_reader_method_with_arity_1
-    assert_array_similarity [:electrolytes, :warpdrive, :shields], @spaceship.all_flags('flags')
+    assert_array_similarity [:electrolytes, :warpdrive, :shields],
+                            @spaceship.all_flags("flags")
   end
 
   def test_should_define_an_all_flags_reader_method_with_arity_0
-    assert_array_similarity [:electrolytes, :warpdrive, :shields], @spaceship.all_flags
+    assert_array_similarity [:electrolytes, :warpdrive, :shields],
+                            @spaceship.all_flags
   end
 
   def test_should_define_a_selected_flags_reader_method_with_arity_1
-    assert_array_similarity [], @spaceship.selected_flags('flags')
+    assert_array_similarity [], @spaceship.selected_flags("flags")
 
     @spaceship.warpdrive = true
-    assert_array_similarity [:warpdrive], @spaceship.selected_flags('flags')
+    assert_array_similarity [:warpdrive],
+                            @spaceship.selected_flags("flags")
 
     @spaceship.electrolytes = true
-    assert_array_similarity [:electrolytes, :warpdrive], @spaceship.selected_flags('flags')
+    assert_array_similarity [:electrolytes, :warpdrive],
+                            @spaceship.selected_flags("flags")
 
     @spaceship.warpdrive = false
     @spaceship.electrolytes = false
-    assert_array_similarity [], @spaceship.selected_flags('flags')
+    assert_array_similarity [], @spaceship.selected_flags("flags")
   end
 
   def test_should_define_a_selected_flags_reader_method_with_arity_0
@@ -614,7 +766,8 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     assert_array_similarity [:warpdrive], @spaceship.selected_flags
 
     @spaceship.electrolytes = true
-    assert_array_similarity [:electrolytes, :warpdrive], @spaceship.selected_flags
+    assert_array_similarity [:electrolytes, :warpdrive],
+                            @spaceship.selected_flags
 
     @spaceship.warpdrive = false
     @spaceship.electrolytes = false
@@ -622,7 +775,7 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
   end
 
   def test_should_define_a_select_all_flags_method_with_arity_1
-    @spaceship.select_all_flags('flags')
+    @spaceship.select_all_flags("flags")
     assert @spaceship.warpdrive
     assert @spaceship.shields
     assert @spaceship.electrolytes
@@ -640,7 +793,7 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     @spaceship.shields = true
     @spaceship.electrolytes = true
 
-    @spaceship.unselect_all_flags('flags')
+    @spaceship.unselect_all_flags("flags")
 
     assert !@spaceship.warpdrive
     assert !@spaceship.shields
@@ -660,19 +813,19 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
   end
 
   def test_should_define_an_has_flag_method_with_arity_1
-    assert !@spaceship.has_flag?('flags')
+    assert !@spaceship.has_flag?("flags")
 
     @spaceship.warpdrive = true
-    assert @spaceship.has_flag?('flags')
+    assert @spaceship.has_flag?("flags")
 
     @spaceship.shields = true
-    assert @spaceship.has_flag?('flags')
+    assert @spaceship.has_flag?("flags")
 
     @spaceship.electrolytes = true
-    assert @spaceship.has_flag?('flags')
+    assert @spaceship.has_flag?("flags")
 
-    @spaceship.unselect_all_flags('flags')
-    assert !@spaceship.has_flag?('flags')
+    @spaceship.unselect_all_flags("flags")
+    assert !@spaceship.has_flag?("flags")
   end
 
   def test_should_define_an_has_flag_method_with_arity_0
@@ -694,17 +847,20 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
   # --------------------------------------------------
 
   def test_should_define_a_customized_all_flags_reader_method
-    assert_array_similarity [:hyperspace, :warpdrive], @small_spaceship.all_bits
+    assert_array_similarity [:hyperspace, :warpdrive],
+                            @small_spaceship.all_bits
   end
 
   def test_should_define_a_customized_selected_flags_reader_method
     assert_array_similarity [], @small_spaceship.selected_bits
 
     @small_spaceship.warpdrive = true
-    assert_array_similarity [:warpdrive], @small_spaceship.selected_bits
+    assert_array_similarity [:warpdrive],
+                            @small_spaceship.selected_bits
 
     @small_spaceship.hyperspace = true
-    assert_array_similarity [:hyperspace, :warpdrive], @small_spaceship.selected_bits
+    assert_array_similarity [:hyperspace, :warpdrive],
+                            @small_spaceship.selected_bits
 
     @small_spaceship.warpdrive = false
     @small_spaceship.hyperspace = false
@@ -761,25 +917,33 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
   # --------------------------------------------------
 
   def test_should_define_a_customized_all_flags_reader_method_with_2_columns
-    assert_array_similarity [:hyperspace, :warpdrive], @big_spaceship.all_bits
-    assert_array_similarity [:dajanatroj, :jeanlucpicard], @big_spaceship.all_commanders
+    assert_array_similarity [:hyperspace, :warpdrive],
+                            @big_spaceship.all_bits
+    assert_array_similarity [:dajanatroj, :jeanlucpicard],
+                            @big_spaceship.all_commanders
   end
 
   def test_should_define_a_customized_selected_flags_reader_method_with_2_columns
-    assert_array_similarity [], @big_spaceship.selected_bits
-    assert_array_similarity [], @big_spaceship.selected_commanders
+    assert_array_similarity [],
+                            @big_spaceship.selected_bits
+    assert_array_similarity [],
+                            @big_spaceship.selected_commanders
 
     @big_spaceship.warpdrive = true
     @big_spaceship.jeanlucpicard = true
-    assert_array_similarity [:warpdrive], @big_spaceship.selected_bits
-    assert_array_similarity [:jeanlucpicard], @big_spaceship.selected_commanders
+    assert_array_similarity [:warpdrive],
+                            @big_spaceship.selected_bits
+    assert_array_similarity [:jeanlucpicard],
+                            @big_spaceship.selected_commanders
 
     @big_spaceship.hyperspace = true
     @big_spaceship.hyperspace = true
     @big_spaceship.jeanlucpicard = true
     @big_spaceship.dajanatroj = true
-    assert_array_similarity [:hyperspace, :warpdrive], @big_spaceship.selected_bits
-    assert_array_similarity [:dajanatroj, :jeanlucpicard], @big_spaceship.selected_commanders
+    assert_array_similarity [:hyperspace, :warpdrive],
+                            @big_spaceship.selected_bits
+    assert_array_similarity [:dajanatroj, :jeanlucpicard],
+                            @big_spaceship.selected_commanders
 
     @big_spaceship.warpdrive = false
     @big_spaceship.hyperspace = false
@@ -903,12 +1067,12 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
   end
 
   def test_should_respect_true_values_like_active_record
-    [true, 1, '1', 't', 'T', 'true', 'TRUE'].each do |true_value|
+    [true, 1, "1", "t", "T", "true", "TRUE"].each do |true_value|
       @spaceship.warpdrive = true_value
       assert @spaceship.warpdrive
     end
 
-    [false, 0, '0', 'f', 'F', 'false', 'FALSE'].each do |false_value|
+    [false, 0, "0", "f", "F", "false", "FALSE"].each do |false_value|
       @spaceship.warpdrive = false_value
       assert !@spaceship.warpdrive
     end
@@ -918,7 +1082,7 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     assert_nothing_raised do
       eval(<<-EOF
         class SpaceshipWithoutFlagsColumn1 < ActiveRecord::Base
-          self.table_name = 'spaceships_without_flags_column'
+          self.table_name = "spaceships_without_flags_column"
           include FlagShihTzu
 
           has_flags 1 => :warpdrive,
@@ -936,7 +1100,7 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     assert_raises FlagShihTzu::IncorrectFlagColumnException do
       eval(<<-EOF
         class SpaceshipWithNonIntegerColumn1 < ActiveRecord::Base
-          self.table_name ='spaceships_with_non_integer_column'
+          self.table_name ="spaceships_with_non_integer_column"
           include FlagShihTzu
 
           has_flags 1 => :warpdrive,
@@ -954,18 +1118,18 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     assert_nothing_raised do
       eval(<<-EOF
         class SpaceshipWithoutFlagsColumn2 < ActiveRecord::Base
-          self.table_name = 'spaceships_without_flags_column'
+          self.table_name = "spaceships_without_flags_column"
           include FlagShihTzu
 
           has_flags 1 => :warpdrive,
                     2 => :shields,
                     3 => :electrolytes,
-                    :check_for_column => true
+                    check_for_column: true
         end
       EOF
       )
     end
-    assert !SpaceshipWithoutFlagsColumn2.send(:check_flag_column, 'flags')
+    assert !SpaceshipWithoutFlagsColumn2.send(:check_flag_column, "flags")
     assert !SpaceshipWithoutFlagsColumn2.method_defined?(:warpdrive)
   end
 
@@ -973,13 +1137,13 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     assert_raises FlagShihTzu::IncorrectFlagColumnException do
       eval(<<-EOF
         class SpaceshipWithNonIntegerColumn2 < ActiveRecord::Base
-          self.table_name ='spaceships_with_non_integer_column'
+          self.table_name ="spaceships_with_non_integer_column"
           include FlagShihTzu
 
           has_flags 1 => :warpdrive,
                     2 => :shields,
                     3 => :electrolytes,
-                    :check_for_column => true
+                    check_for_column: true
         end
       EOF
       )
@@ -992,13 +1156,13 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     assert_nothing_raised do
       eval(<<-EOF
         class SpaceshipWithoutFlagsColumn3 < ActiveRecord::Base
-          self.table_name = 'spaceships_without_flags_column'
+          self.table_name = "spaceships_without_flags_column"
           include FlagShihTzu
 
           has_flags 1 => :warpdrive,
                     2 => :shields,
                     3 => :electrolytes,
-                    :check_for_column => false
+                    check_for_column: false
         end
       EOF
       )
@@ -1011,13 +1175,13 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     assert_nothing_raised do
       eval(<<-EOF
         class SpaceshipWithNonIntegerColumn3 < ActiveRecord::Base
-          self.table_name ='spaceships_with_non_integer_column'
+          self.table_name ="spaceships_with_non_integer_column"
           include FlagShihTzu
 
           has_flags 1 => :warpdrive,
                     2 => :shields,
                     3 => :electrolytes,
-                    :check_for_column => false
+                    check_for_column: false
         end
       EOF
       )
@@ -1027,7 +1191,8 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
   end
 
   def test_column_guessing_for_default_column_2
-    assert_equal 'flags', @spaceship.class.determine_flag_colmn_for(:warpdrive)
+    assert_equal "flags",
+                 @spaceship.class.determine_flag_colmn_for(:warpdrive)
   end
 
   def test_column_guessing_for_default_column_1
@@ -1037,8 +1202,10 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
   end
 
   def test_column_guessing_for_2_columns
-    assert_equal 'commanders', @big_spaceship.class.determine_flag_colmn_for(:jeanlucpicard)
-    assert_equal 'bits', @big_spaceship.class.determine_flag_colmn_for(:warpdrive)
+    assert_equal "commanders",
+                 @big_spaceship.class.determine_flag_colmn_for(:jeanlucpicard)
+    assert_equal "bits",
+                 @big_spaceship.class.determine_flag_colmn_for(:warpdrive)
   end
 
   def test_update_flag_without_updating_instance!
@@ -1053,7 +1220,8 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     assert_equal true, my_spaceship.update_flag!(:jeanlucpicard, false)
     assert_equal true, my_spaceship.update_flag!(:warpdrive, true)
 
-    # Not updating the instance here, so it won't reflect the result of the SQL update until after reloaded
+    # Not updating the instance here,
+    #   so it won't reflect the result of the SQL update until after reloaded
     assert_equal true, my_spaceship.jeanlucpicard
     assert_equal false, my_spaceship.warpdrive
 
@@ -1075,7 +1243,8 @@ class FlagShihTzuInstanceMethodsTest < Test::Unit::TestCase
     assert_equal true, my_spaceship.update_flag!(:jeanlucpicard, false, true)
     assert_equal true, my_spaceship.update_flag!(:warpdrive, true, true)
 
-    # Updating the instance here, so it will reflect the result of the SQL update before and after reload
+    # Updating the instance here,
+    #   so it will reflect the result of the SQL update before and after reload
     assert_equal false, my_spaceship.jeanlucpicard
     assert_equal true, my_spaceship.warpdrive
 
@@ -1230,12 +1399,12 @@ class FlagShihTzuDerivedClassTest < Test::Unit::TestCase
   end
 
   def test_should_respect_true_values_like_active_record
-    [true, 1, '1', 't', 'T', 'true', 'TRUE'].each do |true_value|
+    [true, 1, "1", "t", "T", "true", "TRUE"].each do |true_value|
       @spaceship.warpdrive = true_value
       assert @spaceship.warpdrive
     end
 
-    [false, 0, '0', 'f', 'F', 'false', 'FALSE'].each do |false_value|
+    [false, 0, "0", "f", "F", "false", "FALSE"].each do |false_value|
       @spaceship.warpdrive = false_value
       assert !@spaceship.warpdrive
     end
@@ -1250,8 +1419,10 @@ class FlagShihTzuDerivedClassTest < Test::Unit::TestCase
   end
 
   def test_should_return_a_sql_set_method_for_flag
-    assert_equal "flags = flags | 1",  Spaceship.send( :sql_set_for_flag, :warpdrive, 'flags', true)
-    assert_equal "flags = flags & ~1", Spaceship.send( :sql_set_for_flag, :warpdrive, 'flags', false)
+    assert_equal "flags = flags | 1",
+                 Spaceship.send(:sql_set_for_flag, :warpdrive, "flags", true)
+    assert_equal "flags = flags & ~1",
+                 Spaceship.send(:sql_set_for_flag, :warpdrive, "flags", false)
   end
 
 end
@@ -1259,12 +1430,16 @@ end
 class FlagShihTzuClassMethodsTest < Test::Unit::TestCase
 
   def test_should_track_columns_used_by_FlagShihTzu
-    assert_equal Spaceship.flag_columns, ['flags']
-    assert_equal SpaceshipWith2CustomFlagsColumn.flag_columns, ['bits', 'commanders']
-    assert_equal SpaceshipWith3CustomFlagsColumn.flag_columns, ['engines', 'weapons', 'hal3000']
+    assert_equal Spaceship.flag_columns, ["flags"]
+    assert_equal SpaceshipWith2CustomFlagsColumn.flag_columns,
+                 ["bits", "commanders"]
+    assert_equal SpaceshipWith3CustomFlagsColumn.flag_columns,
+                 ["engines", "weapons", "hal3000"]
     if (ActiveRecord::VERSION::MAJOR >= 3)
-      assert_equal SpaceshipWithValidationsAnd3CustomFlagsColumn.flag_columns, ["engines", "weapons", "hal3000"]
-      assert_equal SpaceshipWithSymbolAndStringFlagColumns.flag_columns, ['peace', 'love', 'happiness']
+      assert_equal SpaceshipWithValidationsAnd3CustomFlagsColumn.flag_columns,
+                   ["engines", "weapons", "hal3000"]
+      assert_equal SpaceshipWithSymbolAndStringFlagColumns.flag_columns,
+                   ["peace", "love", "happiness"]
     end
   end