juvet 0.0.4 → 0.0.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7d3483a72f0e396a5f010a3ad73f5a1d888e89d2
-  data.tar.gz: 551045f13d3322dd05a2a1bd6574e9d32f37e07b
+  metadata.gz: 5d0de0f49adf12f3bb1ee241861332316fb1beb2
+  data.tar.gz: 5bfb76bc640fd10ed6ff0d079e34f033879aee12
 SHA512:
-  metadata.gz: 1dcd42919b068c0e841e1f7fa4052f6c5323f9d6071fb70b142dce80908f25835cd05661738cb9a2fdcd697c4ab4a09d0c8c1aee80f12f2c0cd657aa39c0e628
-  data.tar.gz: bd7ad250abe13a6c48caa4e0ec24d80090c669a5c86937f3faaaef49076c11ecadb398cb45eeda81a1927fffbf574e362421cbe11982e95576c501b81b655aa0
+  metadata.gz: e2a10cd29cdba17de6dd041a388abf3b21c349fd94987bfd96d33f77db9aba836daa5b0ecffed619a37958de921296bce8d0baf8bc68d5cdad7c914353f93840
+  data.tar.gz: c3702f246548de6bda6d1f6283f30ddc487b558f97fda747d6f3f18ea4b690e44c8ebb49f0985efb8c52d94455134f890614c84858136061ccac9a2c53885bfe

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    juvet (0.0.4)
+    juvet (0.0.5)
       redis
 
 GEM

data/lib/juvet/adapters/null_adapter.rb

@@ -4,6 +4,10 @@ module Juvet
       def initialize(collection=nil, options={})
       end
 
+      def all
+        []
+      end
+
       def create(entity)
         entity
       end

data/lib/juvet/adapters/redis_adapter.rb

@@ -11,6 +11,12 @@ module Juvet
         @redis = Redis.new options
       end
 
+      def all
+        redis.keys("#{collection_prefix}:*").map do |key|
+          find_by_key(key)
+        end
+      end
+
       def create(entity)
         update_attributes entity
       end
@@ -20,9 +26,7 @@ module Juvet
       end
 
       def find(id)
-        attributes = redis.get(collection_key(id))
-        return nil if attributes.nil?
-        collection.entity.new ({ id: id }).merge(JSON.load(attributes))
+        find_by_key collection_key(id)
       end
 
       def persist(entity)
@@ -37,7 +41,21 @@ module Juvet
       private
 
       def collection_key(id)
-        "#{Juvet::String.new(collection.name).underscore}:#{id}"
+        "#{collection_prefix}:#{id}"
+      end
+
+      def collection_prefix
+        Juvet::String.new(collection.name).underscore
+      end
+
+      def find_by_key(key)
+        attributes = redis.get key
+        return nil if attributes.nil?
+        collection.entity.new ({ id: id_from_key(key) }).merge(JSON.load(attributes))
+      end
+
+      def id_from_key(key)
+        key.gsub("#{collection_prefix}:", "")
       end
 
       def key(entity)

data/lib/juvet/bot.rb

@@ -5,6 +5,10 @@ module Juvet
     attr_reader :id
 
     class << self
+      def all!
+        BotRepository.all
+      end
+
       def create(id, attributes={})
         new attributes.merge(id: id)
       end

data/lib/juvet/repository.rb

@@ -19,6 +19,10 @@ module Juvet
         @adapter = adapter
       end
 
+      def all
+        adapter.all
+      end
+
       def create(entity)
         adapter.create entity
       end

data/lib/juvet/version.rb

@@ -1,3 +1,3 @@
 module Juvet
-  VERSION = "0.0.4"
+  VERSION = "0.0.5"
 end

data/spec/juvet/adapters/null_adapter_spec.rb

@@ -2,6 +2,12 @@ describe Juvet::Adapters::NullAdapter do
   let(:entity) { Object.new }
   subject { described_class.new }
 
+  describe "#all" do
+    it "simply returns an empty array" do
+      expect(subject.all).to be_empty
+    end
+  end
+
   describe "#create" do
     it "simply returns the entity" do
       expect(subject.create(entity)).to eq entity

data/spec/juvet/adapters/redis_adapter_spec.rb

@@ -14,13 +14,24 @@ describe Juvet::Adapters::RedisAdapter do
   end
 
   let(:collection) { double(:collection, name: :entities, entity: Entity) }
-  let(:entity) { Entity.new id: 123 }
+  let(:entity) { Entity.new id: "123" }
   let(:url) { "redis://localhost:6379" }
   subject { described_class.new collection, url: url, db: 15 }
 
   before(:all) { @redis = Redis.new db: 15 }
   after(:all) { @redis.flushdb }
 
+  describe "#all" do
+    it "returns all the entities for a given collection" do
+      entity1 = subject.create(entity)
+      entity2 = subject.create(Entity.new(id: "1234"))
+
+      result = subject.all
+
+      expect(result.map(&:id)).to eq ["1234", "123"]
+    end
+  end
+
   describe "#create" do
     it "stores the entity in the redis store" do
       result = subject.create entity
@@ -48,7 +59,7 @@ describe Juvet::Adapters::RedisAdapter do
     end
 
     it "returns nil if the entity can't be found" do
-      result = subject.find 789
+      result = subject.find "789"
 
       expect(result).to be_nil
     end
@@ -84,7 +95,7 @@ describe Juvet::Adapters::RedisAdapter do
     end
 
     it "throws an exception if the entity is not already stored" do
-      entity = Entity.new id: 456
+      entity = Entity.new id: "456"
 
       expect { subject.update entity }.to raise_error Juvet::EntityNotFoundError
     end

data/spec/juvet/bot_spec.rb

@@ -25,6 +25,13 @@ describe Juvet::Bot do
     end
   end
 
+  describe ".all!" do
+    it "returns all the bots from the repository" do
+      expect(Juvet::BotRepository).to receive(:all)
+      described_class.all!
+    end
+  end
+
   describe ".create!" do
     it "stores the created instance in the repository" do
       expect(Juvet::BotRepository).to receive(:create).with Juvet::Bot

data/spec/juvet/repository_spec.rb

@@ -7,6 +7,13 @@ describe Juvet::Repository do
     expect(Repository.adapter).to be_instance_of Juvet::Adapters::NullAdapter
   end
 
+  describe ".all" do
+    it "passes the retrieval to the adapter" do
+      expect(Repository.adapter).to receive(:all)
+      Repository.all
+    end
+  end
+
   describe ".create" do
     it "passes creation to the adapter" do
       object = Object.new

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: juvet
 version: !ruby/object:Gem::Version
-  version: 0.0.4
+  version: 0.0.5
 platform: ruby
 authors:
 - Jamie Wright
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-02-16 00:00:00.000000000 Z
+date: 2016-02-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: redis