data-migration 1.1.0 → 1.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 37438001715cc268d2f647d68ee27c6a49d3a4c879187d61a2779523b1d34fb9
-  data.tar.gz: 55d7277c0848aa1395ab004ec986a3d268e18feebbe5fdbe7766b5658ef2a561
+  metadata.gz: 9a8f110139404aa9a93001f7d7d62df44dfa39ed17c6467c3042b71071f0f5a9
+  data.tar.gz: 17ea84c1d597d6ba5907f0841bee35cf231225f9ac6febfa0dfa4b6c697105d3
 SHA512:
-  metadata.gz: 1d655d2e86118b04fdb4f778e3afa611caab5dc95bd048e71f24baf6235b4c7c8fad1cdc2a70d8a15fb6f5491f942d7f272f91bb2b618f9e1541dfae4a443310
-  data.tar.gz: 72542f7c60d054c70b62b39468edb645c1a3cf777ea07fd30ca710f90bdc9aa8ca66d89979ace38ed2eb48f97740757c836cda7a31169e83031a7b84609c542c
+  metadata.gz: 2cb6a04608ff5fa007f3bc3e5e6fd819413d73b9041ee8ec542de029719d6c089706528e3ec72e9d0442d75596709cb2646f004c00bce6a9100a063be2e91eab
+  data.tar.gz: 53b024625cc45959ac7cc94c5ccb5bf22f1ff463a04cfea4f847a43292b688092aaf9f3e757193fd15c17c47f2a9539b208be3153e043cca313f3b78682ff36c

data/CHANGELOG.md

@@ -1,7 +1,13 @@
-# 1.1.0
+# CHANGELOG
+
+## 1.2.0
+
+- Wrap Task.status enum with ActiveRecord version check
+
+## 1.1.0
 
 - Fix gem main file name to be `data-migration.rb` instead of `data_migration.rb`
 
-# 1.0.0
+## 1.0.0
 
 - Initial version

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    data-migration (1.1.0)
+    data-migration (1.2.0)
       activejob (> 5)
       activerecord (> 5)
       activesupport (> 5)

data/lib/data-migration.rb

@@ -3,7 +3,7 @@ require "data_migration/job"
 require "data_migration/task"
 
 module DataMigration
-  VERSION = "1.1.0".freeze
+  VERSION = "1.2.0".freeze
 
   module_function
 
@@ -17,7 +17,7 @@ module DataMigration
 
   def notify(message, context: {})
     if Object.const_defined?(:ActionReporter)
-      ActionReporter.notify(message, context:)
+      ActionReporter.notify(message, context: context.presence || {})
     elsif Object.const_defined?(:Rails)
       Rails.logger.info("#{message} #{context.inspect}")
     end

data/lib/data_migration/config.rb

@@ -35,7 +35,7 @@ module DataMigration
 
     attr_writer :operator_resolver
     def operator_resolver(&block)
-      if block_given?
+      if block.present?
         @operator_resolver = block
       else
         @operator_resolver ||= -> do
@@ -50,7 +50,7 @@ module DataMigration
 
     attr_writer :monitoring_context
     def monitoring_context(&block)
-      if block_given?
+      if block.present?
         @monitoring_context = block
       else
         @monitoring_context ||= ->(migration) do

data/lib/data_migration/job.rb

@@ -18,7 +18,7 @@ module DataMigration
         return
       end
 
-      task.job_check_in!(job_id, job_args:, job_kwargs:)
+      task.job_check_in!(job_id, job_args: job_args, job_kwargs: job_kwargs)
 
       require migration_path
       klass_name = migration_name.gsub(/^[0-9_]+/, "").camelize

data/lib/data_migration/task.rb

@@ -2,6 +2,7 @@ require "active_record"
 
 module DataMigration
   class JobConcurrencyLimitError < StandardError; end
+
   class JobConflictError < StandardError; end
 
   def self.tasks_table_name
@@ -18,16 +19,21 @@ module DataMigration
       self.jobs_limit ||= DataMigration.config.default_jobs_limit
     end
 
-    enum :status, {
+    STATUS_OPTIONS = {
       started: "started",
       performing: "performing",
       paused: "paused",
       completed: "completed"
     }
+    if ActiveRecord::VERSION::MAJOR >= 7
+      enum :status, STATUS_OPTIONS
+    else
+      enum status: STATUS_OPTIONS
+    end
 
     validates :name, presence: true
-    validates :pause_minutes, numericality: { greater_than_or_equal_to: 0, only_integer: true }, if: -> { pause_minutes.present? }
-    validates :jobs_limit, numericality: { greater_than_or_equal_to: 0, only_integer: true }, if: -> { jobs_limit.present? }
+    validates :pause_minutes, numericality: {greater_than_or_equal_to: 0, only_integer: true}, if: -> { pause_minutes.present? }
+    validates :jobs_limit, numericality: {greater_than_or_equal_to: 0, only_integer: true}, if: -> { jobs_limit.present? }
     validate :file_should_exist
 
     after_save do
@@ -50,7 +56,7 @@ module DataMigration
     end
 
     def self.perform_now(name, **kwargs)
-      create!(name:).perform_now(**kwargs)
+      create!(name: name).perform_now(**kwargs)
     end
 
     def perform_now(**perform_args)
@@ -59,7 +65,7 @@ module DataMigration
     end
 
     def self.perform_later(name, **kwargs)
-      create!(name:).perform_later(**kwargs)
+      create!(name: name).perform_later(**kwargs)
     end
 
     def perform_later(**perform_args)
@@ -68,7 +74,7 @@ module DataMigration
     end
 
     def self.prepare(name, pause_minutes: nil, jobs_limit: nil)
-      create!(name:, pause_minutes:, jobs_limit:)
+      create!(name: name, pause_minutes: pause_minutes, jobs_limit: jobs_limit)
     end
 
     def self.root_path

data/spec/fixtures/data_migrations/20241206200114_create_batch_users.rb

@@ -4,6 +4,6 @@ class CreateBatchUsers
 
     User.find_or_create_by(email: "test_#{index}@example.com")
 
-    enqueue(index: index + 1, background:)
+    enqueue(index: index + 1, background: background)
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: data-migration
 version: !ruby/object:Gem::Version
-  version: 1.1.0
+  version: 1.2.0
 platform: ruby
 authors:
 - Andrei Makarov
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2024-12-09 00:00:00.000000000 Z
+date: 2024-12-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails