rod-rest 0.0.1.1 → 0.5.0

data/test/spec/proxy.rb

@@ -35,7 +35,7 @@ module Rod
 
       let(:car_type)            { "Test::Car" }
       let(:mercedes_300_hash)   { { rod_id: mercedes_300_id, name: mercedes_300_name, type: car_type,
-                                    owner: { rod_id: schumaher_id, type: person_type}, drivers: { count: drivers_count } } }
+                                    owner: schumaher_hash, drivers: { count: drivers_count } } }
       let(:mercedes_300_id)     { 1 }
       let(:mercedes_300_name)   { "Mercedes 300" }
 
@@ -74,13 +74,61 @@ module Rod
           mercedes_300.name.should == mercedes_300_name
         end
 
-        it "has an valid 'owner' singular association" do
+        it "has a valid 'owner' singular association" do
           mercedes_300.owner.should == owner_object
         end
 
+        it "caches 'owner' singular association" do
+          mercedes_300.owner
+          mercedes_300.owner
+          expect(client).to have_received.fetch_object(schumaher_hash) { schumaher_object }.once
+        end
+
         it "has a valid 'drivers' plural association" do
           mercedes_300.drivers.should == collection_proxy
         end
+
+        it "caches 'drivers' plural association" do
+          mercedes_300.drivers
+          mercedes_300.drivers
+          expect(collection_proxy_factory).to have_received.new(anything,drivers_association_name,drivers_count,client) { collection_proxy }.once
+        end
+
+        describe "#inspect" do
+          it "reports that it is a proxy" do
+            mercedes_300.inspect.should match(/Proxy/)
+          end
+
+          it "reports the type of the object" do
+            mercedes_300.inspect.should match(/#{car_type}/)
+          end
+
+          it "reports the id of the object" do
+            mercedes_300.inspect.should match(/#{mercedes_300_id}/)
+          end
+
+          it "reports the name of the car" do
+            mercedes_300.inspect.should match(/name:#{mercedes_300_name}/)
+          end
+
+          it "reports the owner of the car" do
+            mercedes_300.inspect.should match(/owner:#{person_type}:#{schumaher_id}/)
+          end
+
+          it "reports the drivers of the car" do
+            mercedes_300.inspect.should match(/drivers\[#{drivers_count}\]/)
+          end
+        end
+
+        describe "#to_s" do
+          it "reports the type of the object" do
+            mercedes_300.to_s.should match(/#{car_type}/)
+          end
+
+          it "reports the id of the object" do
+            mercedes_300.to_s.should match(/#{mercedes_300_id}/)
+          end
+        end
       end
     end
   end

data/test/spec/proxy_cache.rb

@@ -0,0 +1,80 @@
+require 'bundler/setup'
+require_relative 'test_helper'
+require 'rod/rest/proxy_cache'
+
+module Rod
+  module Rest
+    describe ProxyCache do
+      let(:cache)     { ProxyCache.new }
+      let(:object)    { object = stub!.rod_id { object_id }.subject
+                        stub(object).type { object_type }
+                        object
+      }
+      let(:object_id)     { 1 }
+      let(:object_type)   { "Car" }
+      let(:description)   { {rod_id: object_id, type: object_type} }
+
+      describe "#store" do
+        it "stores objects" do
+          cache.store(object).should == object
+        end
+
+        it "raises InvalidData if the object cannot be stored" do
+          lambda { cache.store(Object.new) }.should raise_error(InvalidData)
+        end
+      end
+
+      describe "#has_key?" do
+        context "without object in the cache" do
+          it "returns false" do
+            cache.has_key?(description).should == false
+          end
+        end
+
+        context "with object in the cache" do
+          before do
+            cache.store(object)
+          end
+
+          it "returns true" do
+            cache.has_key?(description).should == true
+          end
+        end
+
+        context "with invalid description" do
+          let(:description) { Object.new }
+
+          it "raises InvalidData exception" do
+            lambda { cache.has_key?(description) }.should raise_error(InvalidData)
+          end
+        end
+      end
+
+      describe "#[description]" do
+        context "with object in the cache" do
+          before do
+            cache.store(object)
+          end
+
+          it "returns the object" do
+            cache[description].should == object
+          end
+        end
+
+        context "without object in the cache" do
+          it "raises CacheMissed exception" do
+            lambda { cache[description] }.should raise_error(CacheMissed)
+          end
+        end
+
+        context "with invalid description" do
+          let(:description) { Object.new }
+
+          it "raises InvalidData exception" do
+            lambda { cache[description] }.should raise_error(InvalidData)
+          end
+        end
+      end
+    end
+  end
+end

data/test/spec/proxy_factory.rb

@@ -5,7 +5,7 @@ require 'rod/rest/proxy_factory'
 module Rod
   module Rest
     describe ProxyFactory do
-      let(:factory)           { ProxyFactory.new(metadata,client,proxy_class: proxy_class) }
+      let(:factory)           { ProxyFactory.new(metadata,client,proxy_class: proxy_class,cache: cache) }
       let(:metadata)          { [car_metadata,person_metadata] }
       let(:car_metadata)      { stub!.name { car_type }.subject }
       let(:person_metadata)   { stub!.name { person_type }.subject }
@@ -17,6 +17,7 @@ module Rod
                                 stub(klass).new(person_metadata,client) { person_proxy_factory }
                                 klass
       }
+      let(:cache)             { nil }
       let(:car_proxy_factory)     { stub!.new(mercedes_300_hash) { mercedes_300 }.subject }
       let(:person_proxy_factory)  { stub!.new(schumaher_hash) { schumaher }.subject }
       let(:mercedes_300_hash)     { { rod_id: mercedes_300_id, type: car_type } }
@@ -39,6 +40,20 @@ module Rod
       it "raises UnknownResource for unknown resource type" do
         lambda { factory.build(unknown_hash) }.should raise_error(UnknownResource)
       end
+
+      describe "with cache" do
+        let(:cache)     { cache = mock!.has_key?(mercedes_300_hash) { false }.subject
+                          mock(cache).has_key?(mercedes_300_hash) { true }
+                          mock(cache).[](mercedes_300_hash) { mercedes_300 }
+                          cache
+        }
+
+        it "doesn't create objects twice" do
+          factory.build(mercedes_300_hash)
+          factory.build(mercedes_300_hash)
+          expect(car_proxy_factory).to have_received.new(mercedes_300_hash).once
+        end
+      end
     end
   end
 end

data/test/spec/resource_metadata.rb

@@ -90,6 +90,18 @@ module Rod
         it "allows to iterate over the indexed properties" do
           resource_metadata.indexed_properties.should respond_to(:each)
         end
+
+        describe "#inspect" do
+          it "reports the description of the metadata" do
+            resource_metadata.inspect.should == description.inspect
+          end
+        end
+
+        describe "#to_s" do
+          it "reports the description of the metadata" do
+            resource_metadata.to_s.should == description.to_s
+          end
+        end
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: rod-rest
 version: !ruby/object:Gem::Version
-  version: 0.0.1.1
+  version: 0.5.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-27 00:00:00.000000000 Z
+date: 2014-02-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: sinatra
@@ -121,6 +121,7 @@ files:
 - Gemfile.lock
 - Rakefile
 - Readme.md
+- changelog.txt
 - lib/rod/rest.rb
 - lib/rod/rest/api.rb
 - lib/rod/rest/client.rb
@@ -132,6 +133,7 @@ files:
 - lib/rod/rest/naming.rb
 - lib/rod/rest/property_metadata.rb
 - lib/rod/rest/proxy.rb
+- lib/rod/rest/proxy_cache.rb
 - lib/rod/rest/proxy_factory.rb
 - lib/rod/rest/resource_metadata.rb
 - rod-rest.gemspec
@@ -143,6 +145,7 @@ files:
 - test/spec/metadata.rb
 - test/spec/property_metadata.rb
 - test/spec/proxy.rb
+- test/spec/proxy_cache.rb
 - test/spec/proxy_factory.rb
 - test/spec/resource_metadata.rb
 - test/spec/test_helper.rb
@@ -171,3 +174,4 @@ signing_key:
 specification_version: 3
 summary: REST API for ROD
 test_files: []
+has_rdoc: