data_migrate 8.4.2 → 9.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7cc2467bde93f842786d6a5bb5077e8a1ef595a80e5868a03ae960552a232faa
-  data.tar.gz: 452420d3616deaaa6776511bb6cabbe765f92106d19e67b017fafb717ca0d760
+  metadata.gz: 2acff14ddfe78e35e593865b31aaa8f5964c7bc958b7f40f7019333288efaa4e
+  data.tar.gz: 3098c184b2bb1ba2f2ed75181c31938800c0c0313431b6b1749ff4b103ee0f33
 SHA512:
-  metadata.gz: aa097fbb8c8560e888984965d6ce10e9b09fcda1031c18aeccbe9e25b46db7baa3c9821e9ea18468a2df24aba7cb94412fc3fb15aa88e2a110f71018697db323
-  data.tar.gz: f9f1bc42960a703223edd39c22285e6dfa0c541bf723152992da61b24a689c1b90e30dee0d2902b15a489d1606b681eb03b4b6d6723fccf932745b20d8f71f72
+  metadata.gz: ba3d909404e128ddb3f7319c83301a7f7395e13226e47c2c35f858535e32d15dd25bb89b475841edf013643407b29a56ebe0eb89f86d1f2f14c66178d868894f
+  data.tar.gz: 7039baeee12e08f77f7cfa680b1841a9137291aa8c3e7c299fe779d8343182a7f851e582b9614d8674add0df9727d38d2918ae726f17a888cd07d4bb556d0360

data/.github/workflows/build.yml

@@ -0,0 +1,35 @@
+name: CI
+
+on:
+  - push
+  - pull_request
+
+jobs:
+  test:
+    strategy:
+      fail-fast: false
+      matrix:
+        os: [ "ubuntu-20.04" ]
+        ruby:
+          - '2.7'
+          - '3.0'
+          - '3.1'
+          - '3.2'
+        gemfile:
+          - gemfiles/rails_6.0.gemfile
+          - gemfiles/rails_6.1.gemfile
+          - gemfiles/rails_7.0.gemfile
+    runs-on: ubuntu-latest
+    env:
+      BUNDLE_GEMFILE: ${{ matrix.gemfile }}
+      RAILS_ENV: test
+    steps:
+      - name: Checkout code
+        uses: actions/checkout@v3
+      - name: Set up Ruby
+        uses: ruby/setup-ruby@v1
+        with:
+          ruby-version: ${{ matrix.ruby }}
+          bundler-cache: true
+      - name: Run tests
+        run: bundle exec rspec

data/.github/workflows/gempush.yml

@@ -3,7 +3,7 @@ name: Ruby Gem
 on:
   push:
     branches:
-      - master
+      - main
 
 jobs:
   build:

data/.travis.yml

@@ -2,6 +2,7 @@ language: ruby
 rvm:
   - 2.7
   - 3.0
+  - 3.2
 script: bundle exec rspec
 gemfile:
   - gemfiles/rails_5.2.gemfile
@@ -12,3 +13,5 @@ jobs:
   exclude:
     - rvm: 3.0
       gemfile: gemfiles/rails_5.2.gemfile
+    - rvm: 3.2
+      gemfile: gemfiles/rails_5.2.gemfile

data/Changelog.md

@@ -1,5 +1,14 @@
 # Changelog
 
