migration_data 0.4.0 → 0.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 8ea11a5103b28e5486a3e9568d4788036c2705d038648af2dba3ce67f97fef48
-  data.tar.gz: 476de225ff2914003630c272755e113e610b8b96729c9e1c02e4723a7037ed2d
+  metadata.gz: fcfa827b7ff6fe4bf4509b6c4a5d148a52ad5328312de09c482b6532d3bdf893
+  data.tar.gz: 7ce0546a76606344b3b17c0e26bfae78724c66556e3c928cefc8a2919910489c
 SHA512:
-  metadata.gz: 73c45622d5caace7a6841f556a6a10719722488fc63733aca8165e26c23cf61149707b053002c6de37ac67dd535d9a289ff80310b75e18eaa1ce4f24aaf45903
-  data.tar.gz: bb7b355049417e35eb0bb359111eace4be2b33faf0f3262be48c85b108331af214f5f2d628bb1798eb273e61cf0cbdb4dcd8cfc0b9c93cfea3df9c1a984807b5
+  metadata.gz: bb1ed960aadd4db399fd6a9d0b1881484bfa494cbe654cc995114b6c23e0031d78ae77d9b1e2f888c72df885f77c33d69e9d8bb3e4f8a9df67f34b084fee6146
+  data.tar.gz: 6ed2163de0401cf380e786a823358d714f948eb968a3132c33a1672dfaac5a2e83e7c04b6b74cb0a14c767ab36ab1254a227cee71c7b3ce0ea9eef7c0380ea37

data/README.md

@@ -81,7 +81,21 @@ describe CreateUsers do
 end
 ```
 
-The helper to load migrations `require_migration` is defined in the `migration_data/testing`. So you should to require it to have access to this convinient require extension.
+The helper to load migrations `require_migration` is defined in the `migration_data/testing`. So you should to require it to have access to this convenient require extension.
+
+## Skipping data migrations execution
+
+At some point, one might realize that data migrations should not run on particular environments, e.g. `test`.
+
+On performing migrations in `test` environment, a data migration might try to add the same data that has already been added by seeds. In that case, migrations might fail with a duplication error.
+
+Use `MigrationData.config.skip = true` to skip data migrations execution. One might put this code in an initializer, e.g. `config/initializers/migration_data.rb`:
+
+```ruby
+if Rails.env.test?
+  MigrationData.config.skip = true
+end
+```
 
 ## Contributing
 

data/lib/migration_data.rb

@@ -1,8 +1,6 @@
 require 'active_record'
 require 'migration_data/version'
 require 'migration_data/active_record/migration'
-
-require 'rake'
-import File.join(File.dirname(__FILE__), 'tasks', 'db.rake')
+require 'migration_data/config'
 
 ActiveRecord::Migration.send(:include, MigrationData::ActiveRecord::Migration)

data/lib/migration_data/active_record/migration.rb

@@ -15,6 +15,8 @@ module MigrationData
         base.class_eval do
           def exec_migration_with_data(conn, direction)
             origin_exec_migration(conn, direction)
+            ::ActiveRecord::Base.connection.schema_cache.clear!
+            return if MigrationData.config.skip_data_on_test
             data if direction == :up && respond_to?(:data)
             rollback if direction == :down && respond_to?(:rollback)
           end

data/lib/migration_data/config.rb

@@ -0,0 +1,6 @@
+module MigrationData
+  @config = Struct.new(:skip_data_on_test).new(false)
+  def self.config
+    @config
+  end
+end

data/lib/migration_data/testing.rb

@@ -17,6 +17,8 @@ end
 def all_migrations(path)
   if Rails::VERSION::MAJOR >= 5 && Rails::VERSION::MINOR >= 2
     ActiveRecord::MigrationContext.new(path).migrations
+  elsif Rails::VERSION::MAJOR > 5
+    ActiveRecord::MigrationContext.new(path, nil).migrations
   else
     ActiveRecord::Migrator.migrations(path)
   end

data/lib/migration_data/version.rb

@@ -1,3 +1,3 @@
 module MigrationData
-  VERSION = '0.4.0'
+  VERSION = '0.5.0'
 end

data/test/migration_test.rb

@@ -38,4 +38,24 @@ describe MyMigration do
       assert_nil @migration.rolled_back_data
     end
   end
+
+  describe "#migrate with Rails.env = test and skip_data_on_test = true" do
+    before do
+      @old_skip, MigrationData.config.skip_data_on_test = MigrationData.config.skip_data_on_test, true
+    end
+
+    after do
+      MigrationData.config.skip_data_on_test = @old_skip
+    end
+
+    it "doesn't runs #data" do
+      @migration.migrate(:up)
+      refute @migration.migrated_data
+    end
+
+    it "doesn't runs #rollback" do
+      @migration.migrate(:down)
+      refute @migration.rolled_back_data
+    end
+  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: migration_data
 version: !ruby/object:Gem::Version
-  version: 0.4.0
+  version: 0.5.0
 platform: ruby
 authors:
 - Andrey Koleshko
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2020-01-27 00:00:00.000000000 Z
+date: 2020-02-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -72,6 +72,7 @@ files:
 - Rakefile
 - lib/migration_data.rb
 - lib/migration_data/active_record/migration.rb
+- lib/migration_data/config.rb
 - lib/migration_data/testing.rb
 - lib/migration_data/version.rb
 - migration_data.gemspec