csv_record 1.9.0 → 2.0.0

data/lib/csv_record/writer.rb

@@ -1,105 +1,103 @@
-module CsvRecord
-  module Writer
-    module ClassMethods
-      def __create__(attributes={})
-        instance = self.build attributes
-        result = instance.save
-        instance
-      end
+module CsvRecord::Writer
+  module ClassMethods
+    def __create__(attributes={})
+      instance = self.build attributes
+      result = instance.save
+      instance
+    end
 
-      [:attr_accessor, :attr_writer].each do |custom_accessor|
-        define_method custom_accessor do |*args|
-          @relevant_instance_variables ||= []
-          args.each { |arg| @relevant_instance_variables << arg }
-          super *args
-        end
+    [:attr_accessor, :attr_writer].each do |custom_accessor|
+      define_method custom_accessor do |*args|
+        @relevant_instance_variables ||= []
+        args.each { |arg| @relevant_instance_variables << arg }
+        super *args
       end
-
-      alias :create :__create__
     end
 
-    module InstanceMethods
-      def self.included(receiver)
-        receiver.send :attr_accessor, :id
-      end
+    alias :create :__create__
+  end
 
-      def __save__(validate=true)
-        if (not validate) || self.valid?
-          self.new_record? ? self.append_registry : self.update_registry
-        else
-          false
-        end
-      end
+  module InstanceMethods
+    def self.included(receiver)
+      receiver.send :attr_accessor, :id
+    end
 
-      def new_record?
-        self.created_at.nil? || self.id.nil?
+    def __save__(validate=true)
+      if (not validate) || self.valid?
+        self.new_record? ? self.append_registry : self.update_registry
+      else
+        false
       end
+    end
 
-      def __update_attribute__(field, value)
-        self.public_send "#{field}=", value
-        self.save false
-      end
+    def new_record?
+      self.created_at.nil? || self.id.nil?
+    end
+
+    def __update_attribute__(field, value)
+      self.public_send "#{field}=", value
+      self.save false
+    end
 
-      def __update_attributes__(params={validate: true})
-        validate = params[:validate]
-        params.delete(:validate)
+    def __update_attributes__(params={validate: true})
+      validate = params[:validate]
+      params.delete(:validate)
 
-        params.each do |field, value|
-          self.public_send "#{field}=", value
-        end
-        self.save validate
+      params.each do |field, value|
+        self.public_send "#{field}=", value
       end
+      self.save validate
+    end
 
-      def __destroy__
-        self.class.parse_database_file do |row|
-          new_row = row
-          new_row = nil if self.id.to_i == row.field('id').to_i
-          new_row
-        end
-        empty_fields
-        true
+    def __destroy__
+      self.class.parse_database_file do |row|
+        new_row = row
+        new_row = nil if self.id.to_i == row.field('id').to_i
+        new_row
       end
+      empty_fields
+      true
+    end
 
-      protected
+    protected
 
-      def calculate_id
-        @id = self.class.count + 1
-      end
+    def calculate_id
+      @id = self.class.count + 1
+    end
 
-      def append_registry
-        set_created_at
-        write_object
-      end
+    def append_registry
+      set_created_at
+      write_object
+    end
 
-      def update_registry
-        set_updated_at
-        self.class.parse_database_file do |row|
-          new_row = row
-          new_row = self.values if self.id.to_i == row.field('id').to_i
-          new_row
-        end
-        true
+    def update_registry
+      set_updated_at
+      self.class.parse_database_file do |row|
+        new_row = row
+        new_row = self.values if self.id.to_i == row.field('id').to_i
+        new_row
       end
+      true
+    end
 
-      def __write_object__
-        calculate_id
-        self.class.open_database_file CsvRecord::Connector::APPEND_MODE do |csv|
-          csv << values
-        end
-        true
+    def __write_object__
+      calculate_id
+      self.class.open_database_file CsvRecord::Connector::APPEND_MODE do |csv|
+        csv << values
       end
+      true
+    end
 
