lhm 1.2.0 → 2.2.0

data/spec/unit/printer_spec.rb

@@ -0,0 +1,79 @@
+require File.expand_path(File.dirname(__FILE__)) + '/unit_helper'
+
+require 'lhm/printer'
+
+describe Lhm::Printer do
+  include UnitHelper
+
+  describe 'percentage printer' do
+
+    before(:each) do
+      @printer = Lhm::Printer::Percentage.new
+    end
+
+    it 'prints the percentage' do
+      mock = MiniTest::Mock.new
+      10.times do |i|
+        mock.expect(:write, :return_value) do |message|
+          assert_match /^\r/, message.first
+          assert_match /#{i}\/10/, message.first
+        end
+      end
+
+      @printer.instance_variable_set(:@output, mock)
+      10.times { |i| @printer.notify(i, 10) }
+      mock.verify
+    end
+
+    it 'always print a bigger message' do
+      @length = 0
+      mock = MiniTest::Mock.new
+      3.times do |i|
+        mock.expect(:write, :return_value) do |message|
+          assert message.first.length >= @length
+          @length = message.first.length
+        end
+      end
+
+      @printer.instance_variable_set(:@output, mock)
+      @printer.notify(10, 100)
+      @printer.notify(0, 100)
+      @printer.notify(1, 1000000)
+      @printer.notify(0, 0)
+      @printer.notify(0, nil)
+
+      mock.verify
+    end
+
+    it 'prints the end message' do
+      mock = MiniTest::Mock.new
+      mock.expect(:write, :return_value, [String])
+      mock.expect(:write, :return_value, ["\n"])
+
+      @printer.instance_variable_set(:@output, mock)
+      @printer.end
+
+      mock.verify
+    end
+  end
+
+  describe 'dot printer' do
+
+    before(:each) do
+      @printer = Lhm::Printer::Dot.new
+    end
+
+    it 'prints the dots' do
+      mock  = MiniTest::Mock.new
+      10.times do
+        mock.expect(:write, :return_value, ['.'])
+      end
+
+      @printer.instance_variable_set(:@output, mock)
+      10.times { @printer.notify }
+
+      mock.verify
+    end
+
+  end
+end

data/spec/unit/sql_helper_spec.rb

@@ -6,27 +6,27 @@ require File.expand_path(File.dirname(__FILE__)) + '/unit_helper'
 require 'lhm/sql_helper'
 
 describe Lhm::SqlHelper do
-  it "should name index with a single column" do
+  it 'should name index with a single column' do
     Lhm::SqlHelper.
       idx_name(:users, :name).
-      must_equal("index_users_on_name")
+      must_equal('index_users_on_name')
   end
 
-  it "should name index with multiple columns" do
+  it 'should name index with multiple columns' do
     Lhm::SqlHelper.
       idx_name(:users, [:name, :firstname]).
-      must_equal("index_users_on_name_and_firstname")
+      must_equal('index_users_on_name_and_firstname')
   end
 
-  it "should name index with prefixed column" do
+  it 'should name index with prefixed column' do
     Lhm::SqlHelper.
-      idx_name(:tracks, ["title(10)", "album"]).
-      must_equal("index_tracks_on_title_and_album")
+      idx_name(:tracks, ['title(10)', 'album']).
+      must_equal('index_tracks_on_title_and_album')
   end
 
-  it "should quote column names in index specification" do
+  it 'should quote column names in index specification' do
     Lhm::SqlHelper.
-      idx_spec(["title(10)", "album"]).
-      must_equal("`title`(10), `album`")
+      idx_spec(['title(10)', 'album']).
+      must_equal('`title`(10), `album`')
   end
 end

data/spec/unit/table_spec.rb

@@ -8,26 +8,26 @@ require 'lhm/table'
 describe Lhm::Table do
   include UnitHelper
 
-  describe "names" do
-    it "should name destination" do
-      @table = Lhm::Table.new("users")
-      @table.destination_name.must_equal "lhmn_users"
+  describe 'names' do
+    it 'should name destination' do
+      @table = Lhm::Table.new('users')
+      @table.destination_name.must_equal 'lhmn_users'
     end
   end
 
-  describe "constraints" do
-    it "should be satisfied with a single column primary key called id" do
-      @table = Lhm::Table.new("table", "id")
+  describe 'constraints' do
+    it 'should be satisfied with a single column primary key called id' do
+      @table = Lhm::Table.new('table', 'id')
       @table.satisfies_primary_key?.must_equal true
     end
 
