active_type 0.1.0

data/spec/active_type/record_spec.rb

@@ -0,0 +1,173 @@
+require 'spec_helper'
+
+module RecordSpec
+
+  class Record < ActiveType::Record
+
+    attribute :virtual_string, :string
+    attribute :virtual_integer, :integer
+    attribute :virtual_time, :datetime
+    attribute :virtual_date, :date
+    attribute :virtual_boolean, :boolean
+
+  end
+
+  class RecordWithValidations < Record
+
+    validates :persisted_string, :presence => true
+    validates :virtual_string, :presence => true
+
+  end
+
+
+  class RecordWithOverrides < Record
+
+    attribute :overridable_test, :string
+
+    def overridable_test
+      super + super
+    end
+
+  end
+
+  class RecordCopy < ActiveType::Record
+    self.table_name = 'records'
+
+    attribute :virtual_string, :string
+
+  end
+
+  class OtherRecord < ActiveType::Record
+  end
+end
+
+
+describe ActiveType::Record do
+
+  subject { RecordSpec::Record.new }
+
+  it 'is a ActiveRecord::Base' do
+    subject.should be_a(ActiveRecord::Base)
+  end
+
+  it 'is an abstract class' do
+    ActiveType::Record.should be_abstract_class
+  end
+
+  describe 'constructors' do
+    subject { RecordSpec::Record }
+
+    it_should_behave_like 'ActiveRecord-like constructors', { :persisted_string => "string", :persisted_integer => 100, :persisted_time => Time.now, :persisted_date => Date.today, :persisted_boolean => true }
+
+    it_should_behave_like 'ActiveRecord-like constructors', { :virtual_string => "string", :virtual_integer => 100, :virtual_time => Time.now, :virtual_date => Date.today, :virtual_boolean => true }
+
+  end
+
+  describe 'mass assignment' do
+    it_should_behave_like 'ActiveRecord-like mass assignment', { :persisted_string => "string", :persisted_integer => 100, :persisted_time => Time.now, :persisted_date => Date.today, :persisted_boolean => true }
+
+    it_should_behave_like 'ActiveRecord-like mass assignment', { :virtual_string => "string", :virtual_integer => 100, :virtual_time => Time.now, :virtual_date => Date.today, :virtual_boolean => true }
+  end
+
+  describe 'accessors' do
+    it_should_behave_like 'ActiveRecord-like accessors', { :persisted_string => "string", :persisted_integer => 100, :persisted_time => Time.now, :persisted_date => Date.today, :persisted_boolean => true }
+
+    it_should_behave_like 'ActiveRecord-like accessors', { :virtual_string => "string", :virtual_integer => 100, :virtual_time => Time.now, :virtual_date => Date.today, :virtual_boolean => true }
+  end
+
+  describe 'overridable attributes' do
+
+    subject { RecordSpec::RecordWithOverrides.new }
+
+    it 'is possible to override attributes with super' do
+      subject.overridable_test = "test"
+
+      subject.overridable_test.should == "testtest"
+    end
+  end
+
+  describe 'attribute name validation' do
+    it 'crashes when trying to define an invalid attribute name' do
+      klass = Class.new(ActiveType::Record)
+      expect {
+        klass.class_eval do
+          attribute :"<attr>", :string
+        end
+      }.to raise_error(ActiveType::InvalidAttributeNameError)
+    end
+  end
+
+  describe '.reset_column_information' do
+    it 'does not affect virtual attributes' do
+      RecordSpec::RecordCopy.new.persisted_string = "string"
+      RecordSpec::RecordCopy.reset_column_information
+
+      expect do
+        RecordSpec::RecordCopy.new.virtual_string = "string"
+      end.to_not raise_error
+    end
+  end
+
+  context 'coercible' do
+    describe 'string columns' do
+      it_should_behave_like 'a coercible string column', :persisted_string
+      it_should_behave_like 'a coercible string column', :virtual_string
+    end
+
+    describe 'integer columns' do
+      it_should_behave_like 'a coercible integer column', :persisted_integer
+      it_should_behave_like 'a coercible integer column', :virtual_integer
+    end
+
+    describe 'date columns' do
+      it_should_behave_like 'a coercible date column', :persisted_date
+      it_should_behave_like 'a coercible date column', :virtual_date
+    end
+
+    describe 'time columns' do
+      it_should_behave_like 'a coercible time column', :persisted_time
+      it_should_behave_like 'a coercible time column', :virtual_time
+    end
+
+    describe 'boolean columns' do
+      it_should_behave_like 'a coercible boolean column', :persisted_boolean
+      it_should_behave_like 'a coercible boolean column', :virtual_boolean
+    end
+  end
+
+  describe 'validations' do
+    subject { RecordSpec::RecordWithValidations.new }
+
+    it { should have(1).error_on(:persisted_string) }
+    it { should have(1).error_on(:virtual_string) }
+  end
+
+  describe 'undefined columns' do
+    it 'raises an error when trying to access an undefined virtual attribute' do
+      expect do
+        subject.read_virtual_attribute('foo')
+      end.to raise_error(ActiveType::MissingAttributeError)
+    end
+  end
+
+  describe 'persistence' do
+
+    it 'persists to the database' do
+      subject.persisted_string = "persisted string"
+      subject.save.should be_true
+
+      subject.class.find(subject.id).persisted_string.should == "persisted string"
+    end
+  end
+
+  describe 'isolation' do
+    it 'does not let column information bleed into different models' do
+      record = RecordSpec::Record.new
+      other_record = RecordSpec::OtherRecord.new
+
+      record.should_not respond_to(:other_string)
+      other_record.should_not respond_to(:persisted_string)
+    end
+  end
+
+end

