csv_record 2.0.0 → 2.1.0

data/test/csv_record/reader_test.rb

@@ -1,7 +1,5 @@
 require_relative '../test_helper'
 
-require_relative '../models/jedi'
-
 describe CsvRecord::Reader do
   describe 'initializing class methods' do
     it ('responds to fields') { Jedi.must_respond_to :fields }
@@ -18,8 +16,8 @@ describe CsvRecord::Reader do
     it ('responds to !=') { klass.must_respond_to :!= }
   end
 
-  describe 'validating the methods behavior' do
-    it 'building an instance' do
+  describe '#build' do
+    it 'traditionally' do
       new_jedi = Jedi.build(
         name: 'Yoda the red',
         age: 148,
@@ -29,167 +27,175 @@ describe CsvRecord::Reader do
       new_jedi.must_be_instance_of Jedi
     end
 
-    it 'Check the current fields' do
-      Jedi.fields.must_equal [:id, :created_at, :updated_at, :name, :age, :midi_chlorians, "jedi_order_id"]
+    it 'with a block' do
+      new_jedi = Jedi.build do |jedi|
+        jedi.name = 'Yoda the red',
+        jedi.age = 148,
+        jedi.midi_chlorians = '0.5k'
+      end
+      new_jedi.wont_be_nil
+      new_jedi.must_be_instance_of Jedi
     end
+  end
 
-    it 'Check the current values' do
-      luke.values.must_equal [nil, nil, nil, "Luke Skywalker", 18, "12k", 0]
+  it '.fields' do
+    [:id, :created_at, :updated_at, :name, :age, :midi_chlorians, "jedi_order_id"].each do |field|
+      Jedi.fields.must_include field
     end
+  end
 
-    it 'Check the current attributes' do
-      expected_result = {:id=>nil, :created_at=>nil, :updated_at=>nil, :name=>"Luke Skywalker", age: 18, midi_chlorians: '12k', 'jedi_order_id' => 0}
-      luke.attributes.must_equal expected_result
-    end
+  it '.values' do
+    luke.values.must_equal [nil, nil, nil, "Luke Skywalker", 18, "12k", 0]
+  end
+
+  describe 'retrieving resources' do
+    before { luke.save }
 
     it "Retrieves all registries" do
-      luke.save
       Jedi.all.size.must_equal 1
       yoda.save
       Jedi.all.size.must_equal 2
     end
 
     it "counting the registries" do
-      luke.save
       Jedi.count.must_equal 1
       yoda.save
       Jedi.count.must_equal 2
     end
 
     it 'checking to_param' do
-      luke.save
       luke.to_param.wont_be_nil
       luke.to_param.must_equal '1'
     end
+  end
 
-    describe '==' do
-      before do
-        yoda.save
-        luke.save
-        jedi_council.save
-      end
+  describe '==' do
+    before do
+      yoda.save
+      luke.save
+      jedi_council.save
+    end
 
-      it 'comparing with the same registry' do
-        (yoda == yoda).must_equal true
-      end
+    it 'comparing with the same registry' do
+      (yoda == yoda).must_equal true
+    end
 
-      it 'comparing with a diferent registry' do
-        (yoda == luke).must_equal false
-      end
+    it 'comparing with a diferent registry' do
+      (yoda == luke).must_equal false
+    end
 
-      it 'comparing with another class registry' do
-        (yoda == jedi_council).must_equal false
-      end
+    it 'comparing with another class registry' do
+      (yoda == jedi_council).must_equal false
     end
+  end
 
-    describe '!=' do
-      before do
-        yoda.save
-        luke.save
-        jedi_council.save
-      end
+  describe '!=' do
+    before do
+      yoda.save
+      luke.save
+      jedi_council.save
+    end
 
-      it 'comparing with the same registry' do
-        (yoda != yoda).must_equal false
-      end
+    it 'comparing with the same registry' do
+      (yoda != yoda).must_equal false
+    end
 
-      it 'comparing with a diferent registry' do
-        (yoda != luke).must_equal true
-      end
+    it 'comparing with a diferent registry' do
+      (yoda != luke).must_equal true
+    end
 
-      it 'comparing with another class registry' do
-        (yoda != jedi_council).must_equal true
-      end
+    it 'comparing with another class registry' do
+      (yoda != jedi_council).must_equal true
     end
+  end
 
-    describe 'simple query' do
-      let (:jedis) { [] }
+  describe 'simple query' do
+    let (:jedis) { [] }
 
-      before do
-        3.times do
-          jedis << luke.clone
-          jedis.last.save
-        end
+    before do
+      3.times do
+        jedis << luke.clone
+        jedis.last.save
       end
+    end
 
-      it 'querying by id' do
-        Jedi.find(jedis.first.id).wont_be_nil
-        Jedi.find(jedis.first.id).must_be_instance_of Jedi
-      end
+    it 'querying by id' do
+      Jedi.find(jedis.first.id).wont_be_nil
+      Jedi.find(jedis.first.id).must_be_instance_of Jedi
+    end
 
-      it 'querying by object' do
-        Jedi.find(jedis.first).wont_be_nil
-        Jedi.find(jedis.first.id).must_be_instance_of Jedi
-      end
+    it 'querying by object' do
+      Jedi.find(jedis.first).wont_be_nil
+      Jedi.find(jedis.first.id).must_be_instance_of Jedi
+    end
 
-      it 'gets the first' do
-        first_jedi = Jedi.first
-        first_jedi.wont_be_nil
-        first_jedi.must_be_instance_of Jedi
-      end
+    it 'gets the first' do
+      first_jedi = Jedi.first
+      first_jedi.wont_be_nil
+      first_jedi.must_be_instance_of Jedi
+    end
 
-      it 'gets the last' do
-        last_jedi = Jedi.last
-        last_jedi.wont_be_nil
-        last_jedi.must_be_instance_of Jedi
-      end
+    it 'gets the last' do
+      last_jedi = Jedi.last
+      last_jedi.wont_be_nil
+      last_jedi.must_be_instance_of Jedi
     end
+  end
 
-    describe 'complex queries' do
-      before do
-        luke.save
-        qui_gon_jinn.save
-      end
+  describe 'complex queries' do
+    before do
+      luke.save
+      qui_gon_jinn.save
+    end
 
-      it 'with a single parameter' do
-        result = Jedi.where age: 37
-        result.wont_be_empty
-        result.first.age.must_equal '37'
-      end
+    it 'with a single parameter' do
+      result = Jedi.where age: 37
+      result.wont_be_empty
+      result.first.age.must_equal '37'
+    end
 
-      it 'with multiple parameters' do
-        result = Jedi.where age: 37, name: 'Qui-Gon Jinn', midi_chlorians: '3k'
-        result.wont_be_empty
-        result.first.age.must_equal '37'
-      end
+    it 'with multiple parameters' do
+      result = Jedi.where age: 37, name: 'Qui-Gon Jinn', midi_chlorians: '3k'
+      result.wont_be_empty
+      result.first.age.must_equal '37'
+    end
 
-      it 'with invalid parameter' do
-        result = Jedi.where age: 22, name: 'Obi Wan Kenobi'
-        result.must_be_empty
-      end
+    it 'with invalid parameter' do
+      result = Jedi.where age: 22, name: 'Obi Wan Kenobi'
+      result.must_be_empty
     end
+  end
 
-    describe 'dynamic finders' do
-      before do
-        luke.save
-        yoda.save
-      end
+  describe 'dynamic finders' do
+    before do
+      luke.save
+      yoda.save
+    end
 
-      let (:properties) { Jedi.fields }
+    let (:properties) { Jedi.fields }
 
-      it 'respond to certain dynamic methods' do
-        Jedi.must_respond_to "find_by_#{properties.sample}"
-      end
+    it 'respond to certain dynamic methods' do
+      Jedi.must_respond_to "find_by_#{properties.sample}"
+    end
 
-      it 'finding with a single field' do
-        found_jedis = Jedi.find_by_age 852
-        found_jedis.wont_be_empty
-        found_jedis.first.must_be_instance_of Jedi
-      end
+    it 'finding with a single field' do
+      found_jedis = Jedi.find_by_age 852
+      found_jedis.wont_be_empty
+      found_jedis.first.must_be_instance_of Jedi
+    end
 
-      it 'finding with multiple fields' do
-        conditions = []
-        properties.each_with_index do |property, i|
-          values = []
-          i.times do |k|
-            conditions[i] = conditions[i] ? "#{conditions[i]}_and_#{properties[k]}" : properties[k]
-            values << luke.send(properties[k])
-          end
-          if conditions[i]
-            found_jedis = Jedi.public_send "find_by_#{conditions[i]}", *values
-            found_jedis.wont_be_empty
-            found_jedis.first.must_be_instance_of Jedi
-          end
+    it 'finding with multiple fields' do
+      conditions = []
+      properties.each_with_index do |property, i|
+        values = []
+        i.times do |k|
+          conditions[i] = conditions[i] ? "#{conditions[i]}_and_#{properties[k]}" : properties[k]
+          values << luke.send(properties[k].name)
+        end
+        if conditions[i]
+          found_jedis = Jedi.public_send "find_by_#{conditions[i]}", *values
+          found_jedis.wont_be_empty
+          found_jedis.first.must_be_instance_of Jedi
         end
       end
     end

data/test/csv_record/timestamps_test.rb

@@ -10,19 +10,16 @@ describe CsvRecord::Timestamps do
   describe 'defines on create' do
     before do
       Timecop.freeze(Time.now.utc)
+      luke.save
     end
 
-    after do
-      Timecop.return
-    end
+    after { Timecop.return }
 
     it 'sets the time wich the object was created' do
-      luke.save
       luke.created_at.must_equal Time.now.utc
     end
 
     it 'sets the updated time on created' do
-      luke.save
       luke.updated_at.must_equal Time.now.utc
     end
   end

data/test/csv_record/validation_test.rb

@@ -1,8 +1,4 @@
 require_relative '../test_helper'
-require 'pry'
-require_relative '../models/jedi'
-require_relative '../models/jedi_order'
-require_relative '../models/custom_errors_class'
 
 describe CsvRecord::Validations do
   let (:invalid_jedi) { Jedi.new }
@@ -13,12 +9,14 @@ describe CsvRecord::Validations do
   end
 
   describe 'initializing instance methods' do
-    it ('responds to valid?') { Jedi.new.must_respond_to :valid? }
-    it ('responds to invalid?') { Jedi.new.must_respond_to :invalid? }
-    it ('responds to errors') { Jedi.new.must_respond_to :errors }
+    let (:jedi) { Jedi.new }
+
+    it ('responds to valid?') { jedi.must_respond_to :valid? }
+    it ('responds to invalid?') { jedi.must_respond_to :invalid? }
+    it ('responds to errors') { jedi.must_respond_to :errors }
   end
 
-  describe 'validates_presence_of' do
+  describe '.validates_presence_of' do
     it 'all fields valid' do
       yoda.valid?.must_equal true
     end
@@ -37,7 +35,7 @@ describe CsvRecord::Validations do
     end
   end
 
-  describe 'invalid?' do
+  describe '#invalid?' do
     it 'invalid object' do
       invalid_jedi.invalid?.must_equal true
     end
@@ -47,7 +45,7 @@ describe CsvRecord::Validations do
     end
   end
 
-  describe 'errors' do
+  describe '#errors' do
     it 'wont be empty when invalid' do
       invalid_jedi.valid?
       invalid_jedi.errors.wont_be_empty
@@ -77,7 +75,7 @@ describe CsvRecord::Validations do
     end
   end
 
-  describe 'validates_uniqueness_of' do
+  describe '.validates_uniqueness_of' do
     before { yoda.save }
 
     let :fake_yoda do
@@ -89,12 +87,13 @@ describe CsvRecord::Validations do
     end
   end
 
-  describe 'custom_validator' do
-    it ('responds to validate') { CustomErrorsClass.must_respond_to :validate }
-
+  describe '.custom_validator' do
     let(:custom_error_class) { CustomErrorsClass.new }
+
     before { custom_error_class.valid? }
 
+    it ('responds to validate') { CustomErrorsClass.must_respond_to :validate }
+
     it 'adding a custom validator' do
       custom_error_class.errors.must_include :custom_error
     end

data/test/csv_record/writer_test.rb

@@ -10,8 +10,6 @@ describe CsvRecord::Writer do
   describe 'initializing instance methods' do
     it ('responds to save') { luke.must_respond_to :save }
     it ('responds to new_record?') { luke.must_respond_to :new_record? }
-    it ('responds to calculate_id') { luke.must_respond_to :calculate_id }
-    it ('responds to write_object') { luke.must_respond_to :write_object }
     it ('responds to id') { luke.must_respond_to :id }
     it ('responds to update_attribute') { luke.must_respond_to :update_attribute }
     it ('responds to update_attributes') { luke.must_respond_to :update_attributes }
@@ -19,7 +17,7 @@ describe CsvRecord::Writer do
   end
 
   describe 'validating the methods behavior' do
-    describe 'create' do
+    describe '.create' do
       it "Creates more than one registry" do
         luke.save
         yoda.save
@@ -44,9 +42,18 @@ describe CsvRecord::Writer do
         yoda.save
         yoda.id.must_equal 2
       end
+
+      it 'should take a block' do
+        jedi = Jedi.create do |jedi|
+          jedi.name = 'Lukas Alexandre'
+          jedi.age = '24'
+          jedi.midi_chlorians = '99k'
+        end
+        jedi.new_record?.must_equal false
+      end
     end
 
-    describe 'new_record' do
+    describe '#new_record' do
       it "Checks whether is a new record" do
         luke.new_record?.must_equal true
         luke.save
@@ -60,21 +67,33 @@ describe CsvRecord::Writer do
     end
 
     describe 'updates' do
-      it "Updates a single field" do
-        luke.save
-        luke.update_attribute :age, 24
-        Jedi.find(luke).age.must_equal '24'
+      before { luke.save }
+
+      describe '#update_attribute' do
+        it "Updates a single field" do
+          luke.update_attribute :age, 24
+          Jedi.find(luke).age.must_equal '24'
+        end
       end
 
-      it "Updates multiple fields at the same time" do
-        luke.save
-        luke.update_attributes name: 'lukas', midi_chlorians: '99999k'
-        Jedi.find(luke).name.must_equal 'lukas'
-        Jedi.find(luke).midi_chlorians.must_equal '99999k'
+      describe '#update_attributes' do
+        it "Updates multiple fields at the same time" do
+          luke.update_attributes name: 'lukas', midi_chlorians: '99999k'
+          Jedi.find(luke).name.must_equal 'lukas'
+          Jedi.find(luke).midi_chlorians.must_equal '99999k'
+        end
+
+        it 'should take a block' do
+          luke.update_attributes do |jedi|
+            jedi.age = 25
+            jedi.name = 'Lukas Skywalker'
+          end
+          luke.age.must_equal 25
+          luke.name.must_equal 'Lukas Skywalker'
+        end
       end
 
       it "Updates multiple fields using save" do
-        luke.save
         luke.name = 'lukas'
         luke.age = 24
         luke.save
@@ -84,20 +103,43 @@ describe CsvRecord::Writer do
       end
     end
 
-    describe 'destroy' do
-      it 'remove the object from the database' do
+    describe '#destroy' do
+      before do
         luke.save
         luke.destroy
+      end
+
+      it 'remove the object from the database' do
         Jedi.count.must_equal 0
       end
 
       it 'by destroying the object its timestamps and id should be empty' do
-        luke.save
-        luke.destroy
         luke.id.must_be_nil
         luke.created_at.must_be_nil
         luke.updated_at.must_be_nil
       end
     end
   end
+
+  describe '.store_as' do
+    it 'uses the default class name' do
+      Jedi.table_name.must_equal 'jedis'
+    end
+
+    it 'uses a custom database name' do
+      CustomizedClass.table_name.must_equal 'wierd_names'
+    end
+  end
+
+  describe '.mapping' do
+    it 'checking the fields initialization' do
+      CustomizedClass.fields.must_include :wierd_field
+      CustomizedClass.doppelganger_fields.wont_include :name
+    end
+
+    it 'checking the recovery of the custom field' do
+      CustomizedClass.create name: 'Ruppert'
+      CustomizedClass.first.name.must_equal 'Ruppert'
+    end
+  end
 end

data/test/helpers.rb

@@ -0,0 +1,7 @@
+module Helpers
+  BASE_PATH = File.expand_path("../fixtures", __FILE__)
+
+  def fetch_fixture_path(path)
+    File.join(BASE_PATH, path)
+  end
+end

data/test/models/customized_class.rb

@@ -0,0 +1,11 @@
+class CustomizedClass
+  include CsvRecord::Document
+
+  store_as :wierd_name
+
+  mapping :name => :wierd_field
+
+  attr_accessor :name
+end
+
+# raise UndefienedCsvField unless self.fields.include? key

data/test/monkey_patches/object_test.rb

@@ -41,16 +41,14 @@ describe Object do
       'abc'.to_param.must_equal 'abc'
     end
 
-    it 'passing a string' do
+    it 'passing a float' do
       (15.50).to_param.must_equal 15.50
     end
   end
 
   describe 'underscored_class_name' do
     it 'custom class' do
-      new_class = Class.new
-      MyCustomClass = new_class
-
+      MyCustomClass = Class.new
       MyCustomClass.new.underscored_class_name.must_equal 'my_custom_class'
     end
   end

data/test/test_helper.rb

@@ -3,15 +3,18 @@ require 'minitest/autorun'
 require 'turn'
 require 'csv_record'
 
-module TestHelper
-  BASE_PATH = File.expand_path("../fixtures", __FILE__)
+require_relative 'helpers'
 
-  def fetch_fixture_path(path)
-    File.join(BASE_PATH, path)
-  end
-end
+require_relative 'models/jedi'
+require_relative 'models/jedi_order'
+require_relative 'models/padawan'
+require_relative 'models/callback_test_class'
+require_relative 'models/custom_errors_class'
+require_relative 'models/customized_class'
 
 class MiniTest::Spec
+  include Helpers
+
   after :each do
     FileUtils.rm_rf 'db'
   end