hammerspace 0.1.4 → 0.1.5

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    NGZkYjk4NzQ0MTlkYjI5Mzc1NWM2YjVmMTlhY2JmZTI1MWJlMmYwNQ==
+    MmM2ZmNiMjg1OGZjOWMwYjQ4MmE3NDgyNDAxZmU5M2FiNjE5OGM0Ng==
   data.tar.gz: !binary |-
-    ZWZkNDM3NWU2MmE4YTNjZmY0M2FlNWMwNzM3OGVlOTc3MmU5MTk5OA==
+    MDAzNzdhYzMwYmRlYTI1N2EzMDBlMjQyMWJiODE4MzJhOWZhMDgyMQ==
 !binary "U0hBNTEy":
   metadata.gz: !binary |-
-    MTAyYjVhMWE2Y2Q1OGI5YTFkYmRkM2ZkZDdkMmFjYTE2ZmMxYmQyNDI4YzU2
-    NmQyZWFiMDBkZWUxNjFkYTQ5M2Y2MGNlMTgxNmQ5ZGM1MjhkY2RkNjk5OTg0
-    ODcxNmYyOWQ2ZWMyN2RhN2UzNTU4OGI2YmM2NjczNWIyNzg2OTQ=
+    MjUxNjUyMGI4ZDIxNjFiOGE4ZWY4MjEzNWNmODBkOTBmZDY3M2UzOTY3YjEz
+    MDIxZWQ0M2M4YjE0OWQ3M2FiOWIxNjRlYmRkOGE5NTY3ZWZmYTM0N2FjYjA1
+    ZDBjYTZlZjkwNmUxZDdjZmFmMzJjNmM5MDdkOWFjYTgwY2I4Nzk=
   data.tar.gz: !binary |-
-    YzVmZjkyNDU2MTllNjNjZTZkMTgyMGM5MzNkOTc0OGQ5NzVkNmUxYzNhNDFi
-    ZGNiMGIzMzFkNjNmMGVhNWRmZjk1MzFhYjc3ZWVhZGRjNWQ0YWJiNzJjNjc5
-    YTNmNmE3Y2QzMDU1ODE0OGRiMmE2NTU4ZWUyMjZmMzFiMTQ1NTk=
+    YjdmNGIxZDY0NTgwNjhmNjg3MGIwZGU5YTY4ODJiZWUyODdiZWI4OWNkNzc2
+    ZmY0YTcwNjlkZTY1ZmYzNzRjYmMxNGRjZjU0OTA0MTYwNmIwYmVlMDNmMDZl
+    YTNmNGI5ZTVhYzk1YjY1OGI2ZmZjNmFhMjVjYWIzNmJlOTJhYzI=

data/CHANGELOG.md

@@ -1,3 +1,6 @@
+# v0.1.5
+* Avoid an unnecessary call to Gnista::Hash#include? on get.
+
 # v0.1.4
 * Upgrade to gnista 0.0.5.
 * Remove work around for gnista bug.

data/lib/hammerspace/backend/sparkey.rb

@@ -49,7 +49,10 @@ module Hammerspace
         close_logwriter
         open_hash
 
-        return @hash[key] if @hash && @hash.include?(key)
+        if @hash
+          value = @hash[key]
+          return value if value
+        end
         frontend.default(key)
       end
 

data/lib/hammerspace/version.rb

@@ -1,3 +1,3 @@
 module Hammerspace
-  VERSION = '0.1.4'
+  VERSION = '0.1.5'
 end

data/spec/features/hash_spec.rb

@@ -246,6 +246,12 @@ describe Hammerspace do
           hash.close
         end
 
+        it "returns nil if no default is set and key does not exist" do
+          hash = Hammerspace.new(path, options)
+          hash['foo'].should be_nil
+          hash.close
+        end
+
       end
 
       describe "#[]=" do
@@ -260,6 +266,38 @@ describe Hammerspace do
           hash.close
         end
 
+        it "does not allow non-string keys" do
+          hash = Hammerspace.new(path, options)
+          expect {
+            hash[8] = 'foo'
+          }.to raise_error TypeError
+          hash.close
+        end
+
+        it "does not allow non-string values" do
+          hash = Hammerspace.new(path, options)
+          expect {
+            hash['foo'] = 8
+          }.to raise_error TypeError
+          hash.close
+        end
+
+        it "does not allow nil keys" do
+          hash = Hammerspace.new(path, options)
+          expect {
+            hash[nil] = 'foo'
+          }.to raise_error TypeError
+          hash.close
+        end
+
+        it "does not allow nil values" do
+          hash = Hammerspace.new(path, options)
+          expect {
+            hash['foo'] = nil
+          }.to raise_error TypeError
+          hash.close
+        end
+
       end
 
       describe "#assoc" do

data/spec/lib/hammerspace/hash_spec.rb

@@ -73,8 +73,8 @@ describe Hammerspace::Hash do
     it "unsets default" do
       hash = Hammerspace::Hash.new(path, options)
       hash.default = 'bar'
-      hash.default_proc = p
-      hash.default('foo').should be_nil
+      hash.default_proc = lambda { |h,k| k }
+      hash.default('foo').should == 'foo'
     end
 
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: hammerspace
 version: !ruby/object:Gem::Version
-  version: 0.1.4
+  version: 0.1.5
 platform: ruby
 authors:
 - Jon Tai
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-03-26 00:00:00.000000000 Z
+date: 2015-11-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gnista