lhm-shopify 3.4.0 → 3.4.1

data/spec/integration/lhm_spec.rb

@@ -17,7 +17,7 @@ describe Lhm do
       end
 
       slave do
-        table_read(:users).columns['logins'].must_equal({
+        value(table_read(:users).columns['logins']).must_equal({
           :type           => 'int(12)',
           :is_nullable    => 'YES',
           :column_default => '0',
@@ -35,7 +35,7 @@ describe Lhm do
       end
 
       slave do
-        table_read(:custom_primary_key).columns['logins'].must_equal({
+        value(table_read(:custom_primary_key).columns['logins']).must_equal({
           :type           => 'int(12)',
           :is_nullable    => 'YES',
           :column_default => '0',
@@ -53,7 +53,7 @@ describe Lhm do
       end
 
       slave do
-        table_read(:composite_primary_key).columns['logins'].must_equal({
+        value(table_read(:composite_primary_key).columns['logins']).must_equal({
           :type           => 'int(12)',
           :is_nullable    => 'YES',
           :column_default => '0',
@@ -82,7 +82,7 @@ describe Lhm do
         end
 
         slave do
-          connection.primary_key('users').must_equal(['username', 'id'])
+          value(connection.primary_key('users')).must_equal(['username', 'id'])
         end
       end
 
@@ -104,7 +104,7 @@ describe Lhm do
 
         it 'migrates the existing data' do
           slave do
-            count_all(:permissions).must_equal(11)
+            value(count_all(:permissions)).must_equal(11)
           end
         end
       end
@@ -120,7 +120,7 @@ describe Lhm do
 
         it 'migrates all data' do
           slave do
-            count_all(:permissions).must_equal(13)
+            value(count_all(:permissions)).must_equal(13)
           end
         end
       end
@@ -132,7 +132,7 @@ describe Lhm do
       end
 
       slave do
-        table_read(:users).columns['logins'].must_equal({
+        value(table_read(:users).columns['logins']).must_equal({
           :type => 'int(12)',
           :is_nullable => 'YES',
           :column_default => '0',
@@ -150,7 +150,7 @@ describe Lhm do
       end
 
       slave do
-        count_all(:users).must_equal(23)
+        value(count_all(:users)).must_equal(23)
       end
     end
 
@@ -170,7 +170,7 @@ describe Lhm do
       end
 
       slave do
-        index_on_columns?(:users, [:comment, :created_at]).must_equal(true)
+        value(index_on_columns?(:users, [:comment, :created_at])).must_equal(true)
       end
     end
 
@@ -180,7 +180,7 @@ describe Lhm do
       end
 
       slave do
-        index?(:users, :my_index_name).must_equal(true)
+        value(index?(:users, :my_index_name)).must_equal(true)
       end
     end
 
@@ -190,7 +190,7 @@ describe Lhm do
       end
 
       slave do
-        index_on_columns?(:users, :group).must_equal(true)
+        value(index_on_columns?(:users, :group)).must_equal(true)
       end
     end
 
@@ -200,7 +200,7 @@ describe Lhm do
       end
 
       slave do
-        index_on_columns?(:users, :comment, :unique).must_equal(true)
+        value(index_on_columns?(:users, :comment, :unique)).must_equal(true)
       end
     end
 
@@ -210,7 +210,7 @@ describe Lhm do
       end
 
       slave do
-        index_on_columns?(:users, [:username, :created_at]).must_equal(false)
+        value(index_on_columns?(:users, [:username, :created_at])).must_equal(false)
       end
     end
 
@@ -220,7 +220,7 @@ describe Lhm do
       end
 
       slave do
-        index?(:users, :index_with_a_custom_name).must_equal(false)
+        value(index?(:users, :index_with_a_custom_name)).must_equal(false)
       end
     end
 
@@ -230,7 +230,7 @@ describe Lhm do
       end
 
       slave do
-        index?(:users, :index_with_a_custom_name).must_equal(false)
+        value(index?(:users, :index_with_a_custom_name)).must_equal(false)
       end
     end
 
@@ -240,7 +240,7 @@ describe Lhm do
       end
 
       slave do
-        table_read(:users).columns['flag'].must_equal({
+        value(table_read(:users).columns['flag']).must_equal({
           :type => 'tinyint(1)',
           :is_nullable => 'YES',
           :column_default => nil,
@@ -256,7 +256,7 @@ describe Lhm do
       end
 
       slave do
-        table_read(:users).columns['comment'].must_equal({
+        value(table_read(:users).columns['comment']).must_equal({
           :type => 'varchar(20)',
           :is_nullable => 'NO',
           :column_default => 'none',
@@ -274,7 +274,7 @@ describe Lhm do
       end
 
       slave do
-        table_read(:small_table).columns['id'].must_equal({
+        value(table_read(:small_table).columns['id']).must_equal({
           :type => 'int(5)',
           :is_nullable => 'NO',
           :column_default => nil,
@@ -295,7 +295,7 @@ describe Lhm do
       slave do
         table_data = table_read(:users)
         assert_nil table_data.columns['username']
-        table_read(:users).columns['login'].must_equal({
+        value(table_read(:users).columns['login']).must_equal({
           :type => 'varchar(255)',
           :is_nullable => 'YES',
           :column_default => nil,
@@ -305,7 +305,7 @@ describe Lhm do
 
         result = select_one('SELECT login from users')
         result = result['login'] if result.respond_to?(:has_key?)
-        result.must_equal('a user')
+        value(result).must_equal('a user')
       end
     end
 
@@ -320,7 +320,7 @@ describe Lhm do
       slave do
         table_data = table_read(:users)
         assert_nil table_data.columns['group']
-        table_read(:users).columns['fnord'].must_equal({
+        value(table_read(:users).columns['fnord']).must_equal({
           :type => 'varchar(255)',
           :is_nullable => 'YES',
           :column_default => 'Superfriends',
@@ -330,7 +330,7 @@ describe Lhm do
 
         result = select_one('SELECT `fnord` from users')
         result = result['fnord'] if result.respond_to?(:has_key?)
-        result.must_equal('Superfriends')
+        value(result).must_equal('Superfriends')
       end
     end
 
@@ -347,7 +347,7 @@ describe Lhm do
       slave do
         table_data = table_read(:users)
         assert_nil table_data.columns['username']
-        table_read(:users).columns['user_name'].must_equal({
+        value(table_read(:users).columns['user_name']).must_equal({
              :type => 'varchar(255)',
              :is_nullable => 'YES',
              :column_default => nil,
@@ -357,7 +357,7 @@ describe Lhm do
 
         result = select_one('SELECT `user_name` from users')
         result = result['user_name'] if result.respond_to?(:has_key?)
-        result.must_equal('a user')
+        value(result).must_equal('a user')
       end
     end
 
@@ -375,7 +375,7 @@ describe Lhm do
       slave do
         table_data = table_read(:users)
         assert_nil table_data.columns['reference']
-        table_read(:users).columns['ref'].must_equal({
+        value(table_read(:users).columns['ref']).must_equal({
            :type => 'int(11)',
            :is_nullable => 'YES',
            :column_default => nil,
@@ -385,7 +385,7 @@ describe Lhm do
 
         result = select_one('SELECT `ref` from users')
         result = result['ref'] if result.respond_to?(:has_key?)
-        result.must_equal(10)
+        value(result).must_equal(10)
       end
     end
 
@@ -402,7 +402,7 @@ describe Lhm do
       slave do
         table_data = table_read(:users)
         assert_nil table_data.columns['group']
-        table_read(:users).columns['fnord'].must_equal({
+        value(table_read(:users).columns['fnord']).must_equal({
            :type => 'varchar(255)',
            :is_nullable => 'YES',
            :column_default => nil,
@@ -412,7 +412,7 @@ describe Lhm do
 
         result = select_one('SELECT `fnord` from users')
         result = result['fnord'] if result.respond_to?(:has_key?)
-        result.must_equal(nil)
+        assert_nil(result)
       end
     end
 
@@ -427,7 +427,7 @@ describe Lhm do
       slave do
         table_data = table_read(:users)
         assert_nil table_data.columns['username']
-        table_read(:users).columns['user_name'].must_equal({
+        value(table_read(:users).columns['user_name']).must_equal({
           :type => 'varchar(255)',
           :is_nullable => 'YES',
           :column_default => nil,
@@ -437,7 +437,7 @@ describe Lhm do
 
         result = select_one('SELECT `user_name` from users')
         result = result['user_name'] if result.respond_to?(:has_key?)
-        result.must_equal('a user')
+        value(result).must_equal('a user')
       end
     end
 
@@ -454,7 +454,7 @@ describe Lhm do
       slave do
         table_data = table_read(:users)
         assert_nil table_data.columns['username']
-        table_read(:users).columns['user_name'].must_equal({
+        value(table_read(:users).columns['user_name']).must_equal({
           :type => 'varchar(255)',
           :is_nullable => 'NO',
           :column_default => nil,
@@ -464,7 +464,7 @@ describe Lhm do
 
         result = select_one('SELECT `user_name` from users')
         result = result['user_name'] if result.respond_to?(:has_key?)
-        result.must_equal('a user')
+        value(result).must_equal('a user')
       end
     end
 
@@ -501,7 +501,7 @@ describe Lhm do
       slave do
         table_data = table_read(:users)
         assert_nil table_data.columns['fnord']
-        table_read(:users).columns['group'].must_equal({
+        value(table_read(:users).columns['group']).must_equal({
           :type => 'varchar(255)',
           :is_nullable => 'YES',
           :column_default => 'Superfriends',
@@ -525,11 +525,11 @@ describe Lhm do
       end
 
       slave do
-        table_read(:lines).columns.must_include 'by'
-        table_read(:lines).columns.wont_include 'lines'
-        index_on_columns?(:lines, ['between'], :unique).must_equal true
-        index_on_columns?(:lines, ['by']).must_equal false
-        count_all(:lines).must_equal(2)
+        value(table_read(:lines).columns).must_include 'by'
+        value(table_read(:lines).columns).wont_include 'lines'
+        value(index_on_columns?(:lines, ['between'], :unique)).must_equal true
+        value(index_on_columns?(:lines, ['by'])).must_equal false
+        value(count_all(:lines)).must_equal(2)
       end
     end
 
@@ -554,7 +554,7 @@ describe Lhm do
         insert.join
 
         slave do
-          count_all(:users).must_equal(60)
+          value(count_all(:users)).must_equal(60)
         end
       end
 
@@ -577,7 +577,7 @@ describe Lhm do
         delete.join
 
         slave do
-          count_all(:users).must_equal(40)
+          value(count_all(:users)).must_equal(40)
         end
       end
     end

data/spec/integration/lock_wait_timeout_spec.rb

@@ -24,7 +24,7 @@ describe Lhm do
     session_innodb_lock_wait_timeout = connection.select_one("SHOW SESSION VARIABLES LIKE 'innodb_lock_wait_timeout'")['Value'].to_i
     session_lock_wait_timeout = connection.select_one("SHOW SESSION VARIABLES LIKE 'lock_wait_timeout'")['Value'].to_i
 
-    session_lock_wait_timeout.must_equal global_lock_wait_timeout + Lhm::Invoker::LOCK_WAIT_TIMEOUT_DELTA
-    session_innodb_lock_wait_timeout.must_equal global_innodb_lock_wait_timeout + Lhm::Invoker::LOCK_WAIT_TIMEOUT_DELTA
+    value(session_lock_wait_timeout).must_equal global_lock_wait_timeout + Lhm::Invoker::LOCK_WAIT_TIMEOUT_DELTA
+    value(session_innodb_lock_wait_timeout).must_equal global_innodb_lock_wait_timeout + Lhm::Invoker::LOCK_WAIT_TIMEOUT_DELTA
   end
 end

data/spec/integration/locked_switcher_spec.rb

@@ -32,8 +32,8 @@ describe Lhm::LockedSwitcher do
       switcher.run
 
       slave do
-        data_source_exists?(@origin).must_equal true
-        table_read(@migration.archive_name).columns.keys.must_include 'origin'
+        value(data_source_exists?(@origin)).must_equal true
+        value(table_read(@migration.archive_name).columns.keys).must_include 'origin'
       end
     end
 
@@ -42,8 +42,8 @@ describe Lhm::LockedSwitcher do
       switcher.run
 
       slave do
-        data_source_exists?(@destination).must_equal false
-        table_read(@origin.name).columns.keys.must_include 'destination'
+        value(data_source_exists?(@destination)).must_equal false
+        value(table_read(@origin.name).columns.keys).must_include 'destination'
       end
     end
   end

data/spec/integration/table_spec.rb

@@ -15,28 +15,24 @@ describe Lhm::Table do
       end
 
       it 'should parse primary key' do
-        @table.pk.must_equal('pk')
+        value(@table.pk).must_equal('pk')
       end
 
       it 'should parse indices' do
-        @table.
-          indices['index_custom_primary_key_on_id'].
-          must_equal(['id'])
+        value(@table.indices['index_custom_primary_key_on_id']).must_equal(['id'])
       end
 
       it 'should parse columns' do
-        @table.
-          columns['id'][:type].
-          must_match(/(bigint|int)\(\d+\)/)
+        value(@table.columns['id'][:type]).must_match(/(bigint|int)\(\d+\)/)
       end
 
       it 'should return true for method that should be renamed' do
-        @table.satisfies_id_column_requirement?.must_equal true
+        value(@table.satisfies_id_column_requirement?).must_equal true
       end
 
       it 'should support bigint tables' do
         @table = table_create(:bigint_table)
-        @table.satisfies_id_column_requirement?.must_equal true
+        value(@table.satisfies_id_column_requirement?).must_equal true
       end
     end
 
@@ -47,7 +43,7 @@ describe Lhm::Table do
 
       it 'should return false for a non-int id column' do
         @table = table_create(:wo_id_int_column)
-        @table.satisfies_id_column_requirement?.must_equal false
+        value(@table.satisfies_id_column_requirement?).must_equal false
       end
     end
   end
@@ -60,15 +56,15 @@ describe Lhm::Table do
       end
 
       it 'should parse table name in show create table' do
-        @table.name.must_equal('users')
+        value(@table.name).must_equal('users')
       end
 
       it 'should parse primary key' do
-        @table.pk.must_equal('id')
+        value(@table.pk).must_equal('id')
       end
 
       it 'should parse column type in show create table' do
-        @table.columns['username'][:type].must_equal('varchar(255)')
+        value(@table.columns['username'][:type]).must_equal('varchar(255)')
       end
 
       it 'should parse column metadata' do
@@ -76,15 +72,11 @@ describe Lhm::Table do
       end
 
       it 'should parse indices' do
-        @table.
-          indices['index_users_on_username_and_created_at'].
-          must_equal(['username', 'created_at'])
+        value(@table.indices['index_users_on_username_and_created_at']).must_equal(['username', 'created_at'])
       end
 
       it 'should parse index' do
-        @table.
-          indices['index_users_on_reference'].
-          must_equal(['reference'])
+        value(@table.indices['index_users_on_reference']).must_equal(['reference'])
       end
     end
   end

data/spec/unit/atomic_switcher_spec.rb

@@ -20,12 +20,10 @@ describe Lhm::AtomicSwitcher do
 
   describe 'atomic switch' do
     it 'should perform a single atomic rename' do
-      @switcher.
-        atomic_switch.
-        must_equal(
-          "rename table `origin` to `#{ @migration.archive_name }`, " \
-          '`destination` to `origin`'
-        )
+      value(@switcher.atomic_switch).must_equal(
+        "rename table `origin` to `#{ @migration.archive_name }`, " \
+        '`destination` to `origin`'
+      )
     end
   end
 end

data/spec/unit/entangler_spec.rb

@@ -34,7 +34,7 @@ describe Lhm::Entangler do
         values (`NEW`.`info`, `NEW`.`tags`)
       }
 
-      @entangler.entangle.must_include strip(ddl)
+      value(@entangler.entangle).must_include strip(ddl)
     end
 
     it 'should create an update trigger to the destination table' do
@@ -45,7 +45,7 @@ describe Lhm::Entangler do
         values (`NEW`.`info`, `NEW`.`tags`)
       }
 
-      @entangler.entangle.must_include strip(ddl)
+      value(@entangler.entangle).must_include strip(ddl)
     end
 
     it 'should create a delete trigger to the destination table' do
@@ -56,7 +56,7 @@ describe Lhm::Entangler do
         where `destination`.`id` = OLD.`id`
       }
 
-      @entangler.entangle.must_include strip(ddl)
+      value(@entangler.entangle).must_include strip(ddl)
     end
 
     it 'should retry trigger creation when it hits a lock wait timeout' do
@@ -101,24 +101,24 @@ describe Lhm::Entangler do
       end
 
       it 'should use truncated names' do
-        @entangler.trigger(:ins).length.must_be :<=, 64
-        @entangler.trigger(:upd).length.must_be :<=, 64
-        @entangler.trigger(:del).length.must_be :<=, 64
+        value(@entangler.trigger(:ins).length).must_be :<=, 64
+        value(@entangler.trigger(:upd).length).must_be :<=, 64
+        value(@entangler.trigger(:del).length).must_be :<=, 64
       end
     end
   end
 
   describe 'removal' do
     it 'should remove insert trigger' do
-      @entangler.untangle.must_include('drop trigger if exists `lhmt_ins_origin`')
+      value(@entangler.untangle).must_include('drop trigger if exists `lhmt_ins_origin`')
     end
 
     it 'should remove update trigger' do
-      @entangler.untangle.must_include('drop trigger if exists `lhmt_upd_origin`')
+      value(@entangler.untangle).must_include('drop trigger if exists `lhmt_upd_origin`')
     end
 
     it 'should remove delete trigger' do
-      @entangler.untangle.must_include('drop trigger if exists `lhmt_del_origin`')
+      value(@entangler.untangle).must_include('drop trigger if exists `lhmt_del_origin`')
     end
   end
 end

data/spec/unit/intersection_spec.rb

@@ -18,7 +18,7 @@ describe Lhm::Intersection do
     destination.columns['retained'] = varchar
 
     intersection = Lhm::Intersection.new(origin, destination)
-    intersection.destination.include?('dropped').must_equal(false)
+    value(intersection.destination.include?('dropped')).must_equal(false)
   end
 
   it 'should have unchanged columns' do
@@ -30,7 +30,7 @@ describe Lhm::Intersection do
     destination.columns['retained'] = varchar
 
     intersection = Lhm::Intersection.new(origin, destination)
-    intersection.destination.must_equal(['retained'])
+    value(intersection.destination).must_equal(['retained'])
   end
 
   it 'should have renamed columns' do
@@ -41,8 +41,8 @@ describe Lhm::Intersection do
     destination.columns['new_name'] = varchar
 
     intersection = Lhm::Intersection.new(origin, destination, { 'old_name' => 'new_name' })
-    intersection.origin.must_equal(['old_name'])
-    intersection.destination.must_equal(['new_name'])
+    value(intersection.origin).must_equal(['old_name'])
+    value(intersection.destination).must_equal(['new_name'])
   end
 
   def varchar

data/spec/unit/lhm_spec.rb

@@ -11,9 +11,9 @@ describe Lhm do
   describe 'logger' do
 
     it 'should use the default parameters if no logger explicitly set' do
-      Lhm.logger.must_be_kind_of Logger
-      Lhm.logger.level.must_equal Logger::INFO
-      Lhm.logger.instance_eval { @logdev }.dev.must_equal STDOUT
+      value(Lhm.logger).must_be_kind_of Logger
+      value(Lhm.logger.level).must_equal Logger::INFO
+      value(Lhm.logger.instance_eval { @logdev }.dev).must_equal STDOUT
     end
 
     it 'should use s new logger if set' do
@@ -21,9 +21,9 @@ describe Lhm do
       l.level = Logger::ERROR
       Lhm.logger = l
 
-      Lhm.logger.level.must_equal Logger::ERROR
-      Lhm.logger.instance_eval { @logdev }.dev.must_be_kind_of File
-      Lhm.logger.instance_eval { @logdev }.dev.path.must_equal 'omg.ponies'
+      value(Lhm.logger.level).must_equal Logger::ERROR
+      value(Lhm.logger.instance_eval { @logdev }.dev).must_be_kind_of File
+      value(Lhm.logger.instance_eval { @logdev }.dev.path).must_equal 'omg.ponies'
     end
   end
 end

data/spec/unit/locked_switcher_spec.rb

@@ -20,32 +20,27 @@ describe Lhm::LockedSwitcher do
 
   describe 'uncommitted' do
     it 'should disable autocommit first' do
-      @switcher.
-        statements[0..1].
-        must_equal([
-          'set @lhm_auto_commit = @@session.autocommit',
-          'set session autocommit = 0'
-        ])
+      value(@switcher.statements[0..1]).must_equal([
+        'set @lhm_auto_commit = @@session.autocommit',
+        'set session autocommit = 0'
+      ])
     end
 
     it 'should reapply original autocommit settings at the end' do
-      @switcher.
-        statements[-1].
-        must_equal('set session autocommit = @lhm_auto_commit')
+      value(@switcher.statements[-1])
+        .must_equal('set session autocommit = @lhm_auto_commit')
     end
   end
 
   describe 'switch' do
     it 'should lock origin and destination table, switch, commit and unlock' do
-      @switcher.
-        switch.
-        must_equal([
-          'lock table `origin` write, `destination` write',
-          "alter table `origin` rename `#{ @migration.archive_name }`",
-          'alter table `destination` rename `origin`',
-          'commit',
-          'unlock tables'
-        ])
+      value(@switcher.switch).must_equal([
+        'lock table `origin` write, `destination` write',
+        "alter table `origin` rename `#{ @migration.archive_name }`",
+        'alter table `destination` rename `origin`',
+        'commit',
+        'unlock tables'
+      ])
     end
   end
 end