data/spec/integration/sign_in_spec.rb

@@ -0,0 +1,98 @@
+require 'spec_helper'
+
+module SignInSpec
+
+  class SignIn < ActiveType::Object
+    attribute :email, :string
+    attribute :password, :string
+
+    validates :email, :presence => true
+    validates :password, :presence => true
+
+    validate :if => :password do |sign_in|
+      errors.add(:password, 'is not correct') unless sign_in.password == "correct password"
+    end
+
+    after_save :set_session
+
+    def set_session
+    end
+  end
+
+end
+
+describe SignInSpec::SignIn do
+
+  describe 'with missing credentials' do
+
+    it 'is invalid' do
+      subject.should_not be_valid
+    end
+
+    it 'has errors' do
+      subject.valid?
+      subject.errors[:email].should == ["can't be blank"]
+      subject.errors[:password].should == ["can't be blank"]
+    end
+
+    it 'does not save' do
+      subject.save.should be_false
+    end
+
+    it 'does not set the session' do
+      subject.should_not_receive :set_session
+      subject.save
+    end
+
+  end
+
+  describe 'with invalid credentials' do
+
+    before do
+      subject.email = "email"
+      subject.password = "incorrect password"
+    end
+
+    it 'is invalid' do
+      subject.should_not be_valid
+    end
+
+    it 'has errors' do
+      subject.valid?
+      subject.errors[:password].should == ["is not correct"]
+    end
+
+    it 'does not save' do
+      subject.save.should be_false
+    end
+
+    it 'does not set the session' do
+      subject.should_not_receive :set_session
+      subject.save
+    end
+
+  end
+
+  describe 'with valid credentials' do
+
+    before do
+      subject.email = "email"
+      subject.password = "correct password"
+    end
+
+    it 'is invalid' do
+      subject.should be_valid
+    end
+
+    it 'does save' do
+      subject.save.should be_true
+    end
+
+    it 'sets the session' do
+      subject.should_receive :set_session
+      subject.save
+    end
+
+  end
+
+end

data/spec/integration/sign_up_spec.rb

@@ -0,0 +1,90 @@
+require 'spec_helper'
+
+module SignUpSpec
+
+  class User < ActiveType::Record
+    validates :email, :presence => true
+    validates :password, :presence => true
+  end
+
+
+  class SignUp < ActiveType::Record[User]
+    attribute :terms, :boolean
+
+    validates :terms, :acceptance => {:allow_nil => false, :accept => true}
+
+    after_create :send_welcome_email
+
+    def send_welcome_email
+    end
+  end
+
+end
+
+
+describe SignUpSpec::User do
+
+  it 'is valid without a password confirmation' do
+    subject.email = "email"
+    subject.password = "password"
+
+    subject.should be_valid
+  end
+
+end
+
+
+describe SignUpSpec::SignUp do
+
+  it 'is invalid without an email' do
+    subject.password = "password"
+    subject.terms = true
+
+    subject.should_not be_valid
+    subject.errors['email'].should == ["can't be blank"]
+  end
+
+  it 'is invalid without accepted terms' do
+    subject.email = "email"
+    subject.password = "password"
+
+    subject.should_not be_valid
+    subject.errors['terms'].should == ["must be accepted"]
+  end
+
+  context 'with invalid data' do
+
+    it 'does not save' do
+      subject.save.should be_false
+    end
+
+    it 'does not send an email' do
+      subject.should_not_receive :send_welcome_email
+      subject.save
+    end
+
+  end
+
+  context 'with valid data' do
+
+    before do
+      subject.email = "email"
+      subject.password = "password"
+      subject.terms = "1"
+    end
+
+    it 'does save' do
+      subject.valid?
+      subject.save.should be_true
+    end
+
+    it 'sends the email' do
+      subject.should_receive :send_welcome_email
+
+      subject.save
+    end
+
+  end
+
+
+end

