remotable 0.3.0 → 0.4.0.beta2

data/test/bespoke_test.rb

@@ -6,8 +6,8 @@ require "rr"
 
 class BespokeTest < ActiveSupport::TestCase
   include RR::Adapters::TestUnit
-  
-  
+
+
   teardown do
     def model.new_resource
       BespokeResource.new
@@ -16,61 +16,61 @@ class BespokeTest < ActiveSupport::TestCase
       nil
     end
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   # Finding                                                                   #
   # ========================================================================= #
-  
+
   test "should be able to find resources by different attributes" do
     new_tenant_slug = "not_found"
     assert_equal 0, BespokeTenant.where(:slug => new_tenant_slug).count,
       "There's not supposed to be a BespokeTenant with the slug #{new_tenant_slug}."
-    
+
     def model.find_by(attribute, value)
       BespokeResource.new(:slug => value)
     end
-    
+
     new_tenant = BespokeTenant.find_by_slug(new_tenant_slug)
     assert_not_nil new_tenant, "A remote @tenant was not found with the slug #{new_tenant_slug.inspect}"
   end
-  
+
   test "should create a record locally when fetching a new remote resource" do
     new_tenant_slug = "17"
     assert_equal 0, BespokeTenant.where(:slug => new_tenant_slug).count,
       "There's not supposed to be a BespokeTenant with the slug #{new_tenant_slug}."
-    
+
     def model.find_by(attribute, value)
       BespokeResource.new(:slug => value)
     end
-    
+
     assert_difference "BespokeTenant.count", +1 do
       new_tenant = BespokeTenant.find_by_slug(new_tenant_slug)
       assert_not_nil new_tenant, "A remote tenant was not found with the id #{new_tenant_slug.inspect}"
     end
   end
-  
+
   test "if a resource is neither local nor remote, raise an exception with the bang method" do
     new_tenant_slug = "not_found3"
     assert_equal 0, BespokeTenant.where(:slug => new_tenant_slug).count,
       "There's not supposed to be a BespokeTenant with the slug #{new_tenant_slug}."
-    
+
     assert_raises ActiveRecord::RecordNotFound do
       BespokeTenant.find_by_slug!(new_tenant_slug)
     end
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   # Updating                                                                 #
   # ========================================================================= #
-  
+
   test "should update a record remotely when updating one locally" do
     @tenant = Factory(:bespoke_tenant)
     new_name = "Totally Wonky"
-    
+
     # RemoteTenant.run_simulation do |s|
     #   s.show(@tenant.remote_id, {
     #     :id => @tenant.remote_id,
@@ -78,89 +78,89 @@ class BespokeTest < ActiveSupport::TestCase
     #     :church_name => @tenant.name
     #   })
     #   s.update(@tenant.remote_id)
-    #   
+    #
     #   @tenant.nosync = false
     #   @tenant.name = "Totally Wonky"
     #   assert_equal true, @tenant.any_remote_changes?
-    #   
+    #
     #   @tenant.save!
-    #   
+    #
     #   pending "Not sure how to test that an update happened"
     # end
   end
-  
+
   test "should fail to update a record locally when failing to update one remotely" do
     @tenant = Factory(:bespoke_tenant, :nosync => false)
-    
+
     def model.find_by(attribute, value)
       BespokeResource.new(attribute => value)
     end
-    
+
     def resource.save
       false
     end
-    
+
     def resource.errors
       {:name => ["is already taken"]}
     end
-    
+
     @tenant.name = "Totally Wonky"
     assert_raises(ActiveRecord::RecordInvalid) do
       @tenant.save!
     end
     assert_equal ["is already taken"], @tenant.errors[:name]
   end
-  
-  
-  
-  
+
+
+
+
   # ========================================================================= #
   # Creating                                                                  #
   # ========================================================================= #
-  
+
   test "should create a record remotely when creating one locally" do
     @tenant = BespokeTenant.new({
       :slug => "brand_new",
       :name => "Brand New"
     })
