seomoz-ripple 1.0.0.pre

data/spec/ripple/validations/associated_validator_spec.rb

@@ -0,0 +1,77 @@
+require 'spec_helper'
+
+describe Ripple::Validations::AssociatedValidator do
+  context 'for a one association' do
+    require 'support/models/family'
+
+    let(:child)  { Child.new  }
+    let(:parent) { Parent.new }
+    before(:each) { parent.child = child }
+
+    it "is invalid when the associated record is invalid" do
+      child.should_not be_valid
+      parent.should_not be_valid
+    end
+
+    it "includes the associated record's validation error messages in the error message" do
+      parent.valid?
+      parent.errors[:child].size.should == 1
+      parent.errors[:child].first.should =~ /is invalid \(Name can't be blank and Age can't be blank\)/
+    end
+
+    it "is valid when the associated record is valid" do
+      child.name = 'Coen'
+      child.age = 1
+      child.should be_valid
+      parent.should be_valid
+    end
+  end
+
+  context 'for a many association' do
+    require 'support/models/team'
+
+    let(:team) { Team.new }
+    let(:ichiro) { Player.new(:name => 'Ichiro', :position => 'Right Field') }
+    let(:satchel_paige) { Player.new(:position => 'Pitcher') }
+    let(:unknown) { Player.new }
+
+    before(:each) do
+      team.players << ichiro
+      team.players << satchel_paige
+      team.players << unknown
+    end
+
+    it 'is invalid when the associated records are invalid' do
+      satchel_paige.should_not be_valid
+      unknown.should_not be_valid
+      team.should_not be_valid
+    end
+
+    it "includes the associated records' validation error messages in the error message" do
+      team.valid?
+      team.errors[:players].size.should == 1
+
+      expected_errors = [
+        "for player 2: Name can't be blank",
+        "for player 3: Name can't be blank and Position can't be blank"
+      ]
+
+      expected_errors = expected_errors.join('; ')
+      team.errors[:players].first.should == "are invalid (#{expected_errors})"
+    end
+
+    it 'is valid when the associated records are all valid' do
+      ichiro.position = 'Right Field'
+      ichiro.should be_valid
+
+      satchel_paige.name = 'Satchel Paige'
+      satchel_paige.should be_valid
+
+      unknown.name = 'John Doe'
+      unknown.position = 'First Base'
+      unknown.should be_valid
+
+      team.should be_valid
+    end
+  end
+end

data/spec/ripple/validations_spec.rb

