keyp 0.0.5 → 0.0.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f6d6251ce2d9255c0f752a0e7981934c8bb3c5e4
-  data.tar.gz: 494b0741cf77583966b6a3317830ac8b53c1a778
+  metadata.gz: bff8c5655d1b350b28208c20add4b0b3cf884fd2
+  data.tar.gz: 53a122d43fda4ec7267bd77aea3626dc2e16dba6
 SHA512:
-  metadata.gz: cfb27e7d76e1813682f996029cc55026dfca82ed34fa70113dce8785fcbe3b992cd624c58e183bcc41527eee751d0fca832c33cc78810b27d91e14d4ab2b57b9
-  data.tar.gz: 3a3fb3beb457d303817333042c657b13d91c6ebc2d8104ade092d94e2e77d022909573621f6ec5665e18d7d68bc0bad393e762acfa7113402baa08152faebd0b
+  metadata.gz: cf0bbf2bc59d7a578afd69641032677ab6ff61f7d78331603a63e30fab9bd3dfd68e88756e9c74d0566f189d438bf92ebe91c9ad43ee7bef74b9379d5dac793b
+  data.tar.gz: baddf04bce4de7c751da7acf783a9527d00c31690cfa7d1a7697c402509c5cd912a9f2d1b4266d95d9f0eec08e79438b4628c2b1e3fd155559d04ef1797b2cc1

data/README.md

@@ -93,7 +93,7 @@ TODO: Write more detailed documentation, For now, see the quick start above and
     $ keyp --help
 
 ## Release Notes
-
+* v 0.0.5 - Fixed iso8601 error
 * v 0.0.4 - Fixed missing file error
 
 ## Development plan/Features to implement

data/lib/keyp/bag.rb

@@ -47,7 +47,7 @@ module Keyp
       # not the most efficient thing, but simpler and safe enough for now
 
       unless File.exist? keypfile
-        puts "Keyper.initialize, create_bag #{keypfile}"
+        puts "Bag.initialize, create_bag #{keypfile}"
         Keyp::create_bag(name)
       end
       file_data = load(keypfile)
@@ -78,8 +78,10 @@ module Keyp
         # TODO: check if data has been modified
         # maybe there is a way hash tells us its been modified. If not then
         # just check if key,val is already in hash and matches
-        @data[key] = value
-        @dirty = true
+        unless @data.key?(key) && @data[key] == value
+          @data[key] = value
+          @dirty = true
+        end
       else
         raise "Bag #{@name} is read only"
       end
@@ -134,9 +136,8 @@ module Keyp
       overwrite = options[:overwrite] || false
       pattern = options[:sysvar] if options.key?(:sysvar)
 
-      pattern ||= '(...)'
-
-      bag.data.each do |key,value|
+      pattern ||= /(...)/
+      @data.each do |key,value|
         if pattern.match(key)
           # TODO: add overwrite checking
           ENV[key] = value

data/lib/keyp/version.rb

@@ -1,3 +1,3 @@
 module Keyp
-  VERSION = "0.0.5"
+  VERSION = "0.0.6"
 end

data/spec/bag_spec.rb

@@ -23,6 +23,7 @@ describe Keyp::Bag do
       @bag = Keyp::Bag.new 'grue_eats_you'
       @bag['LIGHTS'] = 'out'
     end
+
     it "should return a non-empty hash" do
       @bag.data.size.should > 0
     end
@@ -31,12 +32,10 @@ describe Keyp::Bag do
   context "environment variables" do
     before (:each) do
       # TODO: pseudo random bag name generation to a helper
-      bag_name = "grue_eats_you_when_it_is_dark_#{Time.now.strftime("%Y%m%d%H%M")}"
+      bag_name = "grue_eats_you_when_it_is_dark_#{Time.now.strftime("%Y%m%d%H%M%S%L")}"
       @bag = Keyp::Bag.new bag_name
     end
-    it "should copy all vars"
-=begin
-    do
+    it "should copy all vars"  do
       testvars = {
           'ALPHA' => 'First in the phonetic alphabet',
           'BRAVO' => 'Second in the phonetic alphabet',
@@ -45,12 +44,58 @@ describe Keyp::Bag do
 
       testvars.each { |key, value| @bag[key] = value }
 
-      @bag.add_to_env
+      added = @bag.add_to_env
       testvars.each do |key, value|
         ENV[key].should_not == nil
         ENV[key].should == value
       end
     end
-=end
   end
+
+  context "bag state" do
+
+    before (:each) do
+      # TODO: pseudo random bag name generation to a helper
+      bag_name = "grue_eats_you_when_it_is_dark_#{Time.now.strftime("%Y%m%d%H%M%S%L")}"
+      @bag = Keyp::Bag.new bag_name
+    end
+
+    it 'should not set the dirty flag when no items are in the bag' do
+      @bag.empty?.should == true
+      @bag.dirty.should == false
+    end
+
+    it 'should set the dirty flag if a new key is created' do
+      @bag['KEY1'] = 'value1'
+      @bag.empty?.should == false
+      @bag.dirty.should == true
+
+    end
+
+    it 'should not set the dirty flag after save' do
+      @bag.empty?.should == true
+      @bag['KEY1'] = 'value1'
+      @bag.save
+      @bag.dirty.should == false
+    end
+    it 'should set the dirty flag if a key is given a new value' do
+      @bag.empty?.should == true
+      @bag['KEY1'] = 'value1'
+      @bag.dirty.should == true
+    end
+
+    it 'should not set the dirty flag if a key is assigned a value equal to its existing value' do
+      @bag.empty?.should == true
+      @bag['KEY1'] = 'value1'
+      @bag.save
+      @bag.dirty.should == false
+      @bag['KEY1'] = 'value1'
+      @bag.dirty.should == false
+    end
+  end
+
+  it 'should return a key with data member'
+  it 'should return a key acting as a hash'
+  it 'should allow assigning a key if not read only'
+  it 'should not allow assigning a key if read only'
 end

data/spec/keyp_spec.rb

@@ -5,7 +5,7 @@ describe Keyp do
   context "CONSTANTS" do
     it 'should return correct version string' do
       #Keyp.version_string.should == "Keyp version #{Keyp::VERSION}"
-      Keyp::VERSION.should == '0.0.5'
+      Keyp::VERSION.should == '0.0.6'
     end
 
     it 'should specify default store' do
@@ -44,20 +44,14 @@ describe Keyp do
 
     it "should say a bag does not exist" do
       # TODO: add bag name generation to a helper
-      bag_name = "grue_eats_you_when_it_is_dark_#{Time.now.strftime("%Y%m%d%H%M")}"
+      bag_name = "grue_eats_you_when_it_is_dark_#{Time.now.strftime("%Y%m%d%H%M%S%L")}"
       Keyp.exist?(bag_name).should_not == true
     end
 
   end
 
 
-  it 'should return a key with data member'
-  it 'should return a key acting as a hash'
-  it 'should allow assigning a key if not read only'
-  it 'should not allow assigning a key if read only'
-  it 'should set the dirty flag if a new key is created'
-  it 'should set the dirty flag if a key is given a new value'
-  it 'should not set the dirty flag if a key is assigned a value equal to its existing value'
+
 
   # Braindump of tests
   # Should show no keys if empty bag

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: keyp
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - John Baldwin
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-04-01 00:00:00.000000000 Z
+date: 2014-04-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: gli