-    
+
     assert_nil @tenant.remote_resource
     @tenant.save!
     assert_not_nil @tenant.remote_resource
   end
-  
+
   test "should fail to create a record locally when failing to create one remotely" do
     @tenant = BespokeTenant.new({
       :slug => "brand_new",
       :name => "Brand New"
     })
-    
+
     def model.new_resource
       resource = BespokeResource.new
       def resource.save; false; end
       def resource.errors; {:name => ["is already taken"]}; end
       resource
     end
-    
+
     assert_raises(ActiveRecord::RecordInvalid) do
       @tenant.save!
     end
-    
+
     assert_equal ["is already taken"], @tenant.errors[:name]
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   # Destroying                                                                #
   # ========================================================================= #
-  
+
   test "should destroy a record remotely when destroying one locally" do
     @tenant = Factory(:bespoke_tenant, :nosync => false)
     mock(resource).destroy { true }
     @tenant.destroy
   end
-  
+
   test "should fail to destroy a record locally when failing to destroy one remotely" do
     @tenant = Factory(:bespoke_tenant, :nosync => false)
     mock(resource).destroy { raise StandardError }
@@ -168,30 +168,30 @@ class BespokeTest < ActiveSupport::TestCase
       @tenant.destroy
     end
   end
-  
+
   test "should delete a local record when a remote record has been deleted" do
     @tenant = Factory(:bespoke_tenant, :expires_at => 1.year.ago)
-    
+
     def model.find_by(remote_attr, value)
       nil
     end
-    
+
     assert_difference "BespokeTenant.count", -1 do
       @tenant = BespokeTenant.find_by_slug(@tenant.slug)
       assert_equal nil, @tenant
     end
   end
-  
-  
-  
+
+
+
 private
-  
+
   def model
     BespokeTenant.remote_model
   end
-  
+
   def resource
     @tenant.remote_resource
   end
-  
+
 end

data/test/nosync_test.rb

@@ -4,27 +4,27 @@ require "support/active_resource"
 
 
 class NoSyncTest < ActiveSupport::TestCase
-  
-  
-  
+
+
+
   test "nosync? should be false by default" do
     assert_equal false, Tenant.new.nosync?
   end
-  
+
   test "nosync? should be true if remotable is turned off globally" do
     Remotable.nosync do
       assert_equal true, Tenant.new.nosync?
       assert_equal true, RemoteWithoutKey.new.nosync?
     end
   end
-  
+
   test "nosync? should be true if remotable is turned off for the model" do
     Tenant.nosync do
       assert_equal true, Tenant.new.nosync?
       assert_equal false, RemoteWithoutKey.new.nosync?
     end
   end
-  
+
   test "nosync? should be false if syncing is resumed temporarily on a model that prevents it by default" do
     Tenant.nosync!
     assert_equal true, Tenant.new.nosync?
@@ -33,10 +33,10 @@ class NoSyncTest < ActiveSupport::TestCase
     end
     assert_equal true, Tenant.new.nosync?
   end
-  
+
   test "nosync? should take the value further up the chain if a model's value is temporarily cleared" do
     assert_not_nil Tenant.remote_model
-    
+
     Remotable.nosync!
     Tenant.nosync(false) do
       Tenant.nosync(nil) do
@@ -48,22 +48,22 @@ class NoSyncTest < ActiveSupport::TestCase
     end
     assert_equal true, Tenant.nosync?
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   # Finding                                                                   #
   # ========================================================================= #
-  
+
   test "should do nothing if a tenant is expired" do
     tenant = Factory(:tenant, :expires_at => 1.year.ago)
-    
+
     Remotable.nosync do
       result = Tenant.find_by_remote_id(tenant.remote_id)
       assert_equal tenant, result
     end
   end
-  
-  
-  
+
+
+
 end

data/test/null_remote_test.rb

