super_stack 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 9a35826dd6aa1c436784d40750be42b5d903a9e8
-  data.tar.gz: 4036c864e855b5a9f420953ef0d99f4e83acbc36
+  metadata.gz: 2d7e71745423eac71f773457ba8d641c694b238b
+  data.tar.gz: 545f45196d81921479682642d7c292e283ab83f6
 SHA512:
-  metadata.gz: 79a519c5e1a9f202ce7ff2d2be502f94adb204a6b6c9e52775e56d6d0c0dd679add921e772c250bafcbba89c925872bbd7d10dc74964231b1e4af91281968579
-  data.tar.gz: b0a89dffd813a711c242498bcda46882c1f3b5dd941fe79056aa423cf57c4bed0888e88946b151af11b312bce2430eceeaf6ca75a1ef21a95fb1f3f61ddf4143
+  metadata.gz: 721a23e798806a3eada2ab1ca9bb002fb548115e4b47f1ad0c8a0df13a123c5effa13bfbcbd5f3e36c3ec9a0e8fa5ae3006c171f6171dfa48e5e3e4578219487
+  data.tar.gz: bc6b2e86889c0524fe31e0e97e7f32521fe9c89dacba1fc8317400b9867d8348f3b1c2e8f244f2af96119255a0c0486d90519cfc9011272398a5f388d29795f8

data/lib/super_stack/manager.rb

@@ -8,13 +8,29 @@ module SuperStack
     DEFAULT_PRIORITY_INTERVAL = 10
     DEFAULT_MERGE_POLICY = SuperStack::MergePolicies::StandardMergePolicy
 
-    attr_reader :layers
+    attr_reader :layers, :write_layer
 
     def initialize
       @layers = {}
       self.default_merge_policy = DEFAULT_MERGE_POLICY
     end
 
+    def write_layer=(layer_or_layer_name)
+      if layer_or_layer_name.is_a?(String) || layer_or_layer_name.is_a?(Symbol)
+        layer_name = layer_or_layer_name.to_s
+        raise 'Invalid write layer' unless layers.keys.include? layer_name
+        layer_or_layer_name = layers[layer_name]
+      end
+      # At that point layer_or_layer_name can only contain a layer object
+      raise 'Invalid write layer' unless layers.values.include? layer_or_layer_name
+      @write_layer = layer_or_layer_name
+    end
+
+    def []=(key,value)
+      raise 'No write layer specified' if write_layer.nil?
+      write_layer[key] = value
+    end
+
     def [](filter=nil)
       layers = to_a
       return [] if layers.empty?

data/lib/super_stack/version.rb

@@ -1,3 +1,3 @@
 module SuperStack
-  VERSION = "0.1.0"
+  VERSION = "0.2.0"
 end

data/spec/manager_spec.rb

@@ -68,19 +68,47 @@ describe SuperStack::Manager do
   end
 
   it 'should allow to reload all layers at once' do
-      subject.add_layer layer1
-      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.reload_layers}.not_to raise_error
-      expect(subject[:foo] == :bar).to be_falsey
-      expect(subject[:bar] == :foo).to be_truthy
+    subject.add_layer layer1
+    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.reload_layers}.not_to raise_error
+    expect(subject[:foo] == :bar).to be_falsey
+    expect(subject[:bar] == :foo).to be_truthy
+  end
+
+  it 'should be possible to specify a write layer using its name or itself' do
+    override = SuperStack::Layer.new
+    override.name = :override
+    subject << override
+    subject << layer1
+    expect {subject.write_layer = :override}.not_to raise_error
+    expect {subject.write_layer = 'override'}.not_to raise_error
+    expect {subject.write_layer = override}.not_to raise_error
   end
 
+  it 'should not be possible to modify if no write layer has been specified' do
+    subject << layer1
+    subject << layer2
+    expect {subject[:foo] = :bar}.to raise_error
+  end
+
+  it 'should push all modifications to the write layer' do
+    override = SuperStack::Layer.new
+    override.name = :override
+    subject << override
+    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
+  end
+
+
 
   SuperStack::MergePolicies.list.each do |policy|
     it "should provide a merged view of the layers according to the merge policy: #{policy}" do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: super_stack
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Laurent B.