data_migrater 0.2.0 → 0.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a13040b011706340281f05ccfd9bec1d0f1dccbd
-  data.tar.gz: d125de74e4823997d445badbb4b1fc9e35c10265
+  metadata.gz: d18ad057ff627de12fb89e183f32f399fc5982bf
+  data.tar.gz: 71d9bc74af6bedeb81e810512584b20649d26aa0
 SHA512:
-  metadata.gz: 62f37e23c30eacde7c410c3d27f5f82d73a9fadd6ac52d2ea526db44c852b3a8faaf48c99b85ff43c4b81829d6f2c20bc1dcbf0d688716dca4156e6a0a6a49f7
-  data.tar.gz: 429b22415c18dff3dfd8991c2894d662db8a8aae6d1919312e8e7b3db134a090e790201e1f661efac9e8ac9006a7e9904cb1b1d4722b1513683b8b0eb86eb1be
+  metadata.gz: acd8775f066a903ed373309b4c06c934301aaa76ec1a41bcfe769271d3d2c33c69d8f0559bc7d35bdfb91921978e17bf909bfaa7c0de831acd35391759443988
+  data.tar.gz: 7b8c18ec30448b76280a87bca602a335133d11f11b09d2aa4344ae98a0eda6824a0d49620b4883df1574721fc9bebee4f6ecc9560a08f10b3c62cc28543dc0a4

data/CHANGELOG.md

@@ -1,3 +1,7 @@
+# 0.3.0
+
+- Added support for CSV.
+
 # 0.2.0
 
 - Added support for custom logger.

data/README.md

