ar-octopus 0.4.0 → 0.5.0

data/sample_app/features/support/env.rb

@@ -1,6 +1,6 @@
 # IMPORTANT: This file is generated by cucumber-rails - edit at your own peril.
-# It is recommended to regenerate this file in the future when you upgrade to a 
-# newer version of cucumber-rails. Consider adding your own code to a new file 
+# It is recommended to regenerate this file in the future when you upgrade to a
+# newer version of cucumber-rails. Consider adding your own code to a new file
 # instead of editing this one. Cucumber will automatically load all features/**/*.rb
 # files.
 
@@ -24,7 +24,7 @@ require 'aruba/cucumber'
 # steps to use the XPath syntax.
 Capybara.default_selector = :css
 
-# If you set this to false, any error raised from within your app will bubble 
+# If you set this to false, any error raised from within your app will bubble
 # up to your step definition and out to cucumber unless you catch it somewhere
 # on the way. You can make Rails rescue errors and render error pages on a
 # per-scenario basis by tagging a scenario or feature with the @allow-rescue tag.
@@ -36,15 +36,15 @@ Capybara.default_selector = :css
 ActionController::Base.allow_rescue = false
 
 # If you set this to true, each scenario will run in a database transaction.
-# You can still turn off transactions on a per-scenario basis, simply tagging 
+# You can still turn off transactions on a per-scenario basis, simply tagging
 # a feature or scenario with the @no-txn tag. If you are using Capybara,
 # tagging with @culerity or @javascript will also turn transactions off.
 #
 # If you set this to false, transactions will be off for all scenarios,
 # regardless of whether you use @no-txn or not.
 #
-# Beware that turning transactions off will leave data in your database 
-# after each scenario, which can lead to hard-to-debug failures in 
+# Beware that turning transactions off will leave data in your database
+# after each scenario, which can lead to hard-to-debug failures in
 # subsequent scenarios. If you do this, we recommend you create a Before
 # block that will explicitly put your database in a known state.
 Cucumber::Rails::World.use_transactional_fixtures = true
@@ -58,14 +58,14 @@ if defined?(ActiveRecord::Base)
   end
 end
 
-After do 
+After do
   `rm #{Rails.root.to_s}/db/america.sqlite3`
   `rm #{Rails.root.to_s}/db/asia.sqlite3`
   `rm #{Rails.root.to_s}/db/development.sqlite3`
   `rm #{Rails.root.to_s}/db/europe.sqlite3`
 end
 
-Before do 
+Before do
   `rm #{Rails.root.to_s}/db/america.sqlite3`
   `rm #{Rails.root.to_s}/db/asia.sqlite3`
   `rm #{Rails.root.to_s}/db/development.sqlite3`

data/sample_app/lib/tasks/cucumber.rake

@@ -1,6 +1,6 @@
 # IMPORTANT: This file is generated by cucumber-rails - edit at your own peril.
-# It is recommended to regenerate this file in the future when you upgrade to a 
-# newer version of cucumber-rails. Consider adding your own code to a new file 
+# It is recommended to regenerate this file in the future when you upgrade to a
+# newer version of cucumber-rails. Consider adding your own code to a new file
 # instead of editing this one. Cucumber will automatically load all features/**/*.rb
 # files.
 

data/sample_app/public/javascripts/effects.js