+## 9.0.0
+
+Ruby 3.2 support [mehanoid](https://github.com/mehanoid)
+Rails 5.2 is no longer supported
+
+## 8.5.0
+
+Allow custom templates [bazay](https://github.com/bazay)
+
 ## 8.4.0
 
 Avoid Globally Accessible Functions for All Rake Tasks [berniechiu](https://github.com/berniechiu)

data/README.md

@@ -108,6 +108,7 @@ You can override this setting in `config/initializers/data_migrate.rb`
 ```ruby
 DataMigrate.configure do |config|
   config.data_migrations_path = 'db/awesomepath/'
+  config.data_template_path = Rails.root.join("lib", "awesomepath", "custom_data_migration.rb")
   config.db_configuration = {
     'host' => '127.0.0.1',
     'database' => 'awesome_database',

data/data_migrate.gemspec

@@ -19,7 +19,7 @@ Gem::Specification.new do |s|
     activerecord
     railties
   ].each do |rails_gem|
-    s.add_dependency(rails_gem, '>= 5.0')
+    s.add_dependency(rails_gem, '>= 6.0')
   end
 
   s.add_development_dependency "appraisal"
@@ -38,13 +38,4 @@ Gem::Specification.new do |s|
   s.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
   s.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]
-
-  s.post_install_message = <<-POST_INSTALL_MESSAGE
-#{"*" * 80}
-data-migrate: IMPORTANT: Breaking change introduced for migrations from v2.
-
-Failure to run the migration can have serious consequences.
-See Readme for more info.
-#{"*" * 80}
-POST_INSTALL_MESSAGE
 end

data/lib/data_migrate/config.rb

@@ -12,12 +12,21 @@ module DataMigrate
   end
 
   class Config
-    attr_accessor :data_migrations_path, :db_configuration, :spec_name
+    attr_accessor :data_migrations_path, :data_template_path, :db_configuration, :spec_name
+
+    DEFAULT_DATA_TEMPLATE_PATH = "data_migration.rb"
 
     def initialize
       @data_migrations_path = "db/data/"
+      @data_template_path = DEFAULT_DATA_TEMPLATE_PATH
       @db_configuration = nil
       @spec_name = nil
     end
+
+    def data_template_path=(value)
+      @data_template_path = value.tap do |path|
+        raise ArgumentError, "File not found: '#{path}'" unless path == DEFAULT_DATA_TEMPLATE_PATH || File.exist?(path)
+      end
+    end
   end
 end

data/lib/data_migrate/version.rb

@@ -1,3 +1,3 @@
 module DataMigrate
-  VERSION = "8.4.2".freeze
+  VERSION = "9.0.0".freeze
 end

data/lib/data_migrate.rb

@@ -19,4 +19,7 @@ else
 end
 
 module DataMigrate
+  def self.root
+    File.dirname(__FILE__)
+  end
 end

data/lib/generators/data_migrate.rb

@@ -1,9 +1,22 @@
 require 'rails/generators/named_base'
+
 module DataMigrate
   module Generators
     class DataMigrationGenerator < Rails::Generators::NamedBase #:nodoc:
-      def self.source_root
-         @_data_migrate_source_root ||= File.expand_path(File.join(File.dirname(__FILE__), generator_name, 'templates'))
+      class << self
+        def source_root
+          build_data_migrate_source_root
+        end
+
+        private
+
+        def build_data_migrate_source_root
+          if DataMigrate.config.data_template_path == DataMigrate::Config::DEFAULT_DATA_TEMPLATE_PATH
+            File.expand_path(File.join(File.dirname(__FILE__), generator_name, 'templates'))
+          else
+            File.expand_path(File.dirname(DataMigrate.config.data_template_path))
+          end
+        end
       end
     end
   end

data/lib/generators/data_migration/data_migration_generator.rb

@@ -14,7 +14,7 @@ module DataMigrate
 
       def create_data_migration
         set_local_assigns!
-        migration_template "data_migration.rb", data_migrations_file_path
+        migration_template template_path, data_migrations_file_path
       end
 
       protected
@@ -26,6 +26,10 @@ module DataMigrate
         end
       end
 
+      def template_path
+        DataMigrate.config.data_template_path
+      end
+
       def migration_base_class_name
         "ActiveRecord::Migration[#{ActiveRecord::Migration.current_version}]"
       end

data/spec/data_migrate/config_spec.rb

@@ -1,12 +1,16 @@
 require "spec_helper"
 
 describe DataMigrate::Config do
-
   it "sets default data_migrations_path path", :no_override do
     expect(DataMigrate.config.data_migrations_path).to eq "db/data/"
   end
 
+  it "sets default data_template_path path", :no_override do
+    expect(DataMigrate.config.data_template_path).to eq DataMigrate::Config::DEFAULT_DATA_TEMPLATE_PATH
+  end
+
   describe "data migration path configured" do
+    subject { DataMigrate.config.data_migrations_path }
     before do
       @before = DataMigrate.config.data_migrations_path
       DataMigrate.configure do |config|
@@ -20,8 +24,43 @@ describe DataMigrate::Config do
       end
     end
 
-    it do
-      expect(DataMigrate.config.data_migrations_path).to eq "db/awesome/"
+    it "equals the custom data migration path" do
+      is_expected.to eq "db/awesome/"
+    end
+  end
+
+  describe "data template path configured" do
+    subject { DataMigrate.config.data_template_path }
+
+    before do
+      @before = DataMigrate.config.data_template_path
+      DataMigrate.configure do |config|
+        config.data_template_path = data_template_path
+      end
+    end
+
+    let(:data_template_path) do
+      File.join(DataMigrate.root, "generators", "data_migration", "templates", "data_migration.rb")
+    end
+
+    after do
+      DataMigrate.configure do |config|
+        config.data_template_path = @before
+      end
+    end
+
+    it "equals the custom data template path" do
+      is_expected.to eq data_template_path
+    end
+
+    context "when path does not exist" do
+      subject { DataMigrate.config.data_template_path = invalid_path }
+
+      let(:invalid_path) { "lib/awesome/templates/data_migration.rb" }
+
+      it "checks that file exists on setting config var" do
+        expect { subject }.to raise_error { ArgumentError.new("File not found: '#{data_template_path}'") }
+      end
     end
   end
 end

data/spec/generators/data_migration/data_migration_generator_spec.rb

@@ -4,7 +4,8 @@ require 'rails/generators/migration'
 require 'generators/data_migration/data_migration_generator'
 
 describe DataMigrate::Generators::DataMigrationGenerator do
-  let(:subject) { DataMigrate::Generators::DataMigrationGenerator }
+  subject { DataMigrate::Generators::DataMigrationGenerator }
+
   describe :next_migration_number do
     it "next migration" do
       Timecop.freeze("2016-12-03 22:15:26 -0800") do
@@ -19,14 +20,18 @@ describe DataMigrate::Generators::DataMigrationGenerator do
   end
 
   describe :migration_base_class_name do
-    let(:subject) { DataMigrate::Generators::DataMigrationGenerator.new(['my_migration']) }
+    subject { generator.send(:migration_base_class_name) }
+
+    let(:generator) { DataMigrate::Generators::DataMigrationGenerator.new(['my_migration']) }
+
     it "returns the correct base class name" do
-      expect(subject.send(:migration_base_class_name)).to eq("ActiveRecord::Migration[#{ActiveRecord::Migration.current_version}]")
+      is_expected.to eq("ActiveRecord::Migration[#{ActiveRecord::Migration.current_version}]")
     end
   end
 
   describe :create_data_migration do
-    let(:subject) { DataMigrate::Generators::DataMigrationGenerator.new(['my_migration']) }
+    subject { DataMigrate::Generators::DataMigrationGenerator.new(['my_migration']) }
+
     let(:data_migrations_file_path) { 'abc/my_migration.rb' }
 
     context 'when custom data migrations path has a trailing slash' do
@@ -35,7 +40,7 @@ describe DataMigrate::Generators::DataMigrationGenerator do
       end
 
       it 'returns correct file path' do
-        expect(subject).to receive(:migration_template).with(
+        is_expected.to receive(:migration_template).with(
           'data_migration.rb', data_migrations_file_path
         )
 
@@ -49,7 +54,7 @@ describe DataMigrate::Generators::DataMigrationGenerator do
       end
 
       it 'returns correct file path' do
-        expect(subject).to receive(:migration_template).with(
+        is_expected.to receive(:migration_template).with(
           'data_migration.rb', data_migrations_file_path
         )
 
@@ -57,4 +62,40 @@ describe DataMigrate::Generators::DataMigrationGenerator do
       end
     end
   end
+
+  describe ".source_root" do
+    subject { described_class.source_root }
+
+    let(:default_source_root) do
+      File.expand_path(
+        File.dirname(File.join(DataMigrate.root, "generators", "data_migration", "templates", "data_migration.rb"))
+      )
+    end
+
+    it { is_expected.to eq default_source_root }
+
+    context "when DateMigrate.config.data_template_path is set" do
+      before do
+        @before = DataMigrate.config.data_template_path
+        DataMigrate.configure do |config|
+          config.data_template_path = data_template_path
+        end
+      end
+
+      let(:data_template_path) do
+        File.join(DataMigrate.root, "generators", "data_migration", "templates", "data_migration.rb")
+      end
+      let(:expected_source_root) { File.dirname(data_template_path) }
+
+      after do
+        DataMigrate.configure do |config|
+          config.data_template_path = @before
+        end
+      end
+
+      it "reads directory from config data template path" do
+        is_expected.to eq expected_source_root
+      end
+    end
+  end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: data_migrate
 version: !ruby/object:Gem::Version
-  version: 8.4.2
+  version: 9.0.0
 platform: ruby
 authors:
 - Andrew J Vargo
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2022-12-14 00:00:00.000000000 Z
+date: 2023-02-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
@@ -18,28 +18,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: railties
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: appraisal
   requirement: !ruby/object:Gem::Requirement
@@ -189,6 +189,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".github/workflows/build.yml"
 - ".github/workflows/gempush.yml"
 - ".gitignore"
 - ".hound.yml"
@@ -270,13 +271,7 @@ homepage: https://github.com/ilyakatz/data-migrate
 licenses:
 - MIT
 metadata: {}
-post_install_message: |
-  ********************************************************************************
-  data-migrate: IMPORTANT: Breaking change introduced for migrations from v2.
-
-  Failure to run the migration can have serious consequences.
-  See Readme for more info.
-  ********************************************************************************
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib