data_migrate 3.5.0 → 4.0.0

data/spec/data_migrate/schema_dumper_spec.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "spec_helper"
+
 describe DataMigrate::SchemaDumper do
   let(:subject) { DataMigrate::SchemaDumper }
   let(:db_config) do
@@ -14,15 +16,15 @@ describe DataMigrate::SchemaDumper do
 
   describe :dump do
     before do
-      expect(DataMigrate::DataMigrator).
+      allow(DataMigrate::DataMigrator).
         to receive(:db_config) { db_config }.at_least(:once)
       ActiveRecord::Base.establish_connection(db_config)
 
-      ActiveRecord::Base.connection.initialize_schema_migrations_table
+      ActiveRecord::SchemaMigration.create_table
       DataMigrate::DataMigrator.assure_data_schema_table
 
       ActiveRecord::Base.connection.execute <<-SQL
-        INSERT INTO #{DataMigrate::DataMigrator.schema_migrations_table_name}
+        INSERT INTO #{DataMigrate::DataSchemaMigration.table_name}
         VALUES #{fixture_file_timestamps.map { |t| "(#{t})" }.join(', ')}
       SQL
     end

data/spec/data_migrate/schema_migration_spec.rb

@@ -0,0 +1,69 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe DataMigrate::SchemaMigration do
+  let(:migration_path) {
+    if Rails::VERSION::MAJOR == 5
+      if Rails::VERSION::MINOR == 2
+        "spec/db/migrate/5.2"
+      else
+        "spec/db/migrate/5.0"
+      end
+    else
+      "spec/db/migrate/4.2"
+    end
+  }
+
+  let(:subject) { DataMigrate::SchemaMigration }
+  let(:db_config) do
+    {
+      adapter: "sqlite3",
+      database: "spec/db/test.db"
+    }
+  end
+  let(:fixture_file_timestamps) do
+    %w[20091231235959 20101231235959 20111231235959]
+  end
+
+  before do
+    ActiveRecord::Base.establish_connection(db_config)
+    ActiveRecord::SchemaMigration.create_table
+  end
+
+  after do
+    ActiveRecord::Migration.drop_table("schema_migrations")
+  end
+
+  describe :pending_schema_migrations do
+    it "list sorted schema migrations" do
+      expect(subject).to receive(:migrations_paths) {
+        migration_path
+      }
+      migrations = subject.pending_schema_migrations
+
+      expect(migrations.count).to eq 2
+      expect(migrations[0][:version]).to eq(20131111111111)
+      expect(migrations[1][:version]).to eq(20202020202011)
+    end
+  end
+
+  describe :run do
+    it do
+      expect {
+        subject.run(:up, migration_path, 20202020202011)
+      }.to output(/20202020202011 DbMigration: migrating/).to_stdout
+      versions = ActiveRecord::SchemaMigration.normalized_versions
+      expect(versions.first).to eq("20202020202011")
+    end
+
+    it "undo migration" do
+      subject.run(:up, migration_path, 20202020202011)
+      expect {
+        subject.run(:down, migration_path, 20202020202011)
+      }.to output(/Undoing DbMigration/).to_stdout
+      versions = ActiveRecord::SchemaMigration.normalized_versions
+      expect(versions.count).to eq(0)
+    end
+  end
+end

data/spec/data_migrate/status_service_spec.rb

@@ -1,5 +1,4 @@
 # frozen_string_literal: true
-
 require "spec_helper"
 
 describe DataMigrate::StatusService do
@@ -12,44 +11,42 @@ describe DataMigrate::StatusService do
   end
   let(:service) { DataMigrate::StatusService }
 
-  describe :dump do
+  context "table does not exists" do
     before do
       ActiveRecord::Base.establish_connection(db_config)
     end
 
-    describe "table does not exist" do
-      it "table doesn't exist" do
-        expect_any_instance_of(service).to receive(:table_name) { "bogus"}
-        stream = StringIO.new
-        service.dump(ActiveRecord::Base.connection, stream)
-        stream.rewind
+    it "show error message" do
+      allow_any_instance_of(service).to receive(:table_name) { "bogus"}
+      stream = StringIO.new
+
+      service.dump(ActiveRecord::Base.connection, stream)
 