@@ -150,7 +150,7 @@ var Effect = {
   toggle: function(element, effect, options) {
     element = $(element);
     effect  = (effect || 'appear').toLowerCase();
-    
+
     return Effect[ Effect.PAIRS[ effect ][ element.visible() ? 1 : 0 ] ](element, Object.extend({
       queue: { position:'end', scope:(element.id || 'global'), limit: 1 }
     }, options || {}));

data/spec/config/shards.yml

@@ -1,18 +1,19 @@
 mysql: &mysql
-  adapter: <%= 'mysql2' %>
+  adapter: mysql
+  username: <%= ENV['MYSQL_USER'] || 'root' %>
   host: localhost
 
 octopus:
   shards:
     alone_shard:
-      database: octopus_shard5
+      database: octopus_shard_5
       <<: *mysql
 
     postgresql_shard:
       adapter: postgresql
-      username: <%= ENV['POSTGRES_USER'] || 'postgres'%>
+      username: <%= ENV['POSTGRES_USER'] || 'postgres' %>
       password:
-      database: octopus_shard1
+      database: octopus_shard_1
       encoding: unicode
 
     sqlite_shard:
@@ -21,37 +22,38 @@ octopus:
 
     history_shards:
       aug2009:
-        database: octopus_shard2
+        database: octopus_shard_2
         <<: *mysql
       aug2010:
-        database: octopus_shard3
+        database: octopus_shard_3
         <<: *mysql
       aug2011:
-        database: octopus_shard4
+        database: octopus_shard_4
         <<: *mysql
 
     country_shards:
       canada:
-        database: octopus_shard2
+        database: octopus_shard_2
         <<: *mysql
       brazil:
-        database: octopus_shard3
+        database: octopus_shard_3
         <<: *mysql
       russia:
-        database: octopus_shard4
+        database: octopus_shard_4
         <<: *mysql
 
+    protocol_shard: postgres://<%= ENV['POSTGRES_USER'] || 'postgres' %>@localhost:5432/octopus_shard_2
 
 production_raise_error:
   shards:
     history_shards:
       duplicated_shard_name:
-        database: octopus_shard5
+        database: octopus_shard_5
         <<: *mysql
 
     country_shards:
       duplicated_shard_name:
-        database: octopus_shard4
+        database: octopus_shard_4
         <<: *mysql
 
 production_replicated:
@@ -60,16 +62,16 @@ production_replicated:
 
   shards:
     slave1:
-      database: octopus_shard2
+      database: octopus_shard_2
       <<: *mysql
     slave2:
-      database: octopus_shard3
+      database: octopus_shard_3
       <<: *mysql
     slave3:
-      database: octopus_shard4
+      database: octopus_shard_4
       <<: *mysql
     slave4:
-      database: octopus_shard5
+      database: octopus_shard_5
       <<: *mysql
 
 
@@ -77,16 +79,16 @@ production_fully_replicated:
   replicated: true
   shards:
     slave1:
-      database: octopus_shard2
+      database: octopus_shard_2
       <<: *mysql
     slave2:
-      database: octopus_shard3
+      database: octopus_shard_3
       <<: *mysql
     slave3:
-      database: octopus_shard4
+      database: octopus_shard_4
       <<: *mysql
     slave4:
-      database: octopus_shard5
+      database: octopus_shard_5
       <<: *mysql
 
 octopus_rails:
@@ -98,32 +100,40 @@ octopus_rails:
 
   staging:
     slave1:
-      database: octopus_shard2
+      database: octopus_shard_2
       <<: *mysql
     slave2:
-      database: octopus_shard3
+      database: octopus_shard_3
       <<: *mysql
 
   production:
     slave3:
-      database: octopus_shard4
+      database: octopus_shard_4
       <<: *mysql
     slave4:
-      database: octopus_shard5
+      database: octopus_shard_5
       <<: *mysql
 
 not_entire_sharded:
   entire_sharded: false
   shards:
     europe:
-      database: octopus_shard2
+      database: octopus_shard_2
       <<: *mysql
     canada:
-      database: octopus_shard3
+      database: octopus_shard_3
       <<: *mysql
     brazil:
-      database: octopus_shard4
+      database: octopus_shard_4
       <<: *mysql
     russia:
-      database: octopus_shard5
+      database: octopus_shard_5
       <<: *mysql
+
+modify_config:
+  replicated: true
+  shards:
+    modify_config_read:
+      adapter: modify_config
+      database: octopus_shard_1
+      host: localhost

data/spec/migrations/11_add_field_in_all_slaves.rb

@@ -1,6 +1,6 @@
 class AddFieldInAllSlaves < ActiveRecord::Migration
   using(:slave1, :slave2, :slave3, :slave4)
-  
+
   def self.up
     Cat.create!(:name => "Slaves")
   end

data/spec/migrations/12_create_users_using_block.rb

@@ -4,7 +4,7 @@ class CreateUsersUsingBlock < ActiveRecord::Migration
       User.create!(:name => "UsingBlock1")
       User.create!(:name => "UsingBlock2")
     end
-    
+
     Octopus.using(:canada) do
       User.create!(:name => "UsingCanada")
       User.create!(:name => "UsingCanada2")
@@ -15,7 +15,7 @@ class CreateUsersUsingBlock < ActiveRecord::Migration
     Octopus.using(:brazil) do
       User.delete_all()
     end
-    
+
     Octopus.using(:canada) do
       User.delete_all()
     end

data/spec/migrations/13_create_users_using_block_and_using.rb

@@ -1,11 +1,11 @@
 class CreateUsersUsingBlockAndUsing < ActiveRecord::Migration
   using(:brazil)
-  
+
   def self.up
     Octopus.using(:canada) do
       User.create!(:name => "Canada")
     end
-    
+
     User.create!(:name => "Brazil")
   end
 

data/spec/migrations/14_create_users_on_shards_of_a_group_with_versions.rb

@@ -0,0 +1,11 @@
+class CreateUsersOnShardsOfAGroupWithVersions < ActiveRecord::Migration
+  using_group(:country_shards)
+
+  def self.up
+    User.create!(:name => "Group")
+  end
+
+  def self.down
+    User.delete_all()
+  end
+end

data/spec/migrations/1_create_users_on_master.rb

@@ -1,4 +1,4 @@
-class CreateUsersOnMaster < ActiveRecord::Migration  
+class CreateUsersOnMaster < ActiveRecord::Migration
   def self.up
     User.create!(:name => "Master")
   end

data/spec/migrations/2_create_users_on_canada.rb

@@ -1,6 +1,6 @@
 class CreateUsersOnCanada < ActiveRecord::Migration
   using(:canada)
-  
+
   def self.up
     User.create!(:name => "Sharding")
   end

data/spec/migrations/3_create_users_on_both_shards.rb

@@ -1,6 +1,6 @@
 class CreateUsersOnBothShards < ActiveRecord::Migration
   using(:brazil, :canada)
-  
+
   def self.up
     User.create!(:name => "Both")
   end

data/spec/migrations/4_create_users_on_shards_of_a_group.rb

@@ -1,6 +1,6 @@
 class CreateUsersOnShardsOfAGroup < ActiveRecord::Migration
   using_group(:country_shards)
-  
+
   def self.up
     User.create!(:name => "Group")
   end

data/spec/migrations/5_create_users_on_multiples_groups.rb

@@ -1,6 +1,6 @@
 class CreateUsersOnMultiplesGroups < ActiveRecord::Migration
   using_group('country_shards', 'history_shards')
-  
+
   def self.up
     User.create!(:name => "MultipleGroup")
   end

data/spec/migrations/6_raise_exception_with_invalid_shard_name.rb

@@ -1,6 +1,6 @@
 class RaiseExceptionWithInvalidShardName < ActiveRecord::Migration
   using(:amazing_shard)
-  
+
   def self.up
     User.create!(:name => "Error")
   end

data/spec/migrations/7_raise_exception_with_invalid_multiple_shard_names.rb

@@ -1,6 +1,6 @@
 class RaiseExceptionWithInvalidMultipleShardNames < ActiveRecord::Migration
   using(:brazil, :invalid_shard)
-  
+
   def self.up
     User.create!(:name => "Error")
   end

data/spec/migrations/8_raise_exception_with_invalid_group_name.rb

@@ -1,6 +1,6 @@
 class RaiseExceptionWithInvalidGroupName < ActiveRecord::Migration
   using_group(:invalid_group)
-  
+
   def self.up
     User.create!(:name => "Error")
   end

data/spec/migrations/9_raise_exception_with_multiple_invalid_group_names.rb

@@ -1,6 +1,6 @@
 class RaiseExceptionWithMultipleInvalidGroupNames < ActiveRecord::Migration
   using_group(:country_shards,:invalid_group)
-  
+
   def self.up
     User.create!(:name => "Error")
   end

data/spec/octopus/association_spec.rb

@@ -1,13 +1,13 @@
-require File.expand_path(File.dirname(__FILE__) + '/../spec_helper')
+require "spec_helper"
 
-describe Octopus::Association do
+describe Octopus::Association, :shards => [:brazil, :master, :canada] do
   describe "when you have a 1 x 1 relationship" do
     before(:each) do
       @computer_brazil = Computer.using(:brazil).create!(:name => "Computer Brazil")
       @computer_master = Computer.create!(:name => "Computer Brazil")
       @keyboard_brazil = Keyboard.using(:brazil).create!(:name => "Keyboard Brazil", :computer => @computer_brazil)
       @keyboard_master = Keyboard.create!(:name => "Keyboard Master", :computer => @computer_master)
-    end    
+    end
 
     it "should find the models" do
       @keyboard_master.computer.should == @computer_master
@@ -23,7 +23,7 @@ describe Octopus::Association do
       @keyboard_brazil.computer_id.should ==  new_computer_brazil.id
       @keyboard_brazil.computer.should ==  new_computer_brazil
       new_computer_brazil.save()
-      new_computer_brazil.reload      
+      new_computer_brazil.reload
       new_computer_brazil.keyboard.should == @keyboard_brazil
     end
 
@@ -46,6 +46,17 @@ describe Octopus::Association do
       k.computer_id.should == c.id
       k.computer.should == c
     end
+
+    if Octopus.rails3?
+      it "should include models" do
+        c = Computer.using(:brazil).create!(:name => "Computer Brazil")
+        k = c.create_keyboard(:name => "Building keyboard")
+        c.save()
+        k.save()
+
+        Computer.includes(:keyboard).find(c.id).should == c
+      end
+    end
   end
 
   describe "when you have a N x N reliationship" do
@@ -56,7 +67,7 @@ describe Octopus::Association do
       @permission_master = Permission.using(:master).create!(:name => "Master Permission")
       @brazil_role.permissions << @permission_brazil
       @brazil_role.save()
-      Client.using(:master).create!(:name => "teste")        
+      Client.using(:master).create!(:name => "teste")
     end
 
     it "should find all models in the specified shard" do
@@ -122,12 +133,12 @@ describe Octopus::Association do
 
       it "create" do
         role = @permission_brazil_2.roles.create(:name => "Builded Role")
-        @permission_brazil_2.roles.to_set.should == [role].to_set          
+        @permission_brazil_2.roles.to_set.should == [role].to_set
       end
 
       it "create" do
         role = @permission_brazil_2.roles.create!(:name => "Builded Role")
-        @permission_brazil_2.roles.to_set.should == [role].to_set          
+        @permission_brazil_2.roles.to_set.should == [role].to_set
       end
 
       it "count" do
@@ -163,37 +174,37 @@ describe Octopus::Association do
 
       it "delete_all" do
         role = @permission_brazil_2.roles.create(:name => "Builded Role")
-        @permission_brazil_2.roles.empty?.should be_false     
-        @permission_brazil_2.roles.delete_all                
+        @permission_brazil_2.roles.empty?.should be_false
+        @permission_brazil_2.roles.delete_all
         @permission_brazil_2.roles.empty?.should be_true
       end
 
       it "destroy_all" do
         role = @permission_brazil_2.roles.create(:name => "Builded Role")
-        @permission_brazil_2.roles.empty?.should be_false     
-        @permission_brazil_2.roles.destroy_all                
+        @permission_brazil_2.roles.empty?.should be_false
+        @permission_brazil_2.roles.destroy_all
         @permission_brazil_2.roles.empty?.should be_true
       end
 
       it "find" do
         role = @permission_brazil_2.roles.create(:name => "Builded Role")
         @permission_brazil_2.roles.find(:first).should == role
-        @permission_brazil_2.roles.destroy_all                
+        @permission_brazil_2.roles.destroy_all
         @permission_brazil_2.roles.find(:first).should be_nil
       end
 
       it "exists?" do
         role = @permission_brazil_2.roles.create(:name => "Builded Role")
         @permission_brazil_2.roles.exists?(role).should be_true
-        @permission_brazil_2.roles.destroy_all                
-        @permission_brazil_2.roles.exists?(role).should be_false     
+        @permission_brazil_2.roles.destroy_all
+        @permission_brazil_2.roles.exists?(role).should be_false
       end
 
       it "clear" do
         role = @permission_brazil_2.roles.create(:name => "Builded Role")
-        @permission_brazil_2.roles.empty?.should be_false     
-        @permission_brazil_2.roles.clear                
-        @permission_brazil_2.roles.empty?.should be_true          
+        @permission_brazil_2.roles.empty?.should be_false
+        @permission_brazil_2.roles.clear
+        @permission_brazil_2.roles.empty?.should be_true
       end
 
       it "delete" do
@@ -277,12 +288,12 @@ describe Octopus::Association do
 
       it "create" do
         role = @new_brazil_programmer.projects.create(:name => "New VB App :-/")
-        @new_brazil_programmer.projects.to_set.should == [role].to_set          
+        @new_brazil_programmer.projects.to_set.should == [role].to_set
       end
 
       it "create" do
         role = @new_brazil_programmer.projects.create!(:name => "New VB App :-/")
-        @new_brazil_programmer.projects.to_set.should == [role].to_set          
+        @new_brazil_programmer.projects.to_set.should == [role].to_set
       end
 
       it "count" do
@@ -318,37 +329,37 @@ describe Octopus::Association do
 
       it "delete_all" do
         role = @new_brazil_programmer.projects.create(:name => "New VB App :-/")
-        @new_brazil_programmer.projects.empty?.should be_false     
-        @new_brazil_programmer.projects.delete_all                
+        @new_brazil_programmer.projects.empty?.should be_false
+        @new_brazil_programmer.projects.delete_all
         @new_brazil_programmer.projects.empty?.should be_true
       end
 
       it "destroy_all" do
         role = @new_brazil_programmer.projects.create(:name => "New VB App :-/")
-        @new_brazil_programmer.projects.empty?.should be_false     
-        @new_brazil_programmer.projects.destroy_all                
+        @new_brazil_programmer.projects.empty?.should be_false
+        @new_brazil_programmer.projects.destroy_all
         @new_brazil_programmer.projects.empty?.should be_true
       end
 
       it "find" do
         role = @new_brazil_programmer.projects.create(:name => "New VB App :-/")
         @new_brazil_programmer.projects.find(:first).should == role
-        @new_brazil_programmer.projects.destroy_all                
+        @new_brazil_programmer.projects.destroy_all
         @new_brazil_programmer.projects.find(:first).should be_nil
       end
 
       it "exists?" do
         role = @new_brazil_programmer.projects.create(:name => "New VB App :-/")
         @new_brazil_programmer.projects.exists?(role).should be_true
-        @new_brazil_programmer.projects.destroy_all                
-        @new_brazil_programmer.projects.exists?(role).should be_false     
+        @new_brazil_programmer.projects.destroy_all
+        @new_brazil_programmer.projects.exists?(role).should be_false
       end
 
       it "clear" do
         role = @new_brazil_programmer.projects.create(:name => "New VB App :-/")
-        @new_brazil_programmer.projects.empty?.should be_false     
-        @new_brazil_programmer.projects.clear                
-        @new_brazil_programmer.projects.empty?.should be_true          
+        @new_brazil_programmer.projects.empty?.should be_false
+        @new_brazil_programmer.projects.clear
+        @new_brazil_programmer.projects.empty?.should be_true
       end
 
       it "delete" do
@@ -370,7 +381,7 @@ describe Octopus::Association do
       @item_brazil = Item.using(:brazil).create!(:name => "Brazil Item", :client => @brazil_client)
       @item_master = Item.create!(:name => "Master Item", :client => @master_client)
       @brazil_client = Client.using(:brazil).find_by_name("Brazil Client")
-      Client.using(:master).create!(:name => "teste")        
+      Client.using(:master).create!(:name => "teste")
     end
 
     it "should find all models in the specified shard" do
@@ -383,7 +394,7 @@ describe Octopus::Association do
     end
 
     it "should raise error if you try to add a record from a different shard" do
-      lambda do 
+      lambda do
         @brazil_client.items << Item.using(:canada).create!(:name => "New User")
       end.should raise_error("Association Error: Records are from different shards")
     end
@@ -410,7 +421,7 @@ describe Octopus::Association do
     describe "it should works when using" do
       before(:each) do
         @item_brazil_2 = Item.using(:brazil).create!(:name => "Brazil Item 2")
-        @brazil_client.items.to_set.should == [@item_brazil].to_set 
+        @brazil_client.items.to_set.should == [@item_brazil].to_set
       end
 
 
@@ -439,7 +450,7 @@ describe Octopus::Association do
 
       it "create" do
         item = @brazil_client.items.create(:name => "Builded Item")
-        @brazil_client.items.to_set.should == [@item_brazil, item].to_set          
+        @brazil_client.items.to_set.should == [@item_brazil, item].to_set
       end
 
       it "count" do
@@ -449,20 +460,20 @@ describe Octopus::Association do
       end
 
       it "size" do
-        @brazil_client.items.size.should == 1          
+        @brazil_client.items.size.should == 1
         item = @brazil_client.items.create(:name => "Builded Item")
-        @brazil_client.items.size.should == 2          
+        @brazil_client.items.size.should == 2
       end
 
       it "create!" do
         item = @brazil_client.items.create!(:name => "Builded Item")
-        @brazil_client.items.to_set.should == [@item_brazil, item].to_set                    
+        @brazil_client.items.to_set.should == [@item_brazil, item].to_set
       end
 
       it "length" do
-        @brazil_client.items.length.should == 1          
+        @brazil_client.items.length.should == 1
         item = @brazil_client.items.create(:name => "Builded Item")
-        @brazil_client.items.length.should == 2                    
+        @brazil_client.items.length.should == 2
       end
 
       it "empty?" do
@@ -482,37 +493,37 @@ describe Octopus::Association do
       end
 
       it "delete_all" do
-        @brazil_client.items.empty?.should be_false     
-        @brazil_client.items.delete_all                
+        @brazil_client.items.empty?.should be_false
+        @brazil_client.items.delete_all
         @brazil_client.items.empty?.should be_true
       end
 
       it "destroy_all" do
-        @brazil_client.items.empty?.should be_false     
-        @brazil_client.items.destroy_all                
+        @brazil_client.items.empty?.should be_false
+        @brazil_client.items.destroy_all
         @brazil_client.items.empty?.should be_true
       end
 
       it "find" do
         @brazil_client.items.find(:first).should == @item_brazil
-        @brazil_client.items.destroy_all                
+        @brazil_client.items.destroy_all
         @brazil_client.items.find(:first).should be_nil
       end
 
       it "exists?" do
         @brazil_client.items.exists?(@item_brazil).should be_true
-        @brazil_client.items.destroy_all                
-        @brazil_client.items.exists?(@item_brazil).should be_false     
+        @brazil_client.items.destroy_all
+        @brazil_client.items.exists?(@item_brazil).should be_false
       end
 
       it "uniq" do
-        @brazil_client.items.uniq.should == [@item_brazil]                
-      end        
+        @brazil_client.items.uniq.should == [@item_brazil]
+      end
 
       it "clear" do
-        @brazil_client.items.empty?.should be_false     
-        @brazil_client.items.clear                
-        @brazil_client.items.empty?.should be_true          
+        @brazil_client.items.empty?.should be_false
+        @brazil_client.items.clear
+        @brazil_client.items.empty?.should be_true
       end
     end
   end
@@ -524,7 +535,7 @@ describe Octopus::Association do
       @comment_brazil = Comment.using(:brazil).create!(:name => "Brazil Comment", :commentable => @brazil_client)
       @comment_master = Comment.create!(:name => "Master Comment", :commentable => @master_client)
       @brazil_client = Client.using(:brazil).find_by_name("Brazil Client")
-      Client.using(:master).create!(:name => "teste")        
+      Client.using(:master).create!(:name => "teste")
     end
 
     it "should find all models in the specified shard" do
@@ -549,7 +560,7 @@ describe Octopus::Association do
     describe "it should works when using" do
       before(:each) do
         @comment_brazil_2 = Comment.using(:brazil).create!(:name => "Brazil Comment 2")
-        @brazil_client.comments.to_set.should == [@comment_brazil].to_set 
+        @brazil_client.comments.to_set.should == [@comment_brazil].to_set
       end
 
       it "update_attributes" do
@@ -577,7 +588,7 @@ describe Octopus::Association do
 
       it "create" do
         comment = @brazil_client.comments.create(:name => "Builded Comment")
-        @brazil_client.comments.to_set.should == [@comment_brazil, comment].to_set          
+        @brazil_client.comments.to_set.should == [@comment_brazil, comment].to_set
       end
 
       it "count" do
@@ -587,20 +598,20 @@ describe Octopus::Association do
       end
 
       it "size" do
-        @brazil_client.comments.size.should == 1          
+        @brazil_client.comments.size.should == 1
         comment = @brazil_client.comments.create(:name => "Builded Comment")
-        @brazil_client.comments.size.should == 2          
+        @brazil_client.comments.size.should == 2
       end
 
       it "create!" do
         comment = @brazil_client.comments.create!(:name => "Builded Comment")
-        @brazil_client.comments.to_set.should == [@comment_brazil, comment].to_set                    
+        @brazil_client.comments.to_set.should == [@comment_brazil, comment].to_set
       end
 
       it "length" do
-        @brazil_client.comments.length.should == 1          
+        @brazil_client.comments.length.should == 1
         comment = @brazil_client.comments.create(:name => "Builded Comment")
-        @brazil_client.comments.length.should == 2                    
+        @brazil_client.comments.length.should == 2
       end
 
       it "empty?" do
@@ -620,38 +631,45 @@ describe Octopus::Association do
       end
 
       it "delete_all" do
-        @brazil_client.comments.empty?.should be_false     
-        @brazil_client.comments.delete_all                
+        @brazil_client.comments.empty?.should be_false
+        @brazil_client.comments.delete_all
         @brazil_client.comments.empty?.should be_true
       end
 
       it "destroy_all" do
-        @brazil_client.comments.empty?.should be_false     
-        @brazil_client.comments.destroy_all                
+        @brazil_client.comments.empty?.should be_false
+        @brazil_client.comments.destroy_all
         @brazil_client.comments.empty?.should be_true
       end
 
       it "find" do
         @brazil_client.comments.find(:first).should == @comment_brazil
-        @brazil_client.comments.destroy_all                
+        @brazil_client.comments.destroy_all
         @brazil_client.comments.find(:first).should be_nil
       end
 
       it "exists?" do
         @brazil_client.comments.exists?(@comment_brazil).should be_true
-        @brazil_client.comments.destroy_all                
-        @brazil_client.comments.exists?(@comment_brazil).should be_false     
+        @brazil_client.comments.destroy_all
+        @brazil_client.comments.exists?(@comment_brazil).should be_false
       end
 
       it "uniq" do
-        @brazil_client.comments.uniq.should == [@comment_brazil]                
-      end        
+        @brazil_client.comments.uniq.should == [@comment_brazil]
+      end
 
       it "clear" do
-        @brazil_client.comments.empty?.should be_false     
-        @brazil_client.comments.clear                
-        @brazil_client.comments.empty?.should be_true          
+        @brazil_client.comments.empty?.should be_false
+        @brazil_client.comments.clear
+        @brazil_client.comments.empty?.should be_true
       end
     end
   end
+
+  it "block" do
+    @brazil_role = Role.using(:brazil).create!(:name => "Brazil Role")
+    @brazil_role.permissions.build(:name => "ok").name.should == "ok"
+    @brazil_role.permissions.create(:name => "ok").name.should == "ok"
+    @brazil_role.permissions.create!(:name => "ok").name.should == "ok"
+  end
 end