caricature 0.2.0 → 0.3.0

data/README

@@ -2,28 +2,28 @@ Caricature - Bringing simple mocking to the DLR
 ===============================================
 
 This project aims to make interop between IronRuby objects and .NET objects easier.
-The idea is that it integrates nicely with bacon and later rspec and that it transparently lets you mock ironruby ojbects
+The idea is that it integrates nicely with bacon and later rspec and that it transparently lets you isolate ironruby ojbects
 as well as CLR objects/interfaces.
 Caricature handles interfaces, interface inheritance, CLR objects, CLR object instances, Ruby classes and instances of Ruby classes.
 
 From the start I wanted to do away with names like mock, stub, record, replay, verify etc.
 Instead I took the advice from Roy Osherhove and went with a name of Isolation for creating a mock.
 
-An Isolation will create what in Rhino.Mocks would be called a DynamicMock (but can be a partial too) :)
+An Isolation will create what in Rhino.Mocks would be called a DynamicMock (but can be a partial too :) )
 In Moq it would be the Loose mocking strategy.
 
-The naming of the methods for creating mocks is the one that JP Boodhoo proposed WhenToldTo and WasToldTo.
-To specify a stub/expectation on a mock you have one and only one way of doing that with the method called when_told_to.
+The naming of the methods for creating isolations is the one that JP Boodhoo proposed WhenToldTo and WasToldTo.
+To specify a stub/expectation on an isolation you have one and only one way of doing that with the method called when_told_to.
 Then only if you're interested in asserting if a method has been called you can use the was_told_to? method for this.
 
 
-mock = Isolation.for(Ninja)
-mock.when_told_to(:attack) do |exp|
+isolation = Isolation.for(Ninja)
+isolation.when_told_to(:attack) do |exp|
   exp.with(:shuriken)
   exp.return(3)
 end
 
-Battle.new(mock)
+Battle.new(isolation)
 battle.combat
 
-mock.was_told_to?(:attack).should.be.true?
+isolation.was_told_to?(:attack).should.be.true?

data/README.markdown

