pulse-meter 0.1.9 → 0.1.10

data/lib/pulse-meter/mixins/dumper.rb

@@ -10,12 +10,23 @@ module PulseMeter
       module InstanceMethods
         # Serializes object and saves it to Redis
         # @raise [DumpError] if dumping fails for any reason
-        def dump!
+        def dump!(safe = true)
           ensure_storability!
-          serialized_obj = self.to_yaml
-          redis.hset(DUMP_REDIS_KEY, self.name, serialized_obj)
-        rescue
-          raise DumpError, "object cannot be dumped"
+          serialized_obj = to_yaml
+          if safe
+            unless redis.hsetnx(DUMP_REDIS_KEY, name, serialized_obj)
+              stored = self.class.restore(name)
+              unless stored.class == self.class
+                raise DumpConflictError, "Attempt to create sensor #{name} of class #{self.class} but it already has class #{stored.class}"
+              end
+            end
+          else
+            redis.hset(DUMP_REDIS_KEY, name, serialized_obj)
+          end
+        rescue DumpError, RestoreError => exc
+          raise exc
+        rescue StandardError => exc
+          raise DumpError, "object cannot be dumped: #{exc}"
         end
 
         # Ensures that object is dumpable

data/lib/pulse-meter/sensor.rb

@@ -37,6 +37,9 @@ module PulseMeter
   # Exception to be raised when sensor cannot be dumped
   class DumpError < SensorError; end
 
+  # Exception to be raised on attempts of using the same key for different sensors
+  class DumpConflictError < DumpError; end
+
   # Exception to be raised when sensor cannot be restored
   class RestoreError < SensorError; end
 

data/lib/pulse-meter/version.rb

@@ -1,3 +1,3 @@
 module PulseMeter
-  VERSION = "0.1.9"
+  VERSION = "0.1.10"
 end

data/spec/pulse_meter/mixins/dumper_spec.rb

@@ -8,19 +8,23 @@ describe PulseMeter::Mixins::Dumper do
   class Bad < Base; end
 
   class Good < Base
-    attr_accessor :foo
-    def name; foo.to_s; end
+    attr_accessor :some_value
+    attr_accessor :name
 
     def redis; PulseMeter.redis; end
 
-    def initialize(foo)
-      @foo = foo
+    def initialize(name)
+      @name = name.to_s
+      @some_value = name
     end
   end
 
+  class GoodButAnother < Good; end
+
   let(:bad_obj){ Bad.new }
   let(:good_obj){ Good.new(:foo) }
   let(:another_good_obj){ Good.new(:bar) }
+  let(:good_obj_of_another_type){ GoodButAnother.new(:quux) }
   let(:redis){ PulseMeter.redis }
 
   describe '#dump' do
@@ -57,6 +61,23 @@ describe PulseMeter::Mixins::Dumper do
         expect {good_obj.dump!}.to change {redis.hlen(Good::DUMP_REDIS_KEY)}.by(1)
       end
     end
+
+    context "when dump is safe" do
+      it "should not overwrite stored objects of the same type" do
+        good_obj.some_value = 123
+        good_obj.dump!
+        good_obj.some_value = 321
+        good_obj.dump!
+        Base.restore(good_obj.name).some_value.should == 123
+      end
+
+      it "should raise DumpConflictError exception if sensor with the same name but different type already exists" do
+        good_obj.name = "duplicate_name"
+        good_obj_of_another_type.name = "duplicate_name"
+        good_obj.dump!
+        expect{good_obj_of_another_type.dump!}.to raise_exception(PulseMeter::DumpConflictError)
+      end
+    end
   end
 
   describe ".restore" do
@@ -77,14 +98,14 @@ describe PulseMeter::Mixins::Dumper do
 
       it "should restore object data" do
         restored = Base.restore(good_obj.name)
-        restored.foo.should == good_obj.foo
+        restored.some_value.should == good_obj.some_value
       end
 
       it "should restore last dumped object" do
-        good_obj.foo = :bar
-        good_obj.dump!
+        good_obj.some_value = :bar
+        good_obj.dump!(false)
         restored = Base.restore(good_obj.name)
-        restored.foo.should == :bar
+        restored.some_value.should == :bar
       end
     end
   end
@@ -106,8 +127,8 @@ describe PulseMeter::Mixins::Dumper do
       end
 
       it "should return list of registered objects" do
-        good_obj.dump!
-        another_good_obj.dump!
+        good_obj.dump!(false)
+        another_good_obj.dump!(false)
         Base.list_names.should =~ [good_obj.name, another_good_obj.name]
       end
     end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: pulse-meter
 version: !ruby/object:Gem::Version
-  version: 0.1.9
+  version: 0.1.10
   prerelease: 
 platform: ruby
 authors:
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-06-06 00:00:00.000000000 Z
+date: 2012-06-08 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gon-sinatra