-    it "should not be satisfied with a primary key unless called id" do
-      @table = Lhm::Table.new("table", "uuid")
+    it 'should not be satisfied with a primary key unless called id' do
+      @table = Lhm::Table.new('table', 'uuid')
       @table.satisfies_primary_key?.must_equal false
     end
 
-    it "should not be satisfied with multicolumn primary key" do
-      @table = Lhm::Table.new("table", ["id", "secondary"])
+    it 'should not be satisfied with multicolumn primary key' do
+      @table = Lhm::Table.new('table', ['id', 'secondary'])
       @table.satisfies_primary_key?.must_equal false
     end
   end

data/spec/unit/throttler_spec.rb

@@ -0,0 +1,73 @@
+require File.expand_path(File.dirname(__FILE__)) + '/unit_helper'
+
+require 'lhm/throttler'
+
+describe Lhm::Throttler do
+  include UnitHelper
+
+  before :each do
+    @mock = Class.new do
+      extend Lhm::Throttler
+    end
+  end
+
+  describe '#setup_throttler' do
+    describe 'when passing a key' do
+      before do
+        @mock.setup_throttler(:time_throttler, :delay => 2)
+      end
+
+      it 'instantiates the time throttle' do
+        @mock.throttler.class.must_equal Lhm::Throttler::Time
+      end
+
+      it 'returns 2 seconds as time' do
+        @mock.throttler.timeout_seconds.must_equal 2
+      end
+    end
+
+    describe 'when passing an instance' do
+
+      before do
+        @instance = Class.new(Lhm::Throttler::Time) do
+          def timeout_seconds
+            0
+          end
+        end.new
+
+        @mock.setup_throttler(@instance)
+      end
+
+      it 'returns the instace given' do
+        @mock.throttler.must_equal @instance
+      end
+
+      it 'returns 0 seconds as time' do
+        @mock.throttler.timeout_seconds.must_equal 0
+      end
+    end
+
+    describe 'when passing a class' do
+
+      before do
+        @klass = Class.new(Lhm::Throttler::Time)
+        @mock.setup_throttler(@klass)
+      end
+
+      it 'has the same class as given' do
+        @mock.throttler.class.must_equal @klass
+      end
+    end
+  end
+
+  describe '#throttler' do
+
+    it 'returns the default Time based' do
+      @mock.throttler.class.must_equal Lhm::Throttler::Time
+    end
+
+    it 'should default to 100 milliseconds' do
+      @mock.throttler.timeout_seconds.must_equal 0.1
+    end
+  end
+end

data/spec/unit/unit_helper.rb

@@ -1,18 +1,6 @@
 # Copyright (c) 2011 - 2013, SoundCloud Ltd., Rany Keddo, Tobias Bielohlawek, Tobias
 # Schmidt
-
-require File.expand_path(File.dirname(__FILE__)) + "/../bootstrap"
-
-begin
-  require 'active_record'
-  begin
-    require 'mysql2'
-  rescue LoadError
-    require 'mysql'
-  end
-rescue LoadError
-  require 'dm-core'
-end
+require 'test_helper'
 
 module UnitHelper
   def fixture(name)

metadata

@@ -1,8 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: lhm
 version: !ruby/object:Gem::Version
-  version: 1.2.0
-  prerelease: 
+  version: 2.2.0
 platform: ruby
 authors:
 - SoundCloud
@@ -12,38 +11,34 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-22 00:00:00.000000000 Z
+date: 2015-01-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.10.0
+        version: 5.0.8
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - '='
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: 2.10.0
+        version: 5.0.8
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
-    none: false
     requirements:
-    - - ! '>='
+    - - ">="
       - !ruby/object:Gem::Version
         version: '0'
 description: Migrate large tables without downtime by copying to a temporary table
@@ -55,12 +50,14 @@ executables:
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- .travis.yml
+- ".gitignore"
+- ".rubocop.yml"
+- ".travis.yml"
 - CHANGELOG.md
 - LICENSE
 - README.md
 - Rakefile
+- bin/lhm-config.sh
 - bin/lhm-kill-queue
 - bin/lhm-spec-clobber.sh
 - bin/lhm-spec-grants.sh
