Merge pull request #1 from Leafly-com/jackjennings-table-name-config

Add configuration for table name

README.md

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

lib/data_migrate/config.rb

@@ -12,11 +12,12 @@ def config
   end
 
   class Config
-    attr_accessor :data_migrations_path, :data_template_path, :db_configuration, :spec_name
+    attr_accessor :data_migrations_table_name, :data_migrations_path, :data_template_path, :db_configuration, :spec_name
 
     DEFAULT_DATA_TEMPLATE_PATH = "data_migration.rb"
 
     def initialize
+      @data_migrations_table_name = "data_migrations"
       @data_migrations_path = "db/data/"
       @data_template_path = DEFAULT_DATA_TEMPLATE_PATH
       @db_configuration = nil

lib/data_migrate/data_schema_migration.rb

@@ -5,7 +5,7 @@ class << self
 
       def instance
         @instance ||= Class.new(::ActiveRecord::SchemaMigration) do
-          define_singleton_method(:table_name) { ActiveRecord::Base.table_name_prefix + 'data_migrations' + ActiveRecord::Base.table_name_suffix }
+          define_singleton_method(:table_name) { ActiveRecord::Base.table_name_prefix + DataMigrate.config.data_migrations_table_name + ActiveRecord::Base.table_name_suffix }
           define_singleton_method(:primary_key) { "version" }
         end
       end

spec/data_migrate/data_schema_migration_spec.rb

@@ -6,6 +6,27 @@
     it "returns correct table name" do
       expect(subject.table_name).to eq("data_migrations")
     end
+
+    describe "data migrations table name configured" do
+      let(:data_migrations_table_name) { "my_app_data_template_migrations"}
+
+      before do
+        @before = DataMigrate.config.data_migrations_table_name
+        DataMigrate.configure do |config|
+          config.data_migrations_table_name = data_migrations_table_name
+        end
+      end
+
+      after do
+        DataMigrate.configure do |config|
+          config.data_migrations_table_name = @before
+        end
+      end
+
+      it "returns correct table name" do
+        expect(subject.table_name).to eq(data_migrations_table_name)
+      end
+    end
   end
 
   describe :index_name do