-        expected = "Data migrations table does not exist"
-        expect(stream.read).to include expected
-      end
+      stream.rewind
+      expected = "Data migrations table does not exist"
+      expect(stream.read).to include expected
     end
   end
 
-  describe :dump do
-    context "table exists"
+  context "table exists" do
     let(:fixture_file_timestamps) do
       %w[20091231235959 20101231235959 20111231235959]
     end
 
     before do
-      expect(DataMigrate::DataMigrator).
+      allow(DataMigrate::DataMigrator).
         to receive(:db_config) { db_config }.at_least(:once)
       ActiveRecord::Base.establish_connection(db_config)
 
-      ActiveRecord::Base.connection.initialize_schema_migrations_table
+      ActiveRecord::SchemaMigration.create_table
       DataMigrate::DataMigrator.assure_data_schema_table
 
       ActiveRecord::Base.connection.execute <<-SQL
-        INSERT INTO #{DataMigrate::DataMigrator.schema_migrations_table_name}
+        INSERT INTO #{DataMigrate::DataSchemaMigration.table_name}
         VALUES #{fixture_file_timestamps.map { |t| "(#{t})" }.join(', ')}
       SQL
 
-      expect_any_instance_of(service).to receive(:root_folder) { "spec" }
+      allow_any_instance_of(service).to receive(:root_folder) { "spec" }
     end
 
     after do
@@ -70,7 +67,7 @@ describe DataMigrate::StatusService do
       service.dump(ActiveRecord::Base.connection, stream)
       stream.rewind
 
-      expected = "   up     20101231235959  *** NO FILE ***"
+      expected = "   up     20101231235959  ********** NO FILE **********"
       s = stream.read
       expect(s).to include expected
     end

data/spec/data_migrate/tasks/data_migrate_tasks_spec.rb

@@ -0,0 +1,50 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+describe DataMigrate::Tasks::DataMigrateTasks do
+  let(:db_config) do
+    {
+      adapter: "sqlite3",
+      database: "spec/db/test.db"
+    }
+  end
+
+  before do
+    if Rails::VERSION::MAJOR == 5
+      if Rails::VERSION::MINOR == 2
+        allow(DataMigrate::Tasks::DataMigrateTasks).to receive(:migrations_paths) {
+          "spec/db/data"
+        }
+      else
+        allow(DataMigrate::Tasks::DataMigrateTasks).to receive(:migrations_paths) {
+          "spec/db/5.0"
+        }
+      end
+    else
+      allow(DataMigrate::Tasks::DataMigrateTasks).to receive(:migrations_paths) {
+        "spec/db/4.2"
+      }
+    end
+    allow(DataMigrate::DataMigrator).to receive(:db_config) { db_config }
+    ActiveRecord::Base.establish_connection(db_config)
+  end
+
+  after do
+    ActiveRecord::Migration.drop_table("data_migrations")
+  end
+
+  describe :migrate do
+    it do
+      expect {
+        DataMigrate::Tasks::DataMigrateTasks.migrate
+      }.to output(/20091231235959 SomeName: migrating/).to_stdout
+    end
+
+    it do
+      expect {
+        DataMigrate::Tasks::DataMigrateTasks.migrate
+      }.to output(/20171231235959 SuperUpdate: migrating/).to_stdout
+    end
+  end
+end

data/spec/db/4.2/20091231235959_some_name.rb

@@ -0,0 +1,9 @@
+class SomeName < ActiveRecord::Migration
+  def up
+    puts "Doing data migration"
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

data/spec/db/4.2/20171231235959_super_update.rb

@@ -0,0 +1,9 @@
+class SuperUpdate < ActiveRecord::Migration
+  def up
+    puts "Doing data migration"
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

data/spec/db/5.0/20091231235959_some_name.rb

@@ -0,0 +1,9 @@
+class SomeName < ActiveRecord::Migration[5.0]
+  def up
+    puts "Doing data migration"
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

data/spec/db/5.0/20171231235959_super_update.rb

@@ -0,0 +1,9 @@
+class SuperUpdate < ActiveRecord::Migration[5.0]
+  def up
+    puts "Doing data migration"
+  end
+
+  def down
+    raise ActiveRecord::IrreversibleMigration
+  end
+end

