camcorder 0.0.1 → 0.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: de005ffb4a08afb00954a1998d9258193f3bbb73
-  data.tar.gz: 4d9d1e6af8707afc904d62cd8605ce40734ef1dc
+  metadata.gz: cf6593d537d37f4630e5e39874ae2431c22bf955
+  data.tar.gz: 69afb405dd9c9d6171fba3e4ccf5502c8e352d44
 SHA512:
-  metadata.gz: 94cef4dab926fa7768e79a1c9c50003368b2ce14aa2c4a5757d8a2d5f467c9cbbd871bc7aa94fd43bd80f88ae8ac0fe47d335fb929123cbaae53c06695fa6505
-  data.tar.gz: ca4ed1520d25bcc8eba17733a289c76d563f48a564e3e4f08af4d441028f41be32f5ac8796672e96bdb17316aecee38dd82ee4512150929194fb8052e770556f
+  metadata.gz: 2b4d58e302d98fd9374afa2aeae275d73509c6ac32b18f63b8919d6343f85fd015a0f1abb72764e678d70ffafa88bbc86dc2e270bb3891afd216fd7a79517abf
+  data.tar.gz: 4268ffb55fee733246ce27cc38578ae35f26f03f3f504c7f484fb369ee3f7b4bd35559a40200859bdb696c807396bc8f5f354f84109c35236bfb0b6ee7444533

data/README.md

@@ -50,21 +50,23 @@ imap.disconnect
 
 Add this line to your application's Gemfile:
 
-    gem 'camcorder'
+```ruby
+gem 'camcorder'
+```
 
 And then execute:
 
-    $ bundle
+    bundle
 
 Or install it yourself as:
 
-    $ gem install camcorder
+    gem install camcorder
 
 ## Usage with Rspec
 
 In your `spec_helper.rb` file:
 
-```
+```ruby
 require 'camcorder/rspec'
 
 Camcorder.configuration.recordings_dir = 'spec/recordings'
@@ -72,7 +74,7 @@ Camcorder.configuration.recordings_dir = 'spec/recordings'
 
 ## Contributing
 
