csv_record 1.5.0 → 1.6.0

data/README.md

@@ -33,6 +33,8 @@ class Car
   include CsvRecord::Document
 
   attr_accessor :year, :make, :model, :description, :price
+
+  validates_presence_of :price, :model
 end
 ```
 
@@ -48,7 +50,7 @@ Car.create( # save the new record in the database
 )
 
 car.save # save the record in the database (either creating or changing)
-
+car.valid? # verifies its validity, specially if it is declared some of the available validators (e.x:validates_presence_of).
 car.update_attribute :year, 1999 # update a single field of an object
 car.update_attributes year: 1999, model: 'E762' # update multiple fields at the same time
 
@@ -102,7 +104,7 @@ car.company # #<Company:0x007f9b249b24d8>
 ```
 
 ###Has Many
-Extending the previous example, you can use the `has_many` method to stablish the inverse relationship:
+Extending the previous example, you can use the `has_many` method to establish the inverse relationship:
 ```ruby
 class Company
   include CsvRecord::Document
@@ -134,6 +136,31 @@ end
 ```
 `obj` refers to the instance you are in
 
+##Validations
+
+Helpers available:
+
+`validates_presence_of`: Ensures if the specified attributes were filled
+
+```ruby
+class Company
+  include CsvRecord::Document
+  validates_presence_of :name
+  attr_accessor :name
+end
+
+company = Company.create :name => ''
+company.save
+# => false
+
+company = Company.create :name => ''
+company.valid?
+# => false
+company.invalid?
+# => true
+
+```
+
 ##Precautions
 CsvRecord creates a `db` folder in the root of your application. Be sure that it has permission to do so.
 

data/Rakefile

@@ -4,7 +4,7 @@ require "rake/testtask"
 
 Rake::TestTask.new do |t|
   t.libs.push "lib"
-  t.test_files = FileList['test/csv_record/*_test.rb']
+  t.test_files = FileList['test/csv_record/*_test.rb', 'test/monkey_patches/*_test.rb']
   t.verbose = true
 end
 

data/csv_record.gemspec

@@ -18,6 +18,7 @@ Gem::Specification.new do |gem|
   gem.add_dependency 'activesupport'
 
   gem.add_development_dependency 'rake'
+  gem.add_development_dependency 'pry'
   gem.add_development_dependency 'timecop'
   gem.add_development_dependency 'turn'
 end

data/lib/csv_record/callbacks.rb

@@ -1,5 +1,6 @@
 module CsvRecord
   module Callbacks
