celluloid-redis 0.0.1 → 0.0.2

data/CHANGES.md

@@ -1,3 +1,7 @@
+0.0.2
+-----
+* Bugfix for handling Errno::ECONNRESET in the same way as redis-rb
+
 0.0.1
 -----
 * Initial (pre-)release

data/README.md

@@ -1,5 +1,6 @@
 Celluloid::Redis
 ================
+[![Gem Version](https://badge.fury.io/rb/celluloid-redis.png)](http://rubygems.org/gems/celluloid-redis)
 [![Build Status](https://secure.travis-ci.org/celluloid/celluloid-redis.png?branch=master)](http://travis-ci.org/celluloid/celluloid-redis)
 [![Dependency Status](https://gemnasium.com/celluloid/celluloid-redis.png)](https://gemnasium.com/celluloid/celluloid-redis)
 [![Code Climate](https://codeclimate.com/github/celluloid/celluloid-redis.png)](https://codeclimate.com/github/celluloid/celluloid-redis)

data/lib/celluloid/redis/version.rb

@@ -1,5 +1,5 @@
 module Celluloid
   module Redis
-    VERSION = "0.0.1"
+    VERSION = "0.0.2"
   end
 end

data/lib/redis/connection/celluloid.rb

@@ -51,6 +51,7 @@ class Redis
 
       def read
         line = @sock.gets
+        raise Errno::ECONNRESET unless line
         reply_type = line.slice!(0, 1)
         format_reply(reply_type, line)
 
@@ -99,4 +100,4 @@ class Redis
   end
 end
 
-Redis::Connection.drivers << Redis::Connection::Celluloid
+Redis::Connection.drivers << Redis::Connection::Celluloid

data/spec/redis/connection/celluloid_spec.rb

@@ -14,4 +14,11 @@ describe Redis::Connection::Celluloid do
     redis.set(example_key, example_value)
     redis.get(example_key).should eq example_value
   end
-end
+
+  it "cleanly shuts down an instance" do
+    with_new_instance do |instance|
+      redis = Redis.new(:port => instance.port, :driver => :celluloid)
+      expect { redis.shutdown }.not_to raise_error
+    end
+  end
+end

data/spec/redis_instance.rb

@@ -0,0 +1,35 @@
+class RedisInstance
+  def initialize(opts = {})
+    @opts = {:port => 6400, :cmd => 'redis-server'}.merge(opts)
+  end
+
+  def host
+    @opts[:host]
+  end
+
+  def port
+    @opts[:port]
+  end
+
+  def stop 
+    return unless @pid
+    begin
+      Process.kill('SIGTERM', @pid)
+    rescue Errno::ESRCH
+    end
+  end
+
+  def run
+    @pid = spawn(@opts[:cmd], "--port #{port}", "--daemonize yes")
+    wait_until_useable
+  end
+
+  private
+  def wait_until_useable
+    begin
+      TCPSocket.open('localhost', port)
+    rescue Errno::ECONNREFUSED
+      retry
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,4 +1,15 @@
 require "celluloid/redis"
 require "redis/connection/celluloid"
+require "redis_instance"
 require "coveralls"
 Coveralls.wear!
+
+def with_new_instance(opts = {})
+  begin
+    instance = RedisInstance.new(opts)
+    instance.run
+    yield instance
+  ensure
+    instance.stop
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: celluloid-redis
 version: !ruby/object:Gem::Version
-  version: 0.0.1
+  version: 0.0.2
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-14 00:00:00.000000000 Z
+date: 2013-04-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis
@@ -129,6 +129,7 @@ files:
 - lib/celluloid/redis/version.rb
 - lib/redis/connection/celluloid.rb
 - spec/redis/connection/celluloid_spec.rb
+- spec/redis_instance.rb
 - spec/spec_helper.rb
 - tasks/rspec.rake
 homepage: https://github.com/celluloid/celluloid-redis
@@ -146,7 +147,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1868622521016118762
+      hash: 2624804133874232203
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -155,7 +156,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1868622521016118762
+      hash: 2624804133874232203
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23
@@ -164,4 +165,5 @@ specification_version: 3
 summary: celluloid-redis provides a redis-rb connection class using Celluloid::IO
 test_files:
 - spec/redis/connection/celluloid_spec.rb
+- spec/redis_instance.rb
 - spec/spec_helper.rb