poro_repository 0.0.1 → 0.0.2

data/README.md

@@ -31,7 +31,7 @@ Storing different entities separately
 
 ```ruby
 class Contact
-  attr_accessor :id, :name
+  attr_accessor :id, :name, :company
 end
 
 class Company
@@ -62,12 +62,18 @@ loaded_contact.company == loaded_company #=> true
 loaded_contact.company.equal?(loaded_company) #=> true ; same object
 ```
 
-Caveats
+Loading all records (it's all or nothing at the moment - no filtering)
+
+```ruby
+repo = PoroRepository.new("/repository/path")
+repo.load_all('Contact') #=> Array<Contact>
+```
+
+Limitations
 -------
 
 * No consideration has been given to concurrency. If you need concurrency, you
-  should probably use something else.
-* At least at this stage, it is only really suitable for storing hundreds of
-  objects, not thousands or millions.
-* It's still very incomplete and is missing most features required for it to
-  be generally useful.
+  could implement a mutex around repo access in your app, but you should
+  probably just use something else.
+* There is currently no way to efficiently search for records - you must load
+  up all records of a particular type, and filter them in the application.

data/lib/poro_repository/weak_hash.rb

@@ -17,7 +17,7 @@ class PoroRepository::WeakHash < Hash
 
   def [] key
     ref = super(key)
-    self.delete(key) if !ref.weakref_alive?
+    self.delete(key) if ref && !ref.weakref_alive?
     ref
   end
 

data/lib/poro_repository.rb

@@ -65,6 +65,12 @@ class PoroRepository
     id
   end
 
+  def load_all type
+    all_ids_for_type(type).collect do |id|
+      load_record type, id
+    end
+  end
+
   private
 
   def open_for_write path, &block
@@ -162,4 +168,11 @@ class PoroRepository
     end.flatten.compact
   end
 
+  def all_ids_for_type type
+    glob = record_path type, '*'
+    Dir[glob].collect do |path|
+      path.split('/').last # id
+    end
+  end
+
 end

data/poro_repository.gemspec

@@ -3,7 +3,7 @@ $:.push File.expand_path("../lib", __FILE__)
 
 Gem::Specification.new do |s|
   s.name        = "poro_repository"
-  s.version     = "0.0.1"
+  s.version     = "0.0.2"
   s.authors     = ["Joel Plane"]
   s.email       = ["joel.plane@gmail.com"]
   s.homepage    = "https://github.com/joelplane/poro_repository"

data/spec/poro_repository_spec.rb

@@ -164,4 +164,31 @@ describe PoroRepository do
 
   end
 
+  describe "#load_all" do
+
+    let(:object1) do
+      TestObject.new.tap do |o|
+        o.type = 'RecordType'
+        o.some_field = 42
+      end
+    end
+
+    let(:object2) do
+      TestObject.new.tap do |o|
+        o.type = 'RecordType'
+        o.some_field = 43
+      end
+    end
+
+    specify do
+      object_id1 = subject.save_record object1
+      object_id2 = subject.save_record object2
+      all_records = subject.load_all('RecordType').sort_by(&:some_field)
+      all_records.collect(&:some_field).should == [42, 43]
+      all_records.collect do |r|
+        subject.send(:id_from_record, r)
+      end.should == [object_id1, object_id2]
+    end
+  end
+
 end

data/spec/test_object.rb

@@ -1,3 +1,3 @@
 class TestObject
-  attr_accessor :id, :type, :name, :terms, :contact, :due_days
+  attr_accessor :id, :type, :name, :terms, :contact, :due_days, :some_field
 end

data/spec/weak_hash_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+require 'ostruct'
+
+describe PoroRepository::WeakHash do
+
+  subject do
+    PoroRepository::WeakHash.new.tap do |h|
+      h[:key1] = 'value1'
+      h[:key2] = 'value2'
+    end
+  end
+
+  describe do
+    specify "accessing non-existent key returns nil" do
+      subject[:non_existent_key].should be_nil
+    end
+  end
+
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: poro_repository
 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-10-06 00:00:00.000000000 Z
+date: 2013-10-12 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rspec
@@ -45,6 +45,7 @@ files:
 - spec/poro_repository_spec.rb
 - spec/spec_helper.rb
 - spec/test_object.rb
+- spec/weak_hash_spec.rb
 homepage: https://github.com/joelplane/poro_repository
 licenses: []
 post_install_message: 
@@ -73,3 +74,4 @@ test_files:
 - spec/poro_repository_spec.rb
 - spec/spec_helper.rb
 - spec/test_object.rb
+- spec/weak_hash_spec.rb