-      def empty_fields
-        %w(id created_at updated_at).each do |field|
-          self.public_send "#{field}=", nil
-        end
+    def empty_fields
+      %w(id created_at updated_at).each do |field|
+        self.public_send "#{field}=", nil
       end
-
-      alias :save :__save__
-      alias :write_object :__write_object__
-      alias :update_attribute :__update_attribute__
-      alias :destroy :__destroy__
-      alias :update_attributes :__update_attributes__
     end
+
+    alias :save :__save__
+    alias :write_object :__write_object__
+    alias :update_attribute :__update_attribute__
+    alias :destroy :__destroy__
+    alias :update_attributes :__update_attributes__
   end
 end

data/test/csv_record/callbacks_test.rb

@@ -1,5 +1,4 @@
 require_relative '../test_helper'
-require_relative '../models/car'
 require_relative '../models/callback_test_class'
 
 describe CsvRecord::Callbacks do
@@ -85,6 +84,7 @@ describe CsvRecord::Callbacks do
     it 'after_destroy' do
       object_destroyed.after_destroy_called.must_equal true
     end
+
     it 'before_destroy' do
       object_destroyed.before_destroy_called.must_equal true
     end

data/test/csv_record/condition_test.rb

@@ -0,0 +1,35 @@
+require_relative '../test_helper'
+
+describe CsvRecord::Condition do
+  let (:condition) { CsvRecord::Condition.new :age, 37 }
+
+  describe 'checking query initialization' do
+    it 'field should be "age"' do
+      condition.field.must_equal :age
+    end
+
+    it 'value should be "37"' do
+      condition.value.must_equal 37
+    end
+  end
+
+  describe '.create_from_hashes' do
+    it 'with one' do
+      conditions = CsvRecord::Condition.create_from_hashes age: 37
+      conditions.size.must_equal 1
+      conditions.first.must_be_instance_of CsvRecord::Condition
+    end
+
+    it 'more than one' do
+      conditions = CsvRecord::Condition.create_from_hashes age: 37, midi_chlorians: '3k'
+      conditions.size.must_equal 2
+      conditions.first.must_be_instance_of CsvRecord::Condition
+    end
+  end
+
+  describe '#to_code' do
+    it 'trasforming the field and value' do
+      condition.to_code.must_equal "attributes['age'] == '37'"
+    end
+  end
+end

data/test/csv_record/connector_test.rb

@@ -1,30 +1,30 @@
 require_relative '../test_helper'
 
-require_relative '../models/car'
+require_relative '../models/jedi'
 
 describe CsvRecord::Connector do
   describe 'initializing methods' do
-    it ('responds to initialize_db_directory') { Car.must_respond_to :initialize_db_directory }
-    it ('responds to initialize_db') { Car.must_respond_to :initialize_db }
-    it ('responds to db_initialized?') { Car.must_respond_to :db_initialized? }
-    it ('responds to open_database_file') { Car.must_respond_to :open_database_file }
+    it ('responds to initialize_db_directory') { Jedi.must_respond_to :initialize_db_directory }
+    it ('responds to initialize_db') { Jedi.must_respond_to :initialize_db }
+    it ('responds to db_initialized?') { Jedi.must_respond_to :db_initialized? }
+    it ('responds to open_database_file') { Jedi.must_respond_to :open_database_file }
   end
 
   describe 'validating the methods behavior' do
     it "Creates the database folder" do
-      Car.initialize_db_directory.wont_be_nil
+      Jedi.initialize_db_directory.wont_be_nil
       Dir.exists?('db').must_equal true
     end
 
     it "Checks the database initialization state" do
-      Car.db_initialized?.must_equal false
-      car.save
-      Car.db_initialized?.must_equal true
+      Jedi.db_initialized?.must_equal false
+      luke.save
+      Jedi.db_initialized?.must_equal true
     end
 
     it "Creates the database file" do
-      car.save
-      File.exists?(Car::DATABASE_LOCATION).must_equal true
+      luke.save
+      File.exists?(Jedi::DATABASE_LOCATION).must_equal true
     end
   end
 end