data/spec/shared_examples/accessors.rb

@@ -0,0 +1,24 @@
+shared_examples_for "ActiveRecord-like accessors" do |attributes|
+
+  it 'allows to read and write' do
+    attributes.each do |key, value|
+      subject.send("#{key}=", value)
+      subject.send(key).should == value
+    end
+  end
+
+  it 'allows to read via []' do
+    attributes.each do |key, value|
+      subject.send("#{key}=", value)
+      subject[key].should == value
+    end
+  end
+
+  it 'allows to write via []=' do
+    attributes.each do |key, value|
+      subject[key] = value
+      subject.send(key).should == value
+    end
+  end
+
+end

data/spec/shared_examples/belongs_to.rb

@@ -0,0 +1,17 @@
+shared_examples_for 'a belongs_to association' do |association, klass|
+
+  let(:record) { klass.create }
+
+  it 'sets the id when assigning a record' do
+    subject.send("#{association}=", record)
+
+    subject.send("#{association}_id").should == record.id
+  end
+
+  it 'sets the record when assigning an id' do
+    subject.send("#{association}_id=", record.id)
+
+    subject.send("#{association}").should == record
+  end
+
+end

data/spec/shared_examples/coercible_columns.rb

@@ -0,0 +1,203 @@
+module TimeConversionSpec
+  class Record < ActiveRecord::Base
+  end
+end
+
+shared_examples_for 'a coercible string column' do |column|
+
+  it 'is nil by default' do
+    subject.send(column).should be_nil
+  end
+
+  it 'leaves strings alone' do
+    subject.send(:"#{column}=", "string")
+
+    subject.send(column).should == "string"
+  end
+
+  it 'does not convert blank' do
+    subject.send(:"#{column}=", "")
+
+    subject.send(column).should == ""
+  end
+
+end
+
+
+shared_examples_for 'a coercible integer column' do |column|
+
+  it 'is nil by default' do
+    subject.send(column).should be_nil
+  end
+
+  it 'leaves integers alone' do
+    subject.send(:"#{column}=", 10)
+
+    subject.send(column).should == 10
+  end
+
+  it 'converts strings to integers' do
+    subject.send(:"#{column}=", "10")
+
+    subject.send(column).should == 10
+  end
+
+  it 'converts blank to nil' do
+    subject.send(:"#{column}=", "")
+
+    subject.send(column).should be_nil
+  end
+
+end
+
+
+shared_examples_for 'a coercible date column' do |column|
+
+  it 'is nil by default' do
+    subject.send(column).should be_nil
+  end
+
+  it 'leaves dates alone' do
+    date = Date.today
+    subject.send(:"#{column}=", date)
+
+    subject.send(column).should == date
+  end
+
+  it 'converts strings to dates' do
+    subject.send(:"#{column}=", "2010-10-01")
+
+    subject.send(column).should == Date.new(2010, 10, 1)
+  end
+
+  it 'converts blank to nil' do
+    subject.send(:"#{column}=", "")
+
+    subject.send(column).should be_nil
+  end
+
+end
+
+
+shared_examples_for 'a coercible time column' do |column|
+
+  around do |example|
+    begin
+      old_time_zone = Time.zone
+      old_time_zone_aware_attributes = ActiveRecord::Base.time_zone_aware_attributes
+      old_default_timezone = ActiveRecord::Base.default_timezone
+      example.run
+    ensure
+      Time.zone = old_time_zone
+      ActiveRecord::Base.time_zone_aware_attributes = old_time_zone_aware_attributes
+      ActiveRecord::Base.default_timezone = old_default_timezone
+      subject.class.reset_column_information
+    end
+  end
+
+  def it_should_convert_like_active_record(column)
+    time = "2010-10-01 12:15"
+    TimeConversionSpec::Record.reset_column_information
+    subject.class.reset_column_information
+
+    comparison = TimeConversionSpec::Record.new
+    subject.send(:"#{column}=", time)
+    comparison.persisted_time = time
+
+    result = subject.send(column)
+    result.should == comparison.persisted_time
+    result.zone.should == comparison.persisted_time.zone
+  end
+
+
+  it 'is nil by default' do
+    subject.send(column).should be_nil
+  end
+
+  it 'leaves times alone' do
+    time = Time.now
+    subject.send(:"#{column}=", time)
+
+    subject.send(column).should == time
+  end
+
+  it 'converts strings to times' do
+    subject.send(:"#{column}=", "2010-10-01 12:15")
+
+    subject.send(column).should == Time.new(2010, 10, 1, 12, 15)
+  end
+
+  it 'behaves consistently with ActiveRecord' do
+    Time.zone = 'Hawaii'
+
+    it_should_convert_like_active_record(column)
+  end
+
+  it 'behaves consistently with ActiveRecord if time_zone_aware_attributes is set' do
+    Time.zone = 'Hawaii'
+    ActiveRecord::Base.time_zone_aware_attributes = true
+
+    it_should_convert_like_active_record(column)
+  end
+
+  it 'behaves consistently with ActiveRecord if default_timezone is :utc' do
+    Time.zone = 'Hawaii'
+    ActiveRecord::Base.default_timezone = :utc
+
+    it_should_convert_like_active_record(column)
+  end
+
+  it 'behaves consistently with ActiveRecord if time_zone_aware_attributes is set, default_timezone is :utc' do
+    Time.zone = 'Hawaii'
+    ActiveRecord::Base.default_timezone = :utc
+    ActiveRecord::Base.time_zone_aware_attributes = true
+
+    it_should_convert_like_active_record(column)
+  end
+
+end
+
+
+shared_examples_for 'a coercible boolean column' do |column|
+
+  it 'is nil by default' do
+    subject.send(column).should be_nil
+  end
+
+  it 'leaves booleans alone' do
+    subject.send(:"#{column}=", true)
+
+    subject.send(column).should == true
+  end
+
+  it 'converts 1 to true' do
+    subject.send(:"#{column}=", "1")
+
+    subject.send(column).should == true
+  end
+
+  it 'converts 0 to false' do
+    subject.send(:"#{column}=", "0")
+
+    subject.send(column).should == false
+  end
+
+  it 'converts "" to nil' do
+    subject.send(:"#{column}=", "")
+
+    subject.send(column).should be_nil
+  end
+
+  it 'converts "true" to true' do
+    subject.send(:"#{column}=", "true")
+
+    subject.send(column).should == true
+  end
+
+  it 'converts "false" to false' do
+    subject.send(:"#{column}=", "false")
+
+    subject.send(column).should == false
+  end
+
+end