+
     CALLBACKS = [
       :after_create,
       :after_initialize,

data/lib/csv_record/document.rb

@@ -7,6 +7,7 @@ require 'csv_record/timestamps'
 require 'csv_record/callbacks'
 require 'csv_record/helpers'
 require 'csv_record/associations'
+require 'csv_record/validations'
 
 module CsvRecord
 
@@ -23,9 +24,11 @@ module CsvRecord
       receiver.extend         CsvRecord::Writer::ClassMethods
       receiver.extend         CsvRecord::Reader::ClassMethods
       receiver.extend         CsvRecord::Associations
+      receiver.extend         CsvRecord::Validations::ClassMethods
       receiver.send :include, CsvRecord::Writer::InstanceMethods
       receiver.send :include, CsvRecord::Reader::InstanceMethods
       receiver.send :include, CsvRecord::Timestamps
+      receiver.send :include, CsvRecord::Validations::InstanceMethods
       receiver.send :include, CsvRecord::Callbacks
     end
   end

data/lib/csv_record/helpers.rb

@@ -1,12 +1,12 @@
 class Object
   def integer?
-    self.to_s =~ /^\d+$/
-    !$0.empty?
+    /(?<number>^\d+$)/ =~ self.to_s
+    not number.nil?
   end
 
   def float?
-    self.to_s =~ /^\d+\.\d+$/
-    !$0.empty?
+    /(?<number>^\d+\.\d+)$/ =~ self.to_s
+    not number.nil?
   end
 
   def to_param
@@ -32,4 +32,10 @@ class String
     tr("-", "_").
     downcase
   end
+end
+
+class Array
+  def add(obj)
+    self << obj
+  end
 end

data/lib/csv_record/validations.rb

@@ -0,0 +1,43 @@
+module CsvRecord
+  module Validations
+    module ClassMethods
+      attr_writer :fields_to_validate
+
+      def fields_to_validate
+        @fields_to_validate || []
+      end
+
+      def __validates_presence_of__(*attr_names)
+        @fields_to_validate = attr_names
+      end
+
+      alias :validates_presence_of :__validates_presence_of__
+    end
+
+    module InstanceMethods
+      def __valid__?
+        validate_each(self.class.fields_to_validate)
+        errors.empty?
+      end
+
+      def invalid?
+        not self.__valid__?
+      end
+
+      def errors
+        @errors || []
+      end
+
+      alias :valid? :__valid__?
+
+      private
+      def validate_each(attributes)
+        attributes.each do |attribute|
+          if self.public_send(attribute).nil?
+            @errors = self.errors.add attribute
+          end
+        end
+      end
+    end
+  end
+end

data/lib/csv_record/version.rb

@@ -1,3 +1,3 @@
 module CsvRecord
-  VERSION = "1.5.0"
+  VERSION = "1.6.0"
 end

data/lib/csv_record/writer.rb

@@ -26,7 +26,11 @@ module CsvRecord
       end
 
       def __save__
-        self.new_record? ? self.append_registry : self.update_registry
+        if self.valid?
+          self.new_record? ? self.append_registry : self.update_registry
+        else
+          false
+        end
       end
 
       def new_record?
@@ -59,9 +63,6 @@ module CsvRecord
 
       def calculate_id
         @id = self.class.count + 1
-        # if self.respond_to? :jedi_order_id
-        #   p "#{self.class} -> #{@id} -> #{self.jedi_order_id}"
-        # end
       end
 
       def append_registry

data/test/csv_record/associations_test.rb

@@ -1,5 +1,4 @@
 require_relative '../test_helper'
-
 require_relative '../models/jedi'
 require_relative '../models/jedi_order'
 

data/test/csv_record/validation_test.rb

@@ -0,0 +1,77 @@
+require_relative '../test_helper'
+require 'pry'
+require_relative '../models/jedi'
+require_relative '../models/jedi_order'
+
+describe CsvRecord::Validations do
+  describe 'initializing class methods' do
+    it ('responds to validates_presence_of') { Jedi.must_respond_to :validates_presence_of }
+    it ('responds to valid?') { Jedi.new.must_respond_to :valid? }
+    it ('responds to invalid?') { Jedi.new.must_respond_to :invalid? }
+  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
+      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
+      yoda.age = nil
+      yoda.valid?.wont_equal true
+      yoda.save.wont_equal true
+    end
+
+    it "doesn't save without both" do
+      yoda.age = nil
+      yoda.name = nil
+      yoda.valid?.wont_equal true
+      yoda.save.wont_equal true
+    end
+  end
+
+  describe 'default methods' do
+    let (:invalid_jedi) { Jedi.new }
+
+    describe 'invalid?' do
+      it 'invalid object' do
+        invalid_jedi.invalid?.must_equal true
+      end
+
+      it 'valid object' do
+        yoda.invalid?.must_equal false
+      end
+    end
+
+    describe 'errors' do
+      it 'wont be empty when invalid' do
+        invalid_jedi.valid?
+        invalid_jedi.errors.wont_be_empty
+      end
+
+      it 'should have two erros' do
+        invalid_jedi.valid?
+        invalid_jedi.errors.length.must_equal 2
+      end
+    end
+  end
+end

data/test/models/jedi.rb

@@ -5,6 +5,8 @@ class Jedi
 
   belongs_to :jedi_order
 
+  validates_presence_of :name, :age
+
   def initialize(params={})
     params.each do |key, value|
       self.public_send("#{key}=", value)

data/test/monkey_patches/array_test.rb

@@ -0,0 +1,14 @@
+require_relative '../test_helper'
+
+describe Array do
+  describe 'add' do
+    it ('responds to add') { Object.new.must_respond_to :integer? }
+
+    it 'adding' do
+      arr = Array.new
+      arr.add(:testing)
+      arr.wont_be_empty
+      arr.last.must_equal :testing
+    end
+  end
+end

data/test/monkey_patches/object_test.rb

@@ -0,0 +1,57 @@
+require_relative '../test_helper'
+
+describe Object do
+  describe 'monkey patch' do
+    it ('responds to integer?') { Object.new.must_respond_to :integer? }
+    it ('responds to float?') { Object.new.must_respond_to :float? }
+    it ('responds to to_param?') { Object.new.must_respond_to :to_param }
+    it ('responds to underscored_class_name') { Object.new.must_respond_to :underscored_class_name }
+  end
+
+  describe 'integer?' do
+    it 'passing an integer' do
+      15.integer?.must_equal true
+    end
+
+    it 'passing a string' do
+      'abc'.integer?.must_equal false
+    end
+  end
+
+  describe 'float?' do
+    it 'passing a float' do
+      (15.50).float?.must_equal true
+    end
+
+    it 'passing an integer' do
+      15.float?.must_equal false
+    end
+
+    it 'passing a string' do
+      'abc'.float?.must_equal false
+    end
+  end
+
+  describe 'to_param?' do
+    it 'in an integer' do
+      15.to_param.must_equal 15
+    end
+
+    it 'passing a string' do
+      'abc'.to_param.must_equal 'abc'
+    end
+
+    it 'passing a string' 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.new.underscored_class_name.must_equal 'my_custom_class'
+    end
+  end
+end

data/test/test_helper.rb

@@ -1,7 +1,6 @@
 require 'minitest/spec'
 require 'minitest/autorun'
 require 'turn'
-
 require 'csv_record'
 
 module TestHelper

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: csv_record
 version: !ruby/object:Gem::Version
-  version: 1.5.0
+  version: 1.6.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-11-10 00:00:00.000000000 Z
+date: 2012-11-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -43,6 +43,22 @@ dependencies:
     - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: timecop
   requirement: !ruby/object:Gem::Requirement
@@ -97,6 +113,7 @@ files:
 - lib/csv_record/helpers.rb
 - lib/csv_record/reader.rb
 - lib/csv_record/timestamps.rb
+- lib/csv_record/validations.rb
 - lib/csv_record/version.rb
 - lib/csv_record/writer.rb
 - test/csv_record/associations_test.rb
@@ -107,11 +124,14 @@ files:
 - test/csv_record/helpers_test.rb
 - test/csv_record/reader_test.rb
 - test/csv_record/timestamps_test.rb
+- test/csv_record/validation_test.rb
 - test/csv_record/writer_test.rb
 - test/models/callback_test_class.rb
 - test/models/car.rb
 - test/models/jedi.rb
 - test/models/jedi_order.rb
+- test/monkey_patches/array_test.rb
+- test/monkey_patches/object_test.rb
 - test/test_helper.rb
 homepage: https://github.com/lukasalexandre/csv_record
 licenses: []
@@ -147,9 +167,12 @@ test_files:
 - test/csv_record/helpers_test.rb
 - test/csv_record/reader_test.rb
 - test/csv_record/timestamps_test.rb
+- test/csv_record/validation_test.rb
 - test/csv_record/writer_test.rb
 - test/models/callback_test_class.rb
 - test/models/car.rb
 - test/models/jedi.rb
 - test/models/jedi_order.rb
+- test/monkey_patches/array_test.rb
+- test/monkey_patches/object_test.rb
 - test/test_helper.rb