@@ -28,9 +28,9 @@ rails g data_migrater:create name
 ```
 
 Check your `db/data_migrate` folder, the data migration will be there.
-Next time your application run, all pending data migration will be execute.
+Next time your application run, all pending data migration will be executed.
 
-## Custom Logger
+## Logger
 
 You can send your log to a file including the module `DataMigrater::Logger`.
 
@@ -60,7 +60,97 @@ end
 
 #### Options
 
-`path`: Where the log will be written.
+- `dir`: Directory where log will be;
+- `file`: File name;
+- `path`: Composition of `dir/file` when you want give a fully qualified path. Default: `log/class_name.log`.
+
+## CSV
+
+You can parse CSV on the fly using the module `DataMigrater::CSV`.
+
+```
+class MyDataMigration
+  include DataMigrater::CSV
+
+  def execute
+    # parsed from `db/data_migrate/support/csv/my_data_migration.csv`
+    csv.each do |line|
+      Object.create! line
+    end
+  end
+end
+```
+
+By default, the class name is used and the file is parsed from `db/data_migrate/support/csv` folder. You can change it:
+
+```
+class MyDataMigration
+  include DataMigrater::CSV
+
+  data_csv path: "/tmp/objects.csv"
+
+  def execute
+    # parsed from `/tmp/objects.csv`
+    csv.each do |line|
+      Object.create! line
+    end
+  end
+end
+```
+
+You can process a batch of items using the `chunk_size`:
+
+```
+class MyDataMigration
+  include DataMigrater::CSV
+
+  data_csv chunk_size: 2
+
+  def execute
+    csv.each do |line|
+      # line:
+      #
+      # [
+      #   { first_name: 'Washington', last_name: 'Botelho' },
+      #   { first_name: 'Lucas'     , last_name: 'Souza' }
+      # ]
+
+      Object.create! line
+    end
+  end
+end
+```
+
+You can rename the keys inside the iterated data using the option `key_mapping`:
+
+```
+class MyDataMigration
+  include DataMigrater::CSV
+
+  data_csv key_mapping: { first_name: :first }
+
+  def execute
+    csv.each do |line|
+      Object.create! line # { first: 'Washington', last_name: 'Botelho' }
+    end
+  end
+end
+```
+
+#### Options
+
+- `dir`: Directory where CSV is located;
+- `file`: File name;
+- `path`: Composition of `dir/file` when you want give a fully qualified path. Default: `db/data_migrate/support/csv/class_name.csv`.
+
+---
+
+##### CSV Options:
+
+- `chunk_size`: Batch parse size;
+- `key_mapping`: Key name alias.
+
+For more CSV options, check the project [Smarter CSV](https://github.com/tilo/smarter_csv):
 
 ## Test
 

data/lib/data_migrater.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module DataMigrater
 end
 
 require 'data_migration'
 
 require 'data_migrater/collection'
-# require 'data_migrater/csv'
+require 'data_migrater/csv'
 require 'data_migrater/logger'
 require 'data_migrater/migration'
 require 'data_migrater/migrator'

data/lib/data_migrater/collection.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module DataMigrater
   class Collection
     def initialize(path = "#{Rails.root}/db/data_migrate")
@@ -14,7 +16,7 @@ module DataMigrater
 
     def migration_for(file)
       if file =~ migration_pattern
-        DataMigrater::Migration.new $1.to_i, $2, "#{$1}_#{$2}"
+        DataMigrater::Migration.new Regexp.last_match(1).to_i, Regexp.last_match(2), "#{Regexp.last_match(1)}_#{Regexp.last_match(2)}"
       end
     end
 

data/lib/data_migrater/csv.rb

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+module DataMigrater
+  module CSV
+    extend ActiveSupport::Concern
+
+    require 'smarter_csv'
+
+    included do
+      def csv
+        result = ::SmarterCSV.process(csv_path, csv_options)
+
+        result
+      end
+
+      def csv_path
+        return csv_options[:path] if csv_options[:path].present?
+
+        [csv_dir, csv_file].join '/'
+      end
+
+      private
+
+      def csv_dir
+        csv_options.delete(:dir) || 'db/data_migrate/support/csv'
+      end
+
+      def csv_file
+        csv_options.delete(:file) || "#{self.class.name.underscore}.csv"
+      end
+
+      def csv_options
+        self.class.csv_options
+      end
+    end
+
+    module ClassMethods
+      def data_csv(options = {})
+        @options = options
+      end
+
+      def csv_options
+        @options || {}
+      end
+    end
+  end
+end

data/lib/data_migrater/logger.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module DataMigrater
   module Logger
     extend ActiveSupport::Concern
@@ -6,43 +8,47 @@ module DataMigrater
       def logger
         return @logger if @logger
 
-        @logger = ::Logger.new(path)
+        @logger = ::Logger.new(logger_path)
 
         @logger.formatter = formatter
 
         @logger
       end
 
-      private
+      def logger_path
+        return logger_options[:path] if logger_options[:path].present?
 
-      def default_path
-        "log/#{self.class.name.underscore}.log"
+        [logger_dir, logger_file].join '/'
       end
 
+      private
+
       def formatter
-        -> (severity, datetime, _progname, message) do
+        lambda do |severity, datetime, _progname, message|
           "[#{datetime}] #{severity} #{self.class.name}: #{message}\n"
         end
       end
 
-      def options
-        self.class.options
+      def logger_dir
+        logger_options.delete(:dir) || :log
       end
 
-      def path
-        return default_path unless options
+      def logger_file
+        logger_options.delete(:file) || "#{self.class.name.underscore}.log"
+      end
 
-        options.fetch :path, default_path
+      def logger_options
+        self.class.logger_options
       end
     end
 
-    class_methods do
+    module ClassMethods
       def data_logger(options = {})
         @options = options
       end
 
-      def options
-        @options
+      def logger_options
+        @options || {}
       end
     end
   end

data/lib/data_migrater/migration.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module DataMigrater
   class Migration
     attr_reader :version, :name, :filename

data/lib/data_migrater/migrator.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module DataMigrater
   class Migrator
     def initialize(collection = DataMigrater::Collection.new)

data/lib/data_migrater/version.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 module DataMigrater
-  VERSION = "0.2.0"
+  VERSION = '0.3.0'.freeze
 end

data/lib/data_migration.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class DataMigration < ActiveRecord::Base
   validates :version, uniqueness: true
 end

data/lib/generators/data_migrater/create_generator.rb

@@ -1,10 +1,12 @@
+# frozen_string_literal: true
+
 module DataMigrater
   class CreateGenerator < Rails::Generators::Base
     source_root File.expand_path('../templates', __FILE__)
 
     argument :name, type: :string
 
-    desc "create a skeleton data migration."
+    desc 'create a skeleton data migration.'
 
     def create_migrate_file
       tmpl    = 'db/data_migrate/data_migrate.rb.erb'

data/lib/generators/data_migrater/install_generator.rb

@@ -1,8 +1,10 @@
+# frozen_string_literal: true
+
 module DataMigrater
   class InstallGenerator < Rails::Generators::Base
     source_root File.expand_path('../templates', __FILE__)
 
-    desc "creates an initializer and copy necessary files."
+    desc 'creates an initializer and copy necessary files.'
 
     def create_data_folder
       FileUtils.mkdir_p 'db/data_migrate'

data/lib/generators/data_migrater/templates/config/initializers/data_migrater.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'data_migrater'
 
 DataMigrater::Migrator.new.migrate unless Rails.env.test?

data/lib/generators/data_migrater/templates/db/migrate/create_data_migrations.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 class CreateDataMigrations < ActiveRecord::Migration
   def change
     create_table :data_migrations do |t|

data/spec/collection_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe DataMigrater::Collection do
@@ -9,17 +11,15 @@ describe DataMigrater::Collection do
     allow(Rails).to receive(:root) { '.' }
   end
 
-  after do
-    FileUtils.rm_rf 'db'
-  end
+  after { FileUtils.rm_rf 'db' }
 
   describe '.migrations' do
-    let(:file_1) { '20151205090800_add_column' }
-    let(:file_2) { '20151205090801_rename_column' }
-    let(:file_3) { 'invalid_20151205090802_remove_column' }
-
     subject { described_class.new path }
 
+    let!(:file_1) { '20151205090800_add_column' }
+    let!(:file_2) { '20151205090801_rename_column' }
+    let!(:file_3) { 'invalid_20151205090802_remove_column' }
+
     before do
       FileUtils.touch "#{path}/#{file_1}.rb"
       FileUtils.touch "#{path}/#{file_2}.rb"

data/spec/csv/converters_spec.rb

@@ -0,0 +1,36 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe '#converters' do
+  before do
+    stub_const 'Dummy', Class.new
+
+    Dummy.class_eval { include DataMigrater::CSV }
+    Dummy.class_eval { data_csv dir: 'spec/support/csv', value_converters: { birthday: DateConverter } }
+  end
+
+  class DateConverter
+    def self.convert(value)
+      Date.strptime value, '%d/%m/%y'
+    end
+  end
+
+  it 'returns an array of hash' do
+    expect(Dummy.new.csv).to eq [
+      {
+        first_name: 'Washington',
+        last_name:  'Botelho',
+        username:   'wbotelhos',
+        age:        32,
+        birthday:   Date.strptime('23/10/1984', '%d/%m/%y')
+      },
+
+      {
+        first_name: 'Lucas',
+        last_name:  'Souza',
+        username:   'lucasas'
+      }
+    ]
+  end
+end

data/spec/csv/csv_path_spec.rb

@@ -0,0 +1,82 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe '#csv_path' do
+  context 'with :path' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::CSV }
+      Dummy.class_eval { data_csv path: '/tmp/custom.csv' }
+    end
+
+    it 'uses the given one' do
+      expect(Dummy.new.csv_path).to eq '/tmp/custom.csv'
+    end
+  end
+
+  context 'with :dir' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::CSV }
+      Dummy.class_eval { data_csv dir: :dir }
+    end
+
+    it 'uses the given :dir on path' do
+      expect(Dummy.new.csv_path).to eq 'dir/dummy.csv'
+    end
+  end
+
+  context 'with :file' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::CSV }
+      Dummy.class_eval { data_csv file: 'file.odd' }
+    end
+
+    it 'uses the given :file on path' do
+      expect(Dummy.new.csv_path).to eq 'db/data_migrate/support/csv/file.odd'
+    end
+  end
+
+  context 'with :dir and :file' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::CSV }
+      Dummy.class_eval { data_csv dir: :dir, file: 'file.odd' }
+    end
+
+    it 'uses the given :dir and :file as path' do
+      expect(Dummy.new.csv_path).to eq 'dir/file.odd'
+    end
+  end
+
+  context 'with not :dir nor :file' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::CSV }
+      Dummy.class_eval { data_csv }
+    end
+
+    it 'uses the default :dir and :file as path' do
+      expect(Dummy.new.csv_path).to eq 'db/data_migrate/support/csv/dummy.csv'
+    end
+  end
+
+  context 'with no callback options method' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::CSV }
+    end
+
+    it 'uses the default :dir and :file as path' do
+      expect(Dummy.new.csv_path).to eq 'db/data_migrate/support/csv/dummy.csv'
+    end
+  end
+end

data/spec/csv/csv_spec.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe '#data_csv' do
+  before do
+    stub_const 'Dummy', Class.new
+
+    Dummy.class_eval { include DataMigrater::CSV }
+    Dummy.class_eval { data_csv dir: 'spec/support/csv' }
+  end
+
+  it 'returns an array of hash' do
+    expect(Dummy.new.csv).to eq [
+      {
+        first_name: 'Washington',
+        last_name:  'Botelho',
+        username:   'wbotelhos',
+        age:        32,
+        birthday:   '23/10/1984'
+      },
+
+      {
+        first_name: 'Lucas',
+        last_name:  'Souza',
+        username:   'lucasas'
+      }
+    ]
+  end
+end

data/spec/csv/options_spec.rb

@@ -0,0 +1,30 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe '#options' do
+  before do
+    stub_const 'Dummy', Class.new
+
+    Dummy.class_eval { include DataMigrater::CSV }
+    Dummy.class_eval { data_csv dir: 'spec/support/csv', chunk_size: 1, key_mapping: { first_name: :first } }
+  end
+
+  it 'applies the options on csv gem' do
+    expect(Dummy.new.csv).to eq [
+      [{
+        first:     'Washington',
+        last_name: 'Botelho',
+        username:  'wbotelhos',
+        age:       32,
+        birthday:  '23/10/1984'
+      }],
+
+      [{
+        first:     'Lucas',
+        last_name: 'Souza',
+        username:  'lucasas'
+      }]
+    ]
+  end
+end

data/spec/data_migration_spec.rb

@@ -1,15 +1,17 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe DataMigration do
   let(:version) { '19841023113000' }
 
-  it { expect(DataMigration.new version: version).to be_valid }
+  it { expect(described_class.new(version: version)).to be_valid }
 
   it 'validates uniqueness' do
-    DataMigration.create! version: version
+    described_class.create! version: version
 
-    expect {
-      DataMigration.create! version: version
-    }.to raise_error ActiveRecord::RecordInvalid
+    expect do
+      described_class.create! version: version
+    end.to raise_error ActiveRecord::RecordInvalid
   end
 end

data/spec/logger/data_logger_spec.rb

@@ -1,32 +1,30 @@
-require "spec_helper"
+# frozen_string_literal: true
 
-RSpec.describe "#data_logger" do
-  context "with :path" do
+require 'spec_helper'
+
+RSpec.describe '#data_logger' do
+  context 'with :path' do
     before do
-      stub_const "Dummy", Class.new
+      stub_const 'Dummy', Class.new
 
       Dummy.class_eval { include DataMigrater::Logger }
-      Dummy.class_eval { data_logger path: "custom.log" }
-    end
-
-    after :suite do
-      delete :dummy
+      Dummy.class_eval { data_logger path: 'custom.log' }
     end
 
     subject { Dummy.new }
 
-    it "logs on the given path file with right content" do
-      subject.logger.info "done!"
+    it 'logs on the given path file with right content' do
+      subject.logger.info 'done!'
 
       result = /\[\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} -\d{4}\] INFO Dummy: done!\n/
 
-      expect(File.readlines("custom.log").last).to match result
+      expect(File.readlines('custom.log').last).to match result
     end
   end
 
-  context "with no :path" do
+  context 'with no :path' do
     before do
-      stub_const "Dummy", Class.new
+      stub_const 'Dummy', Class.new
 
       Dummy.class_eval { include DataMigrater::Logger }
       Dummy.class_eval { data_logger }
@@ -34,30 +32,30 @@ RSpec.describe "#data_logger" do
 
     subject { Dummy.new }
 
-    it "logs on log folder with class name with right content" do
-      subject.logger.info "done!"
+    it 'logs on log folder with class name with right content' do
+      subject.logger.info 'done!'
 
       result = /\[\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} -\d{4}\] INFO Dummy: done!\n/
 
-      expect(File.readlines("log/dummy.log").last).to match result
+      expect(File.readlines('log/dummy.log').last).to match result
     end
   end
 
-  context "with no :data_logger" do
+  context 'with no :data_logger' do
     before do
-      stub_const "Dummy", Class.new
+      stub_const 'Dummy', Class.new
 
       Dummy.class_eval { include DataMigrater::Logger }
     end
 
     subject { Dummy.new }
 
-    it "logs on log folder with class name with right content" do
-      subject.logger.info "done!"
+    it 'logs on log folder with class name with right content' do
+      subject.logger.info 'done!'
 
       result = /\[\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} -\d{4}\] INFO Dummy: done!\n/
 
-      expect(File.readlines("log/dummy.log").last).to match result
+      expect(File.readlines('log/dummy.log').last).to match result
     end
   end
 end

data/spec/logger/logger_path_spec.rb

@@ -0,0 +1,82 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+RSpec.describe '#logger_path' do
+  context 'with :path' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::Logger }
+      Dummy.class_eval { data_logger path: '/tmp/custom.log' }
+    end
+
+    it 'uses the given one' do
+      expect(Dummy.new.logger_path).to eq '/tmp/custom.log'
+    end
+  end
+
+  context 'with :dir' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::Logger }
+      Dummy.class_eval { data_logger dir: :dir }
+    end
+
+    it 'uses the given :dir on path' do
+      expect(Dummy.new.logger_path).to eq 'dir/dummy.log'
+    end
+  end
+
+  context 'with :file' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::Logger }
+      Dummy.class_eval { data_logger file: 'file.odd' }
+    end
+
+    it 'uses the given :file on path' do
+      expect(Dummy.new.logger_path).to eq 'log/file.odd'
+    end
+  end
+
+  context 'with :dir and :file' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::Logger }
+      Dummy.class_eval { data_logger dir: :dir, file: 'file.odd' }
+    end
+
+    it 'uses the given :dir and :file as path' do
+      expect(Dummy.new.logger_path).to eq 'dir/file.odd'
+    end
+  end
+
+  context 'with not :dir nor :file' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::Logger }
+      Dummy.class_eval { data_logger }
+    end
+
+    it 'uses the default :dir and :file as path' do
+      expect(Dummy.new.logger_path).to eq 'log/dummy.log'
+    end
+  end
+
+  context 'with no callback options method' do
+    before do
+      stub_const 'Dummy', Class.new
+
+      Dummy.class_eval { include DataMigrater::Logger }
+    end
+
+    it 'uses the default :dir and :file as path' do
+      expect(Dummy.new.logger_path).to eq 'log/dummy.log'
+    end
+  end
+end

data/spec/migrater_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe DataMigrater::Migrator do
@@ -22,14 +24,14 @@ describe DataMigrater::Migrator do
     let(:data_migrations) { [data_migration] }
     let(:insert)          { "insert into data_migrations(version) values('21161023010203')" }
     let(:select)          { 'select version from data_migrations' }
-    let(:version)         { lambda { connection.select_one(select)['version'] } }
+    let(:version)         { -> { connection.select_one(select)['version'] } }
 
-    before do
-      connection.insert_sql insert
-    end
+    before { connection.insert insert }
 
     context 'when has no pending migration' do
       context 'when data migration table does not exists' do
+        subject { described_class.new collection }
+
         let(:data_migration) { double }
         let(:collection)     { double migrations: [data_migration] }
 
@@ -37,10 +39,8 @@ describe DataMigrater::Migrator do
           allow(DataMigration).to receive(:table_exists?) { false }
         end
 
-        subject { described_class.new collection }
-
         it 'does not executes data migration' do
-          expect(data_migration).to_not receive :execute
+          expect(data_migration).not_to receive :execute
 
           subject.migrate
         end
@@ -48,11 +48,11 @@ describe DataMigrater::Migrator do
 
       context 'when data migration table exists' do
         context 'but there is no data migration on collection' do
-          let(:collection) { double migrations: [] }
-
           subject { described_class.new collection }
 
-          it { expect { subject.migrate }.to_not change { version.call } }
+          let(:collection) { double migrations: [] }
+
+          it { expect { subject.migrate }.not_to change { version.call } }
         end
 
         context 'and has data migration on collection' do
@@ -95,7 +95,7 @@ describe DataMigrater::Migrator do
         after  { FileUtils.rm next_migration }
 
         it 'does not run the data migrations' do
-          expect(data_migration).to_not receive :execute
+          expect(data_migration).not_to receive :execute
 
           described_class.new(collection).migrate
         end

data/spec/migration_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe DataMigrater::Migration do
@@ -18,11 +20,11 @@ describe DataMigrater::Migration do
     FileUtils.rm_rf 'db'
   end
 
-  describe "#execute" do
-    let(:migration) { double.as_null_object }
-
+  describe '#execute' do
     subject { described_class.new version, name, filename, path }
 
+    let(:migration) { double.as_null_object }
+
     before do
       allow(AddColumnToUsersTable).to receive(:new) { migration }
     end
@@ -35,7 +37,7 @@ describe DataMigrater::Migration do
       specify { expect(subject.execute).to be_falsy }
 
       it 'does not executes' do
-        expect(migration).to_not receive :execute
+        expect(migration).not_to receive :execute
 
         subject.execute
       end
@@ -51,7 +53,7 @@ describe DataMigrater::Migration do
       it 'creates a new data_migrations' do
         subject.execute
 
-        expect(DataMigration.exists? version: version).to be_truthy
+        expect(DataMigration.exists?(version: version)).to be_truthy
       end
 
       context 'and some error is raised' do
@@ -65,7 +67,7 @@ describe DataMigrater::Migration do
           begin
             subject.execute
           rescue
-            expect(DataMigration.exists? version: version).to be_falsy
+            expect(DataMigration.exists?(version: version)).to be_falsy
           end
         end
       end

data/spec/spec_helper.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ENV['RAILS_ENV'] ||= 'test'
 
 require 'active_record/railtie'

data/spec/support/common.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'rspec/rails'
 
 RSpec.configure do |config|

data/spec/support/csv/dummy.csv

@@ -0,0 +1,3 @@
+first name,last name,username,age,birthday
+Washington,Botelho,wbotelhos,32,23/10/1984
+Lucas,Souza,lucasas,,

data/spec/support/database_cleaner.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 RSpec.configure do |config|
   config.before do
     DataMigration.destroy_all

data/spec/support/local/dummy.csv

@@ -0,0 +1,3 @@
+first name,last name,username,age,birthday
+Washington,Botelho,wbotelhos,32,23/10/1984
+Lucas,Souza,lucasas,,

data/spec/support/migrate.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 ActiveRecord::Schema.define do
   create_table :data_migrations do |t|
     t.string :version, null: false

metadata

@@ -1,43 +1,57 @@
 --- !ruby/object:Gem::Specification
 name: data_migrater
 version: !ruby/object:Gem::Version
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors:
 - GetNinjas
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-03-10 00:00:00.000000000 Z
+date: 2017-05-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activerecord
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.1.14
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.1.14
 - !ruby/object:Gem::Dependency
   name: railties
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.1.14
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '0'
+        version: 4.1.14
+- !ruby/object:Gem::Dependency
+  name: smarter_csv
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.1'
 - !ruby/object:Gem::Dependency
   name: bundler
   requirement: !ruby/object:Gem::Requirement
@@ -161,6 +175,7 @@ files:
 - README.md
 - lib/data_migrater.rb
 - lib/data_migrater/collection.rb
+- lib/data_migrater/csv.rb
 - lib/data_migrater/logger.rb
 - lib/data_migrater/migration.rb
 - lib/data_migrater/migrator.rb
@@ -172,15 +187,20 @@ files:
 - lib/generators/data_migrater/templates/db/data_migrate/data_migrate.rb.erb
 - lib/generators/data_migrater/templates/db/migrate/create_data_migrations.rb
 - spec/collection_spec.rb
-- spec/csv/data_csv_spec.rb
+- spec/csv/converters_spec.rb
+- spec/csv/csv_path_spec.rb
+- spec/csv/csv_spec.rb
+- spec/csv/options_spec.rb
 - spec/data_migration_spec.rb
 - spec/logger/data_logger_spec.rb
+- spec/logger/logger_path_spec.rb
 - spec/migrater_spec.rb
 - spec/migration_spec.rb
 - spec/spec_helper.rb
 - spec/support/common.rb
+- spec/support/csv/dummy.csv
 - spec/support/database_cleaner.rb
-- spec/support/methods.rb
+- spec/support/local/dummy.csv
 - spec/support/migrate.rb
 homepage: https://github.com/getninjas/data_migrater
 licenses:
@@ -202,19 +222,24 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.6.8
+rubygems_version: 2.6.11
 signing_key: 
 specification_version: 4
 summary: A Data Migrator gem
 test_files:
 - spec/collection_spec.rb
-- spec/csv/data_csv_spec.rb
+- spec/csv/converters_spec.rb
+- spec/csv/csv_path_spec.rb
+- spec/csv/csv_spec.rb
+- spec/csv/options_spec.rb
 - spec/data_migration_spec.rb
 - spec/logger/data_logger_spec.rb
+- spec/logger/logger_path_spec.rb
 - spec/migrater_spec.rb
 - spec/migration_spec.rb
 - spec/spec_helper.rb
 - spec/support/common.rb
+- spec/support/csv/dummy.csv
 - spec/support/database_cleaner.rb
-- spec/support/methods.rb
+- spec/support/local/dummy.csv
 - spec/support/migrate.rb

data/spec/csv/data_csv_spec.rb

@@ -1,31 +0,0 @@
-require "spec_helper"
-
-class Dummy
-  include DataMigrater::CSV
-
-  data_logger path: "dummy.log"
-
-  def log
-    logger.info "done!"
-  end
-end
-
-RSpec.describe Dummy, "#data_logger" do
-  subject { Dummy.new }
-
-  context "with a given :path" do
-    it "caches the options" do
-      expect(described_class.options).to eq(path: "dummy.log")
-    end
-
-    it "has the right format" do
-      subject.log
-
-      expect(read).to match /\[\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2} -\d{4}\] INFO Dummy: done!\n/
-    end
-  end
-
-  def read
-    File.readlines(described_class.options[:path]).last
-  end
-end

data/spec/support/methods.rb

@@ -1,3 +0,0 @@
-def delete(file_name)
-  File.delete File.expand_path("../#{file_name}.log", __dir__)
-end