redic 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 982edbdc6b3b3ca417621ca43967117805d0faa9
-  data.tar.gz: 2f4b5a8ee99890f5e432cf5606984a3f683303de
+  metadata.gz: 3ba65ff4580315a8788983af009026bff12e3634
+  data.tar.gz: 24bdd4a1198cdcce1b5b2a961037f1a0708ae803
 SHA512:
-  metadata.gz: b4e520979e52233b4137c02463ea4fd6d1a7a222e3ec6665999fce604ed534a99ef36d05253bcce0255f7fec16a557dfc6b81f78e340c2dceb8a70b6ed9694e5
-  data.tar.gz: 080f253def0a1a5d0c7357a81b989a523a557e40de9b1d254fa0ef96c993b2738ed9e81be8656e27a031ff2ae7c3e1648a18f83c133c8a84c07e52a423965541
+  metadata.gz: 705de80d86e04f31637b4ea76496b69afff0e79e631d8fe5d312c2083d739abb88c1023760372d15d843b621b39d216c16c4e4b4edce88afc83e13f03a347527
+  data.tar.gz: 784f89f58b9bfec889c7f88884fa1ffed28bec17f1b30843f2cf1338dde46bafcf1e908df6c6346fb99aea80aadde5dc4e17e4c974252e0707e6b63da9b082ca

data/lib/redic/client.rb

@@ -39,30 +39,29 @@ class Redic
         raise err, "Can't connect to: %s" % @uri
       end
 
-      authenticate
-      select
-    end
-
-    def authenticate
-      if @uri.password
-        @semaphore.synchronize do
-          write ["AUTH", @uri.password]
-          read
-        end
+      if @uri.scheme == "redis"
+        @uri.password && assert_ok(call("AUTH", @uri.password))
+        @uri.path     && assert_ok(call("SELECT", @uri.path[1..-1]))
       end
     end
 
-    def select
-      if @uri.path
-        @semaphore.synchronize do
-          write ["SELECT", @uri.path[1..-1]]
-          read
-        end
+    def call(*args)
+      @semaphore.synchronize do
+        write(args)
+        read
       end
     end
 
     def connected?
       @connection && @connection.connected?
     end
+
+    def assert(value, error)
+      raise error unless value
+    end
+
+    def assert_ok(reply)
+      assert(reply == "OK", reply)
+    end
   end
 end

data/redic.gemspec

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

data/tests/redic_test.rb

@@ -15,7 +15,48 @@ test "url" do |c|
   assert_equal "redis://localhost:6379/", c.url
 end
 
+test "select db from url" do |c1|
+
+  # Connect to db 1
+  c2 = Redic.new("redis://localhost:6379/2")
+
+  c2.call("FLUSHDB")
+
+  # Set a value in db 0
+  c1.call("SET", "foo", "bar")
+
+  assert_equal(1, c1.call("DBSIZE"))
+  assert_equal(0, c2.call("DBSIZE"))
+end
+
+test "error when selecting db from url" do
+  c2 = Redic.new("redis://localhost:6379/foo")
+
+  assert_raise(RuntimeError) do
+
+    # Connection is lazy, send a command
+    c2.call("PING")
+  end
+end
+
+test "error when authenticating from url" do
+
+  # The correct password for 6380 is "foo"
+  c2 = Redic.new("redis://:foo@localhost:6380/")
+  c2.call("PING")
+
+  # The password provided is wrong
+  c3 = Redic.new("redis://:bar@localhost:6380/")
+
+  assert_raise(RuntimeError) do
+
+    # Connection is lazy, send a command
+    c3.call("PING")
+  end
+end
+
 test "timeout" do |c1|
+
   # Default timeout is 10 seconds
   assert_equal 10_000_000, c1.timeout
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: redic
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - Michel Martens