@@ -5,49 +5,49 @@ require "support/null"
 
 class NullRemoteTest < ActiveSupport::TestCase
   include RR::Adapters::TestUnit
-  
-  
-  
+
+
+
   # ========================================================================= #
   # Finding                                                                   #
   # ========================================================================= #
-  
+
   test "should do nothing if a tenant is expired" do
     tenant = Factory(:null_test_tenant, expires_at: 2.days.ago)
     result = NullTestTenant.find_by_slug(tenant.slug)
     assert_equal tenant, result
   end
-  
+
   test "should raise an exception with the bang method if resource isn't found locally" do
     new_tenant_slug = "not_found3"
     assert_equal 0, NullTestTenant.where(:slug => new_tenant_slug).count,
       "There's not supposed to be a NullTestTenant with the slug #{new_tenant_slug}."
-    
+
     assert_raises ActiveRecord::RecordNotFound do
       NullTestTenant.find_by_slug!(new_tenant_slug)
     end
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   # Updating                                                                 #
   # ========================================================================= #
-  
+
   test "should update a record locally without any interference" do
     tenant = Factory(:null_test_tenant)
     new_name = "Totally Wonky"
-    
+
     tenant.name = new_name
     tenant.save!
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   # Creating                                                                  #
   # ========================================================================= #
-  
+
   test "should create a record locally without any interference" do
     tenant = NullTestTenant.create!({
       :slug => "brand_new",
@@ -55,19 +55,19 @@ class NullRemoteTest < ActiveSupport::TestCase
     })
     assert_not_nil tenant.remote_resource
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   # Destroying                                                                #
   # ========================================================================= #
-  
+
   test "should destroy a record locally without any interference" do
     tenant = Factory(:null_test_tenant)
     tenant.nosync = false
     tenant.destroy
   end
-  
-  
-  
+
+
+
 end

data/test/remotable_test.rb

@@ -4,50 +4,50 @@ require "support/bespoke"
 
 
 class RemotableTest < ActiveSupport::TestCase
-  
-  
-  
+
+
+
   # ========================================================================= #
   #  Keys                                                                     #
   # ========================================================================= #
-  
+
   test "should consider :id to be the remote key if none is specified" do
     assert_equal :id,         RemoteWithoutKey.remote_key
     assert_equal :remote_id,  RemoteWithoutKey.local_key
   end
-  
+
   test "should use a different remote_key if one is supplied" do
     assert_equal :slug, RemoteWithKey.remote_key
     assert_equal :slug, RemoteWithKey.local_key
   end
-  
+
   test "should be able to generate paths for with different attributes" do
     assert_equal "by_slug/value",   Tenant.remote_path_for(:slug, "value")
     assert_equal "by_nombre/value", Tenant.remote_path_for(:name, "value")
   end
-  
+
   test "should be able to generate paths for composite keys" do
     assert_equal "groups/5/tenants/test",   RemoteWithCompositeKey.remote_path_for([:group_id, :slug], [5, "test"])
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   #  Temporary remote models                                                  #
   # ========================================================================= #
-  
+
   test "should support temporary models" do
     assert_nil BespokeTenant.find_by_slug("404")
     assert_not_nil BespokeTenant.with_remote_model(BespokeModel2.new) { BespokeTenant.find_by_slug("404") }
     assert_nil BespokeTenant.find_by_slug("405")
   end
-  
+
   test "should support temporary models and chainable syntax" do
     assert_nil BespokeTenant.find_by_slug("404")
     assert_not_nil BespokeTenant.with_remote_model(BespokeModel2.new).find_by_slug("404")
     assert_nil BespokeTenant.find_by_slug("405")
   end
-  
+
   test "should support setting remote_model to nil" do
     Tenant.with_remote_model(nil) do
       Tenant.remote_model