@@ -0,0 +1,104 @@
+require 'spec_helper'
+
+describe Ripple::Validations do
+  require 'support/models/box'
+
+  before :each do
+    @box = Box.new
+    @client = Ripple.client
+    @client.stub!(:backend).and_return(mock("Backend", :store_object => true))
+  end
+
+  it "should add validation declarations to the class" do
+    [:validates, :validate, :validates_with, :validates_each,
+     :validates_acceptance_of, :validates_confirmation_of, :validates_exclusion_of,
+     :validates_format_of, :validates_inclusion_of, :validates_length_of,
+     :validates_numericality_of, :validates_presence_of].each do |meth|
+      Box.should respond_to(meth)
+    end
+  end
+
+  it "should add validation methods to the instance" do
+    %w{errors valid? invalid?}.each do |meth|
+      @box.should respond_to(meth)
+    end
+  end
+
+  it "should override save to run validations" do
+    @box.should_receive(:valid?).and_return(false)
+    @box.save.should be_false
+  end
+
+  it "should allow skipping validations by passing save :validate => false" do
+    Ripple.client.http.stub!(:perform).and_return(mock_response)
+    @box.should_not_receive(:valid?)
+    @box.save(:validate => false).should be_true
+  end
+
+  describe "when using save! on an invalid record" do
+    before(:each) { @box.stub!(:valid?).and_return(false) }
+
+    it "should raise DocumentInvalid" do
+      lambda { @box.save! }.should raise_error(Ripple::DocumentInvalid)
+    end
+
+    it "should raise an exception that has the invalid document" do
+      begin
+        @box.save!
+      rescue Ripple::DocumentInvalid => invalid
+        invalid.document.should == @box
+      end
+    end
+  end
+
+  it "should not raise an error when save! is called and the document is valid" do
+    @box.stub!(:save).and_return(true)
+    @box.stub!(:valid?).and_return(true)
+    lambda { @box.save! }.should_not raise_error(Ripple::DocumentInvalid)
+  end
+
+  it "should return true from save! when no exception is raised" do
+    @box.stub!(:save).and_return(true)
+    @box.stub!(:valid?).and_return(true)
+    @box.save!.should be_true
+  end
+
+  it "should allow unexpected exceptions to be raised" do
+    robject = mock("robject", :key => @box.key, "data=" => true, "content_type=" => true)
+    robject.should_receive(:store).and_raise(Riak::HTTPFailedRequest.new(:post, 200, 404, {}, "404 not found"))
+    @box.stub!(:robject).and_return(robject)
+    @box.stub!(:valid?).and_return(true)
+    lambda { @box.save! }.should raise_error(Riak::FailedRequest)
+  end
+
+  it "should not raise an error when creating a box with create! succeeds" do
+    @box.stub!(:new?).and_return(false)
+    Box.stub(:create).and_return(@box)
+    lambda { @new_box = Box.create! }.should_not raise_error(Ripple::DocumentInvalid)
+    @new_box.should == @box
+  end
+
+  it "should raise an error when creating a box with create! fails" do
+    @box.stub!(:new?).and_return(true)
+    Box.stub(:create).and_return(@box)
+    lambda { Box.create! }.should raise_error(Ripple::DocumentInvalid)
+  end
+
+  it "should automatically add validations from property options" do
+    Box.property :size, Integer, :inclusion => {:in => 1..30 }
+    @box.size = 0
+    @box.should_not be_valid
+    Box.properties.delete :size
+  end
+
+  it "should run validations at the correct lifecycle state" do
+    Box.property :size, Integer, :inclusion => {:in => 1..30, :on => :update }
+    @box.size = 0
+    @box.should be_valid
+    Box.properties.delete :size
+  end
+
+  after :each do
+    Box.reset_callbacks(:validate)
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,26 @@
+$LOAD_PATH.unshift(File.dirname(__FILE__))
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
+$LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', '..', 'riak-client','lib'))
+
+require 'rubygems' # Use the gems path only for the spec suite
+require 'ripple'
+require 'ripple/conflict/test_helper'
+require 'rspec'
+
+# Only the tests should really get away with this.
+Riak.disable_list_keys_warnings = true
+
+Dir[File.join(File.dirname(__FILE__), "support", "*.rb")].sort.each {|f| require f }
+
+RSpec.configure do |config|
+  config.mock_with :rspec
+  config.after(:each) do
+    $test_server.recycle if $test_server
+  end
+
+  config.treat_symbols_as_metadata_keys_with_true_values = true
+  config.filter_run :focus
+  config.run_all_when_everything_filtered = true
+  config.debug = true
+  config.include Ripple::Conflict::TestHelper
+end

data/spec/support/associations.rb

@@ -0,0 +1 @@
+require File.expand_path("../associations/proxies", __FILE__)

data/spec/support/associations/proxies.rb

@@ -0,0 +1,17 @@
+
+
+class FakeNilProxy < Ripple::Associations::Proxy
+  def find_target; nil end
+end
+
+class FakeBlankProxy < Ripple::Associations::Proxy
+  def find_target; '' end
+end
+
+class FakeNumberProxy < Ripple::Associations::Proxy
+  def find_target; 17 end
+end
+
+class FakeProxy < Ripple::Associations::Proxy
+  def find_target; [1, 2] end
+end

data/spec/support/integration_setup.rb

@@ -0,0 +1,11 @@
+# auto-tag all integration specs with :integration => true
+module IntegrationSpecs
+  def self.included(klass)
+    klass.metadata[:integration] = true
+  end
+end
+
+RSpec.configure do |config|
+  config.include IntegrationSpecs, :example_group => { :file_path => %r{spec/integration} }
+end
+

data/spec/support/mocks.rb

@@ -0,0 +1,4 @@
+
+def mock_response(overrides={})
+  {:headers => {"content-type" => ["application/json"]}, :body => '{}'}.merge(overrides)
+end

data/spec/support/models.rb

@@ -0,0 +1,4 @@
+
+Dir[File.expand_path("../models/*.rb", __FILE__)].sort.each do |file|
+  require "support/models/#{File.basename(file)}"
+end

data/spec/support/models/address.rb

@@ -0,0 +1,12 @@
+
+require 'support/models/note'
+
+class Address
+  include Ripple::EmbeddedDocument
+  property :street, String, :presence => true
+  many :notes
+  embedded_in :user
+end
+
+class SpecialAddress < Address
+end