@@ -82,23 +79,30 @@ files:
 - lib/lhm/locked_switcher.rb
 - lib/lhm/migration.rb
 - lib/lhm/migrator.rb
+- lib/lhm/printer.rb
 - lib/lhm/sql_helper.rb
 - lib/lhm/table.rb
+- lib/lhm/throttler.rb
+- lib/lhm/throttler/time.rb
 - lib/lhm/version.rb
 - spec/.lhm.example
 - spec/README.md
-- spec/bootstrap.rb
 - spec/fixtures/destination.ddl
+- spec/fixtures/lines.ddl
 - spec/fixtures/origin.ddl
+- spec/fixtures/permissions.ddl
 - spec/fixtures/small_table.ddl
+- spec/fixtures/tracks.ddl
 - spec/fixtures/users.ddl
 - spec/integration/atomic_switcher_spec.rb
 - spec/integration/chunker_spec.rb
+- spec/integration/cleanup_spec.rb
 - spec/integration/entangler_spec.rb
 - spec/integration/integration_helper.rb
 - spec/integration/lhm_spec.rb
 - spec/integration/locked_switcher_spec.rb
 - spec/integration/table_spec.rb
+- spec/test_helper.rb
 - spec/unit/active_record_connection_spec.rb
 - spec/unit/atomic_switcher_spec.rb
 - spec/unit/chunker_spec.rb
@@ -106,34 +110,69 @@ files:
 - spec/unit/datamapper_connection_spec.rb
 - spec/unit/entangler_spec.rb
 - spec/unit/intersection_spec.rb
+- spec/unit/lhm_spec.rb
 - spec/unit/locked_switcher_spec.rb
 - spec/unit/migration_spec.rb
 - spec/unit/migrator_spec.rb
+- spec/unit/printer_spec.rb
 - spec/unit/sql_helper_spec.rb
 - spec/unit/table_spec.rb
+- spec/unit/throttler_spec.rb
 - spec/unit/unit_helper.rb
-homepage: http://github.com/soundcloud/large-hadron-migrator
+homepage: http://github.com/soundcloud/lhm
 licenses: []
+metadata: {}
 post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 required_rubygems_version: !ruby/object:Gem::Requirement
-  none: false
   requirements:
-  - - ! '>='
+  - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.24
+rubygems_version: 2.2.2
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: online schema changer for mysql
-test_files: []
+test_files:
+- spec/README.md
+- spec/fixtures/destination.ddl
+- spec/fixtures/lines.ddl
+- spec/fixtures/origin.ddl
+- spec/fixtures/permissions.ddl
+- spec/fixtures/small_table.ddl
+- spec/fixtures/tracks.ddl
+- spec/fixtures/users.ddl
+- spec/integration/atomic_switcher_spec.rb
+- spec/integration/chunker_spec.rb
+- spec/integration/cleanup_spec.rb
+- spec/integration/entangler_spec.rb
+- spec/integration/integration_helper.rb
+- spec/integration/lhm_spec.rb
+- spec/integration/locked_switcher_spec.rb
+- spec/integration/table_spec.rb
+- spec/test_helper.rb
+- spec/unit/active_record_connection_spec.rb
+- spec/unit/atomic_switcher_spec.rb
+- spec/unit/chunker_spec.rb
+- spec/unit/connection_spec.rb
+- spec/unit/datamapper_connection_spec.rb
+- spec/unit/entangler_spec.rb
+- spec/unit/intersection_spec.rb
+- spec/unit/lhm_spec.rb
+- spec/unit/locked_switcher_spec.rb
+- spec/unit/migration_spec.rb
+- spec/unit/migrator_spec.rb
+- spec/unit/printer_spec.rb
+- spec/unit/sql_helper_spec.rb
+- spec/unit/table_spec.rb
+- spec/unit/throttler_spec.rb
+- spec/unit/unit_helper.rb

data/spec/bootstrap.rb

@@ -1,13 +0,0 @@
-# Copyright (c) 2011 - 2013, SoundCloud Ltd., Rany Keddo, Tobias Bielohlawek, Tobias
-# Schmidt
-
-require 'minitest/spec'
-require 'minitest/autorun'
-require 'minitest/mock'
-require "pathname"
-
-$project = Pathname.new(File.dirname(__FILE__) + '/..').cleanpath
-$spec = $project.join("spec")
-$fixtures = $spec.join("fixtures")
-
-$: << $project.join("lib").to_s