data/test/csv_record/query_test.rb

@@ -0,0 +1,58 @@
+require_relative '../test_helper'
+require_relative '../models/jedi'
+
+describe CsvRecord::Query do
+  let (:query) { CsvRecord::Query.new Jedi, age: 37 }
+
+  describe 'checking query initialization' do
+    describe 'setting the conditions' do
+      it 'checking the amount' do
+        query.conditions.length.must_equal 1
+      end
+
+      it 'checking the type' do
+        query.conditions.first.must_be_instance_of CsvRecord::Condition
+      end
+    end
+  end
+
+  describe '#where' do
+    describe 'adding more conditions' do
+      let (:second_query) { CsvRecord::Query.new Jedi, age: 852 }
+
+      it 'to one query' do
+        query.where name: 'Luke Skywalker'
+        query.conditions.length.must_equal 2
+      end
+
+      it 'two queries' do
+        query.where midi_chlorians: '4k'
+
+        query.conditions.length.must_equal 2
+        second_query.conditions.length.must_equal 1
+      end
+    end
+  end
+
+  describe 'lazy querying' do
+    before do
+      yoda.save
+      qui_gon_jinn.save
+    end
+
+    describe 'should trigger the query' do
+      it 'when calling the to_a' do
+        query.to_a.must_be_instance_of Array
+        query.to_a.first.must_be_instance_of Jedi
+      end
+
+      it 'when calling the first' do
+        query.last.must_be_instance_of Jedi
+      end
+
+      it 'when calling the last' do
+        query.first.must_be_instance_of Jedi
+      end
+    end
+  end
+end

data/test/csv_record/reader_test.rb

@@ -1,76 +1,65 @@
 require_relative '../test_helper'
 
-require_relative '../models/car'
 require_relative '../models/jedi'
 
 describe CsvRecord::Reader do
   describe 'initializing class methods' do
-    it ('responds to fields') { Car.must_respond_to :fields }
-    it ('responds to all') { Car.must_respond_to :all }
+    it ('responds to fields') { Jedi.must_respond_to :fields }
+    it ('responds to all') { Jedi.must_respond_to :all }
   end
 
   describe 'initializing instance methods' do
-    it ('responds to values') { Car.new.must_respond_to :values }
-    it ('responds to attributes') { Car.new.must_respond_to :attributes }
-    it ('responds to to_param') { Car.new.must_respond_to :to_param }
-    it ('responds to ==') { Car.new.must_respond_to :== }
-    it ('responds to !=') { Car.new.must_respond_to :!= }
+    let (:klass) { Jedi.new }
+
+    it ('responds to values') { klass.must_respond_to :values }
+    it ('responds to attributes') { klass.must_respond_to :attributes }
+    it ('responds to to_param') { klass.must_respond_to :to_param }
+    it ('responds to ==') { klass.must_respond_to :== }
+    it ('responds to !=') { klass.must_respond_to :!= }
   end
 
   describe 'validating the methods behavior' do
-    let(:second_car) do
-      Car.build(
-        year: 2007,
-        make: 'Chevrolet',
-        model: 'F450',
-        description: 'ac, abs, moon',
-        price: 5000.00
-      )
-    end
-
     it 'building an instance' do