data/spec/support/models/box.rb

@@ -0,0 +1,13 @@
+
+class Box
+  include Ripple::Document
+  property :shape, String
+  many :sides, :class_name => 'BoxSide'
+  timestamps!
+end
+
+class BoxSide
+  include Ripple::EmbeddedDocument
+  embedded_in :box
+  property :material, String
+end

data/spec/support/models/car.rb

@@ -0,0 +1,16 @@
+
+class Car
+  include Ripple::Document
+  
+  property :make, String
+  property :model, String
+
+  one :driver       # linked, key_on :name
+  many :passengers  # linked, standard :key
+  one :engine       # embedded
+  many :seats       # embedded
+  many :wheels
+
+  accepts_nested_attributes_for :driver, :passengers, :engine, :seats
+  accepts_nested_attributes_for :wheels, :reject_if => proc{|attrs| attrs['diameter'] < 12 }, :allow_destroy => true
+end

data/spec/support/models/cardboard_box.rb

@@ -0,0 +1,3 @@
+
+class CardboardBox < Box
+end

data/spec/support/models/clock.rb

@@ -0,0 +1,12 @@
+
+
+class Clock
+  include Ripple::Document
+  timestamps!
+  many :modes
+end
+
+class Mode
+  include Ripple::EmbeddedDocument
+  timestamps!
+end

data/spec/support/models/clock_observer.rb

@@ -0,0 +1,3 @@
+
+class ClockObserver < ActiveModel::Observer
+end

data/spec/support/models/company.rb

@@ -0,0 +1,23 @@
+class Company
+  include Ripple::Document
+  property :name, String
+  many :departments
+  many :invoices # linked
+  one :ceo, :class_name => 'CEO'
+end
+
+class Department
+  include Ripple::EmbeddedDocument
+  property :name, String
+  many :managers
+end
+
+class Manager
+  include Ripple::EmbeddedDocument
+  property :name, String
+end
+
+class CEO
+  include Ripple::EmbeddedDocument
+  property :name, String
+end

data/spec/support/models/customer.rb

@@ -0,0 +1,4 @@
+
+class Customer
+  include Ripple::Document
+end

data/spec/support/models/driver.rb

@@ -0,0 +1,6 @@
+
+class Driver
+  include Ripple::Document
+  property :name, String
+  key_on :name
+end

data/spec/support/models/email.rb

@@ -0,0 +1,4 @@
+
+class Email
+  include Ripple::Document
+end

data/spec/support/models/engine.rb

@@ -0,0 +1,5 @@
+
+class Engine
+  include Ripple::EmbeddedDocument
+  property :displacement, String  
+end

data/spec/support/models/family.rb

@@ -0,0 +1,16 @@
+class Parent
+  include Ripple::Document
+  one :child
+  validates_associated :child
+end
+
+class Child
+  include Ripple::EmbeddedDocument
+  property :name, String, :presence => true
+  property :age, Integer, :presence => true
+  one :gchild, :class_name => 'Grandchild'
+end
+
+class Grandchild
+  include Ripple::EmbeddedDocument
+end

data/spec/support/models/favorite.rb

@@ -0,0 +1,4 @@
+
+class Favorite
+  include Ripple::EmbeddedDocument
+end

data/spec/support/models/invoice.rb

@@ -0,0 +1,7 @@
+
+class Invoice
+  include Ripple::Document
+  one :customer
+  one :note
+  timestamps!
+end

data/spec/support/models/late_invoice.rb

@@ -0,0 +1,3 @@
+
+class LateInvoice < Invoice
+end

data/spec/support/models/ninja.rb

@@ -0,0 +1,9 @@
+require 'support/models/user'
+
+class Ninja < User
+  property :name, String
+
+  def key
+    "ninja-#{name.downcase}"
+  end
+end

data/spec/support/models/note.rb

@@ -0,0 +1,5 @@
+
+class Note
+  include Ripple::EmbeddedDocument
+  property :text, String
+end

data/spec/support/models/page.rb

@@ -0,0 +1,4 @@
+
+class Page
+  include Ripple::Document
+end 

data/spec/support/models/paid_invoice.rb

@@ -0,0 +1,4 @@
+
+class PaidInvoice < Invoice
+  self.bucket_name = "paid"
+end

data/spec/support/models/passenger.rb

@@ -0,0 +1,6 @@
+
+class Passenger
+  include Ripple::Document
+  property :name, String
+
+end