redic 0.0.3 → 0.0.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 826a4619b6aec694f3b74b8221db03a6e4cf8f61
-  data.tar.gz: 7c3635c0daf967b9567c9c9f92b8ad5b7ec09284
+  metadata.gz: 764359a897686829a6256da74c6cb9b09431c145
+  data.tar.gz: 326b89336c17823309e87f0892c674d1d3162649
 SHA512:
-  metadata.gz: 53ee3848f37877d86f789e8883867b5f4d6daec0f6575df993e291b343829089471b986a529d09da358316c136b824209e47bebb23604d85da57ef26e99e53d4
-  data.tar.gz: df61e6d8c6551297490450d0001dc2b68691d0008321bfb296d34ed41048fce8f1f0e28163a13b619870c78d13651d8fe2971136e2ae850a5fe1ea85689bf0c2
+  metadata.gz: b15177f76ea1af68c1dd3b18689310df67d2ab867034df7e0039607236c283c5dff4e29189d1bc43c8e7989ced9a9a1bd502611cd5cdc9b1c84ac61b86244e79
+  data.tar.gz: ebd8fbb06218eff075bed70ffa38ef0b8c48a444c7e41189ba097cadf9ee10f53ce3f7234b7a946f296ab9f79737959a541f06f64c77416fcfc324343984efdc

data/lib/redic/client.rb

@@ -10,9 +10,7 @@ class Redic
     end
 
     def read
-      @connection.read.tap do |reply|
-        raise reply if reply.kind_of?(RuntimeError)
-      end
+      @connection.read
     end
 
     def write(command)
@@ -20,35 +18,18 @@ class Redic
     end
 
     def connect
-      try(3) do
-        establish_connection unless connected?
-
-        @semaphore.synchronize do
-          yield
-        end
-      end
-    end
+      establish_connection unless connected?
 
-  private
-    def try(times)
-      tries = 0
-
-      begin
+      @semaphore.synchronize do
         yield
-      rescue Exception => err
-        if (tries += 1) <= times
-          sleep 0.01
-          retry
-        else
-          raise err, "%s (retries=%d)" % [err.message, tries]
-        end
       end
     end
 
+  private
     def establish_connection
       begin
         @connection = Redic::Connection.new(@uri)
-      rescue Exception => err
+      rescue StandardError => err
         raise err, "Can't connect to: %s" % @uri
       end
 

data/redic.gemspec

@@ -2,7 +2,7 @@
 
 Gem::Specification.new do |s|
   s.name              = "redic"
-  s.version           = "0.0.3"
+  s.version           = "0.0.4"
   s.summary           = "Lightweight Redis Client"
   s.description       = "Lightweight Redis Client"
   s.authors           = ["Michel Martens", "Cyril David"]

data/test/redic_test.rb

@@ -30,7 +30,49 @@ test "multi/exec" do |c|
 end
 
 test "runtime errors" do |c|
-  assert_raise RuntimeError do
-    c.call("KABLAMMO")
+  res = c.call("KABLAMMO")
+
+  assert res.is_a?(RuntimeError)
+end
+
+test "encoding" do |c|
+  Encoding.default_external = "UTF-8"
+
+  c.call("SET", "foo", "שלום")
+
+  assert_equal "Shalom שלום", "Shalom " + c.call("GET", "foo")
+
+end if defined?(Encoding)
+
+test "errors in pipeline" do |c|
+  c.write("SET", "foo", "bar")
+  c.write("INCR", "foo")
+  c.write("GET", "foo")
+
+  res = c.run
+
+  assert "OK" == res[0]
+  assert RuntimeError === res[1]
+  assert "bar" == res[2]
+end
+
+test "thread safety" do |c|
+  c.call("SET", "foo", 1)
+  c.call("SET", "bar", 2)
+
+  foos, bars = nil, nil
+
+  t1 = Thread.new do
+    foos = Array.new(100) { c.call("GET", "foo") }
   end
+
+  t2 = Thread.new do
+    bars = Array.new(100) { c.call("GET", "bar") }
+  end
+
+  t1.join
+  t2.join
+
+  assert_equal ["1"], foos.uniq
+  assert_equal ["2"], bars.uniq
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: redic
 version: !ruby/object:Gem::Version
-  version: 0.0.3
+  version: 0.0.4
 platform: ruby
 authors:
 - Michel Martens
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-07 00:00:00.000000000 Z
+date: 2013-04-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: hiredis