-      new_car = Car.build(
-        year: 2007,
-        make: 'Chevrolet',
-        model: 'F450',
-        description: 'ac, abs, moon',
-        price: 5000.00
+      new_jedi = Jedi.build(
+        name: 'Yoda the red',
+        age: 148,
+        midi_chlorians: '0.5k'
       )
-      new_car.wont_be_nil
-      new_car.must_be_instance_of Car
+      new_jedi.wont_be_nil
+      new_jedi.must_be_instance_of Jedi
     end
 
-    it "Check the current fields" do
-      Car.fields.must_equal [:id, :created_at, :updated_at, :year, :make, :model, :description, :price]
+    it 'Check the current fields' do
+      Jedi.fields.must_equal [:id, :created_at, :updated_at, :name, :age, :midi_chlorians, "jedi_order_id"]
     end
 
-    it "Check the current values" do
-      car.values.must_equal [nil, nil, nil, 1997, 'Ford', 'E350', 'ac, abs, moon', 3000.00]
+    it 'Check the current values' do
+      luke.values.must_equal [nil, nil, nil, "Luke Skywalker", 18, "12k", 0]
     end
 
-    it "Check the current attributes" do
-      expected_result = {id: nil, created_at: nil, updated_at: nil, year: 1997, make: 'Ford', model: 'E350', description: 'ac, abs, moon', price: 3000.0}
-      car.attributes.must_equal expected_result
+    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 "Retrieves all registries" do
-      car.save
-      Car.all.size.must_equal 1
-      second_car.save
-      Car.all.size.must_equal 2
+      luke.save
+      Jedi.all.size.must_equal 1
+      yoda.save
+      Jedi.all.size.must_equal 2
     end
 
     it "counting the registries" do
-      car.save
-      Car.count.must_equal 1
-      second_car.save
-      Car.count.must_equal 2
+      luke.save
+      Jedi.count.must_equal 1
+      yoda.save
+      Jedi.count.must_equal 2
     end
 
     it 'checking to_param' do
-      car.save
-      car.to_param.wont_be_nil
-      car.to_param.must_equal '1'
+      luke.save
+      luke.to_param.wont_be_nil
+      luke.to_param.must_equal '1'
     end
 
     describe '==' do
@@ -114,78 +103,78 @@ describe CsvRecord::Reader do
     end
 
     describe 'simple query' do
-      let (:cars) { [] }
+      let (:jedis) { [] }
 
       before do
         3.times do
-          cars << car.clone
-          cars.last.save
+          jedis << luke.clone
+          jedis.last.save
         end
       end
 
       it 'querying by id' do
-        Car.find(cars.first.id).wont_be_nil
-        Car.find(cars.first.id).must_be_instance_of Car
+        Jedi.find(jedis.first.id).wont_be_nil
+        Jedi.find(jedis.first.id).must_be_instance_of Jedi
       end
 
       it 'querying by object' do
-        Car.find(cars.first).wont_be_nil
-        Car.find(cars.first.id).must_be_instance_of Car
+        Jedi.find(jedis.first).wont_be_nil
+        Jedi.find(jedis.first.id).must_be_instance_of Jedi
       end
 
       it 'gets the first' do
-        first_car = Car.first
-        first_car.wont_be_nil
-        first_car.must_be_instance_of Car
+        first_jedi = Jedi.first
+        first_jedi.wont_be_nil
+        first_jedi.must_be_instance_of Jedi
       end
 
       it 'gets the last' do
-        last_car = Car.last
-        last_car.wont_be_nil
-        last_car.must_be_instance_of Car
+        last_jedi = Jedi.last
+        last_jedi.wont_be_nil
+        last_jedi.must_be_instance_of Jedi
       end
     end
 
     describe 'complex queries' do
       before do
-        car.save
-        second_car.save
+        luke.save
+        qui_gon_jinn.save
       end
 
       it 'with a single parameter' do
-        result = Car.where year: 2007
+        result = Jedi.where age: 37
         result.wont_be_empty
-        result.first.year.must_equal '2007'
+        result.first.age.must_equal '37'
       end
 
       it 'with multiple parameters' do
-        result = Car.where year: 2007, make: 'Chevrolet', model: 'F450'
+        result = Jedi.where age: 37, name: 'Qui-Gon Jinn', midi_chlorians: '3k'
         result.wont_be_empty
-        result.first.year.must_equal '2007'
+        result.first.age.must_equal '37'
       end
 
       it 'with invalid parameter' do
-        result = Car.where year: 2008, make: 'Chevroletion'
+        result = Jedi.where age: 22, name: 'Obi Wan Kenobi'
         result.must_be_empty
       end
     end
 
     describe 'dynamic finders' do
       before do
-        car.save
-        second_car.save
+        luke.save
+        yoda.save
       end
 
-      let (:properties) { Car.fields }
+      let (:properties) { Jedi.fields }
 
       it 'respond to certain dynamic methods' do
-        Car.must_respond_to "find_by_#{properties.sample}"
+        Jedi.must_respond_to "find_by_#{properties.sample}"
       end
 
       it 'finding with a single field' do
-        found_cars = Car.find_by_year 2007
-        found_cars.wont_be_empty
-        found_cars.first.must_be_instance_of Car
+        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
@@ -194,12 +183,12 @@ describe CsvRecord::Reader do
           values = []
           i.times do |k|
             conditions[i] = conditions[i] ? "#{conditions[i]}_and_#{properties[k]}" : properties[k]
-            values << car.send(properties[k])
+            values << luke.send(properties[k])
           end
           if conditions[i]
-            found_cars = Car.public_send "find_by_#{conditions[i]}", *values
-            found_cars.wont_be_empty
-            found_cars.first.must_be_instance_of Car
+            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

@@ -2,16 +2,9 @@ require_relative '../test_helper'
 
 require 'timecop'
 
-require_relative '../models/car'
-
 describe CsvRecord::Timestamps do
-  describe 'initializing instance methods' do
-    it ('responds to created_at') { car.must_respond_to :created_at }
-    it ('responds to set_created_at') { car.must_respond_to :set_created_at }
-  end
-
   describe 'checking if it`s extracting the right fields' do
-    it ('checking created_at') { Car.fields.must_include :created_at }
+    it ('checking created_at') { Jedi.fields.must_include :created_at }
   end
 
   describe 'defines on create' do
@@ -24,22 +17,22 @@ describe CsvRecord::Timestamps do
     end
 
     it 'sets the time wich the object was created' do
-      car.save
-      car.created_at.must_equal Time.now.utc
+      luke.save
+      luke.created_at.must_equal Time.now.utc
     end
 
     it 'sets the updated time on created' do
-      car.save
-      car.updated_at.must_equal Time.now.utc
+      luke.save
+      luke.updated_at.must_equal Time.now.utc
     end
   end
 
   describe 'update on update' do
     it 'sets the updated_at attribute on every save' do
-      car.save
-      previous_time = car.updated_at
-      car.update_attribute :year, '1800'
-      car.updated_at.wont_equal previous_time
+      luke.save
+      previous_time = luke.updated_at
+      luke.update_attribute :age, '18'
+      luke.updated_at.wont_equal previous_time
     end
   end
 end

data/test/csv_record/validation_test.rb

@@ -18,44 +18,23 @@ describe CsvRecord::Validations do
     it ('responds to errors') { Jedi.new.must_respond_to :errors }
   end
 
-  describe 'validates_presence_of :name and :age behavior' do
-    it 'is not valid without name' do
-      yoda.name = nil
-      yoda.valid?.wont_equal true
-    end
-
-    it 'is not valid without age' do
-      yoda.age = nil
-      yoda.valid?.wont_equal true
-    end
-
-    it "is valid with all attributes filled" do
+  describe 'validates_presence_of' do
+    it 'all fields valid' do
       yoda.valid?.must_equal true
     end
 
-    it "saves with both attributes filled" do
-      yoda.save.must_equal true
-    end
-
-    it "doesn't save without name" do
-      yoda.name = nil
-      yoda.valid?.wont_equal true
-      yoda.save.wont_equal true
-    end
-
-    it "doesn't save without age" do
+    it 'one invalid field' do
       yoda.age = nil
-      yoda.valid?.wont_equal true
-      yoda.save.wont_equal true
+      yoda.valid?.must_equal false
     end
 
-    it "doesn't save without both" do
+    it 'all fields invalid' do
       yoda.age = nil
       yoda.name = nil
-      yoda.valid?.wont_equal true
-      yoda.save.wont_equal true
+      yoda.valid?.must_equal false
+      yoda.errors.must_include :age
+      yoda.errors.must_include :name
     end
-
   end
 
   describe 'invalid?' do