data/spec/db/data/20091231235959_some_name.rb

@@ -0,0 +1,9 @@
+class SomeName < ActiveRecord::Migration[5.2]
+  def up
+    puts "Doing data migration"
+  end
+
+  def down
+    puts "Undoing SomeName"
+  end
+end

data/spec/db/data/20171231235959_super_update.rb

@@ -0,0 +1,9 @@
+class SuperUpdate < ActiveRecord::Migration[5.2]
+  def up
+    puts "Doing SuperUpdate"
+  end
+
+  def down
+    puts "Undoing SuperUpdate"
+  end
+end

data/spec/db/migrate/4.2/20131111111111_late_migration.rb

@@ -0,0 +1,9 @@
+class LateMigration < ActiveRecord::Migration
+  def up
+    puts "Doing schema LateMigration"
+  end
+
+  def down
+    puts "Undoing LateMigration"
+  end
+end

data/spec/db/migrate/4.2/20202020202011_db_migration.rb

@@ -0,0 +1,9 @@
+class DbMigration < ActiveRecord::Migration
+  def up
+    puts "Doing schema migration"
+  end
+
+  def down
+    puts "Undoing DbMigration"
+  end
+end

data/spec/db/migrate/5.0/20131111111111_late_migration.rb

@@ -0,0 +1,9 @@
+class LateMigration < ActiveRecord::Migration[5.0]
+  def up
+    puts "Doing schema LateMigration"
+  end
+
+  def down
+    puts "Undoing LateMigration"
+  end
+end

data/spec/db/migrate/5.0/20202020202011_db_migration.rb

@@ -0,0 +1,9 @@
+class DbMigration < ActiveRecord::Migration[5.0]
+  def up
+    puts "Doing schema migration"
+  end
+
+  def down
+    puts "Undoing DbMigration"
+  end
+end

data/spec/db/migrate/5.2/20131111111111_late_migration.rb

@@ -0,0 +1,9 @@
+class LateMigration < ActiveRecord::Migration[5.2]
+  def up
+    puts "Doing schema LateMigration"
+  end
+
+  def down
+    puts "Undoing LateMigration"
+  end
+end

data/spec/db/migrate/5.2/20202020202011_db_migration.rb

@@ -0,0 +1,9 @@
+class DbMigration < ActiveRecord::Migration[5.2]
+  def up
+    puts "Doing schema migration"
+  end
+
+  def down
+    puts "Undoing DbMigration"
+  end
+end

data/spec/spec_helper.rb

@@ -3,3 +3,7 @@ require 'rails'
 require 'data_migrate'
 require 'pry'
 require 'timecop'
+
+if Rails::VERSION::MAJOR == 5 &&  Rails::VERSION::MINOR == 2
+  DataMigrate::DataMigrator.migrations_paths = ["spec/db/data"]
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: data_migrate
 version: !ruby/object:Gem::Version
-  version: 3.5.0
+  version: 4.0.0
 platform: ruby
 authors:
 - Andrew J Vargo
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2018-03-08 00:00:00.000000000 Z
+date: 2018-05-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -17,14 +17,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.1'
+        version: '4.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '4.1'
+        version: '4.2'
 - !ruby/object:Gem::Dependency
   name: appraisal
   requirement: !ruby/object:Gem::Requirement
@@ -185,6 +185,7 @@ files:
 - Gemfile
 - Gemfile.rails5
 - Gemfile.rails5.1
+- Gemfile.rails5.2
 - LICENSE
 - README.md
 - Rakefile
@@ -193,18 +194,25 @@ files:
 - gemfiles/rails_4.2.gemfile
 - gemfiles/rails_5.0.gemfile
 - gemfiles/rails_5.1.gemfile
+- gemfiles/rails_5.2.gemfile
 - lib/capistrano/data_migrate.rb
 - lib/capistrano/data_migrate/migrate.rb
 - lib/data_migrate.rb
 - lib/data_migrate/data_migrator.rb