-1. Fork it ( http://github.com/<my-github-username>/camcorder/fork )
+1. Fork it ( http://github.com/ghempton/camcorder/fork )
 2. Create your feature branch (`git checkout -b my-new-feature`)
 3. Commit your changes (`git commit -am 'Add some feature'`)
 4. Push to the branch (`git push origin my-new-feature`)

data/lib/camcorder/recorder.rb

@@ -1,4 +1,5 @@
 require 'yaml'
+require 'camcorder/recording'
 
 module Camcorder
   
@@ -16,6 +17,7 @@ module Camcorder
     def transaction(&block)
       start
       yield
+    ensure
       commit
     end
 
@@ -38,34 +40,28 @@ module Camcorder
     def record(key, &block)
       if @replaying
         if recordings.has_key?(key)
-          recordings[key]
+          recordings[key].replay
         else
           raise PlaybackError.new(key)
         end
       else
-        result = yield
-        @changed = true
-        if recordings.has_key?(key)
-          if !deep_equals(recordings[key], result)
-            raise RecordingError(key)
+        begin
+          recording = Recording.new
+          result = recording.record(&block)
+        ensure
+          @changed = true
+          if recordings.has_key?(key)
+            if recordings[key] != recording
+              raise RecordingError(key)
+            end
+          else
+            recordings[key] = recording
           end
-        else
-          # Make sure we store a copy of the result so future destructive mutations
-          # do not change this value
-          recordings[key] = deep_clone(result)
+          result
         end
-        result
       end
     end
 
-    def deep_clone(value)
-      YAML.load(YAML.dump(value))
-    end
-    
-    def deep_equals(a, b)
-      YAML.dump(a) == YAML.dump(b)
-    end
-
   end
 
 end

data/lib/camcorder/recording.rb

@@ -0,0 +1,45 @@
+module Camcorder
+  
+  class Recording
+    
+    attr_reader :value
+    attr_reader :behavior
+    
+    def initialize(&block)
+      @value = nil
+      @behavior = nil
+    end
+    
+    def record(&block)
+      res = yield
+      # Make sure we store a copy of the result so future destructive mutations
+      # do not change this value
+      @value = deep_clone(res)
+      @behavior = :return
+      res
+    rescue Exception => e
+      @value = e
+      @behavior = :raise
+      raise e
+    end
+    
+    def replay
+      case @behavior
+      when :return
+        return @value
+      when :raise
+        raise @value
+      end
+    end
+    
+    def deep_clone(value)
+      YAML.load(YAML.dump(value))
+    end
+    
+    def ==(other)
+      YAML.dump(self) == YAML.dump(other)
+    end
+    
+  end
+  
+end

data/lib/camcorder/version.rb

@@ -1,3 +1,3 @@
 module Camcorder
-  VERSION = "0.0.1"
+  VERSION = "0.0.2"
 end

data/spec/camcorder/proxy_spec.rb

@@ -14,6 +14,13 @@ describe Camcorder::Proxy do
       
     end
     
+    it 'should re-raise recorded errors' do
+      
+      expect_any_instance_of(TestObject).to_not receive(:broke_method)
+      expect{subject.broke_method}.to raise_error(StandardError)
+      
+    end
+    
   end
   
   context 'when recording does not exist' do

data/spec/camcorder/recorder_spec.rb

@@ -16,7 +16,9 @@ describe Camcorder::Recorder do
       it 'should return recorded value' do
         subject.transaction do
           result = subject.record 'key' do
-            raise 'should not get here'
+            # Manually recorded
+            'dis be da result'
+            #raise 'should not get here'
           end
           expect(result).to eq('dis be da result')
         end
@@ -25,11 +27,28 @@ describe Camcorder::Recorder do
       it 'should error on unknown key' do
         expect {
           subject.transaction do
-            result = subject.record 'another-key' do
+            subject.record 'another-key' do
               raise 'should not get here'
             end
           end
-        }.to raise_error
+        }.to raise_error(Camcorder::PlaybackError)
+      end
+      
+    end
+    
+    context 'when block raises' do
+    
+      let(:filename) { 'spec/fixtures/haz_errorz.yml'}
+    
+      it 'should reraise from recording' do
+        expect {
+          subject.transaction do
+            subject.record 'key-with-error' do
+              # Manually recorded
+              raise StandardError.new('errorz')
+            end
+          end
+        }.to raise_error(StandardError)
       end
       
     end

data/spec/fixtures/haz_errorz.yml

@@ -0,0 +1,5 @@
+---
+key-with-error: !ruby/object:Camcorder::Recording
+  value: !ruby/exception:StandardError
+    message: errorz
+  behavior: :raise

data/spec/fixtures/should_exist.yml

@@ -1,2 +1,4 @@
 ---
-key: dis be da result
+key: !ruby/object:Camcorder::Recording
+  value: dis be da result
+  behavior: :return

data/spec/recordings/Camcorder/constructor_interception/_new/should_return_a_proxy.yml

@@ -1,5 +1,7 @@
 ---
-TestObject-simple_method-cdc941172e4d28939f6513e81222285c:
-- A
-- B
-- C
+TestObject-simple_method-cdc941172e4d28939f6513e81222285c: !ruby/object:Camcorder::Recording
+  value:
+  - A
+  - B
+  - C
+  behavior: :return

data/spec/recordings/Camcorder/constructor_interception/with_multiple_classes/_new/should_return_proxies.yml

@@ -1,9 +1,13 @@
 ---
-TestObject2-simple_method-4de0fd48a5b0c805e290ecc39c85f9e7:
-- 1
-- 2
-- 3
-TestObject-simple_method-cdc941172e4d28939f6513e81222285c:
-- A
-- B
-- C
+TestObject2-simple_method-4de0fd48a5b0c805e290ecc39c85f9e7: !ruby/object:Camcorder::Recording
+  value:
+  - 1
+  - 2
+  - 3
+  behavior: :return
+TestObject-simple_method-cdc941172e4d28939f6513e81222285c: !ruby/object:Camcorder::Recording
+  value:
+  - A
+  - B
+  - C
+  behavior: :return

data/spec/recordings/Camcorder_Proxy/_methods_with_side_effects/should_re-record_for_each_invocation.yml

@@ -1,3 +1,7 @@
 ---
-TestObject-method_with_side_effects-663f267dd7915410d1fce869a1bb0039: 1
-TestObject-method_with_side_effects-4ad0eb78b76e60e313f8d7b40194d3fc: 2
+TestObject-method_with_side_effects-663f267dd7915410d1fce869a1bb0039: !ruby/object:Camcorder::Recording
+  value: 1
+  behavior: :return
+TestObject-method_with_side_effects-4ad0eb78b76e60e313f8d7b40194d3fc: !ruby/object:Camcorder::Recording
+  value: 2
+  behavior: :return

data/spec/recordings/Camcorder_Proxy/when_recording_exists/should_not_call_methods_on_actual_instance.yml

@@ -1,5 +1,7 @@
 ---
-TestObject-simple_method-cdc941172e4d28939f6513e81222285c:
-- A
-- B
-- C
+TestObject-simple_method-cdc941172e4d28939f6513e81222285c: !ruby/object:Camcorder::Recording
+  value:
+  - A
+  - B
+  - C
+  behavior: :return

data/spec/recordings/Camcorder_Proxy/when_recording_exists/should_re-raise_recorded_errors.yml

@@ -0,0 +1,5 @@
+---
+TestObject-broke_method-cdc941172e4d28939f6513e81222285c: !ruby/object:Camcorder::Recording
+  value: !ruby/exception:StandardError
+    message: errorz
+  behavior: :raise

data/spec/support/test_object.rb

@@ -14,4 +14,8 @@ class TestObject
     @count += 1
   end
   
+  def broke_method
+    raise StandardError.new('errorz')
+  end
+  
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: camcorder
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
 platform: ruby
 authors:
 - Gordon L. Hempton
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-05-20 00:00:00.000000000 Z
+date: 2014-06-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler
@@ -56,16 +56,19 @@ files:
 - lib/camcorder/errors.rb
 - lib/camcorder/proxy.rb
 - lib/camcorder/recorder.rb
+- lib/camcorder/recording.rb
 - lib/camcorder/rspec.rb
 - lib/camcorder/version.rb
 - spec/camcorder/proxy_spec.rb
 - spec/camcorder/recorder_spec.rb
 - spec/camcorder_spec.rb
+- spec/fixtures/haz_errorz.yml
 - spec/fixtures/should_exist.yml
 - spec/recordings/Camcorder/constructor_interception/_new/should_return_a_proxy.yml
 - spec/recordings/Camcorder/constructor_interception/with_multiple_classes/_new/should_return_proxies.yml
 - spec/recordings/Camcorder_Proxy/_methods_with_side_effects/should_re-record_for_each_invocation.yml
 - spec/recordings/Camcorder_Proxy/when_recording_exists/should_not_call_methods_on_actual_instance.yml
+- spec/recordings/Camcorder_Proxy/when_recording_exists/should_re-raise_recorded_errors.yml
 - spec/spec_helper.rb
 - spec/support/test_object.rb
 - spec/support/test_object2.rb
@@ -97,11 +100,13 @@ test_files:
 - spec/camcorder/proxy_spec.rb
 - spec/camcorder/recorder_spec.rb
 - spec/camcorder_spec.rb
+- spec/fixtures/haz_errorz.yml
 - spec/fixtures/should_exist.yml
 - spec/recordings/Camcorder/constructor_interception/_new/should_return_a_proxy.yml
 - spec/recordings/Camcorder/constructor_interception/with_multiple_classes/_new/should_return_proxies.yml
 - spec/recordings/Camcorder_Proxy/_methods_with_side_effects/should_re-record_for_each_invocation.yml
 - spec/recordings/Camcorder_Proxy/when_recording_exists/should_not_call_methods_on_actual_instance.yml
+- spec/recordings/Camcorder_Proxy/when_recording_exists/should_re-raise_recorded_errors.yml
 - spec/spec_helper.rb
 - spec/support/test_object.rb
 - spec/support/test_object2.rb