@@ -56,9 +56,9 @@ class RemotableTest < ActiveSupport::TestCase
       assert_not_nil Tenant.find_by_name("Test 1"), "new tenant was not found"
     end
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   #  Edge cases                                                               #
   #                                                                           #
@@ -68,7 +68,7 @@ class RemotableTest < ActiveSupport::TestCase
   #  Remotable doesn't break when it tries to look up a remote record         #
   #  with a nil key.                                                          #
   # ========================================================================= #
-  
+
   test "should not break when refreshing a record which does not have a foreign key" do
     tenant = Tenant.nosync { Tenant.create!(
       :name => "Test 1",
@@ -76,67 +76,67 @@ class RemotableTest < ActiveSupport::TestCase
       :remote_id => nil,
       :expires_at => 1.day.ago ) }
     assert_equal nil, tenant.remote_id
-    
+
     # Fetching this tenant, Remotable will want to
     # refresh it, but it shouldn't because it can't.
     assert_not_nil Tenant.where(:slug => "test-1").first, "A tenant with the slug \"test-1\" was not found"
   end
-  
+
   test "should not break when updating a record which does not have a foreign key" do
     tenant = Tenant.nosync { Tenant.create!(
       :name => "Test 1",
       :slug => "test-1",
       :remote_id => nil ) }
     assert_equal nil, tenant.remote_id
-    
+
     # Updating this tenatn, Remotable will want to
     # sync it, but it shouldn't because it can't.
     assert_equal true, tenant.update_attributes(:name => "Test 2"), "The tenant was not updated (errors: #{tenant.errors.full_messages.join(", ")})"
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   #  Finders                                                                  #
   # ========================================================================= #
-  
+
   test "should create expected finders" do
     assert_equal true, Tenant.respond_to?(:find_by_name)
     assert_equal true, Tenant.respond_to?(:find_by_slug)
     assert_equal true, RemoteWithoutKey.respond_to?(:find_by_id)
     assert_equal true, RemoteWithCompositeKey.respond_to?(:find_by_group_id_and_slug)
   end
-  
+
   test "should recognize a finder method with a single key" do
     method_details = RemoteWithKey.recognize_remote_finder_method(:find_by_slug)
     assert_not_equal false, method_details
     assert_equal :slug, method_details[:remote_key]
   end
-  
+
   test "should recognize a finder method with a composite key" do
     method_details = RemoteWithCompositeKey.recognize_remote_finder_method(:find_by_group_id_and_slug)
     assert_not_equal false, method_details
     assert_equal [:group_id, :slug], method_details[:remote_key]
   end
-  
-  
-  
+
+
+
   # ========================================================================= #
   #  Validating Models                                                        #
   # ========================================================================= #
-  
+
   test "should raise an exception if a remote model does not respond to all class methods" do
     class Example1 < ActiveRecord::Base; self.table_name = "tenants"; end
     class RemoteModel1; def self.find_by(*args); end; end
     assert_raise(Remotable::InvalidRemoteModel) { Example1.remote_model RemoteModel1 }
   end
-  
+
   test "should raise an exception if a remote resource does not respond to all instance methods" do
     class Example2 < ActiveRecord::Base; self.table_name = "tenants"; end
     class RemoteModel2; def self.new_resource; Object.new; end; end
     assert_raise(Remotable::InvalidRemoteModel) { Example2.remote_model RemoteModel2 }
   end
-  
+
   test "should not raise an exception if remote models are not being validated" do
     Remotable.without_validation do
       class Example4 < ActiveRecord::Base; self.table_name = "tenants"; end
@@ -144,11 +144,11 @@ class RemotableTest < ActiveSupport::TestCase
       assert_nothing_raised { Example4.remote_model RemoteModel4 }
     end
   end
-  
+
   test "should not raise an exception if a remote model responds to all required methods" do
     class Example3 < ActiveRecord::Base; self.table_name = "tenants"; end
     assert_nothing_raised { Example3.remote_model BespokeModel.new }
   end
-  
-  
+
+
 end