super_stack 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 54cbe4ddd3ce5e7164c09042f75841a327cd1a1c
-  data.tar.gz: a01f72c6e8ab135e13cf12d0f9662e1bfa6c75fc
+  metadata.gz: 252c5d3c0c7502cd8d199660fd2366328c1a0b62
+  data.tar.gz: e8f0846916ab97dce8071ceffe9b6bf692ae7c3d
 SHA512:
-  metadata.gz: 771fcfb537611679e55a5fc36c14755ee276f71f3261190245b9299e54ec1f7fa460676f1e5a80056aa673ae9fcc9e9ab438850dd8a02c88bd3fc3aa4f64a83f
-  data.tar.gz: b9a855e2c8e20d9e15181ceb843da436c6b70175f96ed1deb8c0c088a67a65b6024e943e7eb265a96055ec86601195893bb262c13c68a0d81a7f5b1b66a7b64a
+  metadata.gz: 66082adea0c2facc15445c0bbef7e04bd42c66a80465e775456fb127b8f8a7c96c7f3c6bd31558e8966a177fb0883dd30dfe8401b07f3450cd351b1f41132955
+  data.tar.gz: 2f735111b2053ad0363fc775affc39f7ff3a267fdf207cffc463db46b199560e16f75d7df55b860a068d6fed80eb515e486056187a3e58219dbb7ad534b2a275

data/README.md

@@ -39,10 +39,20 @@ the result of the merge of all layers as a hash. It will be done according to th
 policy chosen.
 
 You can directly access to a particular key of the resulting merging by simply doing `manager[:your_key]` like you would
-do with a hash. (`manager` itself has no `Hash` in its ancestors, whereas `manager[]` *is* actually a `Hash`).
+do with a regular `Hash`.
 
-**None of the layers is modified by the manager**.
+`SuperStack::Manager` itself has no `Hash` in its ancestors, whereas `SuperStack::Manager#[]` that you can access
+through `manager[]` *is* actually the `Hash` resulting of the merge of the layers).
 
+**None of the layers is modified by the manager**, although you can nevertheless access them with the
+`SuperStack::Manager#layers` method.
+
+```ruby
+manager = SuperStack::Manager.new
+puts manager.class # =>  SuperStack::Manager
+puts manager[].class # =>  Hash
+puts manager.layers.class # =>  Hash
+```
 
 ### Layers
 
@@ -90,7 +100,7 @@ manager.reload_layers
 ```
 
 Layers can of course be:
-* removed (using the manager's `remove_layer(layer_or_layer_name)` method).
+* Removed (using the manager's `remove_layer(layer_or_layer_name)` method).
 * Enabled or disabled (using the manager's `disable_layer(layer_or_layer_name)` and `enable_layer(layer_or_layer_name)`
   methods).
 

data/lib/super_stack/layer_wrapper.rb

@@ -61,7 +61,7 @@ module SuperStack
 
     def load_from_yaml(file_name)
       begin
-        self.replace Hash[YAML::load(File.open(file_name)).map { |k, v| [k.to_sym, v] }]
+        self.replace Hash[YAML::load(File.open(file_name)).map { |k, v| [k.to_s, v] }]
 
       rescue  NoMethodError => e
         # Empty file...

data/lib/super_stack/manager.rb

@@ -24,7 +24,7 @@ module SuperStack
 
     def []=(key,value)
       raise 'No write layer specified' if write_layer.nil?
-      write_layer[key] = value
+      write_layer[key.to_s] = value
     end
 
     def [](filter=nil)
@@ -44,7 +44,9 @@ module SuperStack
         # Trick to return a bare hash
         {}.merge res
       else
-        res[filter]
+        value = res[filter]
+        value = res[filter.to_s] if filter.is_a? Symbol and value.nil?
+        value
       end
     end
 

data/lib/super_stack/version.rb

@@ -1,3 +1,3 @@
 module SuperStack
-  VERSION = '0.3.0'
+  VERSION = '0.4.0'
 end

data/spec/manager_spec.rb

@@ -81,13 +81,13 @@ describe SuperStack::Manager do
     subject << {bar: :foo}
     subject.add_layer layer3
     subject.add_layer layer4
-    expect(subject[:foo] == :bar).to be_falsey
-    subject.layers['layer3'][:foo] = :bar
-    expect(subject[:foo] == :bar).to be_truthy
-    expect(subject[:bar] == :foo).to be_truthy
+    expect(subject[:foo]).not_to eq :bar
+    subject.layers['layer3']['foo'] = :bar
+    expect(subject[:foo]).to eq :bar
+    expect(subject[:bar]).to eq :foo
     expect {subject.reload_layers}.not_to raise_error
-    expect(subject[:foo] == :bar).to be_falsey
-    expect(subject[:bar] == :foo).to be_truthy
+    expect(subject[:foo]).not_to eq :bar
+    expect(subject[:bar]).to eq :foo
   end
 
   it 'should not be possible to modify the manager if no write layer has been specified' do
@@ -111,8 +111,9 @@ describe SuperStack::Manager do
       subject << layer1
       subject.write_layer = override
       expect {subject[:foo] = :bar}.not_to raise_error
-      expect(subject.layers['override'][:foo] == :bar).to be_truthy
-      expect(subject[:foo] == :bar).to be_truthy
+      expect(subject.layers['override']['foo']).to eq :bar
+      expect(subject[:foo]).to eq :bar
+      expect(subject['foo']).to eq :bar
     end
 
 
@@ -206,11 +207,12 @@ describe SuperStack::Manager do
 
     it 'should remain consistent regarding the write level' do
       subject.write_layer = :layer3
-      expect(subject.write_layer == layer3).to be_truthy
+      expect(subject.write_layer).to eq layer3
       subject[:foo] = :bar
-      expect(layer3[:foo] == :bar).to be_truthy
+      expect(layer3['foo']).to eq :bar
       subject.remove_layer :layer3
-      expect(subject[:foo] == :bar).to be_falsey
+      expect(subject[:foo]).not_to eq :bar
+      expect(subject['foo']).not_to eq :bar
       expect(subject.write_layer).to be_nil
     end
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: super_stack
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Laurent B.
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-01-18 00:00:00.000000000 Z
+date: 2015-01-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: bundler