+- lib/data_migrate/data_migrator_five.rb
 - lib/data_migrate/data_schema.rb
 - lib/data_migrate/data_schema_migration.rb
 - lib/data_migrate/database_tasks.rb
 - lib/data_migrate/migration.rb
+- lib/data_migrate/migration_context.rb
 - lib/data_migrate/migration_five.rb
 - lib/data_migrate/railtie.rb
 - lib/data_migrate/schema_dumper.rb
+- lib/data_migrate/schema_migration.rb
+- lib/data_migrate/schema_migration_five.rb
 - lib/data_migrate/status_service.rb
+- lib/data_migrate/status_service_five.rb
+- lib/data_migrate/tasks/data_migrate_tasks.rb
 - lib/data_migrate/version.rb
 - lib/generators/data_migrate.rb
 - lib/generators/data_migration/data_migration_generator.rb
@@ -213,13 +221,26 @@ files:
 - screenshot.png
 - spec/data_migrate/data_migrator_spec.rb
 - spec/data_migrate/data_schema_migration_spec.rb
-- spec/data_migrate/data_schema_spec.rb
+- spec/data_migrate/data_spec.rb
 - spec/data_migrate/database_tasks_spec.rb
 - spec/data_migrate/migration.rb
+- spec/data_migrate/migration_context_spec.rb
 - spec/data_migrate/schema_dumper_spec.rb
+- spec/data_migrate/schema_migration_spec.rb
 - spec/data_migrate/status_service_spec.rb
+- spec/data_migrate/tasks/data_migrate_tasks_spec.rb
+- spec/db/4.2/20091231235959_some_name.rb
+- spec/db/4.2/20171231235959_super_update.rb
+- spec/db/5.0/20091231235959_some_name.rb
+- spec/db/5.0/20171231235959_super_update.rb
 - spec/db/data/20091231235959_some_name.rb
 - spec/db/data/20171231235959_super_update.rb
+- spec/db/migrate/4.2/20131111111111_late_migration.rb
+- spec/db/migrate/4.2/20202020202011_db_migration.rb
+- spec/db/migrate/5.0/20131111111111_late_migration.rb
+- spec/db/migrate/5.0/20202020202011_db_migration.rb
+- spec/db/migrate/5.2/20131111111111_late_migration.rb
+- spec/db/migrate/5.2/20202020202011_db_migration.rb
 - spec/generators/data_migration/data_migration_generator_spec.rb
 - spec/spec_helper.rb
 - tasks/.gitkeep
@@ -247,19 +268,32 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: data_migrate
-rubygems_version: 2.5.2
+rubygems_version: 2.6.13
 signing_key: 
 specification_version: 4
 summary: Rake tasks to migrate data alongside schema changes.
 test_files:
 - spec/data_migrate/data_migrator_spec.rb
 - spec/data_migrate/data_schema_migration_spec.rb
-- spec/data_migrate/data_schema_spec.rb
+- spec/data_migrate/data_spec.rb
 - spec/data_migrate/database_tasks_spec.rb
 - spec/data_migrate/migration.rb
+- spec/data_migrate/migration_context_spec.rb
 - spec/data_migrate/schema_dumper_spec.rb
+- spec/data_migrate/schema_migration_spec.rb
 - spec/data_migrate/status_service_spec.rb
+- spec/data_migrate/tasks/data_migrate_tasks_spec.rb
+- spec/db/4.2/20091231235959_some_name.rb
+- spec/db/4.2/20171231235959_super_update.rb
+- spec/db/5.0/20091231235959_some_name.rb
+- spec/db/5.0/20171231235959_super_update.rb
 - spec/db/data/20091231235959_some_name.rb
 - spec/db/data/20171231235959_super_update.rb
+- spec/db/migrate/4.2/20131111111111_late_migration.rb
+- spec/db/migrate/4.2/20202020202011_db_migration.rb
+- spec/db/migrate/5.0/20131111111111_late_migration.rb
+- spec/db/migrate/5.0/20202020202011_db_migration.rb
+- spec/db/migrate/5.2/20131111111111_late_migration.rb
+- spec/db/migrate/5.2/20202020202011_db_migration.rb
 - spec/generators/data_migration/data_migration_generator_spec.rb
 - spec/spec_helper.rb