data/spec/shared_examples/constructor.rb

@@ -0,0 +1,30 @@
+shared_examples_for 'ActiveRecord-like constructors' do |attributes|
+
+  it 'return a new record' do
+    subject.new.should be_new_record
+  end
+
+  it 'assigns given attributes' do
+    record = subject.new(attributes)
+
+    attributes.each do |key, value|
+      record.send(key).should == value
+    end
+  end
+
+  if ActiveRecord::VERSION::MAJOR >= 4
+
+    it 'raises on unpermitted parameters' do
+      params = ProtectedParams.new(attributes)
+      expect { subject.new(params) }.to raise_error(ActiveModel::ForbiddenAttributesError)
+    end
+
+    it 'accepts permitted parameters' do
+      params = ProtectedParams.new(attributes)
+      params.permit!
+      expect { subject.new(params) }.to_not raise_error
+    end
+
+  end
+
+end

data/spec/shared_examples/mass_assignment.rb

@@ -0,0 +1,26 @@
+shared_examples_for 'ActiveRecord-like mass assignment' do |attributes|
+
+  it 'assigns all given attributes' do
+    subject.attributes = attributes
+
+    attributes.each do |key, value|
+      subject.send(key).should == value
+    end
+  end
+
+  if ActiveRecord::VERSION::MAJOR >= 4
+
+    it 'raises on unpermitted parameters' do
+      params = ProtectedParams.new(attributes)
+      expect { subject.attributes = params }.to raise_error(ActiveModel::ForbiddenAttributesError)
+    end
+
+    it 'accepts permitted parameters' do
+      params = ProtectedParams.new(attributes)
+      params.permit!
+      expect { subject.attributes = params }.to_not raise_error
+    end
+
+  end
+
+end

data/spec/spec_helper.rb

@@ -0,0 +1,24 @@
+# encoding: utf-8
+
+$: << File.join(File.dirname(__FILE__), "/../../lib" )
+
+require 'active_record'
+require 'active_type'
+
+ActiveRecord::Base.default_timezone = :local
+
+Dir["#{File.dirname(__FILE__)}/support/*"].each {|f| require f}
+Dir["#{File.dirname(__FILE__)}/shared_examples/*"].each {|f| require f}
+
+
+RSpec.configure do |config|
+  config.around do |example|
+    ActiveRecord::Base.transaction do
+      begin
+        example.run
+      ensure
+        raise ActiveRecord::Rollback
+      end
+    end
+  end
+end