@@ -13,12 +13,12 @@ An Isolation will create what in Rhino.Mocks would be called a DynamicMock (but
 In Moq it would be the Loose mocking strategy.
 
 The naming of the methods for creating mocks is the one that JP Boodhoo proposed WhenToldTo and WasToldTo.
-To specify a stub/expectation on a mock you have one and only one way of doing that with the method called when_told_to.
+To specify a stub/expectation on an isolation you have one and only one way of doing that with the method called when_told_to.
 Then only if you're interested in asserting if a method has been called you can use the was_told_to? method for this.
 
 <pre>
-mock = Isolation.for(Ninja)
-mock.when_told_to(:attack) do |exp|
+isolation = Isolation.for(Ninja)
+isolation.when_told_to(:attack) do |exp|
   exp.with(:shuriken)
   exp.return(3)
 end
@@ -26,9 +26,14 @@ end
 Battle.new(mock)
 battle.combat
 
-mock.was_told_to?(:attack).should.be.true?
+isolation.was_told_to?(:attack).should.be.true?
 </pre>
 
+It may be very important to note that when you're going to be isolating CLR classes to be used in other CLR classes
+you still need to obide by the CLR rules. That means if you want to redefine a method you'll need to make sure it's
+marked as virtual. Static types are still governed by static type rules.  I'm working on a solution around those
+problems but for the time being this is how it has to be.
+
 License:
 --------
 

data/spec/integration_spec.rb

@@ -0,0 +1,142 @@
+require File.dirname(__FILE__) + "/bacon_helper"
+
+describe "Full scenarios" do
+
+  describe "when isolating CLR interfaces" do
+    before do
+      @ninja = ClrModels::Ninja.new      
+      @weapon = Caricature::Isolation.for(ClrModels::IWeapon)
+    end
+
+    it "should work without expectations" do
+      @ninja.attack ClrModels::Ninja.new, @weapon
+
+      @weapon.was_told_to?(:attack).should.be.successful
+    end
+
+    it "should work with an expectation with any arguments" do
+      @weapon.when_told_to(:damage).return(5)
+
+      @ninja.is_killed_by(@weapon).should.be.true?
+      @weapon.was_told_to?(:damage).should.be.successful
+    end
+
+#    it "should work with an expectation with any arguments" do
+#      @weapon.when_told_to(:damage).return(5)
+#
+#      @ninja.is_killed_by(@weapon).should.be.true?
+#      @weapon.was_told_to?(:damage).should.be.successful
+#    end
+
+    it "should work with an expectation getting different method call result" do
+      @weapon.when_told_to(:damage).return(2)
+
+      @ninja.is_killed_by(@weapon).should.be.false?
+      @weapon.was_told_to?(:damage).should.be.successful
+    end
+
+    it "should work for an assertion on a specific argument" do
+      @weapon.when_told_to(:damage).return(2)
+
+      @ninja.is_killed_by(@weapon).should.be.false?
+      @weapon.was_told_to?(:damage).should.be.successful
+    end
+
+  end
+
+  describe "when isolating CLR classes" do
+
+    before do
+      @weapon = ClrModels::Sword.new
+      @ninja = Caricature::Isolation.for(ClrModels::Ninja)
+    end
+
+    it "should work without expectations" do
+      result = @weapon.attack @ninja
+      result.should.equal 0
+
+      @ninja.was_told_to?(:survive_attack_with).with(@weapon).should.be.successful
+    end
+
+    it "should work with an expectation for any arguments" do
+      @ninja.when_told_to(:survive_attack_with).return(5)
+
+      result = @weapon.attack @ninja
+      result.should.equal 5
+
+      @ninja.was_told_to?(:survive_attack_with).with(:any).should.be.successful
+    end
+
+    it "should work with an assertion for specific arguments" do
+      @ninja.when_told_to(:survive_attack_with) do |method_should|
+         method_should.return(5)
+      end
+
+      result = @weapon.attack @ninja
+      result.should.equal 5
+
+      @ninja.was_told_to?(:survive_attack_with).with(@weapon).should.be.successful
+    end
+
+    it "should fail for an assertion with wrong arguments" do
+      @ninja.when_told_to(:survive_attack_with) do |method_should|
+         method_should.return(5)
+      end
+
+      result = @weapon.attack @ninja
+      result.should.equal 5
+
+      @ninja.
+        was_told_to?(:survive_attack_with).
+        with(Caricature::Isolation.for(ClrModels::IWeapon)).
+        should.not.be.successful
+    end
+
+  end
+
+  describe "when isolating CLR instances" do
+
+    before do
+      @weapon = ClrModels::Sword.new
+      @ninja = Caricature::Isolation.for(ClrModels::Ninja.new)
+    end
+
+    it "should work without expectations" do
+      result = @weapon.attack @ninja
+      result.should.equal 0
+
+      @ninja.was_told_to?(:survive_attack_with).with(@weapon).should.be.successful
+    end
+
+    it "should work with an expectation for any arguments" do
+      @ninja.when_told_to(:survive_attack_with).return(5)
+
+      result = @weapon.attack @ninja
+      result.should.equal 5
+
+      @ninja.was_told_to?(:survive_attack_with).with(:any).should.be.successful
+    end
+
+    it "should fail for an assertion for specific arguments" do
+      @ninja.when_told_to(:survive_attack_with) do |method_should|
+         method_should.return(5)
+      end
+
+      result = @weapon.attack @ninja
+      result.should.equal 5
+      var = @ninja.was_told_to?(:survive_attack_with).with(:any)
+      @ninja.was_told_to?(:survive_attack_with).with(@weapon).should.be.successful
+    end
+
+    it "should allow to delegate the method call to the real instance (partial mock)" do
+      @ninja.when_told_to(:survive_attack_with).super_after
+
+      result = @weapon.attack @ninja
+      result.should.equal 6
+
+      @ninja.was_told_to?(:survive_attack_with).should.be.successful
+    end
+
+  end
+
+end

data/spec/interop_spec.rb

@@ -30,7 +30,7 @@ describe "Event handling" do
   describe "for an IR generated interface proxy" do
 
     before do
-      @proxy = Caricature::RecordingClrProxy.new ClrModels::IExposingWarrior
+      @proxy = Caricature::ClrIsolator.new ClrModels::IExposingWarrior
     end
 
     # apparently events don't work yet in IronRuby.. keeping this spec here to find out when it does

data/spec/isolation_spec.rb

@@ -59,16 +59,12 @@ describe "Caricature::Isolation" do
     end
 
     it "should create a proxy" do
-      @isolator.proxy.should.not.be == nil
-    end
-
-    it "should create the Ruby objects proxy" do
-      @isolator.proxy.is_clr_proxy?.should.be.false?
+      @isolator.should.not.be == nil
     end
 
     describe "when asked to stub a method" do
 
-      it "should create an expectation" do
+      it "should create an expectation with a block" do
         nm = "What's in a name"
         expectation = @isolator.when_told_to(:name) do |cl|
           cl.return(nm)
@@ -77,41 +73,18 @@ describe "Caricature::Isolation" do
         expectation.has_return_value?.should.be.true?
         expectation.return_value.should.equal nm
       end
-    end
-
-  end
 
-  describe 'when verifying if a call was made' do
-
-    before do
-      @rec = Caricature::MethodCallRecorder.new
-      @rec.record_call :my_method
-      @rec.record_call :my_method, 1, 2, 3
-      @proxy = Caricature::RecordingProxy.new(Soldier, @rec)
-    end
-
-    it "should be successful with any arguments allowed" do
-      iso = Caricature::Isolation.new(@proxy, @rec)
-      iso.was_told_to?(:my_method).should.be.true?
-    end
-
-    it "should be successful with a correct set of arguments provided for my_method" do
-      iso = Caricature::Isolation.new(@proxy, @rec)
-      iso.was_told_to?(:my_method){ |ver| ver.with(1, 2, 3) }.should.be.true?
-    end
-
-    it "should be unsuccessful when a wrong set of arguments is provided" do
-      iso = Caricature::Isolation.new(@proxy, @rec)
-      iso.was_told_to?(:my_method){|ver| ver.with(1, 3, 6) }.should.be.false?
-    end
-
-    it "should be unsuccessful when the wrong method name is provided" do
-      iso = Caricature::Isolation.new(@proxy, @rec)
-      iso.was_told_to?(:some_method).should.be.false?
+      it "should create an expectation without a block" do
+        nm = "What's in a name"
+        expectation = @isolator.when_told_to(:name).return(nm)
+        expectation.method_name.should.equal :name
+        expectation.has_return_value?.should.be.true?
+        expectation.return_value.should.equal nm
+      end
     end
 
   end
-
+    
   describe "when creating an isolation for CLR objects" do
 
     it "should not raise" do
@@ -127,11 +100,28 @@ describe "Caricature::Isolation" do
     end
 
     it "should create a proxy" do
-      @isolator.proxy.should.not.be == nil
+      @isolator.should.not.be == nil
     end
 
-    it "should create the CLR objects proxy" do
-      @isolator.proxy.is_clr_proxy?.should.be.true?
+    describe "when asked to stub a method" do
+
+      it "should create an expectation with a block" do
+        nm = "What's in a name"
+        expectation = @isolator.when_told_to(:name) do |cl|
+          cl.return(nm)
+        end
+        expectation.method_name.should.equal :name
+        expectation.has_return_value?.should.be.true?
+        expectation.return_value.should.equal nm
+      end
+
+      it "should create an expectation without a block" do
+        nm = "What's in a name"
+        expectation = @isolator.when_told_to(:name).return(nm)
+        expectation.method_name.should.equal :name
+        expectation.has_return_value?.should.be.true?
+        expectation.return_value.should.equal nm
+      end
     end
 
   end

data/spec/{proxy_spec.rb → isolator_spec.rb}

@@ -1,21 +1,14 @@
 require File.dirname(__FILE__) + "/bacon_helper"
 
-describe "Caricature::RecordingProxy" do
+describe "Caricature::RubyIsolator" do
 
   before do    
     @subj = Soldier.new
     @recorder = Caricature::MethodCallRecorder.new
-    @proxy = Caricature::RecordingProxy.new(@subj, @recorder)
-  end
-
-  it "should forward existing methods" do
-    @proxy.___super___.name.should.equal @subj.name
-  end
-
-  it "should call to_s on the proxied object" do
-    @proxy.___super___.to_s.should.equal @subj.to_s
+    @proxy = Caricature::RubyIsolator.for(@subj, @recorder)
   end
 
+  
   describe "when invoking a method" do
 
     before do
@@ -48,7 +41,7 @@ describe "Caricature::RecordingClrProxy" do
       @samurai = ClrModels::Samurai.new
       @samurai.name = "Nakiro"
       @recorder = Caricature::MethodCallRecorder.new
-      @proxy = Caricature::RecordingClrProxy.new(@samurai, @recorder)
+      @proxy = Caricature::ClrIsolator.for(@samurai, @recorder)
     end
 
     it "should create a proxy" do
@@ -86,12 +79,13 @@ describe "Caricature::RecordingClrProxy" do
 
     before do
       @recorder = Caricature::MethodCallRecorder.new
-      @proxy = Caricature::RecordingClrProxy.new(ClrModels::Ninja, @recorder)
+      @proxy = Caricature::ClrIsolator.for(ClrModels::Ninja, @recorder)
     end
 
     it "should create a proxy" do
-      @proxy.___super___.class.should.equal ClrModels::Ninja
-      @proxy.___super___.id.should.equal 0
+      
+      @proxy.___super___.class.superclass.should.equal ClrModels::Ninja
+      
     end
 
 
@@ -119,7 +113,7 @@ describe "Caricature::RecordingClrProxy" do
 
     before do
       @recorder = Caricature::MethodCallRecorder.new
-      @proxy = Caricature::RecordingClrProxy.new(ClrModels::IWarrior, @recorder)
+      @proxy = Caricature::ClrInterfaceIsolator.for(ClrModels::IWarrior, @recorder)
     end
 
     it "should create a proxy" do
@@ -163,7 +157,7 @@ describe "Caricature::RecordingClrProxy" do
 
     before do
       @recorder = Caricature::MethodCallRecorder.new
-      @proxy = Caricature::RecordingClrProxy.new(ClrModels::IExposing, @recorder)
+      @proxy = Caricature::ClrInterfaceIsolator.for(ClrModels::IExposing, @recorder)
     end
 
     it "should create an add method for the event" do
@@ -179,7 +173,7 @@ describe "Caricature::RecordingClrProxy" do
   describe "for CLR interface recursion" do
 
     before do
-      @proxy = Caricature::RecordingClrProxy.new(ClrModels::IExposingWarrior, Caricature::MethodCallRecorder.new)
+      @proxy = Caricature::ClrInterfaceIsolator.for(ClrModels::IExposingWarrior, Caricature::MethodCallRecorder.new)
     end
 
     it "should create a method defined on the CLR interface" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification 
 name: caricature
 version: !ruby/object:Gem::Version 
-  version: 0.2.0
+  version: 0.3.0
 platform: ruby
 authors: 
 - Ivan Porto Carrero
@@ -9,7 +9,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2009-05-11 00:00:00 +02:00
+date: 2009-05-15 00:00:00 +02:00
 default_executable: 
 dependencies: []
 
@@ -57,8 +57,9 @@ test_files:
 - spec/bacon_helper.rb
 - spec/core_ext_spec.rb
 - spec/expectation_spec.rb
+- spec/integration_spec.rb
 - spec/interop_spec.rb
 - spec/isolation_spec.rb
+- spec/isolator_spec.rb
 - spec/method_call_spec.rb
-- spec/proxy_spec.rb
 - spec/verification_spec.rb